From c8895dd43b098d635ec0ae55fbbbfe5d1fe8bef9 Mon Sep 17 00:00:00 2001 From: Matthew I Date: Thu, 30 Aug 2012 15:41:48 -0400 Subject: [PATCH] Prevent world creation if the world already exists --- src/guiMainMenu.cpp | 21 +++++++++++++++++---- 1 file changed, 17 insertions(+), 4 deletions(-) diff --git a/src/guiMainMenu.cpp b/src/guiMainMenu.cpp index 4ceecbb5f..5e0a3c38f 100644 --- a/src/guiMainMenu.cpp +++ b/src/guiMainMenu.cpp @@ -48,10 +48,23 @@ struct CreateWorldDestMainMenu : public CreateWorldDest {} void accepted(std::wstring name, std::string gameid) { - if(!string_allowed_blacklist(wide_to_narrow(name), WORLDNAME_BLACKLISTED_CHARS)) + std::string name_narrow = wide_to_narrow(name); + if(!string_allowed_blacklist(name_narrow, WORLDNAME_BLACKLISTED_CHARS)) + { m_menu->displayMessageMenu(wgettext("Cannot create world: Name contains invalid characters")); - else - m_menu->createNewWorld(name, gameid); + return; + } + std::vector worlds = getAvailableWorlds(); + for(std::vector::iterator i = worlds.begin(); + i != worlds.end(); i++) + { + if((*i).name == name_narrow) + { + m_menu->displayMessageMenu(wgettext("Cannot create world: A world by this name already exists")); + return; + } + } + m_menu->createNewWorld(name, gameid); } GUIMainMenu *m_menu; }; @@ -936,4 +949,4 @@ int GUIMainMenu::getTab() void GUIMainMenu::displayMessageMenu(std::wstring msg) { (new GUIMessageMenu(env, parent, -1, menumgr, msg))->drop(); -} \ No newline at end of file +} -- 2.25.1