[IMP] Data filter: Trim whitespace

[Data filter] Trim whitespace and don't bother of caps vs low case when filtering

Task:5375214

Author: matho-odoo

packages/o-spreadsheet-engine/src/helpers/text_helper.ts

@@ -297,6 +297,11 @@ export function toLowerCase(str: string | undefined): string {
   return str ? str.toLowerCase() : "";
 }
 
+/** Removes whitespace from both ends of the string. If the string is undefined, return an empty string */
+export function trim(str: string | undefined): string {
+  return str ? str.trim() : "";
+}
+
 /**
  * Extract the fontSize from a context font string
  * @param font The (context) font string to parse

src/components/filters/filter_menu_value_list/filter_menu_value_list.ts

@@ -1,6 +1,6 @@
 import { SpreadsheetChildEnv } from "@odoo/o-spreadsheet-engine/types/spreadsheet_env";
 import { Component, onWillUpdateProps, useRef, useState } from "@odoo/owl";
-import { deepEquals, positions, toLowerCase } from "../../../helpers";
+import { deepEquals, positions, toLowerCase, trim } from "../../../helpers";
 import { fuzzyLookup } from "../../../helpers/search";
 import { Position } from "../../../types";
 import { FilterMenuValueItem } from "../filter_menu_item/filter_menu_value_item";
@@ -76,12 +76,12 @@ export class FilterMenuValueList extends Component<Props, SpreadsheetChildEnv> {
 
     const cellValues = cells.map((val) => val.cellValue);
     const filterValues = filterValue?.filterType === "values" ? filterValue.hiddenValues : [];
-    const normalizedFilteredValues = new Set(filterValues.map(toLowerCase));
+    const normalizedFilteredValues = new Set(filterValues.map(toLowerCase).map(trim));
 
     const set = new Set<string>();
     const values: (Value & { normalizedValue: string })[] = [];
     const addValue = (value: string) => {
-      const normalizedValue = toLowerCase(value);
+      const normalizedValue = trim(toLowerCase(value));
       if (!set.has(normalizedValue)) {
         values.push({
           string: value || "",