mikekamornikov / UpgradeSpec

src/Cache/Adapter/AdapterTrait.php

Doc Comments +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
      * @param iterable $keys    a list of keys that can obtained in a single operation
      * @param mixed    $default default value to return for keys that do not exist
      *
-     * @return iterable A list of key => value pairs. Cache keys that do not exist or are stale will have $default as value.
+     * @return \Generator A list of key => value pairs. Cache keys that do not exist or are stale will have $default as value.
      */
     public function getMultiple($keys, $default = null)
     {
@@ -88,7 +88,7 @@ 
     }
 
     /**
-     * @param $key
+     * @param string $key
      *
      * @throws InvalidArgumentException
      */

src/Cache/Adapter/File.php

Doc Comments +3 added lines, -3 removed lines

@@ -20,7 +20,7 @@ 
     /**
      * File constructor.
      *
-     * @param $path
+     * @param string $path
      * @param $ttl
      */
     public function __construct($path, $ttl = 3600)
@@ -35,7 +35,7 @@ 
 
     /**
      * @param string $key
-     * @param null   $default
+     * @param File   $default
      *
      * @return bool|mixed|null
      */
@@ -212,7 +212,7 @@ 
     }
 
     /**
-     * @return Generator
+     * @return \RecursiveIteratorIterator
      */
     private function getCacheFolderIterator()
     {

src/Cache/Cache.php

Doc Comments +3 added lines, -3 removed lines

@@ -23,7 +23,7 @@ 
     }
 
     /**
-     * @param $key
+     * @param string $key
      *
      * @return bool
      */
@@ -37,7 +37,7 @@ 
     }
 
     /**
-     * @param $key
+     * @param string $key
      * @param null $default
      *
      * @return mixed
@@ -56,7 +56,7 @@ 
     }
 
     /**
-     * @param $key
+     * @param string $key
      * @param $value
      * @param null $ttl
      *

src/Command/GenerateSpecCommand.php

Doc Comments +1 added lines, -1 removed lines

@@ -124,7 +124,7 @@
     }
 
     /**
-     * @param $buildVersion
+     * @param string $buildVersion
      * @param $upgradeTo
      */
     private function validateVersion($buildVersion, $upgradeTo)

src/Data/Manager.php

Doc Comments +3 added lines, -3 removed lines

@@ -43,7 +43,7 @@ 
      *
      * Examples: 7.6.1 -> 7.6.1.0, 7.7 -> 7.7.1, 7.8 -> 7.8.0.0
      *
-     * @param $flav
+     * @param string $flav
      * @param null $baseVersion
      *
      * @return mixed
@@ -99,7 +99,7 @@ 
     /**
      * Gets all required information to perform health check.
      *
-     * @param $version
+     * @param string $version
      *
      * @return mixed
      */
@@ -111,7 +111,7 @@ 
     /**
      * Gets all required information to perform upgrade.
      *
-     * @param $version
+     * @param string $version
      *
      * @return mixed
      */

Spacing +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
          * all versions with $baseVersion base
          * for example: 7.6.1 -> [7.6.1.0, 7.6.1.1], 7.6 -> [7.6.1.0, 7.6.1.1, 7.6.2].
          */
