1
0
mirror of https://gitlab.com/OpenMW/openmw.git synced 2025-01-29 09:32:45 +00:00

OpenMW-CS verify duplicate RefNums

This commit is contained in:
cykoder 2024-06-24 18:03:59 +00:00 committed by psi29a
parent b001deeff7
commit 57e7bf3748
2 changed files with 12 additions and 0 deletions

View File

@ -45,6 +45,16 @@ void CSMTools::ReferenceCheckStage::perform(int stage, CSMDoc::Messages& message
const CSMWorld::CellRef& cellRef = record.get();
const CSMWorld::UniversalId id(CSMWorld::UniversalId::Type_Reference, cellRef.mId);
// Check RefNum is unique per content file, otherwise can cause load issues
const auto refNum = cellRef.mRefNum;
const auto insertResult = mUsedReferenceIDs.emplace(refNum, cellRef.mId);
if (!insertResult.second)
messages.add(id,
"Duplicate RefNum: " + std::to_string(refNum.mContentFile) + std::string("-")
+ std::to_string(refNum.mIndex) + " shared with cell reference "
+ insertResult.first->second.toString(),
"", CSMDoc::Message::Severity_Error);
// Check reference id
if (cellRef.mRefID.empty())
messages.add(id, "Instance is not based on an object", "", CSMDoc::Message::Severity_Error);
@ -109,6 +119,7 @@ void CSMTools::ReferenceCheckStage::perform(int stage, CSMDoc::Messages& message
int CSMTools::ReferenceCheckStage::setup()
{
mIgnoreBaseRecords = CSMPrefs::get()["Reports"]["ignore-base-records"].isTrue();
mUsedReferenceIDs.clear();
return mReferences.getSize();
}

View File

@ -43,6 +43,7 @@ namespace CSMTools
const CSMWorld::IdCollection<CSMWorld::Cell>& mCells;
const CSMWorld::IdCollection<ESM::Faction>& mFactions;
const CSMWorld::IdCollection<ESM::BodyPart>& mBodyParts;
std::unordered_map<ESM::RefNum, ESM::RefId> mUsedReferenceIDs;
bool mIgnoreBaseRecords;
};
}