mikekamornikov / UpgradeSpec

src/DI/Compiler/ConsoleCommandPass.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
         $application = $container->findDefinition('application');
 
         $taggedCommands = $container->findTaggedServiceIds('console.command');
-        $commands = array_map(function ($id) {
+        $commands = array_map(function($id) {
             return new Reference($id);
         }, array_keys($taggedCommands));
 

src/Cache/Adapter/File.php

Doc Comments +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
 
     /**
      * @param string $key
-     * @param null   $default
+     * @param File   $default
      *
      * @return bool|mixed|null
      */

src/Formatter/FormatterInterface.php

Doc Comments +11 added lines

@@ -4,9 +4,20 @@
 
 interface FormatterInterface
 {
+    /**
+     * @param string $text
+     *
+     * @return string
+     */
     public function asTitle($text, $level = 1);
 
+    /**
+     * @return string
+     */
     public function asBody($text);
 
+    /**
+     * @return string
+     */
     public function getDelimiter();
 }

src/Template/RendererInterface.php

Doc Comments +2 added lines, -1 removed lines

@@ -7,8 +7,9 @@
     /**
      * Renders template.
      *
-     * @param $name
+     * @param string $name
      * @param $arguments
+     * @return string
      */
     public function render($name, $arguments);
 }

src/Updater/Adapter/AdapterInterface.php

Doc Comments +17 added lines

@@ -6,13 +6,30 @@
 
 interface AdapterInterface
 {
+    /**
+     * @return boolean
+     */
     public function hasUpdate();
 
+    /**
+     * @return string
+     */
     public function getOldVersion();
 
+    /**
+     * @return string
+     */
     public function getNewVersion();
 
+    /**
+     * @param string $stability
+     *
+     * @return boolean
+     */
     public function update($stability = Updater::STABILITY_ANY);
 
+    /**
+     * @return boolean
+     */
     public function rollback();
 }

src/Updater/Adapter/HumbugAdapter.php

Doc Comments +3 added lines, -3 removed lines

@@ -32,7 +32,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function getOldVersion()
     {
@@ -40,7 +40,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function getNewVersion()
     {
@@ -75,7 +75,7 @@ 
     }
 
     /**
-     * @param $stability
+     * @param string $stability
      *
      * @return string
      */

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));