Skip documentation node id remapping if node is not changed (#8876)

praveenkuttappan · web-flow · commit 4aea26ad9921 · 2024-08-23T23:12:24.000-04:00
diff --git a/src/dotnet/APIView/APIViewWeb/Helpers/CodeFileHelpers.cs b/src/dotnet/APIView/APIViewWeb/Helpers/CodeFileHelpers.cs
@@ -509,7 +509,7 @@ private static void RemapDocumentationLines(List<ReviewLine> activeLines, Dictio
                 line.DiffKind = diffKind;
                 var newHashId = line.GetTokenNodeIdHash(newParentId, idx);
 
-                if (documentationRowMap.ContainsKey(oldHashId))
+                if (documentationRowMap.ContainsKey(oldHashId) && !newHashId.Equals(oldHashId))
                 {
                     documentationRowMap[newHashId] = documentationRowMap[oldHashId];
                     documentationRowMap.Remove(oldHashId);

Original file line number	Diff line number	Diff line change
`@@ -509,7 +509,7 @@ private static void RemapDocumentationLines(List<ReviewLine> activeLines, Dictio`
`509`	`509`	`line.DiffKind = diffKind;`
`510`	`510`	`var newHashId = line.GetTokenNodeIdHash(newParentId, idx);`
`511`	`511`
`512`		`- if (documentationRowMap.ContainsKey(oldHashId))`
	`512`	`+ if (documentationRowMap.ContainsKey(oldHashId) && !newHashId.Equals(oldHashId))`
`513`	`513`	`{`
`514`	`514`	`documentationRowMap[newHashId] = documentationRowMap[oldHashId];`
`515`	`515`	`documentationRowMap.Remove(oldHashId);`