🚸 Prevent duplicate shooting actions (#204)

Fixes #202
fluttercandies · Oct 4, 2023 · 94f9e9d · 94f9e9d
1 parent 486a840
commit 94f9e9d
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 6 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,6 +6,12 @@ that can be found in the LICENSE file. -->
 
 See the [Migration Guide](guides/migration_guide.md) for the details of breaking changes between versions.
 
+## Unreleased
+
+### Fixes
+
+- Prevent duplicate shooting actions.
+
 ## 4.0.2
 
 ### Fixes

diff --git a/lib/src/states/camera_picker_state.dart b/lib/src/states/camera_picker_state.dart
@@ -883,18 +883,20 @@ class CameraPickerState extends State<CameraPicker>
   }
 
   PointerMoveEventListener? onPointerMove(BoxConstraints c) {
-    if (innerController != null && enablePullToZoomInRecord) {
+    if (innerController != null &&
+        enablePullToZoomInRecord &&
+        controller.value.isRecordingVideo) {
       return (PointerMoveEvent e) => onShootingButtonMove(e, c);
     }
     return null;
   }
 
   GestureTapCallback? get onTap {
-    if (innerController == null) {
+    if (innerController == null || isControllerBusy) {
       return null;
     }
     if (enableTapRecording) {
-      if (innerController?.value.isRecordingVideo ?? false) {
+      if (controller.value.isRecordingVideo) {
         return stopRecordingVideo;
       }
       return () {
@@ -911,7 +913,7 @@ class CameraPickerState extends State<CameraPicker>
   }
 
   String? get onTapHint {
-    if (innerController == null) {
+    if (innerController == null || isControllerBusy) {
       return null;
     }
     if (enableTapRecording) {
@@ -927,7 +929,7 @@ class CameraPickerState extends State<CameraPicker>
   }
 
   GestureLongPressCallback? get onLongPress {
-    if (innerController == null) {
+    if (innerController == null || isControllerBusy) {
       return null;
     }
     if (enableRecording && !enableTapRecording) {
@@ -937,7 +939,7 @@ class CameraPickerState extends State<CameraPicker>
   }
 
   String? get onLongPressHint {
-    if (innerController == null) {
+    if (innerController == null || isControllerBusy) {
       return null;
     }
     if (enableRecording && !enableTapRecording) {