Proper mod compatibility check system

client/CServerHandler.cpp

@@ -704,6 +704,9 @@ void CServerHandler::startCampaignScenario(std::shared_ptr<CampaignState> cs)
 
 void CServerHandler::showServerError(const std::string & txt) const
 {
+	if(auto w = GH.windows().topWindow<CLoadingScreen>())
+		GH.windows().popWindow(w);
+	
 	CInfoWindow::showInfoDialog(txt, {});
 }
 

lib/StartInfo.cpp

@@ -76,7 +76,7 @@ void LobbyInfo::verifyStateBeforeStart(bool ignoreNoHuman) const
 	auto missingMods = CMapService::verifyMapHeaderMods(*mi->mapHeader);
 	ModIncompatibility::ModList modList;
 	for(const auto & m : missingMods)
-		modList.push_back({m.first, m.second.toString()});
+		modList.push_back({m.second.name, m.second.version.toString()});
 	
 	if(!modList.empty())
 		throw ModIncompatibility(std::move(modList));

lib/mapping/CMapHeader.h

@@ -10,7 +10,7 @@
 
 #pragma once
 
-#include "../modding/CModVersion.h"
+#include "../modding/CModInfo.h"
 #include "../LogicalExpression.h"
 #include "../int3.h"
 #include "../MetaString.h"
@@ -19,7 +19,7 @@ VCMI_LIB_NAMESPACE_BEGIN
 
 class CGObjectInstance;
 enum class EMapFormat : uint8_t;
-using ModCompatibilityInfo = std::map<std::string, CModVersion>;
+using ModCompatibilityInfo = std::map<std::string, CModInfo::VerificationInfo>;
 
 /// The hero name struct consists of the hero id and the hero name.
 struct DLL_LINKAGE SHeroName
