mikekamornikov / UpgradeSpec

src/Version/Graph/Dijkstra.php

Doc Comments +6 added lines, -2 removed lines

@@ -72,6 +72,10 @@ 
         return $pairs;
     }
 
+    /**
+     * @param string $source
+     * @param string $target
+     */
     private function getShortestPath($source, $target)
     {
         // array of the best estimates of shortest path to each vertex
@@ -134,7 +138,7 @@ 
     /**
      * @param Version $from
      *
-     * @return OrderedList
+     * @return \Sugarcrm\UpgradeSpec\Version\OrderedList
      */
     private function getOriginVertexes(Version $from)
     {
@@ -152,7 +156,7 @@ 
     /**
      * @param Version $to
      *
-     * @return OrderedList
+     * @return \Sugarcrm\UpgradeSpec\Version\OrderedList
      */
     private function getDestinationVertexes(Version $to)
     {

Spacing +3 added lines, -3 removed lines

@@ -19,7 +19,7 @@ 
     public function __construct(AdjacencyList $adj)
     {
         foreach ($adj as $vertex => $list) {
-            $list = array_map(function (Version $version) use ($vertex) {
+            $list = array_map(function(Version $version) use ($vertex) {
                 return [(string) $version => (new Version($vertex))->isChildOf($version) ? 0 : 1];
             }, iterator_to_array($list));
 
@@ -144,7 +144,7 @@ 
             $aliases = $aliases->merge($from->getAliases());
         }
 
-        return $aliases->filter(function (Version $version) {
+        return $aliases->filter(function(Version $version) {
             return isset($this->graph[(string) $version]);
         });
     }
@@ -156,7 +156,7 @@ 
      */
     private function getDestinationVertexes(Version $to)
     {
-        return $to->getAliases()->filter(function (Version $version) {
+        return $to->getAliases()->filter(function(Version $version) {
             return isset($this->graph[(string) $version]);
         });
     }

src/Data/Provider/SourceCode/LocalUpgradePackages.php

Spacing +4 added lines, -4 removed lines

@@ -81,7 +81,7 @@ 
             $deletedFiles = array_diff(array_merge($deletedFiles, $packageDeletedFiles), $packageModifiedFiles);
         }
 
-        $modifiedFiles = array_keys(array_filter($modifiedFiles, function ($package, $changedFile) use ($buildPath, $packageZips) {
+        $modifiedFiles = array_keys(array_filter($modifiedFiles, function($package, $changedFile) use ($buildPath, $packageZips) {
             if (($buildFile = @file_get_contents($buildPath . DS . $changedFile)) === false) {
                 return false;
             }
@@ -96,11 +96,11 @@ 
             $zip->close();
         }
 
-        $modifiedFiles = array_values(array_filter($modifiedFiles, function ($file) use ($buildPath) {
+        $modifiedFiles = array_values(array_filter($modifiedFiles, 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);
         }));
 
@@ -124,7 +124,7 @@ 
         $packages = $this->getFlavPackages($context->getBuildFlav(), $context->getTargetPath());
         $graph = new Dijkstra(new AdjacencyList($packages));
 
-        $this->suitablePackages = array_map(function (array $hop) use ($packages) {
+        $this->suitablePackages = array_map(function(array $hop) use ($packages) {
             foreach ($packages as $path => $pair) {
                 if ((new OrderedList($hop))->isEqualTo(new OrderedList($pair))) {
                     return $path;