From 4092dbd156f84e00c1603274f534963d9f98667f Mon Sep 17 00:00:00 2001 From: Stefan Dombrowski Date: Tue, 7 Sep 2010 22:22:27 +0200 Subject: Fixing logic that decides if ServerDialog shows up --- src/client.cpp | 26 +++++++++++++------------- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/src/client.cpp b/src/client.cpp index 3946d0f9..5488d8ad 100644 --- a/src/client.cpp +++ b/src/client.cpp @@ -607,19 +607,10 @@ int Client::exec() // If a server was passed on the command line, or branding // provides a server and a blank server list, we skip the // server selection dialog. - if (mOptions.serverName.empty() || mOptions.serverPort == 0 - || !(!branding.getValue("defaultServer","").empty() && - branding.getValue("defaultPort",0) && - branding.getValue("onlineServerList", "").empty())) - { - // Don't allow an alpha opacity - // lower than the default value - Theme::instance()->setMinimumOpacity(0.8f); - - mCurrentDialog = new ServerDialog(&mCurrentServer, - mConfigDir); - } - else + if ((!mOptions.serverName.empty() && mOptions.serverPort) + || (!branding.getValue("defaultServer","").empty() && + branding.getValue("defaultPort",0) && + branding.getValue("onlineServerList", "").empty())) { mState = STATE_CONNECT_SERVER; @@ -628,6 +619,15 @@ int Client::exec() mOptions.serverName.clear(); mOptions.serverPort = 0; } + else + { + // Don't allow an alpha opacity + // lower than the default value + Theme::instance()->setMinimumOpacity(0.8f); + + mCurrentDialog = new ServerDialog(&mCurrentServer, + mConfigDir); + } break; case STATE_CONNECT_SERVER: -- cgit v1.2.3-70-g09d2