@@ -249,7 +249,6 @@ public:
 	void serialize(Handler & h, const int Version)
 	{
 		h & version;
-		if(Version >= 821)
 		h & mods;
 		h & name;
 		h & description;

lib/mapping/CMapService.cpp

@@ -92,14 +92,15 @@ void CMapService::saveMap(const std::unique_ptr<CMap> & map, boost::filesystem::
 
 ModCompatibilityInfo CMapService::verifyMapHeaderMods(const CMapHeader & map)
 {
-	ModCompatibilityInfo modCompatibilityInfo;
 	const auto & activeMods = VLC->modh->getActiveMods();
+	
+	ModCompatibilityInfo modCompatibilityInfo;
 	for(const auto & mapMod : map.mods)
 	{
 		if(vstd::contains(activeMods, mapMod.first))
 		{
 			const auto & modInfo = VLC->modh->getModInfo(mapMod.first);
-			if(modInfo.version.compatible(mapMod.second))
+			if(modInfo.getVerificationInfo().version.compatible(mapMod.second.version))
 				continue;
 		}
 		

lib/mapping/CMapService.h

@@ -10,6 +10,8 @@
 
 #pragma once
 
+#include "../modding/CModInfo.h"
+
 VCMI_LIB_NAMESPACE_BEGIN
 
 class ResourcePath;
@@ -17,12 +19,11 @@ class ResourcePath;
 class CMap;
 class CMapHeader;
 class CInputStream;
-struct CModVersion;
 
 class IMapLoader;
 class IMapPatcher;
 
-using ModCompatibilityInfo = std::map<std::string, CModVersion>;
+using ModCompatibilityInfo = std::map<std::string, CModInfo::VerificationInfo>;
 
 /**
  * The map service provides loading of VCMI/H3 map files. It can

lib/mapping/MapFormatJson.cpp

@@ -342,7 +342,7 @@ namespace TerrainDetail
 
 ///CMapFormatJson
 const int CMapFormatJson::VERSION_MAJOR = 1;
-const int CMapFormatJson::VERSION_MINOR = 2;
+const int CMapFormatJson::VERSION_MINOR = 3;
 
 const std::string CMapFormatJson::HEADER_FILE_NAME = "header.json";
 const std::string CMapFormatJson::OBJECTS_FILE_NAME = "objects.json";
@@ -953,7 +953,18 @@ void CMapLoaderJson::readHeader(const bool complete)
 	if(!header["mods"].isNull())
 	{
 		for(auto & mod : header["mods"].Vector())
-			mapHeader->mods[mod["name"].String()] = CModVersion::fromString(mod["version"].String());
+		{
+			CModInfo::VerificationInfo info;
+			info.version = CModVersion::fromString(mod["version"].String());
+			info.checksum = mod["checksum"].Integer();
+			info.name = mod["name"].String();
+			info.impactsGameplay = true;
+			
+			if(!mod["modId"].isNull())
+				mapHeader->mods[mod["modId"].String()] = info;
+			else
+				mapHeader->mods[mod["name"].String()] = info;
+		}
 	}
 
 	//todo: multilevel map load support
@@ -1294,8 +1305,10 @@ void CMapSaverJson::writeHeader()
 	for(const auto & mod : mapHeader->mods)
 	{
 		JsonNode modWriter;
-		modWriter["name"].String() = mod.first;
+		modWriter["modId"].String() = mod.first;
-		modWriter["version"].String() = mod.second.toString();
+		modWriter["name"].String() = mod.second.name;
+		modWriter["version"].String() = mod.second.version.toString();
+		modWriter["checksum"].Integer() = mod.second.checksum;
 		mods.Vector().push_back(modWriter);
 	}
 

lib/modding/CModHandler.cpp

@@ -56,7 +56,7 @@ bool CModHandler::hasCircularDependency(const TModID & modID, std::set<TModID> c
 	if (vstd::contains(currentList, modID))
 	{
 		logMod->error("Error: Circular dependency detected! Printing dependency list:");
-		logMod->error("\t%s -> ", mod.name);
+		logMod->error("\t%s -> ", mod.getVerificationInfo().name);
 		return true;
 	}
 
@@ -67,7 +67,7 @@ bool CModHandler::hasCircularDependency(const TModID & modID, std::set<TModID> c
 	{
 		if (hasCircularDependency(dependency, currentList))
 		{
-			logMod->error("\t%s ->\n", mod.name); // conflict detected, print dependency list
+			logMod->error("\t%s ->\n", mod.getVerificationInfo().name); // conflict detected, print dependency list
 			return true;
 		}
 	}
@@ -129,7 +129,7 @@ std::vector <TModID> CModHandler::validateAndSortDependencies(std::vector <TModI
 		for(const TModID & dependency : brokenMod.dependencies)
 		{
 			if(!vstd::contains(resolvedModIDs, dependency))
-				logMod->error("Mod '%s' will not work: it depends on mod '%s', which is not installed.", brokenMod.name, dependency);
+				logMod->error("Mod '%s' will not work: it depends on mod '%s', which is not installed.", brokenMod.getVerificationInfo().name, dependency);
 		}
 	}
 	return sortedValidMods;
@@ -212,7 +212,6 @@ void CModHandler::loadMods(bool onlyEssential)
 	}
 
 	coreMod = std::make_unique<CModInfo>(ModScope::scopeBuiltin(), modConfig[ModScope::scopeBuiltin()], JsonNode(JsonPath::builtin("config/gameConfig.json")));
-	coreMod->name = "Original game files";
 }
 
 std::vector<std::string> CModHandler::getAllMods()
@@ -352,7 +351,7 @@ void CModHandler::initializeConfig()
 CModVersion CModHandler::getModVersion(TModID modName) const
 {
 	if (allMods.count(modName))
-		return allMods.at(modName).version;
+		return allMods.at(modName).getVerificationInfo().version;
 	return {};
 }
 
@@ -462,6 +461,7 @@ void CModHandler::afterLoad(bool onlyEssential)
 		modSettings["activeMods"].resolvePointer(pointer) = modEntry.second.saveLocalData();
 	}
 	modSettings[ModScope::scopeBuiltin()] = coreMod->saveLocalData();
+	modSettings[ModScope::scopeBuiltin()]["name"].String() = "Original game files";
 
 	if(!onlyEssential)
 	{
@@ -471,48 +471,64 @@ void CModHandler::afterLoad(bool onlyEssential)
 
 }
 
-void CModHandler::trySetActiveMods(std::vector<TModID> saveActiveMods, const std::map<TModID, CModVersion> & modList)
+void CModHandler::trySetActiveMods(const std::vector<std::pair<TModID, CModInfo::VerificationInfo>> & modList)
 {
-	std::vector<TModID> newActiveMods;
+	auto hasMod = [&modList](const TModID & m) -> bool
-
+	{
-	ModIncompatibility::ModList missingMods;
+		for(auto & i : modList)
+			if(i.first == m)
+				return true;
+		return false;
+	};
 	
 	for(const auto & m : activeMods)
 	{
-		if (vstd::contains(saveActiveMods, m))
+		if(hasMod(m))
 			continue;
 
-		auto & modInfo = allMods.at(m);
+		if(getModInfo(m).checkModGameplayAffecting())
-		if(modInfo.checkModGameplayAffecting())
+			allMods[m].setEnabled(false);
-			missingMods.emplace_back(m, modInfo.version.toString());
 	}
 
-	for(const auto & m : saveActiveMods)
+	std::vector<TModID> newActiveMods;
-	{
+	ModIncompatibility::ModList missingMods;
-		const CModVersion & mver = modList.at(m);
 	
-		if (allMods.count(m) == 0)
+	for(const auto & infoPair : modList)
 	{
-			missingMods.emplace_back(m, mver.toString());
+		auto & remoteModId = infoPair.first;
+		auto & remoteModInfo = infoPair.second;
+		
+		if(!allMods.count(remoteModId))
+		{
+			if(remoteModInfo.impactsGameplay)
+				missingMods.push_back({remoteModInfo.name, remoteModInfo.version.toString()}); //mod is not installed
 			continue;
 		}
 		
-		auto & modInfo = allMods.at(m);
+		auto & localModInfo = getModInfo(remoteModId).getVerificationInfo();
 
-		bool modAffectsGameplay = modInfo.checkModGameplayAffecting();
+		bool modAffectsGameplay = getModInfo(remoteModId).checkModGameplayAffecting();
-		bool modVersionCompatible = modInfo.version.isNull() || mver.isNull() || modInfo.version.compatible(mver);
+		bool modVersionCompatible = localModInfo.version.isNull()
-		bool modEnabledLocally = vstd::contains(activeMods, m);
+			|| remoteModInfo.version.isNull()
-		bool modCanBeEnabled = modEnabledLocally && modVersionCompatible;
+			|| localModInfo.version.compatible(remoteModInfo.version);
+		bool modLocalyEnabled = vstd::contains(activeMods, remoteModId);
 		
-		allMods[m].setEnabled(modCanBeEnabled);
+		if(modVersionCompatible && modAffectsGameplay && modLocalyEnabled)
-
+		{
-		if (modCanBeEnabled)
+			newActiveMods.push_back(remoteModId);
-			newActiveMods.push_back(m);
+			continue;
-
-		if (!modCanBeEnabled && modAffectsGameplay)
-			missingMods.emplace_back(m, mver.toString());
 		}
 		
+		if(modAffectsGameplay || remoteModInfo.impactsGameplay)
+			missingMods.push_back({remoteModInfo.name, remoteModInfo.version.toString()}); //incompatible mod impacts gameplay
+	}
+	
+	if(!missingMods.empty())
+		throw ModIncompatibility(std::move(missingMods));
+	
+	for(auto & m : newActiveMods)
+		allMods[m].setEnabled(true);
+
 	std::swap(activeMods, newActiveMods);
 }
 

lib/modding/CModHandler.h

@@ -9,13 +9,12 @@
  */
 #pragma once
 
-#include "CModVersion.h"
+#include "CModInfo.h"
 
 VCMI_LIB_NAMESPACE_BEGIN
 
 class CModHandler;
 class CModIndentifier;
-class CModInfo;
 class JsonNode;
 class IHandlerBase;
 class CIdentifierStorage;
@@ -52,7 +51,7 @@ class DLL_LINKAGE CModHandler : boost::noncopyable
 	CModVersion getModVersion(TModID modName) const;
 
 	/// Attempt to set active mods according to provided list of mods from save, throws on failure
-	void trySetActiveMods(std::vector<TModID> saveActiveMods, const std::map<TModID, CModVersion> & modList);
+	void trySetActiveMods(const std::vector<std::pair<TModID, CModInfo::VerificationInfo>> & modList);
 
 public:
 	std::shared_ptr<CContentHandler> content; //(!)Do not serialize FIXME: make private
@@ -88,22 +87,22 @@ public:
 		{
 			h & activeMods;
 			for(const auto & m : activeMods)
-			{
+				h & getModInfo(m).getVerificationInfo();
-				CModVersion version = getModVersion(m);
-				h & version;
-			}
 		}
 		else
 		{
 			loadMods();
 			std::vector<TModID> saveActiveMods;
-			std::map<TModID, CModVersion> modVersions;
 			h & saveActiveMods;
 			
-			for(const auto & m : saveActiveMods)
+			std::vector<std::pair<TModID, CModInfo::VerificationInfo>> saveModInfos(saveActiveMods.size());
-				h & modVersions[m];
+			for(int i = 0; i < saveActiveMods.size(); ++i)
+			{
+				saveModInfos[i].first = saveActiveMods[i];
+				h & saveModInfos[i].second;
+			}
 
-			trySetActiveMods(saveActiveMods, modVersions);
+			trySetActiveMods(saveModInfos);
 		}
 	}
 };

lib/modding/CModInfo.cpp

@@ -23,7 +23,6 @@ static JsonNode addMeta(JsonNode config, const std::string & meta)
 }
 
 CModInfo::CModInfo():
-	checksum(0),
 	explicitlyEnabled(false),
 	implicitlyEnabled(true),
 	validation(PENDING)
@@ -33,17 +32,17 @@ CModInfo::CModInfo():
 
 CModInfo::CModInfo(const std::string & identifier, const JsonNode & local, const JsonNode & config):
 	identifier(identifier),
-	name(config["name"].String()),
 	description(config["description"].String()),
 	dependencies(config["depends"].convertTo<std::set<std::string>>()),
 	conflicts(config["conflicts"].convertTo<std::set<std::string>>()),
-	checksum(0),
 	explicitlyEnabled(false),
 	implicitlyEnabled(true),
 	validation(PENDING),
 	config(addMeta(config, identifier))
 {
-	version = CModVersion::fromString(config["version"].String());
+	verificationInfo.name = config["name"].String();
+	verificationInfo.version = CModVersion::fromString(config["version"].String());
+	
 	if(!config["compatibility"].isNull())
 	{
 		vcmiCompatibleMin = CModVersion::fromString(config["compatibility"]["min"].String());
@@ -61,7 +60,7 @@ CModInfo::CModInfo(const std::string & identifier, const JsonNode & local, const
 JsonNode CModInfo::saveLocalData() const
 {
 	std::ostringstream stream;
-	stream << std::noshowbase << std::hex << std::setw(8) << std::setfill('0') << checksum;
+	stream << std::noshowbase << std::hex << std::setw(8) << std::setfill('0') << verificationInfo.checksum;
 
 	JsonNode conf;
 	conf["active"].Bool() = explicitlyEnabled;
@@ -83,9 +82,9 @@ JsonPath CModInfo::getModFile(const std::string & name)
 void CModInfo::updateChecksum(ui32 newChecksum)
 {
 	// comment-out next line to force validation of all mods ignoring checksum
-	if (newChecksum != checksum)
+	if (newChecksum != verificationInfo.checksum)
 	{
-		checksum = newChecksum;
+		verificationInfo.checksum = newChecksum;
 		validation = PENDING;
 	}
 }
@@ -95,7 +94,7 @@ void CModInfo::loadLocalData(const JsonNode & data)
 	bool validated = false;
 	implicitlyEnabled = true;
 	explicitlyEnabled = !config["keepDisabled"].Bool();
-	checksum = 0;
+	verificationInfo.checksum = 0;
 	if (data.getType() == JsonNode::JsonType::DATA_BOOL)
 	{
 		explicitlyEnabled = data.Bool();
@@ -104,7 +103,7 @@ void CModInfo::loadLocalData(const JsonNode & data)
 	{
 		explicitlyEnabled = data["active"].Bool();
 		validated = data["validated"].Bool();
-		checksum  = strtol(data["checksum"].String().c_str(), nullptr, 16);
+		updateChecksum(strtol(data["checksum"].String().c_str(), nullptr, 16));
 	}
 
 	//check compatibility
@@ -112,13 +111,13 @@ void CModInfo::loadLocalData(const JsonNode & data)
 	implicitlyEnabled &= (vcmiCompatibleMax.isNull() || vcmiCompatibleMax.compatible(CModVersion::GameVersion(), true, true));
 
 	if(!implicitlyEnabled)
-		logGlobal->warn("Mod %s is incompatible with current version of VCMI and cannot be enabled", name);
+		logGlobal->warn("Mod %s is incompatible with current version of VCMI and cannot be enabled", verificationInfo.name);
 
 	if (boost::iequals(config["modType"].String(), "translation")) // compatibility code - mods use "Translation" type at the moment
 	{
 		if (baseLanguage != VLC->generaltexth->getPreferredLanguage())
 		{
-			logGlobal->warn("Translation mod %s was not loaded: language mismatch!", name);
+			logGlobal->warn("Translation mod %s was not loaded: language mismatch!", verificationInfo.name);
 			implicitlyEnabled = false;
 		}
 	}
@@ -127,6 +126,8 @@ void CModInfo::loadLocalData(const JsonNode & data)
 		validation = validated ? PASSED : PENDING;
 	else
 		validation = validated ? PASSED : FAILED;
+	
+	verificationInfo.impactsGameplay = checkModGameplayAffecting();
 }
 
 bool CModInfo::checkModGameplayAffecting() const
@@ -171,6 +172,11 @@ bool CModInfo::checkModGameplayAffecting() const
 	return *modGameplayAffecting;
 }
 
+const CModInfo::VerificationInfo & CModInfo::getVerificationInfo() const
+{
+	return verificationInfo;
+}
+
 bool CModInfo::isEnabled() const
 {
 	return implicitlyEnabled && explicitlyEnabled;

lib/modding/CModInfo.h

@@ -30,16 +30,36 @@ public:
 		PASSED
 	};
 	
-	/// identifier, identical to name of folder with mod
+	struct VerificationInfo
-	std::string identifier;
+	{
-
+		/// human-readable mod name
-	/// human-readable strings
 		std::string name;
-	std::string description;
 		
 		/// version of the mod
 		CModVersion version;
 		
+		/// CRC-32 checksum of the mod
+		ui32 checksum = 0;
+		
+		/// for serialization purposes
+		bool impactsGameplay = true;
+		
+		template <typename Handler>
+		void serialize(Handler & h, const int v)
+		{
+			h & name;
+			h & version;
+			h & checksum;
+			h & impactsGameplay;
+		}
+	};
+
+	/// identifier, identical to name of folder with mod
+	std::string identifier;
+
+	/// detailed mod description
+	std::string description;
+
 	/// Base language of mod, all mod strings are assumed to be in this language
 	std::string baseLanguage;
 
@@ -52,9 +72,6 @@ public:
 	/// list of mods that can't be used in the same time as this one
 	std::set <TModID> conflicts;
 
-	/// CRC-32 checksum of the mod
-	ui32 checksum;
-
 	EValidationStatus validation;
 
 	JsonNode config;
@@ -74,6 +91,8 @@ public:
 	/// return true if this mod can affect gameplay, e.g. adds or modifies any game objects
 	bool checkModGameplayAffecting() const;
 	
+	const VerificationInfo & getVerificationInfo() const;
+
 private:
 	/// true if mod is enabled by user, e.g. in Launcher UI
 	bool explicitlyEnabled;
@@ -81,6 +100,8 @@ private:
 	/// true if mod can be loaded - compatible and has no missing deps
 	bool implicitlyEnabled;
 	
+	VerificationInfo verificationInfo;
+
 	void loadLocalData(const JsonNode & data);
 };
 

lib/modding/ContentTypeHandler.cpp

@@ -212,7 +212,8 @@ void CContentHandler::preloadData(CModInfo & mod)
 	bool validate = (mod.validation != CModInfo::PASSED);
 
 	// print message in format [<8-symbols checksum>] <modname>
-	logMod->info("\t\t[%08x]%s", mod.checksum, mod.name);
+	auto & info = mod.getVerificationInfo();
+	logMod->info("\t\t[%08x]%s", info.checksum, info.name);
 
 	if (validate && mod.identifier != ModScope::scopeBuiltin())
 	{
@@ -233,12 +234,12 @@ void CContentHandler::load(CModInfo & mod)
 	if (validate)
 	{
 		if (mod.validation != CModInfo::FAILED)
-			logMod->info("\t\t[DONE] %s", mod.name);
+			logMod->info("\t\t[DONE] %s", mod.getVerificationInfo().name);
 		else
-			logMod->error("\t\t[FAIL] %s", mod.name);
+			logMod->error("\t\t[FAIL] %s", mod.getVerificationInfo().name);
 	}
 	else
-		logMod->info("\t\t[SKIP] %s", mod.name);
+		logMod->info("\t\t[SKIP] %s", mod.getVerificationInfo().name);
 }
 
 const ContentTypeHandler & CContentHandler::operator[](const std::string & name) const

mapeditor/mainwindow.cpp

@@ -338,7 +338,7 @@ bool MainWindow::openMap(const QString & filenameSelect)
 			auto missingMods = CMapService::verifyMapHeaderMods(*header);
 			ModIncompatibility::ModList modList;
 			for(const auto & m : missingMods)
-				modList.push_back({m.first, m.second.toString()});
+				modList.push_back({m.second.name, m.second.version.toString()});
 			
 			if(!modList.empty())
 				throw ModIncompatibility(std::move(modList));

mapeditor/mapcontroller.cpp

@@ -588,7 +588,7 @@ ModCompatibilityInfo MapController::modAssessmentAll()
 			auto handler = VLC->objtypeh->getHandlerFor(primaryID, secondaryID);
 			auto modName = QString::fromStdString(handler->getJsonKey()).split(":").at(0).toStdString();
 			if(modName != "core")
-				result[modName] = VLC->modh->getModInfo(modName).version;
+				result[modName] = VLC->modh->getModInfo(modName).getVerificationInfo();
 		}
 	}
 	return result;
@@ -605,7 +605,7 @@ ModCompatibilityInfo MapController::modAssessmentMap(const CMap & map)
 		auto handler = VLC->objtypeh->getHandlerFor(obj->ID, obj->subID);
 		auto modName = QString::fromStdString(handler->getJsonKey()).split(":").at(0).toStdString();
 		if(modName != "core")
-			result[modName] = VLC->modh->getModInfo(modName).version;
+			result[modName] = VLC->modh->getModInfo(modName).getVerificationInfo();
 	}
 	//TODO: terrains?
 	return result;

