[FIX] Sort: Allow to sort array formula that do not spread

Some array formulas can lead to a single cell result. This can occur
quite easily when manipulating array functions that end up aggregated in
a single result. An easy example,

```javascript
=min(if(A1:B4>1, 2))
```
leads to a single cell result but should be sortable.

closes #7189

Task: 5033192
X-original-commit: f0d974c
Signed-off-by: Adrien Minne (adrm) <adrm@odoo.com>
Signed-off-by: Rémi Rahir (rar) <rar@odoo.com>

Author: rrahir

src/plugins/ui_feature/sort.ts

@@ -1,4 +1,11 @@
-import { isInside, overlap, positions, range, zoneToDimension } from "../../helpers/index";
+import {
+  deepEquals,
+  isInside,
+  overlap,
+  positions,
+  range,
+  zoneToDimension,
+} from "../../helpers/index";
 import { sortCells } from "../../helpers/sort";
 import {
   CellPosition,
@@ -79,9 +86,10 @@ export class SortPlugin extends UIPlugin {
   }
 
   private checkArrayFormulaInSortZone({ sheetId, zone }: SortCommand): CommandResult {
-    const arrayFormulaInZone = positions(zone).some(({ col, row }) =>
-      this.getters.getArrayFormulaSpreadingOn({ sheetId, col, row })
-    );
+    const arrayFormulaInZone = positions(zone).some(({ col, row }) => {
+      const originPosition = this.getters.getArrayFormulaSpreadingOn({ sheetId, col, row });
+      return originPosition && !deepEquals(originPosition, { sheetId, col, row });
+    });
     return arrayFormulaInZone ? CommandResult.SortZoneWithArrayFormulas : CommandResult.Success;
   }
 

tests/helpers/ui_helpers.test.ts

@@ -446,7 +446,7 @@ describe("UI Helpers", () => {
     });
   });
 
-  test("Cannot sort on zone with array formulas", () => {
+  test("Cannot sort on zone with array formulas that spread", () => {
     const raiseError = jest.fn();
     model = createModelFromGrid({ A1: "9", A2: "8", A3: "=CHOOSECOLS(A1:A2, 1)" });
     const env = makeTestEnv({ model, raiseError });
@@ -455,6 +455,20 @@ describe("UI Helpers", () => {
     expect(raiseError).toHaveBeenCalledWith("Cannot sort a zone with array formulas.");
   });
 
+  test("Can sort on zone with array formulas that do not spread", () => {
+    const raiseError = jest.fn();
+    model = createModelFromGrid({ A1: "9", A2: "8", B1: "1", C1: "=MMULT(A1:A2, A1:B1)" });
+    const env = makeTestEnv({ model, raiseError });
+
+    interactiveSortSelection(env, sheetId, toCartesian("C1"), toZone("C1:D2"), "asc");
+    expect(raiseError).toHaveBeenCalledTimes(1);
+    expect(raiseError).toHaveBeenCalledWith("Cannot sort a zone with array formulas.");
+
+    setCellContent(model, "C1", "=MMULT( A1:B1, A1:A2)");
+    interactiveSortSelection(env, sheetId, toCartesian("C1"), toZone("C1:D2"), "asc");
+    expect(raiseError).toHaveBeenCalledTimes(1);
+  });
+
   describe("Sort Merges", () => {
     const raiseError = jest.fn();
     const sheetId: UID = "sheet5";