Merge pull request #2724 from akortunov/factions

Cleanup ownership for items in containers

apps/openmw/mwworld/containerstore.cpp

@@ -290,7 +290,7 @@ MWWorld::ContainerStoreIterator MWWorld::ContainerStore::add (const Ptr& itemPtr
     item.getCellRef().setOwner("");
     item.getCellRef().resetGlobalVariable();
     item.getCellRef().setFaction("");
-    item.getCellRef().setFactionRank(-1);
+    item.getCellRef().setFactionRank(-2);
 
     // must reset the RefNum on the copied item, so that the RefNum on the original item stays unique
     // maybe we should do this in the copy constructor instead?

components/esm/cellref.cpp

@@ -153,14 +153,20 @@ void ESM::CellRef::save (ESMWriter &esm, bool wideRefNum, bool inInventory, bool
         esm.writeHNT("XSCL", scale);
     }
 
+    if (!inInventory)
         esm.writeHNOCString("ANAM", mOwner);
+
     esm.writeHNOCString("BNAM", mGlobalVariable);
     esm.writeHNOCString("XSOL", mSoul);
 
+    if (!inInventory)
+    {
         esm.writeHNOCString("CNAM", mFaction);
-    if (mFactionRank != -2) {
+        if (mFactionRank != -2)
+        {
             esm.writeHNT("INDX", mFactionRank);
         }
+    }
 
     if (mEnchantmentCharge != -1)
         esm.writeHNT("XCHG", mEnchantmentCharge);