feat(power_trait_provider): reflect error state in UI

Yonomi · Dec 10, 2021 · cd3d154 · cd3d154
1 parent 84a96ac
commit cd3d154
Show file tree

Hide file tree

Showing 2 changed files with 34 additions and 15 deletions.
diff --git a/lib/providers/power_trait_provider.dart b/lib/providers/power_trait_provider.dart
@@ -12,8 +12,8 @@ class PowerTraitProvider extends ChangeNotifier {
   final int MAX_RETRIES = 10;
 
   bool _isLoading = false;
-
   bool _isPerformingAction = false;
+  bool _isInErrorState = false;
 
   late String _deviceId;
 
@@ -54,18 +54,22 @@ class PowerTraitProvider extends ChangeNotifier {
     if (!isPerformingAction) {
       setPerformingAction = true;
 
-      await sendPowerMethod(_request, this._deviceId, setOnOff);
-
-      int numRetries = 0;
-      while (getPowerTrait()?.state.value != setOnOff &&
-          numRetries < MAX_RETRIES) {
-        _deviceDetail = await getDetails(_request, _deviceId);
-
-        await Future.delayed(Duration(milliseconds: 750));
-        numRetries++;
+      try {
+        await sendPowerMethod(_request, this._deviceId, setOnOff);
+
+        int numRetries = 0;
+        while (getPowerTrait()?.state.value != setOnOff &&
+            numRetries < MAX_RETRIES) {
+          _deviceDetail = await getDetails(_request, _deviceId);
+
+          await Future.delayed(Duration(milliseconds: 750));
+          numRetries++;
+        }
+        setPerformingAction = false;
+      } catch (error) {
+        setErrorState = true;
+        Future.delayed(Duration(seconds: 1)).then((_) => setErrorState = false);
       }
-
-      setPerformingAction = false;
     }
   }
 
@@ -74,16 +78,29 @@ class PowerTraitProvider extends ChangeNotifier {
   }
 
   set setLoading(bool newIsLoading) {
+    _isPerformingAction = false;
     _isLoading = newIsLoading;
+    _isInErrorState = false;
     notifyListeners();
   }
 
   bool get isLoading => _isLoading;
 
   set setPerformingAction(bool newIsPerformingAction) {
     _isPerformingAction = newIsPerformingAction;
+    _isLoading = false;
+    _isInErrorState = false;
     notifyListeners();
   }
 
   bool get isPerformingAction => _isPerformingAction;
+
+  set setErrorState(bool newIsError) {
+    _isInErrorState = newIsError;
+    _isLoading = false;
+    _isPerformingAction = false;
+    notifyListeners();
+  }
+
+  bool get isInErrorState => _isInErrorState;
 }
diff --git a/lib/traits/trait_based_device_screen.dart b/lib/traits/trait_based_device_screen.dart
@@ -118,12 +118,14 @@ class PowerTraitWidget extends StatelessWidget {
         if (powerDeviceNotifier.isLoading ||
             powerDeviceNotifier.isPerformingAction) {
           return CircularProgressIndicator();
+        } else if (powerDeviceNotifier.isInErrorState) {
+          return Icon(Icons.error);
         } else {
           return Switch(
             value: powerDeviceNotifier.getPowerTrait()?.state.value ?? false,
-            onChanged: (bool newValue) {
-              print("Power Switch value set to: ${newValue}");
-              powerDeviceNotifier.sendPowerOnOffAction(newValue);
+            onChanged: (bool onOff) {
+              print("Power switch value set to: ${onOff}");
+              powerDeviceNotifier.sendPowerOnOffAction(onOff);
             },
           );
         }