Properly Notify when Achievements Login Fails

This commit is contained in:
LillyJadeKatrin 2024-07-24 20:50:54 -04:00
parent 8fac6061ac
commit b7096ef64f
5 changed files with 25 additions and 4 deletions

View File

@ -733,6 +733,7 @@ void AchievementManager::LoginCallback(int result, const char* error_message, rc
{ {
WARN_LOG_FMT(ACHIEVEMENTS, "Failed to login {} to RetroAchievements server.", WARN_LOG_FMT(ACHIEVEMENTS, "Failed to login {} to RetroAchievements server.",
Config::Get(Config::RA_USERNAME)); Config::Get(Config::RA_USERNAME));
AchievementManager::GetInstance().m_update_callback({.failed_login_code = result});
return; return;
} }
@ -744,6 +745,7 @@ void AchievementManager::LoginCallback(int result, const char* error_message, rc
if (!user) if (!user)
{ {
WARN_LOG_FMT(ACHIEVEMENTS, "Failed to retrieve user information from client."); WARN_LOG_FMT(ACHIEVEMENTS, "Failed to retrieve user information from client.");
AchievementManager::GetInstance().m_update_callback({.failed_login_code = RC_INVALID_STATE});
return; return;
} }
@ -762,6 +764,7 @@ void AchievementManager::LoginCallback(int result, const char* error_message, rc
INFO_LOG_FMT(ACHIEVEMENTS, "Attempted to login prior user {}; current user is {}.", INFO_LOG_FMT(ACHIEVEMENTS, "Attempted to login prior user {}; current user is {}.",
user->username, Config::Get(Config::RA_USERNAME)); user->username, Config::Get(Config::RA_USERNAME));
rc_client_logout(client); rc_client_logout(client);
AchievementManager::GetInstance().m_update_callback({.failed_login_code = RC_INVALID_STATE});
return; return;
} }
} }

View File

@ -98,6 +98,7 @@ public:
bool all_leaderboards = false; bool all_leaderboards = false;
std::set<AchievementId> leaderboards{}; std::set<AchievementId> leaderboards{};
bool rich_presence = false; bool rich_presence = false;
int failed_login_code = 0;
}; };
using UpdateCallback = std::function<void(const UpdatedItems&)>; using UpdateCallback = std::function<void(const UpdatedItems&)>;

View File

@ -39,8 +39,24 @@ AchievementSettingsWidget::AchievementSettingsWidget(QWidget* parent) : QWidget(
ToggleHardcore(); ToggleHardcore();
} }
void AchievementSettingsWidget::UpdateData() void AchievementSettingsWidget::UpdateData(int login_failed_code)
{ {
if (login_failed_code != RC_OK)
{
switch (login_failed_code)
{
case RC_INVALID_CREDENTIALS:
m_common_login_failed->setText(tr("Login Failed - Invalid Username/Password"));
break;
case RC_NO_RESPONSE:
m_common_login_failed->setText(tr("Login Failed - No Internet Connection"));
break;
default:
m_common_login_failed->setText(tr("Login Failed - Server Error"));
break;
}
m_common_login_failed->setVisible(true);
}
LoadSettings(); LoadSettings();
} }
@ -245,6 +261,7 @@ void AchievementSettingsWidget::ToggleRAIntegration()
void AchievementSettingsWidget::Login() void AchievementSettingsWidget::Login()
{ {
m_common_login_failed->setVisible(false);
Config::SetBaseOrCurrent(Config::RA_USERNAME, m_common_username_input->text().toStdString()); Config::SetBaseOrCurrent(Config::RA_USERNAME, m_common_username_input->text().toStdString());
AchievementManager::GetInstance().Login(m_common_password_input->text().toStdString()); AchievementManager::GetInstance().Login(m_common_password_input->text().toStdString());
m_common_password_input->setText(QString()); m_common_password_input->setText(QString());

View File

@ -18,7 +18,7 @@ class AchievementSettingsWidget final : public QWidget
Q_OBJECT Q_OBJECT
public: public:
explicit AchievementSettingsWidget(QWidget* parent); explicit AchievementSettingsWidget(QWidget* parent);
void UpdateData(); void UpdateData(int login_failed_code);
private: private:
void OnControllerInterfaceConfigure(); void OnControllerInterfaceConfigure();

View File

@ -36,7 +36,7 @@ AchievementsWindow::AchievementsWindow(QWidget* parent) : QDialog(parent)
}); });
}); });
connect(&Settings::Instance(), &Settings::EmulationStateChanged, this, connect(&Settings::Instance(), &Settings::EmulationStateChanged, this,
[this] { m_settings_widget->UpdateData(); }); [this] { m_settings_widget->UpdateData(0); });
connect(&Settings::Instance(), &Settings::HardcoreStateChanged, this, connect(&Settings::Instance(), &Settings::HardcoreStateChanged, this,
[this] { AchievementsWindow::UpdateData({.all = true}); }); [this] { AchievementsWindow::UpdateData({.all = true}); });
} }
@ -79,7 +79,7 @@ void AchievementsWindow::ConnectWidgets()
void AchievementsWindow::UpdateData(AchievementManager::UpdatedItems updated_items) void AchievementsWindow::UpdateData(AchievementManager::UpdatedItems updated_items)
{ {
m_settings_widget->UpdateData(); m_settings_widget->UpdateData(updated_items.failed_login_code);
if (updated_items.all) if (updated_items.all)
{ {
m_header_widget->UpdateData(); m_header_widget->UpdateData();