fix: fetchPathProperties for API 28 (#1161)

Signed-off-by: Caijinglong <[email protected]>

CHANGELOG.md

@@ -6,6 +6,12 @@ that can be found in the LICENSE file. -->
 
 To know more about breaking changes, see the [Migration Guide][].
 
+## Unreleased
+
+### Fixes
+
+- Fix `fetchPathProperties` exception on android of API 28.
+
 ## 3.2.2
 
 ### Fixes 

android/src/main/kotlin/com/fluttercandies/photo_manager/core/utils/DBUtils.kt

@@ -102,14 +102,14 @@ object DBUtils : IDBUtils {
         option: FilterOption
     ): AssetPathEntity? {
         val args = ArrayList<String>()
+        val where = option.makeWhere(type, args)
         val idSelection: String
         if (pathId == "") {
             idSelection = ""
         } else {
             idSelection = "AND ${MediaStore.MediaColumns.BUCKET_ID} = ?"
             args.add(pathId)
         }
-        val where = option.makeWhere(type, args)
         val selection =
             "${MediaStore.MediaColumns.BUCKET_ID} IS NOT NULL $where $idSelection) GROUP BY (${MediaStore.MediaColumns.BUCKET_ID}"
         val cursor = context.contentResolver.logQuery(

example/lib/page/developer/issues_page/issue_1152.dart

@@ -0,0 +1,69 @@
+import 'package:flutter/material.dart';
+import 'package:oktoast/oktoast.dart';
+import 'package:photo_manager/photo_manager.dart';
+
+import 'issue_index_page.dart';
+
+class Issus1152 extends StatefulWidget {
+  const Issus1152({super.key});
+
+  @override
+  State<Issus1152> createState() => _Issus1152State();
+}
+
+class _Issus1152State extends State<Issus1152> with IssueBase<Issus1152> {
+  @override
+  List<TargetPlatform>? get supportPlatforms => [
+        TargetPlatform.android,
+      ];
+
+  List<RequestType> checkStatus = [];
+
+  @override
+  void initState() {
+    super.initState();
+    PhotoManager.setLog(true);
+  }
+
+  @override
+  Widget build(BuildContext context) {
+    return buildScaffold([
+      buildButton(
+        'Test API 28 get images',
+        reproduce,
+      ),
+      buildLogWidget(),
+    ]);
+  }
+
+  @override
+  int get issueNumber => 1152;
+
+  Future<void> reproduce() async {
+    final pathList = await PhotoManager.getAssetPathList();
+    final noAllPathList = pathList.where((element) => !element.isAll);
+    if (noAllPathList.isEmpty) {
+      showToast('No path found');
+      return;
+    }
+
+    final path = noAllPathList.first;
+
+    // final name = path.name;
+    // final count = await path.assetCountAsync;
+    // addLog('assetCount of "$name": $count');
+
+    // final list = await path.getAssetListPaged(page: 0, size: 50);
+    // addLog('list.length: ${list.length}');
+
+    await fetchNewProperties(path);
+  }
+
+  Future<void> fetchNewProperties(AssetPathEntity path) async {
+    try {
+      await path.obtainForNewProperties();
+    } catch (e) {
+      addLog('error: $e');
+    }
+  }
+}

example/lib/page/developer/issues_page/issue_index_page.dart

@@ -11,6 +11,7 @@ import 'issue_1025.dart';
 import 'issue_1031.dart';
 import 'issue_1051.dart';
 import 'issue_1053.dart';
+import 'issue_1152.dart';
 import 'issue_734.dart';
 import 'issue_918.dart';
 import 'issue_962.dart';
@@ -73,6 +74,7 @@ class IssuePage extends StatelessWidget {
             Issue979(),
             Issus1051(),
             Issus1053(),
+            Issus1152(),
           ],
         ),
       ),