Translator: Attempt to keep old notes when rescan

Author: Wohlstand

Translator/textdata/text_data_processor.cpp

@@ -669,7 +669,76 @@ void TextDataProcessor::importLevel(TranslationData &origin, const QString &path
     }
 
     if(hasStrings)
+    {
+        // Try to merge comments from the previous copy
+        if(origin.levels.contains(shortPath))
+        {
+            // Backup
+            const auto old = origin.levels[shortPath];
+
+            // Copy back the title's note
+            tr.title.note = old.title.note;
+
+            // Import dialogues' notes
+            for(int i = 0; i < old.dialogues.size(); ++i)
+            {
+                auto &di_n = tr.dialogues[i];
+
+                // Check if dialogue equal:
+                if(i >= tr.dialogues.size())
+                    continue; // Not ours
+
+                auto &di_o = old.dialogues[i];
+
+                if(di_o.messages.size() != di_n.messages.size())
+                    continue; // Also not ours
+
+                bool equal = true;
+                // Compare messages
+                for(int m = 0; m < di_o.messages.size(); ++m)
+                {
+                    if(di_o.messages[m].type != di_n.messages[m].type ||
+                       di_o.messages[m].item_index != di_n.messages[m].item_index)
+                    {
+                       equal = false;
+                       break;
+                    }
+                }
+
+                if(!equal)
+                    continue; // Also not ours
+
+                // And now, it's really ours, so, import the note
+                di_n.note = di_o.note;
+            }
+
+            // Import NPC source notes
+            for(auto &no : old.npc)
+            {
+                for(auto &nn : tr.npc)
+                {
+                    if(no.npc_index != nn.npc_index || no.npc_id != nn.npc_id)
+                        continue; // Not ours
+
+                    nn.talk.note = no.talk.note;
+                }
+            }
+
+            // Import NPC source notes
+            for(auto &no : old.events)
+            {
+                for(auto &nn : tr.events)
+                {
+                    if(no.event_index != nn.event_index)
+                        continue; // Not ours
+
+                    nn.message.note = no.message.note;
+                }
+            }
+        }
+
         origin.levels.insert(shortPath, tr);
+    }
     else
     {
         QFileInfo f(path);
@@ -724,7 +793,35 @@ void TextDataProcessor::importWorld(TranslationData &origin, const QString &path
     }
 
     if(hasStrings)
+    {
+        // Try to merge comments from the previous copy
+        if(origin.worlds.contains(shortPath))
+        {
+            // Backup
+            const auto old = origin.worlds[shortPath];
+
+            // Copy back the title's note
+            tr.title.note = old.title.note;
+            // Copy back the credits' note
+            tr.credits.note = old.credits.note;
+
+            // Import NPC source notes
+            for(auto &no : old.level_titles)
+            {
+                for(auto &nn : tr.level_titles)
+                {
+                    if(no.level_index != nn.level_index || no.filename != nn.filename)
+                        continue; // Not ours
+
+                    nn.title.note = no.title.note;
+                }
+            }
+
+            origin.worlds.insert(shortPath, tr);
+        }
+
         origin.worlds.insert(shortPath, tr);
+    }
     else
     {
         QFileInfo f(path);
@@ -906,7 +1003,30 @@ void TextDataProcessor::importScript(TranslationData &origin, const QString &pat
 #undef PARSE_FMT_STR_2
 
     if(hasStrings)
+    {
+        // Try to merge comments from the previous copy
+        if(origin.scripts.contains(shortPath))
+        {
+            // Backup
+            const auto old = origin.scripts[shortPath];
+
+            // Import NPC source notes
+            for(auto &no : old.lines)
+            {
+                for(auto &nn : tr.lines)
+                {
+                    if(no.line != nn.line || no.source != nn.source)
+                        continue; // Not ours
+
+                    nn.translation.note = no.translation.note;
+                }
+            }
+
+            origin.scripts.insert(shortPath, tr);
+        }
+
         origin.scripts.insert(shortPath, tr);
+    }
 }
 
 void TextDataProcessor::recountStats(TranslateProject &proj, TranslationData &tr, bool isOrigin)