mapeditor/mapcontroller.h

@@ -13,10 +13,10 @@
 #include "maphandler.h"
 #include "mapview.h"
 
-#include "../lib/modding/CModVersion.h"
+#include "../lib/modding/CModInfo.h"
 
 VCMI_LIB_NAMESPACE_BEGIN
-using ModCompatibilityInfo = std::map<std::string, CModVersion>;
+using ModCompatibilityInfo = std::map<std::string, CModInfo::VerificationInfo>;
 class EditorObstaclePlacer;
 VCMI_LIB_NAMESPACE_END
 

mapeditor/mapsettings/modsettings.cpp

@@ -47,7 +47,7 @@ void ModSettings::initialize(MapController & c)
 
 	auto createModTreeWidgetItem = [&](QTreeWidgetItem * parent, const CModInfo & modInfo)
 	{
-		auto item = new QTreeWidgetItem(parent, {QString::fromStdString(modInfo.name), QString::fromStdString(modInfo.version.toString())});
+		auto item = new QTreeWidgetItem(parent, {QString::fromStdString(modInfo.getVerificationInfo().name), QString::fromStdString(modInfo.getVerificationInfo().version.toString())});
 		item->setData(0, Qt::UserRole, QVariant(QString::fromStdString(modInfo.identifier)));
 		item->setFlags(item->flags() | Qt::ItemIsUserCheckable);
 		item->setCheckState(0, controller->map()->mods.count(modInfo.identifier) ? Qt::Checked : Qt::Unchecked);
@@ -104,7 +104,7 @@ void ModSettings::update()
 		if(item->checkState(0) == Qt::Checked)
 		{
 			auto modName = item->data(0, Qt::UserRole).toString().toStdString();
-			controller->map()->mods[modName] = VLC->modh->getModInfo(modName).version;
+			controller->map()->mods[modName] = VLC->modh->getModInfo(modName).getVerificationInfo();
 		}
 	};
 

mapeditor/validator.cpp

@@ -174,7 +174,7 @@ std::list<Validator::Issue> Validator::validate(const CMap * map)
 		{
 			if(!map->mods.count(mod.first))
 			{
-				issues.emplace_back(QString(tr("Map contains object from mod \"%1\", but doesn't require it")).arg(QString::fromStdString(VLC->modh->getModInfo(mod.first).name)), true);
+				issues.emplace_back(QString(tr("Map contains object from mod \"%1\", but doesn't require it")).arg(QString::fromStdString(VLC->modh->getModInfo(mod.first).getVerificationInfo().name)), true);
 			}
 		}
 	}