CactiChameleon9/PrismLauncher
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix crashing when there are no accounts added

Browse Source 
The warning message actually displays now when there are no Mojang
accounts to log in with, rather than simply crashing MultiMC.
This commit is contained in:
Andrew 2013-11-22 09:53:27 -06:00
parent abf8408911
commit a332e0d7b1
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
gui/MainWindow.cpp
View File

@ -569,7 +569,8 @@ void MainWindow::doLogin(const QString &errorMsg)
// Tell the user they need to log in at least one account in order to play. // Tell the user they need to log in at least one account in order to play.
CustomMessageBox::selectable(this, tr("No Accounts"), CustomMessageBox::selectable(this, tr("No Accounts"),
tr("In order to play Minecraft, you must have at least one Mojang or Minecraft account logged in to MultiMC. Please add an account."), tr("In order to play Minecraft, you must have at least one Mojang or Minecraft account logged in to MultiMC. Please add an account."),
QMessageBox::Warning)->show(); QMessageBox::Information)->exec();
return;
} }
else else
{ {
@ -612,6 +613,7 @@ void MainWindow::doLogin(const QString &errorMsg)
void MainWindow::launchInstance(BaseInstance *instance, MojangAccountPtr account) void MainWindow::launchInstance(BaseInstance *instance, MojangAccountPtr account)
{ {
Q_ASSERT_X(instance != NULL, "launchInstance", "instance is NULL"); Q_ASSERT_X(instance != NULL, "launchInstance", "instance is NULL");
Q_ASSERT_X(account.get() != nullptr, "launchInstance", "account is NULL");
proc = instance->prepareForLaunch(account); proc = instance->prepareForLaunch(account);
if (!proc) if (!proc)