Fix slash escaping in multiline annos (#846)

anz-bank · Aug 9, 2023 · 5d434ca · 5d434ca
1 parent 8b78c76
commit 5d434ca
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/ts/src/model/model.test.ts b/ts/src/model/model.test.ts
@@ -288,7 +288,7 @@ describe("Roundtrip", () => {
             `),
         StringAnnoEscaped: realign(`
             App:
-                @name = "a \\"value\\""
+                @name = "hello;,?:@&=+$.!~*'()\\"/\\\\ world"
             `),
         MultilineAnno: realign(`
             App:
@@ -298,6 +298,7 @@ describe("Roundtrip", () => {
                     |   across
                     |
                     |    multiple lines
+                    |     with special chars: hello;,?:@&=+$.!~*'()"/\\ world
             `),
         ArrayAnno: realign(`
             App:

diff --git a/ts/src/pbModel/attribute.ts b/ts/src/pbModel/attribute.ts
@@ -27,7 +27,7 @@ export class PbAttribute {
      * character. Therefore we re-esacpe the backslashes to avoid accidentally escaping something else with them.
      */
     static deserializeString(input?: string): string | undefined {
-        return input?.replaceAll("\\", "\\\\");
+        return input?.includes("\n") ? input : input?.replaceAll("\\", "\\\\");
     }
 
     getAnnoValue(): AnnoValue {