fix: too many changes to display

Vinzent03 · Aug 24, 2022 · c4bf4eb · c4bf4eb
1 parent 1c38b91
commit c4bf4eb
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 11 deletions.
diff --git a/src/main.ts b/src/main.ts
@@ -284,6 +284,10 @@ export default class ObsidianGit extends Plugin {
 
                 const status = await this.gitManager.status();
                 this.setState(PluginState.idle);
+                if (status.changed.length + status.staged.length > 500) {
+                    this.displayError("Too many changes to display");
+                    return;
+                }
 
                 new ChangedFilesModal(this, status.changed).open();
             }

diff --git a/src/ui/sidebar/gitView.svelte b/src/ui/sidebar/gitView.svelte
@@ -84,14 +84,24 @@
       };
     }
     if (status) {
-      changeHierarchy = {
-        title: "",
-        children: plugin.gitManager.getTreeStructure(status.changed),
-      };
-      stagedHierarchy = {
-        title: "",
-        children: plugin.gitManager.getTreeStructure(status.staged),
-      };
+      if (status.changed.length + status.staged.length > 500) {
+        status = undefined;
+        if (!plugin.loading) {
+          plugin.displayError("Too many changes to display");
+        }
+      } else {
+        changeHierarchy = {
+          title: "",
+          children: plugin.gitManager.getTreeStructure(status.changed),
+        };
+        stagedHierarchy = {
+          title: "",
+          children: plugin.gitManager.getTreeStructure(status.staged),
+        };
+      }
+    } else {
+      changeHierarchy = undefined;
+      stagedHierarchy = undefined;
     }
     loading = plugin.loading;
   }
@@ -102,14 +112,14 @@
 
   function stageAll() {
     loading = true;
-    plugin.gitManager.stageAll({status: status}).finally(triggerRefresh);
+    plugin.gitManager.stageAll({ status: status }).finally(triggerRefresh);
   }
 
   function unstageAll() {
     loading = true;
-    plugin.gitManager.unstageAll({status:status}).finally(triggerRefresh);
+    plugin.gitManager.unstageAll({ status: status }).finally(triggerRefresh);
   }
-  
+
   function push() {
     loading = true;