🥅 Silents part of thumbnail request exceptions

Author: AlexV525

CHANGELOG.md

@@ -12,6 +12,10 @@ that can be found in the LICENSE file. -->
 - Add Turkish language text delegate. (#331).
 - Allow to confirm 0 assets if there are selected assets previously. (#461)
 
+### Improvements
+
+- Silents part of thumbnail request exceptions.
+
 ### Fixes
 
 - Fix semantics interactions for video preview. (#458)

lib/src/constants/constants.dart

@@ -5,6 +5,8 @@
 import 'package:flutter/painting.dart';
 import 'package:photo_manager/photo_manager.dart';
 
+const String packageName = 'wechat_assets_picker';
+
 const int defaultAssetsPerPage = 80;
 const int defaultMaxAssetsCount = 9;
 

lib/src/delegates/asset_picker_delegate.dart

@@ -8,7 +8,6 @@ import 'package:photo_manager/photo_manager.dart';
 
 import '../constants/config.dart';
 import '../constants/constants.dart';
-import '../internal/methods.dart';
 import '../provider/asset_picker_provider.dart';
 import '../widget/asset_picker.dart';
 import '../widget/asset_picker_page_route.dart';
@@ -165,8 +164,15 @@ class AssetPickerDelegate {
     try {
       PhotoManager.addChangeCallback(callback);
       PhotoManager.startChangeNotify();
-    } catch (e) {
-      realDebugPrint('Error when registering assets callback: $e');
+    } catch (e, s) {
+      FlutterError.presentError(
+        FlutterErrorDetails(
+          exception: e,
+          stack: s,
+          library: packageName,
+          silent: true,
+        ),
+      );
     }
   }
 
@@ -181,8 +187,15 @@ class AssetPickerDelegate {
     try {
       PhotoManager.removeChangeCallback(callback);
       PhotoManager.stopChangeNotify();
-    } catch (e) {
-      realDebugPrint('Error when unregistering assets callback: $e');
+    } catch (e, s) {
+      FlutterError.presentError(
+        FlutterErrorDetails(
+          exception: e,
+          stack: s,
+          library: packageName,
+          silent: true,
+        ),
+      );
     }
   }
 

lib/src/internal/methods.dart

@@ -419,36 +419,48 @@ class DefaultAssetPickerProvider
   Future<Uint8List?> getThumbnailFromPath(
     PathWrapper<AssetPathEntity> path,
   ) async {
-    if (requestType == RequestType.audio) {
-      return null;
-    }
-    final int assetCount = path.assetCount ?? await path.path.assetCountAsync;
-    if (assetCount == 0) {
-      return null;
-    }
-    final List<AssetEntity> assets = await path.path.getAssetListRange(
-      start: 0,
-      end: 1,
-    );
-    if (assets.isEmpty) {
-      return null;
-    }
-    final AssetEntity asset = assets.single;
-    // Obtain the thumbnail only when the asset is image or video.
-    if (asset.type != AssetType.image && asset.type != AssetType.video) {
+    try {
+      if (requestType == RequestType.audio) {
+        return null;
+      }
+      final int assetCount = path.assetCount ?? await path.path.assetCountAsync;
+      if (assetCount == 0) {
+        return null;
+      }
+      final List<AssetEntity> assets = await path.path.getAssetListRange(
+        start: 0,
+        end: 1,
+      );
+      if (assets.isEmpty) {
+        return null;
+      }
+      final AssetEntity asset = assets.single;
+      // Obtain the thumbnail only when the asset is image or video.
+      if (asset.type != AssetType.image && asset.type != AssetType.video) {
+        return null;
+      }
+      final Uint8List? data = await asset.thumbnailDataWithSize(
+        pathThumbnailSize,
+      );
+      final int index = _paths.indexWhere(
+        (PathWrapper<AssetPathEntity> p) => p.path == path.path,
+      );
+      if (index != -1) {
+        _paths[index] = _paths[index].copyWith(thumbnailData: data);
+        notifyListeners();
+      }
+      return data;
+    } catch (e, s) {
+      FlutterError.presentError(
+        FlutterErrorDetails(
+          exception: e,
+          stack: s,
+          library: packageName,
+          silent: true,
+        ),
+      );
       return null;
     }
-    final Uint8List? data = await asset.thumbnailDataWithSize(
-      pathThumbnailSize,
-    );
-    final int index = _paths.indexWhere(
-      (PathWrapper<AssetPathEntity> p) => p.path == path.path,
-    );
-    if (index != -1) {
-      _paths[index] = _paths[index].copyWith(thumbnailData: data);
-      notifyListeners();
-    }
-    return data;
   }
 
   Future<void> getAssetCountFromPath(PathWrapper<AssetPathEntity> path) async {

lib/src/provider/asset_picker_provider.dart

@@ -8,8 +8,8 @@ import 'package:flutter/material.dart';
 import 'package:photo_manager/photo_manager.dart';
 import 'package:video_player/video_player.dart';
 
+import '../../constants/constants.dart';
 import '../../constants/extensions.dart';
-import '../../internal/methods.dart';
 import '../../internal/singleton.dart';
 import '../scale_text.dart';
 
@@ -93,8 +93,15 @@ class _AudioPageBuilderState extends State<AudioPageBuilder> {
       _controller = VideoPlayerController.networkUrl(Uri.parse(url!));
       await controller.initialize();
       controller.addListener(audioPlayerListener);
-    } catch (e) {
-      realDebugPrint('Error when opening audio file: $e');
+    } catch (e, s) {
+      FlutterError.presentError(
+        FlutterErrorDetails(
+          exception: e,
+          stack: s,
+          library: packageName,
+          silent: true,
+        ),
+      );
     } finally {
       isLoaded = true;
       if (mounted) {

lib/src/widget/builder/audio_page_builder.dart

@@ -2,13 +2,13 @@
 // Use of this source code is governed by an Apache license that can be found
 // in the LICENSE file.
 
+import 'dart:developer' as dev;
 import 'dart:io';
 
 import 'package:flutter/material.dart';
 import 'package:photo_manager/photo_manager.dart';
 
 import '../../constants/extensions.dart';
-import '../../internal/methods.dart';
 import '../scale_text.dart';
 
 class LocallyAvailableBuilder extends StatefulWidget {
@@ -65,7 +65,6 @@ class _LocallyAvailableBuilderState extends State<LocallyAvailableBuilder> {
           withSubtype: true,
           progressHandler: _progressHandler,
         );
-        realDebugPrint('Produced file: $file.');
         if (file != null) {
           _isLocallyAvailable = true;
           if (mounted) {
@@ -75,7 +74,10 @@ class _LocallyAvailableBuilderState extends State<LocallyAvailableBuilder> {
       });
     }
     _progressHandler?.stream.listen((PMProgressState s) {
-      realDebugPrint('Handling progress: $s.');
+      assert(() {
+        dev.log('Handling progress: $s.');
+        return true;
+      }());
       if (s.state == PMRequestState.success) {
         _isLocallyAvailable = true;
         if (mounted) {

lib/src/widget/builder/locally_available_builder.dart

@@ -8,8 +8,8 @@ import 'package:flutter/material.dart';
 import 'package:photo_manager/photo_manager.dart';
 import 'package:video_player/video_player.dart';
 
+import '../../constants/constants.dart';
 import '../../delegates/asset_picker_viewer_builder_delegate.dart';
-import '../../internal/methods.dart';
 import '../../internal/singleton.dart';
 import '../scale_text.dart';
 import 'locally_available_builder.dart';
@@ -117,8 +117,15 @@ class _VideoPageBuilderState extends State<VideoPageBuilder> {
       if (widget.hasOnlyOneVideoAndMoment) {
         controller.play();
       }
-    } catch (e) {
-      realDebugPrint('Error when initialize video controller: $e');
+    } catch (e, s) {
+      FlutterError.presentError(
+        FlutterErrorDetails(
+          exception: e,
+          stack: s,
+          library: packageName,
+          silent: true,
+        ),
+      );
       hasErrorWhenInitializing = true;
     } finally {
       if (mounted) {

lib/src/widget/builder/video_page_builder.dart

@@ -7,7 +7,7 @@ library wechat_assets_picker;
 export 'package:photo_manager/photo_manager.dart';
 
 export 'src/constants/config.dart';
-export 'src/constants/constants.dart';
+export 'src/constants/constants.dart' hide packageName;
 export 'src/constants/enums.dart';
 export 'src/constants/typedefs.dart';