🥅 Add more assertions (#411)

Author: AlexV525

CHANGELOG.md

@@ -4,12 +4,18 @@ that can be found in the LICENSE file. -->
 
 # Changelog
 
+## 8.3.2+1
+
+### Improvements
+
+- Add more assertions. (#411)
+
 ## 8.3.2
 
 ### Fixes
 
 - Use `Completer` and more accurate conditions to avoid duplicate load assets
-  when `AssetPickerConfig.pageSize` is smaller than a complete page.
+  when `AssetPickerConfig.pageSize` is smaller than a complete page. (#407)
 
 ## 8.3.1+1
 

lib/src/constants/config.dart

@@ -36,11 +36,13 @@ class AssetPickerConfig {
     this.shouldRevertGrid,
     this.limitedPermissionOverlayPredicate,
     this.pathNameBuilder,
-  })  : assert(maxAssets >= 1, 'maxAssets must be greater than 1.'),
-        assert(
+  })  : assert(
           pickerTheme == null || themeColor == null,
           'pickerTheme and themeColor cannot be set at the same time.',
         ),
+        assert(maxAssets > 0, 'maxAssets must be greater than 0.'),
+        assert(pageSize > 0, 'pageSize must be greater than 0.'),
+        assert(gridCount > 0, 'gridCount must be greater than 0.'),
         assert(
           pageSize % gridCount == 0,
           'pageSize must be a multiple of gridCount.',

lib/src/delegates/asset_picker_builder_delegate.dart

@@ -52,7 +52,8 @@ abstract class AssetPickerBuilderDelegate<Asset, Path> {
     Color? themeColor,
     AssetPickerTextDelegate? textDelegate,
     Locale? locale,
-  })  : assert(
+  })  : assert(gridCount > 0, 'gridCount must be greater than 0.'),
+        assert(
           pickerTheme == null || themeColor == null,
           'Theme and theme color cannot be set at the same time.',
         ),

lib/src/delegates/asset_picker_viewer_builder_delegate.dart

@@ -40,7 +40,8 @@ abstract class AssetPickerViewerBuilderDelegate<Asset, Path> {
     this.maxAssets,
     this.shouldReversePreview = false,
     this.selectPredicate,
-  });
+  })  : assert(maxAssets == null || maxAssets > 0),
+        assert(currentIndex >= 0);
 
   /// [ChangeNotifier] for photo selector viewer.
   /// 资源预览器的状态保持

lib/src/provider/asset_picker_provider.dart

@@ -27,7 +27,8 @@ abstract class AssetPickerProvider<Asset, Path> extends ChangeNotifier {
     this.pageSize = defaultAssetsPerPage,
     this.pathThumbnailSize = defaultPathThumbnailSize,
     List<Asset>? selectedAssets,
-  }) {
+  })  : assert(maxAssets > 0, 'maxAssets must be greater than 0.'),
+        assert(pageSize > 0, 'pageSize must be greater than 0.') {
     if (selectedAssets != null && selectedAssets.isNotEmpty) {
       _selectedAssets = selectedAssets.toList();
     }
@@ -359,6 +360,7 @@ class DefaultAssetPickerProvider
       _hasAssetsToDisplay = _currentAssets.isNotEmpty;
       notifyListeners();
     }
+
     if (_getAssetsFromPathCompleter == null) {
       _getAssetsFromPathCompleter = Completer<void>();
       run().then((_) {

lib/src/provider/asset_picker_viewer_provider.dart

@@ -14,7 +14,7 @@ class AssetPickerViewerProvider<A> extends ChangeNotifier {
   AssetPickerViewerProvider(
     List<A>? assets, {
     this.maxAssets = defaultMaxAssetsCount,
-  }) {
+  }) : assert(maxAssets > 0, 'maxAssets must be greater than 0.') {
     _currentlySelectedAssets = (assets ?? <A>[]).toList();
   }
 

pubspec.yaml

@@ -1,5 +1,5 @@
 name: wechat_assets_picker
-version: 8.3.2
+version: 8.3.2+1
 description: |
   An image picker (also with videos and audio)
   for Flutter projects based on WeChat's UI,