krageon/openmw-tes3coop
1
0
Fork 0
forked from teamnwah/openmw-tes3coop
Code Issues Pull requests Releases Wiki Activity

Expanded inventory check to creatures and NPCs

Browse source
This commit is contained in:
Aesylwinn 2015-12-19 22:02:39 -05:00
parent 4c14117761
commit 9bd14215d7
2 changed files with 49 additions and 34 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

79
apps/opencs/model/tools/referenceablecheck.cpp
View file

@ -399,39 +399,7 @@ void CSMTools::ReferenceableCheckStage::containerCheck(
messages.push_back (std::make_pair (id, container.mId + " has an empty name")); messages.push_back (std::make_pair (id, container.mId + " has an empty name"));
//checking contained items //checking contained items
const std::vector<ESM::ContItem>& itemList = container.mInventory.mList; inventoryListCheck(container.mInventory.mList, messages, id.toString());
for (unsigned i = 0; i < itemList.size(); ++i)
{
std::string itemName = itemList[i].mItem.toString();
CSMWorld::RefIdData::LocalIndex localIndex = mReferencables.searchId(itemName);
if (localIndex.first == -1)
messages.push_back (std::make_pair (id,
container.mId + " contains unreferenced item " + itemName));
else
{
switch (localIndex.second)
{
case CSMWorld::UniversalId::Type_Potion:
case CSMWorld::UniversalId::Type_Apparatus:
case CSMWorld::UniversalId::Type_Armor:
case CSMWorld::UniversalId::Type_Book:
case CSMWorld::UniversalId::Type_Clothing:
case CSMWorld::UniversalId::Type_Ingredient:
case CSMWorld::UniversalId::Type_Light:
case CSMWorld::UniversalId::Type_Lockpick:
case CSMWorld::UniversalId::Type_Miscellaneous:
case CSMWorld::UniversalId::Type_Probe:
case CSMWorld::UniversalId::Type_Repair:
case CSMWorld::UniversalId::Type_Weapon:
case CSMWorld::UniversalId::Type_ItemLevelledList:
break;
default:
messages.push_back (std::make_pair(id,
container.mId + " contains illegal item " + itemName));
}
}
}
// Check that mentioned scripts exist // Check that mentioned scripts exist
scriptCheck<ESM::Container>(container, messages, id.toString()); scriptCheck<ESM::Container>(container, messages, id.toString());
@ -506,6 +474,9 @@ void CSMTools::ReferenceableCheckStage::creatureCheck (
if (creature.mScale == 0) if (creature.mScale == 0)
messages.push_back (std::make_pair (id, creature.mId + " has zero scale value")); messages.push_back (std::make_pair (id, creature.mId + " has zero scale value"));
// Check inventory
inventoryListCheck(creature.mInventory.mList, messages, id.toString());
// Check that mentioned scripts exist // Check that mentioned scripts exist
scriptCheck<ESM::Creature>(creature, messages, id.toString()); scriptCheck<ESM::Creature>(creature, messages, id.toString());
} }
@ -777,6 +748,9 @@ void CSMTools::ReferenceableCheckStage::npcCheck (
//TODO: reputation, Disposition, rank, everything else //TODO: reputation, Disposition, rank, everything else
// Check inventory
inventoryListCheck(npc.mInventory.mList, messages, id.toString());
// Check that mentioned scripts exist // Check that mentioned scripts exist
scriptCheck<ESM::NPC>(npc, messages, id.toString()); scriptCheck<ESM::NPC>(npc, messages, id.toString());
} }
@ -926,6 +900,45 @@ void CSMTools::ReferenceableCheckStage::finalCheck (CSMDoc::Messages& messages)
"There is no player record")); "There is no player record"));
} }
void CSMTools::ReferenceableCheckStage::inventoryListCheck(
const std::vector<ESM::ContItem>& itemList,
CSMDoc::Messages& messages,
const std::string& id)
{
for (size_t i = 0; i < itemList.size(); ++i)
{
std::string itemName = itemList[i].mItem.toString();
CSMWorld::RefIdData::LocalIndex localIndex = mReferencables.searchId(itemName);
if (localIndex.first == -1)
messages.push_back (std::make_pair (id,
id + " contains non-existing item (" + itemName + ")"));
else
{
// Needs to accomodate Containers, Creatures, and NPCs
switch (localIndex.second)
{
case CSMWorld::UniversalId::Type_Potion:
case CSMWorld::UniversalId::Type_Apparatus:
case CSMWorld::UniversalId::Type_Armor:
case CSMWorld::UniversalId::Type_Book:
case CSMWorld::UniversalId::Type_Clothing:
case CSMWorld::UniversalId::Type_Ingredient:
case CSMWorld::UniversalId::Type_Light:
case CSMWorld::UniversalId::Type_Lockpick:
case CSMWorld::UniversalId::Type_Miscellaneous:
case CSMWorld::UniversalId::Type_Probe:
case CSMWorld::UniversalId::Type_Repair:
case CSMWorld::UniversalId::Type_Weapon:
case CSMWorld::UniversalId::Type_ItemLevelledList:
break;
default:
messages.push_back (std::make_pair(id,
id + " contains item of invalid type (" + itemName + ")"));
}
}
}
}
//Templates begins here //Templates begins here

4
apps/opencs/model/tools/referenceablecheck.hpp
View file

@ -47,7 +47,9 @@ namespace CSMTools
//FINAL CHECK //FINAL CHECK
void finalCheck (CSMDoc::Messages& messages); void finalCheck (CSMDoc::Messages& messages);
//TEMPLATE CHECKS //Convenience functions
void inventoryListCheck(const std::vector<ESM::ContItem>& itemList, CSMDoc::Messages& messages, const std::string& id);
template<typename ITEM> void inventoryItemCheck(const ITEM& someItem, template<typename ITEM> void inventoryItemCheck(const ITEM& someItem,
CSMDoc::Messages& messages, CSMDoc::Messages& messages,
const std::string& someID, const std::string& someID,