Xargon/ES-DE
1
0
Fork 0
mirror of https://github.com/RetroDECK/ES-DE.git synced 2024-11-22 06:05:38 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Made the 'fullname' tag in es_systems.cfg mandatory.

Browse source
This commit is contained in:
Leon Styhre 2020-12-26 20:20:45 +01:00
parent 6dfd8e0cfc
commit b1bb9743f2
1 changed files with 11 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
es-app/src/SystemData.cpp
View file

@ -285,7 +285,8 @@ bool SystemData::loadConfig()
cmd = system.child("command").text().get();
// Platform ID list
const std::string platformList = system.child("platform").text().get();
const std::string platformList =
Utils::String::toLower(system.child("platform").text().get());
std::vector<std::string> platformStrs = readList(platformList);
std::vector<PlatformIds::PlatformId> platformIds;
for (auto it = platformStrs.cbegin(); it != platformStrs.cend(); it++) {
@ -312,9 +313,15 @@ bool SystemData::loadConfig()
themeFolder = system.child("theme").text().as_string(name.c_str());
// Validate.
if (name.empty() || path.empty() || extensions.empty() || cmd.empty()) {
LOG(LogError) << "System \"" << name <<
"\" is missing name, path, extension, or command";
if (name.empty()) {
LOG(LogError) <<
"A system in the es_systems.cfg file has no name defined, skipping entry.";
continue;
}
else if (fullname.empty() || path.empty() || extensions.empty() || cmd.empty()) {
LOG(LogError) << "System \"" << name << "\" is missing the fullname, path, "
"extension, or command tag, skipping entry.";
continue;
}