Merge pull request #5152 from mP1/feature/SpreadsheetMetadata-sortSpr…

…eadsheetComparatorContext-reorder-parameters SpreadsheetMetadata.sortSpreadsheetComparatorContext
mP1 · Sep 21, 2024 · b32445f · b32445f
2 parents b97b4d0 + 24c64db
commit b32445f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/main/java/walkingkooka/spreadsheet/engine/BasicSpreadsheetEngine.java b/src/main/java/walkingkooka/spreadsheet/engine/BasicSpreadsheetEngine.java
@@ -501,8 +501,8 @@ private SpreadsheetDelta sortCells0(final SpreadsheetCellRangeReference cellRang
                     movedFromTo::put, // moved cells
                     context.spreadsheetMetadata()
                             .sortSpreadsheetComparatorContext(
-                                    context, // ConverterProvider
                                     context::now, // now supplier
+                                    context, // ConverterProvider
                                     context, // SpreadsheetLabelNameResolver
                                     context // ProviderContext
                             )

diff --git a/src/main/java/walkingkooka/spreadsheet/meta/SpreadsheetMetadata.java b/src/main/java/walkingkooka/spreadsheet/meta/SpreadsheetMetadata.java
@@ -764,9 +764,9 @@ public final ExpressionFunctionProvider expressionFunctionProvider(final Express
     /**
      * Returns a {@link SpreadsheetComparatorContext} which may be used for sorting.
      */
-    public final SpreadsheetComparatorContext sortSpreadsheetComparatorContext(final SpreadsheetProvider spreadsheetProvider,
-                                                                               final Supplier<LocalDateTime> now,
+    public final SpreadsheetComparatorContext sortSpreadsheetComparatorContext(final Supplier<LocalDateTime> now,
                                                                                final SpreadsheetLabelNameResolver resolveIfLabel,
+                                                                               final SpreadsheetProvider spreadsheetProvider,
                                                                                final ProviderContext providerContext) {
         return this.spreadsheetComparatorContext(
                 now, // now supplier