Removed saving of all online ids in players.xml file.

src/config/player_manager.cpp

@@ -190,19 +190,6 @@ void PlayerManager::load()
         return;
     }
 
-    const XMLNode *online = m_player_data->getNode("online-ids");
-    if (online)
-    {
-        for (unsigned int i = 0; i < online->getNumNodes(); i++)
-        {
-            core::stringw s;
-            online->getNode(i)->get("name", &s);
-            if (s.size()>0) m_all_online_ids.push_back(s);
-        }   // for i<online->getNumNodes()
-    }
-    else
-        m_all_online_ids.clear();
-
     m_current_player = NULL;
     std::vector<XMLNode*> player_list;
     m_player_data->getNodes("player", player_list);
@@ -263,14 +250,6 @@ void PlayerManager::save()
         players_file << L"<?xml version=\"1.0\"?>\n";
         players_file << L"<players version=\"1\" >\n";
 
-        players_file << L"    <online-ids>\n";
-        for (unsigned int i = 0; i < m_all_online_ids.size(); i++)
-        {
-            players_file << L"      <id name=\""<<m_all_online_ids[i]
-                         << "\"/>\n";
-        }
-        players_file << L"    </online-ids>\n";
-
         PlayerProfile *player;
         for_in(player, m_all_players)
         {
@@ -446,29 +425,3 @@ void PlayerManager::setCurrentPlayer(PlayerProfile *player)
     }
 }   // setCurrentPlayer
 
-// ----------------------------------------------------------------------------
-/** Adds an online id to the list of all online ids, if that id is not already
- *  in the list. An added id is always moved to the front of the list, so that
- *  the list of online ids shown to the user is sorted by recently-used ids.
- *  \param online_id An online id.
- */
-void PlayerManager::addOnlineId(const core::stringw &online_id)
-{
-    std::vector<core::stringw>::iterator i = std::find(m_all_online_ids.begin(),
-        m_all_online_ids.end(),
-        online_id);
-    if (i == m_all_online_ids.end())
-    {
-        m_all_online_ids.insert(m_all_online_ids.begin(), online_id);
-    }
-    else
-    {
-        // If the online_id is not already at the front of the list,
-        // move it to the front.
-        if (i != m_all_online_ids.begin())
-        {
-            m_all_online_ids.erase(i);
-            m_all_online_ids.insert(m_all_online_ids.begin(), online_id);
-        }
-    }
-}   // addOnlineId

src/config/player_manager.hpp

@@ -58,9 +58,6 @@ private:
 
     PtrVector<PlayerProfile> m_all_players;
 
-    /** A list of all online ids created locally. */
-    std::vector<core::stringw> m_all_online_ids;
-
     /** A pointer to the current player. */
     PlayerProfile* m_current_player;
 
@@ -114,7 +111,6 @@ public:
     static void requestSignOut();
     static Online::XMLRequest *requestSignIn(const irr::core::stringw &username,
                                              const irr::core::stringw &password);
-    void addOnlineId(const core::stringw &online_id);
 
     // ------------------------------------------------------------------------
     /** Returns the current player. */
@@ -157,13 +153,6 @@ public:
         a->increase(key, increase);
     }   // increaseAchievement
     // ------------------------------------------------------------------------
-    /** Returns the list of all online ids. */
-    const std::vector<core::stringw>& getAllOnlineIds() const
-    {
-        return m_all_online_ids;
-    }   // getAllOnlineIds
-    // ------------------------------------------------------------------------
-    // ------------------------------------------------------------------------
 };   // PlayerManager
 
 

src/states_screens/register_screen.cpp

@@ -177,7 +177,6 @@ void RegisterScreen::doRegister()
     }
     else
     {
-        PlayerManager::get()->addOnlineId(username);
         m_info_widget->setDefaultColor();
         new RegistrationDialog();
         return;