mikekamornikov / UpgradeSpec

src/Data/Provider/LocalUpgradePackage.php

Doc Comments +6 added lines, -8 removed lines

@@ -23,9 +23,7 @@ 
     /**
      * LocalUpgradePackage constructor.
      *
-     * @param Cache         $cache
      * @param $zipExtractor
-     * @param String $pathToBuild
      */
     public function __construct()
     {
@@ -60,7 +58,7 @@ 
      *
      * @param String $pathToUpgradePackage
      *
-     * @return mixed
+     * @return string
      */
     public function getListOfDeletedFiles($pathToUpgradePackage)
     {
@@ -71,7 +69,7 @@ 
      *
      * @param String $pathToUpgradePackage
      *
-     * @return mixed
+     * @return string
      */
     public function getListOfModifiedFiles($pathToUpgradePackage)
     {
@@ -82,7 +80,7 @@ 
      *
      * @param String $pathToUpgradePackage
      *
-     * @return mixed
+     * @return string
      */
     public function getListOfScripts($pathToUpgradePackage)
     {
@@ -93,7 +91,7 @@ 
      *
      * @param String $pathToUpgradePackage
      *
-     * @return mixed
+     * @return string
      */
     public function getListOfSQLScripts($pathToUpgradePackage)
     {
@@ -164,7 +162,6 @@ 
     }
 
     /**
-     * @param string $upgradePackagePath
      * @param string version
      */
     public function setExtractedPackagePath($version)
@@ -184,7 +181,8 @@ 
 
     /**
      * Creates directory for extracting zip package
-     * @return bool
+     * @param string $extractedPackagePath
+     * @return boolean|null
      */
     private function initPackageDirectory($extractedPackagePath)
     {

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace Sugarcrm\UpgradeSpec\Data\Provider;
 
 use Symfony\Component\Finder\Finder;
-
 use Symfony\Component\Filesystem\Filesystem;
 use Symfony\Component\Filesystem\Exception\IOExceptionInterface;
 use ZipArchive;

Spacing +1 added lines, -1 removed lines

@@ -201,7 +201,7 @@
             $fs->mkdir($extractedPackagePath);
             return true;
         } catch (IOExceptionInterface $e) {
-            echo "An error occurred while creating upgrade package directory ".$e->getPath();
+            echo "An error occurred while creating upgrade package directory " . $e->getPath();
         }
 
     }

Braces +3 added lines, -2 removed lines

@@ -128,8 +128,9 @@
         $finder->files()->in($needstoBeUpgradedPath);
         foreach ($finder as $file) {
             $fileName = $file->getRelativePathname();
-            if (!file_exists($installedBuildPath . $fileName))
-                $filesLIst[] = $fileName;
+            if (!file_exists($installedBuildPath . $fileName)) {
+                            $filesLIst[] = $fileName;
+            }
         }
         return $filesLIst;
     }