fix: Implemented Exception Handling and solved persistent Shimmer in …

…Explorer Screen (#53)
CCExtractor · Feb 28, 2021 · 8278251 · 8278251
1 parent 0705bfd
commit 8278251
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 16 deletions.
diff --git a/lib/api/api_requests.dart b/lib/api/api_requests.dart
@@ -420,26 +420,31 @@ class ApiRequests {
 
   /// Gets Disk Files
   static Future<List<DiskFile>> getDiskFiles(Api api, String path) async {
-    var response = await api.ioClient.post(Uri.parse(api.explorerPluginUrl),
-        headers: api.getAuthHeader(),
-        body: {
-          'cmd': 'get',
-          'src': path,
-        });
+    try {
+      var response = await api.ioClient.post(Uri.parse(api.explorerPluginUrl),
+          headers: api.getAuthHeader(),
+          body: {
+            'cmd': 'get',
+            'src': path,
+          });
 
-    var files = jsonDecode(response.body)['files'];
+      var files = jsonDecode(response.body)['files'];
 
-    List<DiskFile> diskFiles = [];
+      List<DiskFile> diskFiles = [];
 
-    for (var file in files) {
-      DiskFile diskFile = DiskFile();
+      for (var file in files) {
+        DiskFile diskFile = DiskFile();
 
-      diskFile.isDirectory = file['is_dir'];
-      diskFile.name = file['data']['name'];
-      diskFiles.add(diskFile);
-    }
+        diskFile.isDirectory = file['is_dir'];
+        diskFile.name = file['data']['name'];
+        diskFiles.add(diskFile);
+      }
 
-    return diskFiles;
+      return diskFiles;
+    } on Exception catch (e) {
+      print(e.toString());
+      return null;
+    }
   }
 
   static setTorrentLabel(Api api, String hashValue, {String label}) async {

diff --git a/lib/screens/disk_explorer_screen.dart b/lib/screens/disk_explorer_screen.dart
@@ -21,7 +21,8 @@ class _DiskExplorerState extends State<DiskExplorer> {
       isLoading = true;
     });
     diskFiles = await ApiRequests.getDiskFiles(
-        Provider.of<Api>(context, listen: false), path);
+            Provider.of<Api>(context, listen: false), path) ??
+        [];
     setState(() {
       isLoading = false;
     });