Revert "Add option for sorting servers list"

This reverts commits e35b3bdd0a74464e4b4d3b931ca416f0c7864ba6
and e35b3bdd0a74464e4b4d3b931ca416f0c7864ba6.

Sometimes the text fields and the list show different server
selected, so you don't see to what server you connect. It is
a big security risk sending your user name and password to
the wrong server.

Reviewed-by: Freeyorp, Bertram

1 files changed, 0 insertions, 38 deletions

diff --git a/src/gui/setup_video.cpp b/src/gui/setup_video.cpp
index 0c2ebf83..7f6aecb3 100644
--- a/src/gui/setup_video.cpp
+++ b/src/gui/setup_video.cpp
@@ -150,31 +150,6 @@ public:
     }
 };
 
-const char *SERVLIST_ORDER_BY[2] =
-{
-    N_("Last usage"),
-    N_("Name")
-};
-
-class ServerListOrderListModel : public gcn::ListModel
-{
-public:
-    virtual ~ServerListOrderListModel() { }
-
-    virtual int getNumberOfElements()
-    {
-        return 2;
-    }
-
-    virtual std::string getElementAt(int i)
-    {
-        if (i >= getNumberOfElements())
-            return _("???");
-
-        return SERVLIST_ORDER_BY[i];
-    }
-};
-
 static const char *speechModeToString(Being::Speech mode)
 {
     switch (mode)
@@ -262,7 +237,6 @@ Setup_Video::Setup_Video():
     mParticleDetailSlider(new Slider(0, 3)),
     mParticleDetailField(new Label),
     mFontSize(config.getIntValue("fontSize")),
-    mServerListOrder(config.getIntValue("serverListOrder")),
     mDisableSDLTransparencyCheckBox(
                           new CheckBox(_("Disable transparency (Low CPU mode)"),
                                        mSDLTransparencyDisabled))
@@ -280,14 +254,10 @@ Setup_Video::Setup_Video():
     overlayDetailLabel = new Label(_("Ambient FX"));
     particleDetailLabel = new Label(_("Particle detail"));
     fontSizeLabel = new Label(_("Font size"));
-    serverListOrderLabel = new Label(_("Order servers by"));
 
     mFontSizeListModel = new FontSizeChoiceListModel;
     mFontSizeDropDown = new DropDown(mFontSizeListModel);
 
-    mServerListOrderListModel = new ServerListOrderListModel;
-    mServerListOrderDropDown = new DropDown(mServerListOrderListModel);
-
     mModeList->setEnabled(true);
 
 #ifndef USE_OPENGL
@@ -365,9 +335,6 @@ Setup_Video::Setup_Video():
     mFontSizeDropDown->setSelected(mFontSize - 10);
     mFontSizeDropDown->adjustHeight();
 
-    mServerListOrderDropDown->setSelected(mServerListOrder);
-    mServerListOrderDropDown->adjustHeight();
-
     // Do the layout
     LayoutHelper h(this);
     ContainerPlacer place = h.getPlacer(0, 0);
@@ -414,9 +381,6 @@ Setup_Video::Setup_Video():
 
     place(0, 12, mDisableSDLTransparencyCheckBox, 4);
 
-    place(0, 13, serverListOrderLabel, 3);
-    place(1, 13, mServerListOrderDropDown, 2);
-
     setDimension(gcn::Rectangle(0, 0, 365, 300));
 }
 
@@ -425,7 +389,6 @@ Setup_Video::~Setup_Video()
     delete mModeListModel;
     delete mModeList;
     delete mFontSizeListModel;
-    delete mServerListOrderListModel;
 }
 
 void Setup_Video::apply()
@@ -524,7 +487,6 @@ void Setup_Video::apply()
     // FPS change
     config.setValue("fpslimit", mFps);
     config.setValue("fontSize", mFontSizeDropDown->getSelected() + 10);
-    config.setValue("serverListOrder", mServerListOrderDropDown->getSelected());
 
     // We sync old and new values at apply time
     mFullScreenEnabled = config.getBoolValue("screen");