[git] Fixed duplicated entries for git history on merge operations

Signed-off-by: Federico Bozzini <federico.bozzini@arm.com>
eclipse-theia · Feb 20, 2020 · 04c9fd2 · 04c9fd2
1 parent 2ef80c0
commit 04c9fd2
Showing 1 changed file with 17 additions and 8 deletions.
diff --git a/packages/git/src/node/dugite-git.ts b/packages/git/src/node/dugite-git.ts
@@ -147,14 +147,23 @@ export class CommitDetailsParser extends OutputParser<CommitWithChanges> {
         for (const chunk of chunks) {
             const [sha, email, name, timestamp, authorDateRelative, summary, body, rawChanges] = chunk.trim().split(CommitDetailsParser.ENTRY_DELIMITER);
             const fileChanges = this.nameStatusParser.parse(repositoryUri, (rawChanges || '').trim());
-            changes.push({
-                sha,
-                author: { timestamp, email, name },
-                authorDateRelative,
-                summary,
-                body,
-                fileChanges
-            });
+            const shaChange = changes.find(c => c.sha === sha);
+            if (shaChange !== undefined) {
+                for (const fileChange of fileChanges) {
+                    if (!shaChange.fileChanges.some(c => c.uri === fileChange.uri)) {
+                        shaChange.fileChanges.push(fileChange);
+                    }
+                }
+            } else {
+                changes.push({
+                    sha,
+                    author: { timestamp, email, name },
+                    authorDateRelative,
+                    summary,
+                    body,
+                    fileChanges
+                });
+            }
         }
         return changes;
     }