Implement an override-value when cloning, use when cloning info records

apps/opencs/model/world/collection.hpp

@@ -15,7 +15,6 @@
 
 #include "columnbase.hpp"
 #include "collectionbase.hpp"
-#include "info.hpp"
 #include "land.hpp"
 #include "landtexture.hpp"
 #include "ref.hpp"
@@ -265,13 +264,6 @@ namespace CSMWorld
             CSMWorld::CellRef* ptr = (CSMWorld::CellRef*) &copy.mModified;
             ptr->mRefNum.mIndex = 0;
         }
-        if (type == UniversalId::Type_TopicInfo || type == UniversalId::Type_JournalInfo)
-        {
-            CSMWorld::Info* ptr = (CSMWorld::Info*) &copy.mModified;
-            std::vector<std::string> splitStringContainer;
-            Misc::StringUtils::split(destination, splitStringContainer, "#");
-            if (!splitStringContainer.empty()) ptr->mTopicId = splitStringContainer[0];
-        }
 
         int index = getAppendIndex(destination, type);
         insertRecord(copy, getAppendIndex(destination, type));

apps/opencs/model/world/commands.cpp

@@ -397,6 +397,10 @@ void CSMWorld::CloneCommand::redo()
 {
     mModel.cloneRecord (mIdOrigin, mId, mType);
     applyModifications();
+    for (auto& value : mOverrideValues)
+    {
+        mModel.setData(mModel.getModelIndex (mId, value.first), value.second);
+    }
 }
 
 void CSMWorld::CloneCommand::undo()
@@ -404,6 +408,11 @@ void CSMWorld::CloneCommand::undo()
     mModel.removeRow (mModel.getModelIndex (mId, 0).row());
 }
 
+void CSMWorld::CloneCommand::setOverrideValue(int column, QVariant value)
+{
+    mOverrideValues.emplace_back(std::make_pair(column, value));
+}
+
 CSMWorld::CreatePathgridCommand::CreatePathgridCommand(IdTable& model, const std::string& id, QUndoCommand *parent)
     : CreateCommand(model, id, parent)
 {

apps/opencs/model/world/commands.hpp

@@ -183,6 +183,7 @@ namespace CSMWorld
     class CloneCommand : public CreateCommand
     {
             std::string mIdOrigin;
+            std::vector<std::pair<int, QVariant>> mOverrideValues;
 
         public:
 
@@ -194,6 +195,8 @@ namespace CSMWorld
             void redo() override;
 
             void undo() override;
+
+            void setOverrideValue(int column, QVariant value);
     };
 
     class RevertCommand : public QUndoCommand

apps/opencs/view/world/infocreator.cpp

@@ -40,10 +40,14 @@ void CSVWorld::InfoCreator::configureCreateCommand (CSMWorld::CreateCommand& com
         command.addValue (table.findColumnIndex(CSMWorld::Columns::ColumnId_Rank), -1);
         command.addValue (table.findColumnIndex(CSMWorld::Columns::ColumnId_Gender), -1);
         command.addValue (table.findColumnIndex(CSMWorld::Columns::ColumnId_PcRank), -1);
+        CSMWorld::CloneCommand* cloneCommand = dynamic_cast<CSMWorld::CloneCommand*> (&command);
+        if (cloneCommand) cloneCommand->setOverrideValue(table.findColumnIndex(CSMWorld::Columns::ColumnId_Topic), mTopic->text());
     }
     else
     {
         command.addValue (table.findColumnIndex(CSMWorld::Columns::ColumnId_Journal), mTopic->text());
+        CSMWorld::CloneCommand* cloneCommand = dynamic_cast<CSMWorld::CloneCommand*> (&command);
+        if (cloneCommand) cloneCommand->setOverrideValue(table.findColumnIndex(CSMWorld::Columns::ColumnId_Journal), mTopic->text());
     }
 }