-        $minors = array_filter($versions, function ($minor) use ($baseVersion) {
+        $minors = array_filter($versions, function($minor) use ($baseVersion) {
             return implode('.', array_slice(explode('.', $minor), 0, count(explode('.', $baseVersion)))) === $baseVersion;
         });
 

src/Data/Provider/Memory.php

Doc Comments +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
     /**
      * Gets data from memory by key.
      *
-     * @param $key
+     * @param string $key
      * @param null $default
      *
      * @return mixed|null

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
      */
     public function getReleaseNotes($flav, array $versions)
     {
-        return array_map(function ($version) use ($flav) {
+        return array_map(function($version) use ($flav) {
             return $this->get(sprintf('%s_%s_release_notes', $flav, $version), '');
         }, $versions);
     }

src/Data/Provider/SupportSugarcrm.php

Doc Comments +5 added lines, -5 removed lines

@@ -222,7 +222,7 @@ 
 
     /**
      * Returns the result (response body) of GET request
-     * @param $url
+     * @param string $url
      * @return string
      */
     private function getContent($url)
@@ -233,7 +233,7 @@ 
     }
 
     /**
-     * @param $version
+     * @param string $version
      *
      * @return string
      */
@@ -243,7 +243,7 @@ 
     }
 
     /**
-     * @param $version
+     * @param string $version
      *
      * @return string
      */
@@ -313,7 +313,7 @@ 
      *
      * @param $keyParts
      *
-     * @return mixed
+     * @return string
      */
     private function getCacheKey(array $keyParts)
     {
@@ -327,7 +327,7 @@ 
     /**
      * Lightweight HTML purifier.
      *
-     * @param $html
+     * @param string $html
      * @param string $url
      * @param array  $options
      *

Spacing +8 added lines, -8 removed lines

@@ -73,7 +73,7 @@ 
         }
 
         // sort versions (ASC)
-        usort($versions, function ($v1, $v2) {
+        usort($versions, function($v1, $v2) {
             return version_compare($v1, $v2, '<') ? -1 : (version_compare($v1, $v2, '>') ? 1 : 0);
         });
 
@@ -90,13 +90,13 @@ 
      */
     public function getReleaseNotes($flav, array $versions)
     {
-        $newVersions = array_filter($versions, function ($version) use ($flav) {
+        $newVersions = array_filter($versions, function($version) use ($flav) {
             return !$this->cache->has($this->getCacheKey([$flav, 'release_notes', $version]));
         });
 
-        $requests = function () use ($flav, $newVersions) {
+        $requests = function() use ($flav, $newVersions) {
             foreach ($newVersions as $version) {
-                yield $version => function () use ($flav, $version) {
+                yield $version => function() use ($flav, $version) {
                     return $this->httpClient->getAsync($this->getReleaseNotesUri($flav, $version));
                 };
             }
@@ -104,7 +104,7 @@ 
 
         if ($newVersions) {
             $this->processRequestPool($requests, [
-                'fulfilled' => function (ResponseInterface $response, $version) use ($flav) {
+                'fulfilled' => function(ResponseInterface $response, $version) use ($flav) {
                     $html = $response->getBody()->getContents();
                     $crawler = new Crawler($this->purifyHtml($html, $this->getReleaseNotesUri($flav, $version)));
 
@@ -134,7 +134,7 @@ 
 
                     $this->cache->set($this->getCacheKey([$flav, 'release_notes', $version]), $releaseNote);
                 },
-                'rejected' => function ($reason, $version) {
+                'rejected' => function($reason, $version) {
                     throw new \RuntimeException(
                         sprintf('Can\'t get release notes for version: %s (reason: %s)', $version, $reason)
                     );
@@ -150,7 +150,7 @@ 
             }
         }
 
-        uksort($releaseNotes, function ($v1, $v2) {
+        uksort($releaseNotes, function($v1, $v2) {
             return version_compare($v1, $v2, '<') ? -1 : (version_compare($v1, $v2, '>') ? 1 : 0);
         });
 
@@ -319,7 +319,7 @@ 
     {
         $delimiter = '___';
 
-        return implode($delimiter, array_map(function ($key) {
+        return implode($delimiter, array_map(function($key) {
             return preg_replace('/[^a-zA-Z0-9_\.]+/', '', mb_strtolower($key));
         }, $keyParts));
     }

src/Element/Section/UpgradeChanges.php

Doc Comments +5 added lines, -6 removed lines

@@ -54,8 +54,7 @@ 
     /**
      * Get all available SugarCRM versions (sorted ASC).
      *
-     * @param mixed $flav
-     * @param mixed $pathToUpgradePackage
+     * @param Context $pathToUpgradePackage
      *
      * @return mixed
      */
@@ -75,7 +74,7 @@ 
      *
      * @param string $pathToUpgradePackage
      *
-     * @return mixed
+     * @return string
      */
     public function getListOfDeletedFiles($pathToUpgradePackage)
     {
@@ -87,7 +86,7 @@ 
      *
      * @param string $pathToUpgradePackage
      *
-     * @return mixed
+     * @return string
      */
     public function getListOfModifiedFiles($pathToUpgradePackage)
     {
@@ -99,7 +98,7 @@ 
      *
      * @param string $pathToUpgradePackage
      *
-     * @return mixed
+     * @return string
      */
     public function getListOfScripts($pathToUpgradePackage)
     {
@@ -111,7 +110,7 @@ 
      *
      * @param string $pathToUpgradePackage
      *
-     * @return mixed
+     * @return string
      */
     public function getListOfSQLScripts($pathToUpgradePackage)
     {

src/Updater/Adapter/HumbugAdapter.php

Doc Comments +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
     }
 
     /**
-     * @param $stability
+     * @param string $stability
      *
      * @return string
      */