1.0.2: Fixed the inability to remove default sorts

CHANGELOG.md

@@ -1,3 +1,8 @@
+## 1.0.2
+
+* Fixed the inability to remove the default sorts.
+* Added a test to verify the default sorts behavior.
+
 ## 1.0.1
 
 * Fixed formatting issues

example/pubspec.lock

@@ -68,7 +68,7 @@ packages:
       path: ".."
       relative: true
     source: path
-    version: "1.0.0"
+    version: "1.0.2"
   flutter_test:
     dependency: "direct dev"
     description: flutter

lib/src/application/pagination_controller.dart

@@ -109,7 +109,10 @@ abstract interface class PaginationController with ChangeNotifier {
   /// If no config is present, an empty map will be used instead.
   ///
   /// If the sort set is not empty, the listeners are notified.
-  void clearSorts();
+  ///
+  /// If the [resetDefaults] is set to true, the contents of the respective
+  /// [PaginateConfig.defaultSortBy] will be added to the sort list.
+  void clearSorts({bool resetDefaults = false});
 
   /// Adds a filter by the given [field] with the given [operator] to the query.
   ///
@@ -230,14 +233,20 @@ class PaginationControllerImpl
   }
 
   @override
-  void clearSorts() {
-    if (_sorts.isEmpty) return;
+  void clearSorts({bool resetDefaults = false}) {
+    bool wasChanged = false;
 
-    _sorts
-      ..clear()
-      ..addAll(paginateConfig.defaultSortBy);
+    if (_sorts.isNotEmpty) {
+      _sorts.clear();
+      wasChanged = true;
+    }
 
-    notifyListeners();
+    if (resetDefaults) {
+      _sorts.addAll(paginateConfig.defaultSortBy);
+      wasChanged |= paginateConfig.defaultSortBy.isNotEmpty;
+    }
+
+    if (wasChanged) notifyListeners();
   }
 
   @override

pubspec.yaml

@@ -1,6 +1,6 @@
 name: flutter_nestjs_paginate
 description: "A library providing a reactive paginated view builder compatible with `nestjs-paginate`"
-version: 1.0.1
+version: 1.0.2
 homepage: https://github.com/mitryp/flutter_nestjs_paginate
 issue_tracker: https://github.com/mitryp/flutter_nestjs_paginate/issues
 

test/pagination_controller_test.dart

@@ -461,6 +461,27 @@ void main() {
         expect(controller.sorts, isEmpty);
       });
 
+      test('Sorts are cleared correctly with a non-empty defaultSortBy', () {
+        const defaultSortBy = {testSortName1: testSort};
+        final controller = PaginationController(
+          paginateConfig: const PaginateConfig(
+            sortableColumns: {testSortName1},
+            defaultSortBy: defaultSortBy,
+          ),
+        );
+
+        expect(controller.sorts, hasLength(1));
+
+        controller.clearSorts();
+        expect(controller.sorts, isEmpty);
+
+        controller.clearSorts(resetDefaults: true);
+        expect(controller.sorts, equals(defaultSortBy));
+
+        controller.removeSort(testSortName1);
+        expect(controller.sorts, isEmpty);
+      });
+
       test('silently works correctly', () {
         // without silently controller should notify 4 times here
         controller