Merge pull request #193 from ramunasd/release-chown

changed release actions order

Mage/Task/BuiltIn/Deployment/ReleaseTask.php

@@ -74,21 +74,21 @@ class ReleaseTask extends AbstractTask implements IsReleaseAware, SkipOnOverride
                 }
             }
 
-            // Remove symlink if exists; create new symlink and change owners
-            $tmplink = $currentCopy . '.tmp';
-            $command = 'ln -sfn ' . $currentCopy . ' ' . $tmplink
-                . ' && '
-                . 'mv -T ' . $tmplink . ' ' . $symlink;
-
-            if ($resultFetch && $userGroup != '') {
-                $command .= ' && '
-                    . 'chown -h ' . $userGroup . ' ' . $symlink
+            if ($resultFetch && $userGroup != '') {  
+                $command = 'chown -h ' . $userGroup . ' ' . $symlink
                     . ' && '
                     . 'chown -R ' . $userGroup . ' ' . $currentCopy
                     . ' && '
                     . 'chown ' . $userGroup . ' ' . $releasesDirectory;
+                $result = $this->runCommandRemote($command);
+                if (!$result) {
+                    return $result;
+                }
             }
 
+            // Remove symlink if exists; create new symlink and change owner
+            $tmplink = $currentCopy . '.tmp';
+            $command = "ln -sfn {$currentCopy} {$tmplink} && mv -fT {$tmplink} {$symlink}";
             $result = $this->runCommandRemote($command);
 
             if ($result) {