🐛 Fix invalid path sort (#364)

Author: AlexV525

CHANGELOG.md

@@ -4,6 +4,12 @@ that can be found in the LICENSE file. -->
 
 # Changelog
 
+## 8.0.4
+
+### Fixes
+
+- Fix invalid path sort. (#364)
+
 ## 8.0.3
 
 ### Improvements

example/pubspec.yaml

@@ -1,6 +1,6 @@
 name: wechat_assets_picker_demo
 description: The demo project for the wechat_assets_picker package.
-version: 8.0.3+24
+version: 8.0.4+25
 publish_to: none
 
 environment:

lib/src/delegates/sort_path_delegate.dart

@@ -54,7 +54,7 @@ class CommonSortPathDelegate extends SortPathDelegate<AssetPathEntity> {
         if (a.path.isAll) {
           return -1;
         }
-        if (a.path.isAll) {
+        if (b.path.isAll) {
           return 1;
         }
         if (_isCamera(a.path)) {

pubspec.yaml

@@ -1,6 +1,6 @@
 name: wechat_assets_picker
 description: An audio/video/image picker in pure Dart which is the same with WeChat, support multi picking.
-version: 8.0.3
+version: 8.0.4
 homepage: https://github.com/fluttercandies/flutter_wechat_assets_picker
 
 environment:

test/delegates/sort_path_delegate_test.dart

@@ -0,0 +1,34 @@
+//
+// [Author] Alex (https://github.com/AlexV525)
+// [Date] 2022/9/19 11:52
+//
+
+import 'package:flutter_test/flutter_test.dart';
+import 'package:wechat_assets_picker/wechat_assets_picker.dart';
+
+void main() {
+  test('Sort paths correctly', () {
+    final List<PathWrapper<AssetPathEntity>> paths =
+        <PathWrapper<AssetPathEntity>>[
+      PathWrapper<AssetPathEntity>(
+        path: AssetPathEntity(id: 'id2', name: 'Screenshots'),
+      ),
+      PathWrapper<AssetPathEntity>(
+        path: AssetPathEntity(id: 'id1', name: 'Camera'),
+      ),
+      PathWrapper<AssetPathEntity>(
+        path: AssetPathEntity(id: 'id0', name: 'All', isAll: true),
+      ),
+    ];
+    SortPathDelegate.common.sort(paths);
+    expect(paths[0], (PathWrapper<AssetPathEntity> e) => e.path.isAll);
+    expect(
+      paths[1],
+      (PathWrapper<AssetPathEntity> e) => e.path.name == 'Camera',
+    );
+    expect(
+      paths[2],
+      (PathWrapper<AssetPathEntity> e) => e.path.name == 'Screenshots',
+    );
+  });
+}