diff --git a/Mods/vcmi/Content/config/chinese.json b/Mods/vcmi/Content/config/chinese.json index 943ab8ead..30d03b445 100644 --- a/Mods/vcmi/Content/config/chinese.json +++ b/Mods/vcmi/Content/config/chinese.json @@ -249,7 +249,7 @@ "vcmi.server.errors.existingProcess" : "一个VCMI进程已经在运行，启动新进程前请结束它。", "vcmi.server.errors.modsToEnable" : "{需要启用的mod列表}", "vcmi.server.errors.modsToDisable" : "{需要禁用的mod列表}", - "vcmi.server.errors.unknownEntity" : "加载保存失败！ 在保存的游戏中发现未知实体'%s'！ 保存可能与当前安装的mod版本不兼容！", + "vcmi.server.errors.saveFile.unknownEntity" : "加载保存失败！ 在保存的游戏中发现未知实体'%s'！ 保存可能与当前安装的mod版本不兼容！", "vcmi.server.errors.wrongIdentified" : "你被识别为玩家%s，但预期是玩家%s。", "vcmi.server.errors.notAllowed" : "你无权执行此操作！", diff --git a/Mods/vcmi/Content/config/czech.json b/Mods/vcmi/Content/config/czech.json index f269c3188..80cf61ddc 100644 --- a/Mods/vcmi/Content/config/czech.json +++ b/Mods/vcmi/Content/config/czech.json @@ -249,7 +249,7 @@ "vcmi.server.errors.existingProcess" : "Již běží jiný server VCMI. Prosím, ukončete ho před startem nové hry.", "vcmi.server.errors.modsToEnable" : "{Následující modifikace jsou nutné pro načtení hry}", "vcmi.server.errors.modsToDisable" : "{Následující modifikace musí být zakázány}", - "vcmi.server.errors.unknownEntity" : "Nelze načíst uloženou pozici! Neznámá entita '%s' nalezena v uložené pozici! Uložná pozice nemusí být kompatibilní s aktuálními verzemi modifikací!", + "vcmi.server.errors.saveFile.unknownEntity" : "Nelze načíst uloženou pozici! Neznámá entita '%s' nalezena v uložené pozici! Uložná pozice nemusí být kompatibilní s aktuálními verzemi modifikací!", "vcmi.server.errors.wrongIdentified" : "Byli jste identifikováni jako hráč %s, zatímco byl očekáván hráč %s.", "vcmi.server.errors.notAllowed" : "Nemáte oprávnění provést tuto akci!", diff --git a/Mods/vcmi/Content/config/english.json b/Mods/vcmi/Content/config/english.json index d926c7940..28c1bec88 100644 --- a/Mods/vcmi/Content/config/english.json +++ b/Mods/vcmi/Content/config/english.json @@ -249,7 +249,8 @@ "vcmi.server.errors.existingProcess" : "Another VCMI server process is running. Please terminate it before starting a new game.", "vcmi.server.errors.modsToEnable" : "{Following mods are required}", "vcmi.server.errors.modsToDisable" : "{Following mods must be disabled}", - "vcmi.server.errors.unknownEntity" : "Failed to load save! Unknown entity '%s' found in saved game! Save may not be compatible with currently installed version of mods!", + "vcmi.server.errors.saveFile.unknownEntity" : "Failed to load save! Unknown entity '%s' found in saved game! Save may not be compatible with currently installed version of mods!", + "vcmi.server.errors.campOrMapFile.unknownEntity" : "Failed to load file! Unknown entity '%s' found! File may not be compatible with currently installed version of mods!", "vcmi.server.errors.wrongIdentified" : "You were identified as player %s while expecting %s", "vcmi.server.errors.notAllowed" : "You are not allowed to perform this action!", diff --git a/Mods/vcmi/Content/config/german.json b/Mods/vcmi/Content/config/german.json index 6bd8535d4..a2a956ef5 100644 --- a/Mods/vcmi/Content/config/german.json +++ b/Mods/vcmi/Content/config/german.json @@ -249,7 +249,7 @@ "vcmi.server.errors.existingProcess" : "Es läuft ein weiterer vcmiserver-Prozess, bitte beendet diesen zuerst", "vcmi.server.errors.modsToEnable" : "{Erforderliche Mods um das Spiel zu laden}", "vcmi.server.errors.modsToDisable" : "{Folgende Mods müssen deaktiviert werden}", - "vcmi.server.errors.unknownEntity" : "Spielstand konnte nicht geladen werden! Unbekannte Entität '%s' im gespeicherten Spiel gefunden! Der Spielstand ist möglicherweise nicht mit der aktuell installierten Version der Mods kompatibel!", + "vcmi.server.errors.saveFile.unknownEntity" : "Spielstand konnte nicht geladen werden! Unbekannte Entität '%s' im gespeicherten Spiel gefunden! Der Spielstand ist möglicherweise nicht mit der aktuell installierten Version der Mods kompatibel!", "vcmi.server.errors.wrongIdentified" : "Ihr wurdet als Spieler %s identifiziert, während %s erwartet wurde", "vcmi.server.errors.notAllowed" : "Ihr dürft diese Aktion nicht durchführen!", diff --git a/Mods/vcmi/Content/config/hungarian.json b/Mods/vcmi/Content/config/hungarian.json index 49c254ad4..312a5089b 100644 --- a/Mods/vcmi/Content/config/hungarian.json +++ b/Mods/vcmi/Content/config/hungarian.json @@ -225,7 +225,7 @@ "vcmi.server.errors.existingProcess" : "Egy másik VCMI szerver folyamat fut. Kérjük, zárja be, mielőtt új játékot indítana.", "vcmi.server.errors.modsToEnable" : "{Az alábbi modok szükségesek}", "vcmi.server.errors.modsToDisable" : "{Az alábbi modokat le kell tiltani}", - "vcmi.server.errors.unknownEntity" : "Nem sikerült betölteni a mentést! Ismeretlen entitás '%s' található a mentett játékban! A mentés nem kompatibilis a jelenleg telepített modverziókkal!", + "vcmi.server.errors.saveFile.unknownEntity" : "Nem sikerült betölteni a mentést! Ismeretlen entitás '%s' található a mentett játékban! A mentés nem kompatibilis a jelenleg telepített modverziókkal!", "vcmi.server.errors.wrongIdentified" : "Önt %s játékosként azonosították, miközben %s játékosra számítottak", "vcmi.server.errors.notAllowed" : "Nem engedélyezett művelet!", diff --git a/Mods/vcmi/Content/config/italian.json b/Mods/vcmi/Content/config/italian.json index 9e1a5a621..0b581b723 100644 --- a/Mods/vcmi/Content/config/italian.json +++ b/Mods/vcmi/Content/config/italian.json @@ -227,7 +227,7 @@ "vcmi.server.errors.existingProcess" : "Un altro processo del server VCMI è in esecuzione. Terminarlo prima di avviare una nuova partita.", "vcmi.server.errors.modsToEnable" : "{Le seguenti mod sono richieste}", "vcmi.server.errors.modsToDisable" : "{Le seguenti mod devono essere disattivate}", - "vcmi.server.errors.unknownEntity" : "Impossibile caricare il salvataggio! Entità sconosciuta '%s' trovata nel salvataggio! Il salvataggio potrebbe non essere compatibile con la versione attualmente installata delle mod!", + "vcmi.server.errors.saveFile.unknownEntity" : "Impossibile caricare il salvataggio! Entità sconosciuta '%s' trovata nel salvataggio! Il salvataggio potrebbe non essere compatibile con la versione attualmente installata delle mod!", "vcmi.server.errors.wrongIdentified" : "Sei stato identificato come giocatore %s mentre ci si aspettava %s", "vcmi.server.errors.notAllowed" : "Non ti è permesso eseguire questa azione!", diff --git a/Mods/vcmi/Content/config/polish.json b/Mods/vcmi/Content/config/polish.json index 7b0fe3d68..4997ba2df 100644 --- a/Mods/vcmi/Content/config/polish.json +++ b/Mods/vcmi/Content/config/polish.json @@ -250,7 +250,8 @@ "vcmi.server.errors.modsToDisable" : "{Następujące mody muszą zostać wyłączone}", "vcmi.server.errors.modDependencyLoop" : "Nie udało się wczytać moda {'%s'}!\n Być może znajduje się w pętli zależności", "vcmi.server.errors.notAllowed" : "To działanie nie jest dozwolone!", - "vcmi.server.errors.unknownEntity" : "Nie udało się wczytać zapisu! Nieznany element '%s' znaleziony w pliku zapisu! Zapis może nie być zgodny z aktualnie zainstalowaną wersją modów!", + "vcmi.server.errors.saveFile.unknownEntity" : "Nie udało się wczytać zapisu! Nieznany element '%s' znaleziony w pliku zapisu! Zapis może nie być zgodny z aktualnie zainstalowaną wersją modów!", + "vcmi.server.errors.campOrMapFile.unknownEntity" : "Nie udało się wczytać pliku! Nieznany element '%s' znaleziony w pliku! Plik może nie być zgodny z aktualnie zainstalowaną wersją modów!", "vcmi.server.errors.wrongIdentified" : "Zostałeś zidentyfikowany jako gracz %s natomiast powinieneś być %s", "vcmi.dimensionDoor.seaToLandError" : "Nie jest możliwa teleportacja przez drzwi wymiarów z wód na ląd i na odwrót.", diff --git a/Mods/vcmi/Content/config/portuguese.json b/Mods/vcmi/Content/config/portuguese.json index 949819736..37772b37e 100644 --- a/Mods/vcmi/Content/config/portuguese.json +++ b/Mods/vcmi/Content/config/portuguese.json @@ -244,7 +244,7 @@ "vcmi.server.errors.existingProcess" : "Outro processo do servidor VCMI está em execução. Por favor, termine-o antes de iniciar um novo jogo.", "vcmi.server.errors.modsToEnable" : "{Os seguintes mods são necessários}", "vcmi.server.errors.modsToDisable" : "{Os seguintes mods devem ser desativados}", - "vcmi.server.errors.unknownEntity" : "Falha ao carregar o jogo salvo! Entidade desconhecida '%s' encontrada no jogo salvo! O jogo salvo pode não ser compatível com a versão atualmente instalada dos mods!", + "vcmi.server.errors.saveFile.unknownEntity" : "Falha ao carregar o jogo salvo! Entidade desconhecida '%s' encontrada no jogo salvo! O jogo salvo pode não ser compatível com a versão atualmente instalada dos mods!", "vcmi.server.errors.wrongIdentified" : "Você foi identificado como o jogador %s, mas esperava-se %s", "vcmi.server.errors.notAllowed" : "Você não tem permissão para realizar esta ação!", diff --git a/Mods/vcmi/Content/config/russian.json b/Mods/vcmi/Content/config/russian.json index 1d74878f5..07dcfc1d4 100644 --- a/Mods/vcmi/Content/config/russian.json +++ b/Mods/vcmi/Content/config/russian.json @@ -225,7 +225,7 @@ "vcmi.server.errors.existingProcess" : "Запущен другой процесс сервера VCMI. Пожалуйста, завершите его перед запуском новой игры.", "vcmi.server.errors.modsToEnable" : "{Требуемые моды для загрузки игры}", "vcmi.server.errors.modsToDisable" : "{Необходимо отключить следующие моды}", - "vcmi.server.errors.unknownEntity" : "Не удалось загрузить сохранение! В сохраненной игре обнаружен неизвестный объект '%s'! Возможно, сохранение несовместимо с текущей версией модов!", + "vcmi.server.errors.saveFile.unknownEntity" : "Не удалось загрузить сохранение! В сохраненной игре обнаружен неизвестный объект '%s'! Возможно, сохранение несовместимо с текущей версией модов!", "vcmi.server.errors.wrongIdentified" : "Вы были идентифицированы как игрок %s, ожидалось %s", "vcmi.server.errors.notAllowed" : "Вам запрещено выполнять данное действие!", "vcmi.dimensionDoor.seaToLandError" : "Невозможно телепортироваться с моря на сушу или наоборот с помощью Двери Измерений.", diff --git a/Mods/vcmi/Content/config/spanish.json b/Mods/vcmi/Content/config/spanish.json index 32c6bdfad..cfd4b9730 100644 --- a/Mods/vcmi/Content/config/spanish.json +++ b/Mods/vcmi/Content/config/spanish.json @@ -76,7 +76,7 @@ "vcmi.server.errors.modsToEnable" : "{Se requieren los siguientes mods}", "vcmi.server.errors.modsToDisable" : "{Deben desactivarse los siguientes mods}", "vcmi.server.confirmReconnect" : "¿Quieres reconectar a la última sesión?", - "vcmi.server.errors.unknownEntity" : "Error al cargar la partida guardada. ¡Se encontró una entidad desconocida '%s' en la partida guardada! Es posible que la partida no sea compatible con la versión actualmente instalada de los mods.", + "vcmi.server.errors.saveFile.unknownEntity" : "Error al cargar la partida guardada. ¡Se encontró una entidad desconocida '%s' en la partida guardada! Es posible que la partida no sea compatible con la versión actualmente instalada de los mods.", "vcmi.settingsMainWindow.generalTab.hover" : "General", "vcmi.settingsMainWindow.generalTab.help" : "Cambiar a la pestaña de opciones generales, que contiene ajustes relacionados con el comportamiento general del juego", diff --git a/Mods/vcmi/Content/config/swedish.json b/Mods/vcmi/Content/config/swedish.json index bbecd006d..1cabde601 100644 --- a/Mods/vcmi/Content/config/swedish.json +++ b/Mods/vcmi/Content/config/swedish.json @@ -249,7 +249,7 @@ "vcmi.server.errors.existingProcess" : "En annan VCMI-serverprocess är igång. Vänligen avsluta den innan du startar ett nytt spel.", "vcmi.server.errors.modsToEnable" : "{Följande modd(ar) krävs}", "vcmi.server.errors.modsToDisable" : "{Följande modd(ar) måste inaktiveras}", - "vcmi.server.errors.unknownEntity" : "Misslyckades med att ladda sparat spel! Okänd enhet '%s' hittades i sparat spel! Sparningen kanske inte är kompatibel med den aktuella versionen av moddarna!", + "vcmi.server.errors.saveFile.unknownEntity" : "Misslyckades med att ladda sparat spel! Okänd enhet '%s' hittades i sparat spel! Sparningen kanske inte är kompatibel med den aktuella versionen av moddarna!", "vcmi.server.errors.wrongIdentified" : "Du identifierades som spelare %s när du förväntade dig %s", "vcmi.server.errors.notAllowed" : "Du får inte utföra denna åtgärd!", diff --git a/Mods/vcmi/Content/config/ukrainian.json b/Mods/vcmi/Content/config/ukrainian.json index abe421a74..829f9c04c 100644 --- a/Mods/vcmi/Content/config/ukrainian.json +++ b/Mods/vcmi/Content/config/ukrainian.json @@ -241,7 +241,7 @@ "vcmi.server.errors.existingProcess" : "Працює інший процес vcmiserver, будь ласка, спочатку завершіть його", "vcmi.server.errors.modsToEnable" : "{Потрібні модифікації для завантаження гри}", "vcmi.server.errors.modsToDisable" : "{Модифікації що мають бути вимкнені}", - "vcmi.server.errors.unknownEntity" : "Не вдалося завантажити гру! У збереженій грі знайдено невідомий об'єкт '%s'! Це збереження може бути несумісним зі встановленою версією модифікацій!", + "vcmi.server.errors.saveFile.unknownEntity" : "Не вдалося завантажити гру! У збереженій грі знайдено невідомий об'єкт '%s'! Це збереження може бути несумісним зі встановленою версією модифікацій!", "vcmi.server.errors.wrongIdentified" : "Ви були ідентифіковані як гравець %s, хоча очікували %s", "vcmi.server.errors.notAllowed" : "Ви не можете виконати цю дію!", diff --git a/Mods/vcmi/Content/config/vietnamese.json b/Mods/vcmi/Content/config/vietnamese.json index 7757815e6..915dfaebf 100644 --- a/Mods/vcmi/Content/config/vietnamese.json +++ b/Mods/vcmi/Content/config/vietnamese.json @@ -225,7 +225,7 @@ "vcmi.server.errors.existingProcess" : "Một chương trình máy chủ VCMI khác đang chạy. Hãy đóng nó trước khi bắt đầu một trò chơi mới.", "vcmi.server.errors.modsToEnable" : "{Các mod sau đây là bắt buộc}", "vcmi.server.errors.modsToDisable" : "{Bạn phải tắt các mod sau đây}", - "vcmi.server.errors.unknownEntity" : "Không tải được tệp tin đã lưu! Có lỗi chưa xác định trong tệp tin đã lưu '%s'! Tệp tin có thể không tương thích với phiên bản mod hiện đang cài đặt!", + "vcmi.server.errors.saveFile.unknownEntity" : "Không tải được tệp tin đã lưu! Có lỗi chưa xác định trong tệp tin đã lưu '%s'! Tệp tin có thể không tương thích với phiên bản mod hiện đang cài đặt!", "vcmi.server.errors.wrongIdentified" : "Bạn được chỉ định là người chơi %s trong khi bạn muốn %s", "vcmi.server.errors.notAllowed" : "Bạn không được phép thực hiện hành động này!", diff --git a/client/CServerHandler.cpp b/client/CServerHandler.cpp index 37cca84fd..cb80f843d 100644 --- a/client/CServerHandler.cpp +++ b/client/CServerHandler.cpp @@ -569,24 +569,14 @@ bool CServerHandler::validateGameStart(bool allowOnlyAI) const catch(ModIncompatibility & e) { logGlobal->warn("Incompatibility exception during start scenario: %s", e.what()); - std::string errorMsg; - if(!e.whatMissing().empty()) - { - errorMsg += LIBRARY->generaltexth->translate("vcmi.server.errors.modsToEnable") + '\n'; - errorMsg += e.whatMissing(); - } - if(!e.whatExcessive().empty()) - { - errorMsg += LIBRARY->generaltexth->translate("vcmi.server.errors.modsToDisable") + '\n'; - errorMsg += e.whatExcessive(); - } - showServerError(errorMsg); + + showServerError(e.getFullErrorMsg()); return false; } catch(std::exception & e) { logGlobal->error("Exception during startScenario: %s", e.what()); - showServerError( std::string("Unable to start map! Reason: ") + e.what()); + showServerError(std::string("Unable to start map!\nReason: ") + e.what()); return false; } diff --git a/client/NetPacksLobbyClient.cpp b/client/NetPacksLobbyClient.cpp index a419d60ca..f81006125 100644 --- a/client/NetPacksLobbyClient.cpp +++ b/client/NetPacksLobbyClient.cpp @@ -146,7 +146,6 @@ void ApplyOnLobbyHandlerNetPackVisitor::visitLobbyRestartGame(LobbyRestartGame & void ApplyOnLobbyHandlerNetPackVisitor::visitLobbyPrepareStartGame(LobbyPrepareStartGame & pack) { - handler.client = std::make_unique(); handler.logicConnection->enterLobbyConnectionMode(); } @@ -159,6 +158,7 @@ void ApplyOnLobbyHandlerNetPackVisitor::visitLobbyStartGame(LobbyStartGame & pac handler.si = pack.initializedStartInfo; handler.si->mode = modeBackup; } + handler.client = std::make_unique(); handler.startGameplay(pack.initializedGameState); } diff --git a/lib/mapObjectConstructors/CObjectClassesHandler.cpp b/lib/mapObjectConstructors/CObjectClassesHandler.cpp index 839ed55bf..503ddd2be 100644 --- a/lib/mapObjectConstructors/CObjectClassesHandler.cpp +++ b/lib/mapObjectConstructors/CObjectClassesHandler.cpp @@ -408,9 +408,9 @@ TObjectTypeHandler CObjectClassesHandler::getHandlerFor(const std::string & scop return object->objectTypeHandlers.at(subID.value()); } - std::string errorString = "Failed to find object of type " + type + "::" + subtype; - logGlobal->error(errorString); - throw std::runtime_error(errorString); + std::string objectType = type + "::" + subtype; + logGlobal->error("Failed to find object of type %s", objectType); + throw IdentifierResolutionException(objectType); } TObjectTypeHandler CObjectClassesHandler::getHandlerFor(CompoundMapObjectID compoundIdentifier) const diff --git a/lib/modding/ModIncompatibility.h b/lib/modding/ModIncompatibility.h index e21ad8bff..86bab7c90 100644 --- a/lib/modding/ModIncompatibility.h +++ b/lib/modding/ModIncompatibility.h @@ -9,6 +9,9 @@ */ #pragma once +#include "../lib/texts/CGeneralTextHandler.h" +#include "GameLibrary.h" + VCMI_LIB_NAMESPACE_BEGIN class DLL_LINKAGE ModIncompatibility: public std::exception @@ -49,6 +52,22 @@ public: return messageExcessiveMods; } + std::string getFullErrorMsg() const noexcept + { + std::string errorMsg; + if(!messageMissingMods.empty()) + { + errorMsg += LIBRARY->generaltexth->translate("vcmi.server.errors.modsToEnable") + '\n'; + errorMsg += messageMissingMods; + } + if(!messageExcessiveMods.empty()) + { + errorMsg += LIBRARY->generaltexth->translate("vcmi.server.errors.modsToDisable") + '\n'; + errorMsg += messageExcessiveMods; + } + return errorMsg; + } + private: std::string messageMissingMods; std::string messageExcessiveMods; diff --git a/mapeditor/mainwindow.cpp b/mapeditor/mainwindow.cpp index cdbda068a..a76a1181e 100644 --- a/mapeditor/mainwindow.cpp +++ b/mapeditor/mainwindow.cpp @@ -467,9 +467,19 @@ bool MainWindow::openMap(const QString & filenameSelect) catch(const ModIncompatibility & e) { assert(e.whatExcessive().empty()); - QMessageBox::warning(this, tr("Mods are required"), QString::fromStdString(e.whatMissing())); + auto qstrError = QString::fromStdString(e.getFullErrorMsg()).remove('{').remove('}'); + QMessageBox::warning(this, tr("Mods are required"), qstrError); return false; } + catch(const IdentifierResolutionException & e) + { + MetaString errorMsg; + errorMsg.appendTextID("vcmi.server.errors.campOrMapFile.unknownEntity"); + errorMsg.replaceRawString(e.identifierName); + QMessageBox::critical(this, tr("Failed to open map"), QString::fromStdString(errorMsg.toString())); + return false; + } + catch(const std::exception & e) { QMessageBox::critical(this, tr("Failed to open map"), tr(e.what())); diff --git a/server/CVCMIServer.cpp b/server/CVCMIServer.cpp index 4dc44cb64..72e4f1ccd 100644 --- a/server/CVCMIServer.cpp +++ b/server/CVCMIServer.cpp @@ -230,9 +230,6 @@ bool CVCMIServer::prepareToStartGame() Load::Progress current(1); progressTracking.include(current); - if (lobbyProcessor) - lobbyProcessor->sendGameStarted(); - auto progressTrackingThread = std::thread([this, &progressTracking]() { setThreadName("progressTrackingThread"); @@ -252,43 +249,72 @@ bool CVCMIServer::prepareToStartGame() } }); - gh = std::make_shared(*this); - switch(si->mode) + auto newGH = std::make_shared(*this); + bool started = false; + try { - case EStartMode::CAMPAIGN: - logNetwork->info("Preparing to start new campaign"); - si->startTime = std::time(nullptr); - si->fileURI = mi->fileURI; - si->campState->setCurrentMap(campaignMap); - si->campState->setCurrentMapBonus(campaignBonus); - gh->init(si.get(), progressTracking); - break; - - case EStartMode::NEW_GAME: - logNetwork->info("Preparing to start new game"); - si->startTime = std::time(nullptr); - si->fileURI = mi->fileURI; - gh->init(si.get(), progressTracking); - break; - - case EStartMode::LOAD_GAME: - logNetwork->info("Preparing to start loaded game"); - if(!loadSavedGame(*si)) + switch(si->mode) { - current.finish(); - progressTrackingThread.join(); - return false; - } - break; - default: - logNetwork->error("Wrong mode in StartInfo!"); - assert(0); - break; - } + case EStartMode::CAMPAIGN: + logNetwork->info("Preparing to start new campaign"); + si->startTime = std::time(nullptr); + si->fileURI = mi->fileURI; + si->campState->setCurrentMap(campaignMap); + si->campState->setCurrentMapBonus(campaignBonus); + newGH->init(si.get(), progressTracking); // may throw + started = true; + break; + case EStartMode::NEW_GAME: + logNetwork->info("Preparing to start new game"); + si->startTime = std::time(nullptr); + si->fileURI = mi->fileURI; + newGH->init(si.get(), progressTracking); // may throw + started = true; + break; + case EStartMode::LOAD_GAME: + logNetwork->info("Preparing to start loaded game"); + if(loadSavedGame(*newGH, *si)) + started = true; + break; + default: + logNetwork->error("Wrong mode in StartInfo!"); + assert(0); + break; + } + } + catch(const ModIncompatibility & e) + { + logGlobal->error("Failed to launch game: %s", e.what()); + announceMessage(e.getFullErrorMsg()); + } + catch(const IdentifierResolutionException & e) + { + logGlobal->error("Failed to launch game: %s", e.what()); + MetaString errorMsg; + errorMsg.appendTextID("vcmi.server.errors.campOrMapFile.unknownEntity"); + errorMsg.replaceRawString(e.identifierName); + announceMessage(errorMsg); + } + catch(const std::exception & e) + { + logGlobal->error("Failed to launch game: %s", e.what()); + auto str = MetaString::createFromTextID("vcmi.broadcast.failedLoadGame"); + str.appendRawString(":\n"); + str.appendRawString(e.what()); + announceMessage(str); + } current.finish(); progressTrackingThread.join(); + if (!started) + return false; + + gh = std::move(newGH); + + if(lobbyProcessor) + lobbyProcessor->sendGameStarted(); + return true; } @@ -1066,36 +1092,23 @@ INetworkServer & CVCMIServer::getNetworkServer() return *networkServer; } -bool CVCMIServer::loadSavedGame(const StartInfo &info) +bool CVCMIServer::loadSavedGame(CGameHandler & handler, const StartInfo & info) { try { - gh->load(info); + handler.load(info); } catch(const ModIncompatibility & e) { logGlobal->error("Failed to load game: %s", e.what()); - MetaString errorMsg; - if(!e.whatMissing().empty()) - { - errorMsg.appendTextID("vcmi.server.errors.modsToEnable"); - errorMsg.appendRawString("\n"); - errorMsg.appendRawString(e.whatMissing()); - } - if(!e.whatExcessive().empty()) - { - errorMsg.appendTextID("vcmi.server.errors.modsToDisable"); - errorMsg.appendRawString("\n"); - errorMsg.appendRawString(e.whatExcessive()); - } - announceMessage(errorMsg); + announceMessage(e.getFullErrorMsg()); return false; } catch(const IdentifierResolutionException & e) { logGlobal->error("Failed to load game: %s", e.what()); MetaString errorMsg; - errorMsg.appendTextID("vcmi.server.errors.unknownEntity"); + errorMsg.appendTextID("vcmi.server.errors.saveFile.unknownEntity"); errorMsg.replaceRawString(e.identifierName); announceMessage(errorMsg); return false; diff --git a/server/CVCMIServer.h b/server/CVCMIServer.h index 41b392e90..dbabf63a1 100644 --- a/server/CVCMIServer.h +++ b/server/CVCMIServer.h @@ -55,7 +55,7 @@ class CVCMIServer : public LobbyInfo, public INetworkServerListener, public INet bool runByClient; - bool loadSavedGame(const StartInfo &info); + bool loadSavedGame(CGameHandler & handler, const StartInfo & info); public: // IGameServer impl