Merge pull request #30 from juliansteenbakker/bug/crash-after-permission

bug: add check to resume start call

@@ -50,7 +50,7 @@ class _MobileScannerState extends State<MobileScanner>
   void didChangeAppLifecycleState(AppLifecycleState state) {
   void didChangeAppLifecycleState(AppLifecycleState state) {
     switch (state) {
     switch (state) {
       case AppLifecycleState.resumed:
       case AppLifecycleState.resumed:
-        controller.start();
+        if (!controller.isStarting) controller.start();
         break;
         break;
       case AppLifecycleState.inactive:
       case AppLifecycleState.inactive:
       case AppLifecycleState.paused:
       case AppLifecycleState.paused:

@@ -102,12 +102,18 @@ class MobileScannerController {
   // }
   // }
 
 
   // List<BarcodeFormats>? formats = _defaultBarcodeFormats,
   // List<BarcodeFormats>? formats = _defaultBarcodeFormats,
+  bool isStarting = false;
+
   /// Start barcode scanning. This will first check if the required permissions
   /// Start barcode scanning. This will first check if the required permissions
   /// are set.
   /// are set.
   Future<void> start() async {
   Future<void> start() async {
     ensure('startAsync');
     ensure('startAsync');
-
+    if (isStarting) {
+      throw Exception('mobile_scanner: Called start() while already starting.');
+    }
+    isStarting = true;
     // setAnalyzeMode(AnalyzeMode.barcode.index);
     // setAnalyzeMode(AnalyzeMode.barcode.index);
+
     // Check authorization status
     // Check authorization status
     MobileScannerState state =
     MobileScannerState state =
         MobileScannerState.values[await methodChannel.invokeMethod('state')];
         MobileScannerState.values[await methodChannel.invokeMethod('state')];
@@ -118,6 +124,7 @@ class MobileScannerController {
             result ? MobileScannerState.authorized : MobileScannerState.denied;
             result ? MobileScannerState.authorized : MobileScannerState.denied;
         break;
         break;
       case MobileScannerState.denied:
       case MobileScannerState.denied:
+        isStarting = false;
         throw PlatformException(code: 'NO ACCESS');
         throw PlatformException(code: 'NO ACCESS');
       case MobileScannerState.authorized:
       case MobileScannerState.authorized:
         break;
         break;
@@ -138,11 +145,13 @@ class MobileScannerController {
           'start', arguments);
           'start', arguments);
     } on PlatformException catch (error) {
     } on PlatformException catch (error) {
       debugPrint('${error.code}: ${error.message}');
       debugPrint('${error.code}: ${error.message}');
+      isStarting = false;
       // setAnalyzeMode(AnalyzeMode.none.index);
       // setAnalyzeMode(AnalyzeMode.none.index);
       return;
       return;
     }
     }
 
 
     if (startResult == null) {
     if (startResult == null) {
+      isStarting = false;
       throw PlatformException(code: 'INITIALIZATION ERROR');
       throw PlatformException(code: 'INITIALIZATION ERROR');
     }
     }
 
 
@@ -151,6 +160,7 @@ class MobileScannerController {
         textureId: startResult['textureId'],
         textureId: startResult['textureId'],
         size: toSize(startResult['size']),
         size: toSize(startResult['size']),
         hasTorch: hasTorch);
         hasTorch: hasTorch);
+    isStarting = false;
   }
   }
 
 
   Future<void> stop() async {
   Future<void> stop() async {