Merge pull request #1044 from solidify/bugfix/try-catch-in-mark-attachment-as-processed

Add try-catch block in MarkAttachmentAsProcessed method to warn about already-added attachments

Author: Alexander-Hjelm

src/WorkItemMigrator/Migration.Common/Journal.cs

@@ -1,4 +1,5 @@
-using System;
+using Migration.Common.Log;
+using System;
 using System.Collections.Generic;
 using System.IO;
 
@@ -78,8 +79,17 @@ private void WriteItem(string originId, int wiId, int rev)
 
         public void MarkAttachmentAsProcessed(string attOriginId, string attWiId)
         {
-            ProcessedAttachments.Add(attOriginId, attWiId);
-            WriteAttachment(attOriginId, attWiId);
+            try
+            {
+                ProcessedAttachments.Add(attOriginId, attWiId);
+                WriteAttachment(attOriginId, attWiId);
+            }
+            catch(Exception ex)
+            {
+                Logger.Log(LogLevel.Warning, $"Failed to write attachment to the attachmentsJournal.txt file. An attachment with attOriginId={attOriginId} was already present in the dictionary. (attWiId={attWiId})");
+                Logger.Log(LogLevel.Warning, ex.Message);
+            }
+
         }
 
         private void WriteAttachment(string attOriginId, string attWiId)