♻️ Remove shouldKeptInLocal and update path methods.

Author: AlexV525

lib/src/constants/constants.dart

@@ -8,6 +8,7 @@ export 'package:flutter_common_exports/flutter_common_exports.dart';
 export 'package:photo_manager/photo_manager.dart';
 
 export '../delegates/camera_picker_text_delegate.dart';
+export '../utils/device_utils.dart';
 export 'colors.dart';
 
 class Constants {

lib/src/utils/device_utils.dart

@@ -0,0 +1,30 @@
+///
+/// [Author] Alex (https://github.com/AlexV525)
+/// [Date] 2020/7/17 10:09
+///
+import 'dart:io';
+
+import 'package:device_info/device_info.dart';
+
+export 'package:device_info/device_info.dart';
+
+class DeviceUtils {
+  const DeviceUtils._();
+
+  static final DeviceInfoPlugin _deviceInfoPlugin = DeviceInfoPlugin();
+  static dynamic deviceInfo;
+
+  static Future<void> getDeviceInfo() async {
+    if (Platform.isAndroid) {
+      deviceInfo = await _deviceInfoPlugin.androidInfo;
+    } else if (Platform.isIOS) {
+      deviceInfo = await _deviceInfoPlugin.iosInfo;
+    } else {
+      throw UnsupportedError('Platform not supported');
+    }
+  }
+
+  static bool get isLowerThanAndroidQ {
+    return (deviceInfo as AndroidDeviceInfo).version.sdkInt < 29;
+  }
+}

lib/src/widget/camera_picker.dart

@@ -28,7 +28,6 @@ import 'camera_picker_viewer.dart';
 class CameraPicker extends StatefulWidget {
   CameraPicker({
     Key key,
-    this.shouldKeptInLocal = false,
     this.isAllowRecording = false,
     this.maximumRecordingDuration = const Duration(seconds: 15),
     this.theme,
@@ -40,10 +39,6 @@ class CameraPicker extends StatefulWidget {
             : DefaultCameraPickerTextDelegate());
   }
 
-  /// Whether the taken file should be kept in local.
-  /// 拍照的文件是否应该保存在本地
-  final bool shouldKeptInLocal;
-
   /// Whether the picker can record video.
   /// 选择器是否可以录像
   final bool isAllowRecording;
@@ -75,7 +70,6 @@ class CameraPicker extends StatefulWidget {
     ).push<AssetEntity>(
       SlidePageTransitionBuilder<AssetEntity>(
         builder: CameraPicker(
-          shouldKeptInLocal: shouldKeptInLocal,
           isAllowRecording: isAllowRecording,
           theme: theme,
           textDelegate: textDelegate,
@@ -195,10 +189,6 @@ class CameraPickerState extends State<CameraPicker> {
   /// 当前的相机实例是否已完成初始化
   bool get isInitialized => cameraController?.value?.isInitialized ?? false;
 
-  /// Whether the taken file should be kept in local. (A non-null wrapper)
-  /// 拍照的文件是否应该保存在本地（非空包装）
-  bool get shouldKeptInLocal => widget.shouldKeptInLocal ?? false;
-
   /// Whether the picker can record video. (A non-null wrapper)
   /// 选择器是否可以录像（非空包装）
   bool get isAllowRecording => widget.isAllowRecording ?? false;
@@ -265,25 +255,29 @@ class CameraPickerState extends State<CameraPicker> {
     super.dispose();
   }
 
-  /// Defined the path according to [shouldKeptInLocal], with platforms specification.
-  /// 根据 [shouldKeptInLocal] 及平台规定确定存储路径。
+  /// Defined the path with platforms specification.
+  /// 根据平台存储规范及特性确定存储路径。
   ///
-  /// * When [Platform.isIOS], use [getApplicationDocumentsDirectory].
-  /// * When platform is others: [shouldKeptInLocal] ?
-  ///   * [true] : [getExternalStorageDirectory]'s path
-  ///   * [false]: [getExternalCacheDirectories]'s last path
+  /// * When the platform is not Android, use [getApplicationDocumentsDirectory] .
+  /// * When [Platform.isAndroid] :
+  ///   * SDK < 29: /sdcard/DCIM/camera .
+  ///   * SDK >= 29: ${cacheDir}/ .
   Future<void> initStorePath() async {
-    if (Platform.isIOS) {
-      cacheFilePath = (await getApplicationDocumentsDirectory()).path;
-    } else {
-      if (shouldKeptInLocal) {
-        cacheFilePath = (await getExternalStorageDirectory()).path;
+    /// Get device info before the path initialized.
+    await DeviceUtils.getDeviceInfo();
+
+    if (Platform.isAndroid) {
+      if (DeviceUtils.isLowerThanAndroidQ) {
+        cacheFilePath =
+            '${(await getExternalStorageDirectory()).path}/DCIM/Camera/';
       } else {
-        cacheFilePath = (await getExternalCacheDirectories()).last.path;
+        cacheFilePath = (await getTemporaryDirectory()).path;
       }
+    } else {
+      cacheFilePath = (await getApplicationDocumentsDirectory()).path;
     }
     if (cacheFilePath != null) {
-      cacheFilePath += '/picker';
+      cacheFilePath += '/cameraPicker';
     }
   }
 

lib/src/widget/camera_picker_viewer.dart

@@ -67,7 +67,8 @@ class CameraPickerViewer extends StatefulWidget {
         previewFilePath: previewFilePath,
         theme: theme,
       );
-      final PageRouteBuilder<AssetEntity> pageRoute = PageRouteBuilder<AssetEntity>(
+      final PageRouteBuilder<AssetEntity> pageRoute =
+          PageRouteBuilder<AssetEntity>(
         pageBuilder: (
           BuildContext context,
           Animation<double> animation,
@@ -84,7 +85,8 @@ class CameraPickerViewer extends StatefulWidget {
           return FadeTransition(opacity: animation, child: child);
         },
       );
-      final AssetEntity result = await Navigator.of(context).push<AssetEntity>(pageRoute);
+      final AssetEntity result =
+          await Navigator.of(context).push<AssetEntity>(pageRoute);
       return result;
     } catch (e) {
       realDebugPrint('Error when calling camera picker viewer: $e');
@@ -193,27 +195,38 @@ class _CameraPickerViewerState extends State<CameraPickerViewer> {
     }
   }
 
-  /// When users confirm to use the taken file, create the [AssetEntity], then delete
-  /// the file if not [shouldKeptInLocal]. While the entity might returned null, there's
-  /// no side effects if popping `null` because the parent picker will ignore it.
+  /// When users confirm to use the taken file, create the [AssetEntity].
+  /// While the entity might returned null, there's no side effects if popping `null`
+  /// because the parent picker will ignore it.
   Future<void> createAssetEntityAndPop() async {
     try {
       Future<AssetEntity> saveFuture;
 
       switch (pickerType) {
         case CameraPickerViewType.image:
           final Uint8List data = await previewFile.readAsBytes();
-          saveFuture = PhotoManager.editor.saveImage(data, title: previewFilePath);
-          break;
+          saveFuture = PhotoManager.editor.saveImage(
+            data,
+            title: previewFilePath,
+          );
           break;
         case CameraPickerViewType.video:
-          saveFuture = PhotoManager.editor.saveVideo(previewFile, title: previewFilePath);
+          saveFuture = PhotoManager.editor.saveVideo(
+            previewFile,
+            title: previewFilePath,
+          );
           break;
       }
 
       saveFuture.then((AssetEntity entity) {
-        if (!pickerState.shouldKeptInLocal) {
-          previewFile.delete();
+        if (Platform.isAndroid) {
+          if (!DeviceUtils.isLowerThanAndroidQ && previewFile.existsSync()) {
+            previewFile.delete();
+          }
+        } else {
+          if (previewFile.existsSync()) {
+            previewFile.delete();
+          }
         }
         Navigator.of(context).pop(entity);
       });
@@ -228,10 +241,9 @@ class _CameraPickerViewerState extends State<CameraPickerViewer> {
     return InkWell(
       borderRadius: maxBorderRadius,
       onTap: () {
-        previewFile.delete();
-        widget.pickerState.setState(() {
-          widget.pickerState.takenPictureFilePath = null;
-        });
+        if (previewFile.existsSync()) {
+          previewFile.delete();
+        }
         Navigator.of(context).pop();
       },
       child: Container(
@@ -295,7 +307,9 @@ class _CameraPickerViewerState extends State<CameraPickerViewer> {
                 shape: BoxShape.circle,
               ),
               child: Icon(
-                isControllerPlaying ? Icons.pause_circle_outline : Icons.play_circle_filled,
+                isControllerPlaying
+                    ? Icons.pause_circle_outline
+                    : Icons.play_circle_filled,
                 size: 70.0,
                 color: Colors.white,
               ),
@@ -357,7 +371,8 @@ class _CameraPickerViewerState extends State<CameraPickerViewer> {
               ),
 
           /// Place the button before the actions to ensure it's not blocking.
-          if (pickerType == CameraPickerViewType.video && videoController != null)
+          if (pickerType == CameraPickerViewType.video &&
+              videoController != null)
             playControlButton,
 
           viewerActions,

lib/src/widget/temp_picker.dart

@@ -4,4 +4,3 @@ export 'package:photo_manager/photo_manager.dart';
 
 export 'src/delegates/camera_picker_text_delegate.dart';
 export 'src/widget/camera_picker.dart';
-export 'src/widget/temp_picker.dart';

lib/wechat_camera_picker.dart

@@ -12,6 +12,7 @@ dependencies:
     sdk: flutter
 
   camera: ^0.5.8+2
+  device_info: ^0.4.2+4
   flutter_common_exports: ^0.1.0
   path_provider: ^1.6.11
   photo_manager: ^0.5.6