Merge pull request #223 from isyndicate/release-task-ownership-fix-042915

resultFetch is always false when owner is specified in config
10 years ago · bbefc65888
parent 5bc6a0d1cd a3219e8f68
commit bbefc65888
1 changed files with 7 additions and 9 deletions
--- a/Mage/Task/BuiltIn/Deployment/ReleaseTask.php
+++ b/Mage/Task/BuiltIn/Deployment/ReleaseTask.php
@ -70,18 +70,16 @@ class ReleaseTask extends AbstractTask implements IsReleaseAware, SkipOnOverride
                    if (!empty($infoArray[3])) {
                        $group = $infoArray[3];
                    }
-                    $userGroup = $user . ':' . $group;
                }
+                $userGroup = $user . ':' . $group;
            }

-            if ($resultFetch && $userGroup != '') {
-                $command = 'chown -R ' . $userGroup . ' ' . $currentCopy
-                    . ' && '
-                    . 'chown ' . $userGroup . ' ' . $releasesDirectory;
-                $result = $this->runCommandRemote($command);
-                if (!$result) {
-                    return $result;
-                }
+            $command = 'chown -R ' . $userGroup . ' ' . $currentCopy
+                . ' && '
+                . 'chown ' . $userGroup . ' ' . $releasesDirectory;
+            $result = $this->runCommandRemote($command);
+            if (!$result) {
+                return $result;
            }

            // Switch symlink and change owner