feat: expose Loading state when either retrieving detail or performin…

…g action in Provider
Yonomi · May 20, 2021 · cd33d76 · cd33d76
1 parent b67000c
commit cd33d76
Show file tree

Hide file tree

Showing 2 changed files with 37 additions and 9 deletions.
diff --git a/lib/components/lock_widget.dart b/lib/components/lock_widget.dart
@@ -10,7 +10,7 @@ class LockWidget extends StatelessWidget {
   Widget build(BuildContext context) {
     final lockProvider = Provider.of<LockProvider>(context, listen: true);
 
-    return lockProvider.loading
+    return lockProvider.loadingDetail
         ? Center(child: CircularProgressIndicator())
         : Column(mainAxisAlignment: MainAxisAlignment.start, children: [
             Row(
@@ -27,11 +27,7 @@ class LockWidget extends StatelessWidget {
             Center(
               child: Arc(
                 centerWidget: InkWell(
-                  child: (lockProvider.isLocked)
-                      ? DeviceItemIcon.getLockIcon(
-                          175, WidgetStyleConstants.deviceDetailIconColorActive)
-                      : DeviceItemIcon.getUnlockIcon(175,
-                          WidgetStyleConstants.deviceDetailIconColorInactive),
+                  child: getLockStateIcon(lockProvider),
                   onTap: () {
                     bool setLock = !lockProvider.isLocked;
                     lockProvider.setLockUnlockAction(
@@ -45,4 +41,14 @@ class LockWidget extends StatelessWidget {
             ),
           ]);
   }
+
+  Widget getLockStateIcon(LockProvider lockProvider) {
+    return (lockProvider.loadingDetail || lockProvider.loadingAction)
+        ? Center(child: CircularProgressIndicator())
+        : (lockProvider.isLocked)
+            ? DeviceItemIcon.getLockIcon(
+                175, WidgetStyleConstants.deviceDetailIconColorActive)
+            : DeviceItemIcon.getUnlockIcon(
+                175, WidgetStyleConstants.deviceDetailIconColorInactive);
+  }
 }
diff --git a/lib/providers/lock_provider.dart b/lib/providers/lock_provider.dart
@@ -4,7 +4,8 @@ import 'package:yonomi_platform_sdk/repository/devices/lock_repository.dart';
 import 'package:yonomi_platform_sdk/request/request.dart';
 
 class LockProvider extends ChangeNotifier {
-  bool loading = false;
+  bool loadingDetail = false;
+  bool loadingAction = false;
 
   LockProvider(Request request, String deviceId) {
     _request = request;
@@ -19,14 +20,35 @@ class LockProvider extends ChangeNotifier {
   bool get isLocked => _deviceDetail?.traits?.first?.state?.value ?? false;
 
   Future<void> getDeviceDetail(String deviceId) async {
-    loading = true;
+    loadingDetail = true;
+
+    notifyListeners();
+
     _deviceDetail = await DevicesRepository.getLockDetails(_request, deviceId);
 
-    loading = false;
+    loadingDetail = false;
+
     notifyListeners();
   }
 
   Future<void> setLockUnlockAction(String deviceId, bool setLock) async {
+    loadingAction = true;
+
+    notifyListeners();
+
     await LockRepository.sendLockUnlockAction(_request, deviceId, setLock);
+
+    var maxRetries = 0;
+    while (_deviceDetail?.traits?.first?.state?.value != setLock &&
+        maxRetries < 10) {
+      // Wait more time
+      _deviceDetail =
+          await DevicesRepository.getLockDetails(_request, deviceId);
+      await Future.delayed(Duration(milliseconds: 750));
+      maxRetries++;
+    }
+    loadingAction = false;
+
+    notifyListeners();
   }
 }