continue instead of entire code in condition

jmsche · jmsche · commit 989e725fe107 · 2023-08-23T21:19:28.000+02:00
diff --git a/src/Symfony/Component/AssetMapper/ImportMap/ImportMapManager.php b/src/Symfony/Component/AssetMapper/ImportMap/ImportMapManager.php
@@ -120,18 +120,20 @@ public function downloadMissingPackages(): array
         $packagesToDownload = [];
 
         foreach ($entries as $entry) {
-            if ($entry->isDownloaded && !$this->assetMapper->getAsset($entry->path)) {
-                $parts = self::parsePackageName($entry->url);
-
-                $packagesToDownload[] = new PackageRequireOptions(
-                    $parts['package'],
-                    $parts['version'] ?? throw new RuntimeException(sprintf('Cannot get a version for the "%s" package.', $parts['package'])),
-                    true,
-                    $entry->preload,
-                    $parts['alias'] ?? $parts['package'],
-                    isset($parts['registry']) && $parts['registry'] ? $parts['registry'] : null,
-                );
+            if (!$entry->isDownloaded || $this->assetMapper->getAsset($entry->path)) {
+                continue;
             }
+
+            $parts = self::parsePackageName($entry->url);
+
+            $packagesToDownload[] = new PackageRequireOptions(
+                $parts['package'],
+                $parts['version'] ?? throw new RuntimeException(sprintf('Cannot get a version for the "%s" package.', $parts['package'])),
+                true,
+                $entry->preload,
+                $parts['alias'] ?? $parts['package'],
+                isset($parts['registry']) && $parts['registry'] ? $parts['registry'] : null,
+            );
         }
 
         return $this->require($packagesToDownload);
