From 83f17807d45044f2770f474b77681d43d565dc0f Mon Sep 17 00:00:00 2001
From: Daniel Schreiber <daniel-schreiber@gmx.de>
Date: Wed, 20 Jul 2022 13:26:34 +0200
Subject: [PATCH] fix: do not change translation state for non-updates (changes
 only in collapsible whitespace)

---
 __tests__/merge.test.ts | 35 +++++++++++++++++++++++++++++++++++
 src/merge.ts            |  2 +-
 2 files changed, 36 insertions(+), 1 deletion(-)

diff --git a/__tests__/merge.test.ts b/__tests__/merge.test.ts
index 872c11a..6944733 100644
--- a/__tests__/merge.test.ts
+++ b/__tests__/merge.test.ts
@@ -1172,6 +1172,41 @@ describe('merge', () => {
                 '</xliff>'));
         });
 
+        test('should not update node with only collapsible whitespace change', () => {
+            const sourceFileContent = '<xliff version="2.0" xmlns="urn:oasis:names:tc:xliff:document:2.0" srcLang="de">\n' +
+                '  <file original="ng.template" id="ngi18n">\n' +
+                '    <unit id="ID1">\n' +
+                '      <segment>\n' +
+                '        <source>    source val    </source>\n' +
+                '      </segment>\n' +
+                '    </unit>\n' +
+                '  </file>\n' +
+                '</xliff>';
+            const destFileContent = '<xliff version="2.0" xmlns="urn:oasis:names:tc:xliff:document:2.0" srcLang="de" trgLang="fr-CH">\n' +
+                '  <file original="ng.template" id="ngi18n">\n' +
+                '    <unit id="ID1">\n' +
+                '      <segment state="translated">\n' +
+                '        <source> source val </source>\n' +
+                '        <target>target val</target>\n' +
+                '      </segment>\n' +
+                '    </unit>\n' +
+                '  </file>\n' +
+                '</xliff>';
+
+            const result = merge(sourceFileContent, destFileContent);
+
+            expect(norm(result)).toEqual(norm('<xliff version="2.0" xmlns="urn:oasis:names:tc:xliff:document:2.0" srcLang="de" trgLang="fr-CH">\n' +
+                '  <file original="ng.template" id="ngi18n">\n' +
+                '    <unit id="ID1">\n' +
+                '      <segment state="translated">\n' +
+                '        <source> source val </source>\n' +
+                '        <target>target val</target>\n' +
+                '      </segment>\n' +
+                '    </unit>\n' +
+                '  </file>\n' +
+                '</xliff>'));
+        });
+
         test('should update changed node without updating translation state if disabled', () => {
             const sourceFileContent = '<xliff version="2.0" xmlns="urn:oasis:names:tc:xliff:document:2.0" srcLang="de">\n' +
                 '  <file original="ng.template" id="ngi18n">\n' +
diff --git a/src/merge.ts b/src/merge.ts
index ef5a749..3706329 100644
--- a/src/merge.ts
+++ b/src/merge.ts
@@ -38,7 +38,7 @@ function toString(...nodes: XmlNode[]): string {
 }
 
 function collapseWhitespace(destSourceText: string) {
-    return destSourceText.replace(/\s+/, ' ');
+    return destSourceText.replace(/\s+/g, ' ');
 }
 
 function getUnits(doc: XmlDocument, xliffVersion: '1.2' | '2.0') {