simplify reactive declaration of patch fetching

Signed-off-by: Thomas Scholtes <[email protected]>

Author: geigerzaehler

ui/App/ProjectScreen/Patch.svelte

@@ -31,10 +31,6 @@
   export let view: "commits" | "discussion";
   export let patchCount: number;
 
-  let previousProject = project;
-  let previousId = id;
-  let previousPeerId = peerId;
-
   type PatchStatus =
     | { type: "loading" }
     | { type: "notReplicated" }
@@ -46,27 +42,17 @@
 
   let patchStatus: PatchStatus = { type: "loading" };
 
+  $: projectUrn = project.urn;
   $: {
-    // Show loading indicator if we wanna show a different patch.
-    if (
-      project.urn !== previousProject.urn ||
-      previousId !== id ||
-      previousPeerId !== peerId
-    ) {
-      patchStatus = { type: "loading" };
-      previousProject = project;
-      previousId = id;
-      previousPeerId = peerId;
-    }
-    fetch(project, peerId, id);
+    projectUrn;
+    id;
+    peerId;
+    patchStatus = { type: "loading" };
+    fetch();
   }
 
   const fetchExecutor = mutexExecutor.create();
-  async function fetch(
-    project: Project,
-    peerId: string,
-    id: string
-  ): Promise<void> {
+  async function fetch(): Promise<void> {
     try {
       const result = await fetchExecutor.run(async () => {
         return await patch.getDetails(project, peerId, id);
@@ -90,7 +76,7 @@
   function watchPatchUpdates(): () => void {
     return localPeer.projectEvents.onValue(event => {
       if (event.urn.startsWith(project.urn)) {
-        fetch(project, peerId, id);
+        fetch();
       }
     });
   }