[FIX] formatting: do not show escape character

If you want to use double quotes in a string which is part of a formula, you
have to escape it using a backslash (="hello \"world\"")
However, the backslash is shown when displaying the cell content.

closes #3606

Task: 3698283
X-original-commit: b157972
Signed-off-by: Lucas Lefèvre (lul) <lul@odoo.com>
Signed-off-by: Pierre Rousseau (pro) <pro@odoo.com>

src/helpers/format.ts

@@ -126,6 +126,9 @@ export function formatValue(value: CellValue, { format, locale }: LocaleFormat):
   }
   switch (typeof value) {
     case "string":
+      if (value.includes('\\"')) {
+        return value.replace(/\\"/g, '"');
+      }
       return value;
     case "boolean":
       return value ? "TRUE" : "FALSE";

tests/cells/cell_plugin.test.ts

@@ -110,6 +110,12 @@ describe("getCellText", () => {
     const result = clearCell(model, "A1");
     expect(result).toBeCancelledBecause(CommandResult.NoChanges);
   });
+
+  test("escape character is not display when formatting string", () => {
+    const model = new Model();
+    setCellContent(model, "A1", '="hello \\"world\\""');
+    expect(getEvaluatedCell(model, "A1")?.formattedValue).toBe('hello "world"');
+  });
 });
 
 describe("link cell", () => {

tests/formats/format_helpers.test.ts

@@ -10,6 +10,18 @@ import { FR_LOCALE } from "../test_helpers/constants";
 
 const locale = DEFAULT_LOCALE;
 
+describe("formatValue on string", () => {
+  test("apply on regular strings", () => {
+    expect(formatValue("", { locale })).toBe("");
+    expect(formatValue("test", { locale })).toBe("test");
+    expect(formatValue("test", { locale, format: "#,###.0" })).toBe("test");
+  });
+
+  test("apply on strings with escape characters", () => {
+    expect(formatValue('Hello \\"world\\"', { locale })).toBe('Hello "world"');
+  });
+});
+
 describe("formatValue on number", () => {
   test("apply default format ", () => {
     expect(formatValue(1, { locale })).toBe("1");