fix(fetch): fetch default icons only when fetcher is called

pkgforge · Oct 20, 2024 · fdefcd5 · fdefcd5
1 parent d37bad0
commit fdefcd5
Showing 1 changed file with 13 additions and 13 deletions.
diff --git a/src/registry/mod.rs b/src/registry/mod.rs
@@ -73,6 +73,19 @@ impl PackageRegistry {
                 }
             };
             storage.add_repository(&repo.name, packages);
+        }
+
+        Ok(())
+    }
+
+    pub async fn fetch(&mut self) -> Result<()> {
+        for repo in &CONFIG.repositories {
+            let content = self.fetcher.execute(repo).await?;
+
+            let mut de = rmp_serde::Deserializer::new(&content[..]);
+            let packages = RepositoryPackages::deserialize(&mut de)?;
+
+            self.storage.add_repository(&repo.name, packages);
 
             // fetch default icons
             let icon_futures: Vec<_> = repo
@@ -104,19 +117,6 @@ impl PackageRegistry {
         Ok(())
     }
 
-    pub async fn fetch(&mut self) -> Result<()> {
-        for repo in &CONFIG.repositories {
-            let content = self.fetcher.execute(repo).await?;
-
-            let mut de = rmp_serde::Deserializer::new(&content[..]);
-            let packages = RepositoryPackages::deserialize(&mut de)?;
-
-            self.storage.add_repository(&repo.name, packages);
-        }
-
-        Ok(())
-    }
-
     pub async fn install_packages(
         &self,
         package_names: &[String],