mikekamornikov / UpgradeSpec

src/Element/Section/UpgradeChanges.php

Doc Comments +7 added lines, -4 removed lines

@@ -64,6 +64,9 @@ 
         ]);
     }
 
+    /**
+     * @param string $buildPath
+     */
     private function getChangedFiles($buildPath, $packages)
     {
         $changedFiles = $deletedFiles = $packageZips = [];
@@ -164,8 +167,8 @@ 
     /**
      * Gets flav specific packages
      *
-     * @param $flav
-     * @param $packagesPath
+     * @param string $flav
+     * @param string $packagesPath
      *
      * @return array
      */
@@ -197,8 +200,8 @@ 
      * Calculates an array of possible upgrade chains
      *
      * @param $packages
-     * @param $buildVersion
-     * @param $upgradeTo
+     * @param string $buildVersion
+     * @param string $upgradeTo
      *
      * @return array
      */

Spacing +11 added lines, -11 removed lines

@@ -91,7 +91,7 @@ 
             $deletedFiles = array_diff(array_merge($deletedFiles, $packageDeletedFiles), $packageChangedFiles);
         }
 
-        $changedFiles = array_keys(array_filter($changedFiles, function ($package, $changedFile) use ($buildPath, $packageZips) {
+        $changedFiles = array_keys(array_filter($changedFiles, function($package, $changedFile) use ($buildPath, $packageZips) {
             if (($buildFile = @file_get_contents($buildPath . DS . $changedFile)) === false) {
                 return false;
             }
@@ -104,11 +104,11 @@ 
             $zip->close();
         }
 
-        $changedFiles = array_values(array_filter($changedFiles, function ($file) use ($buildPath) {
+        $changedFiles = array_values(array_filter($changedFiles, function($file) use ($buildPath) {
             return file_exists($buildPath . '/custom/' . $file);
         }));
 
-        $deletedFiles = array_values(array_filter($deletedFiles, function ($file) use ($buildPath) {
+        $deletedFiles = array_values(array_filter($deletedFiles, function($file) use ($buildPath) {
             return file_exists($buildPath . '/custom/' . $file);
         }));
 
@@ -154,7 +154,7 @@ 
             return [];
         }
 
-        usort($chains, function ($a1, $a2) {
+        usort($chains, function($a1, $a2) {
             return count($a1) < count($a2) ? -1 : (count($a1) > count($a2) ? 1 : 0);
         });
 
@@ -207,7 +207,7 @@ 
         $versionMatrix = $this->getVersionMatrix($packages);
         $allVersions = array_keys($versionMatrix);
 
-        $getExistingSubversions = function ($version) use ($allVersions) {
+        $getExistingSubversions = function($version) use ($allVersions) {
             $existingVersions = [];
 
             $fromParts = explode('.', $version);
@@ -222,7 +222,7 @@ 
         };
 
         // init chains with starting versions
-        $chains = array_map(function ($version) use ($buildVersion) {
+        $chains = array_map(function($version) use ($buildVersion) {
             return [$version => $buildVersion];
         }, $getExistingSubversions($buildVersion));
 
@@ -232,7 +232,7 @@ 
         }
 
         // gets last key of assoc array
-        $getLastKey = function ($array) {
+        $getLastKey = function($array) {
             end($array);
 
             return key($array);
@@ -281,7 +281,7 @@ 
             $chains = $fullChains;
         }
 
-        $chains = array_map(function ($chain) use ($versionMatrix) {
+        $chains = array_map(function($chain) use ($versionMatrix) {
             $keys = array_keys($chain);
             $values = array_values($chain);
 
@@ -311,17 +311,17 @@ 
      */
     private function getVersionMatrix($packages)
     {
-        $allVersions = array_unique(call_user_func_array('array_merge', array_map(function ($package) {
+        $allVersions = array_unique(call_user_func_array('array_merge', array_map(function($package) {
             return [$package['from'], $package['to']];
         }, $packages)));
 
         // sort versions (ASC)
-        usort($allVersions, function ($v1, $v2) {
+        usort($allVersions, function($v1, $v2) {
             return version_compare($v1, $v2, '<') ? -1 : (version_compare($v1, $v2, '>') ? 1 : 0);
         });
 
         // create matrix and fill it with zeros
-        $versionMatrix = call_user_func_array('array_merge',array_map(function ($version) use ($allVersions) {
+        $versionMatrix = call_user_func_array('array_merge', array_map(function($version) use ($allVersions) {
             return [$version => array_combine($allVersions, array_fill(0, count($allVersions), 0))];
         }, $allVersions));