1.0.3 (view changelog)

wow-apps · Dec 21, 2017 · 565d34c · 565d34c
1 parent 7061418
commit 565d34c
Showing 1 changed file with 60 additions and 49 deletions.
diff --git a/Service/Packagist.php b/Service/Packagist.php
@@ -265,60 +265,71 @@ private function addPackageVersions(Package &$package, array $packageArray)
                     continue;
                 }
 
-                $packageVersion = new PackageVersion();
-
-                $packageVersion
-                    ->setName($version['name'] ?? '')
-                    ->setDescription($version['description'] ?? '')
-                    ->setKeywords($version['keywords'] ?? [])
-                    ->setHomepage($version['homepage'] ?? '')
-                    ->setVersion($version['version'])
-                    ->setVersionNormalized($version['version_normalized'] ?? '')
-                    ->setLicense($version['license'][0] ?? '')
-                    ->setAuthors(new \ArrayObject())
-                    ->setSource(
-                        new PackageSource(
-                            $version['source']['type'] ?? '',
-                            $version['source']['url'] ?? '',
-                            $version['source']['reference'] ?? ''
-                        )
-                    )
-                    ->setDist(
-                        new PackageDist(
-                            $version['dist']['type'] ?? '',
-                            $version['dist']['url'] ?? '',
-                            $version['dist']['reference'] ?? '',
-                            $version['dist']['shasum'] ?? ''
-                        )
-                    )
-                    ->setType($version['type'] ?? '')
-                    ->setTime($version['time'] ?? '')
-                    ->setAutoload($version['autoload'] ?? [])
-                    ->setRequire(new \ArrayObject())
-                ;
+                $packageVersion = $this->createPackageVersion($version);
 
-                if (!empty($version['authors'])) {
-                    foreach ($version['authors'] as $author) {
-                        $packageVersion->getAuthors()->append(
-                            new PackageAuthor(
-                                $author['name'] ?? '',
-                                $author['email'] ?? '',
-                                $author['homepage'] ?? '',
-                                $author['role'] ?? ''
-                            )
-                        );
-                    }
-                }
+                $package->getVersions()->offsetSet($packageVersion->getVersion(), $packageVersion);
+            }
+        }
+    }
 
-                if (!empty($version['require'])) {
-                    foreach ($version['require'] as $name => $ver) {
-                        $packageVersion->getRequire()->append(new PackageDependency($name, $ver));
-                    }
-                }
+    /**
+     * @param array $version
+     * @return PackageVersion
+     */
+    private function createPackageVersion(array  $version): PackageVersion
+    {
+        $packageVersion = new PackageVersion();
+
+        $packageVersion
+            ->setName($version['name'] ?? '')
+            ->setDescription($version['description'] ?? '')
+            ->setKeywords($version['keywords'] ?? [])
+            ->setHomepage($version['homepage'] ?? '')
+            ->setVersion($version['version'])
+            ->setVersionNormalized($version['version_normalized'] ?? '')
+            ->setLicense($version['license'][0] ?? '')
+            ->setAuthors(new \ArrayObject())
+            ->setSource(
+                new PackageSource(
+                    $version['source']['type'] ?? '',
+                    $version['source']['url'] ?? '',
+                    $version['source']['reference'] ?? ''
+                )
+            )
+            ->setDist(
+                new PackageDist(
+                    $version['dist']['type'] ?? '',
+                    $version['dist']['url'] ?? '',
+                    $version['dist']['reference'] ?? '',
+                    $version['dist']['shasum'] ?? ''
+                )
+            )
+            ->setType($version['type'] ?? '')
+            ->setTime($version['time'] ?? '')
+            ->setAutoload($version['autoload'] ?? [])
+            ->setRequire(new \ArrayObject())
+        ;
 
-                $package->getVersions()->offsetSet($packageVersion->getVersion(), $packageVersion);
+        if (!empty($version['authors'])) {
+            foreach ($version['authors'] as $author) {
+                $packageVersion->getAuthors()->append(
+                    new PackageAuthor(
+                        $author['name'] ?? '',
+                        $author['email'] ?? '',
+                        $author['homepage'] ?? '',
+                        $author['role'] ?? ''
+                    )
+                );
+            }
+        }
+
+        if (!empty($version['require'])) {
+            foreach ($version['require'] as $name => $ver) {
+                $packageVersion->getRequire()->append(new PackageDependency($name, $ver));
             }
         }
+
+        return $packageVersion;
     }
 
     /**