PSR-2 style.

Mage/Task/BuiltIn/Filesystem/LinkSharedFilesTask.php

@@ -5,21 +5,44 @@ use Mage\Task\AbstractTask;
 use Mage\Task\Releases\IsReleaseAware;
 use Mage\Task\SkipException;
 
+/**
+ * Class LinkSharedFilesTask
+ *
+ * @package Mage\Task\BuiltIn\Filesystem
+ * @author Andrey Kolchenko <a.j.kolchenko@baltsoftservice.ru>
+ */
 class LinkSharedFilesTask extends AbstractTask implements IsReleaseAware
 {
 
-    const LINKED_FOLDERS   = 'linked_folders';
-    const LINKED_STRATEGY  = 'linking_strategy';
+    /**
+     * Linked folders parameter name
+     */
+    const LINKED_FOLDERS = 'linked_folders';
+    /**
+     * Linking strategy parameter name
+     */
+    const LINKED_STRATEGY = 'linking_strategy';
 
+    /**
+     * Absolute linked strategy
+     */
     const ABSOLUTE_LINKING = 'absolute';
+    /**
+     * Relative linked strategy
+     */
     const RELATIVE_LINKING = 'relative';
 
+    /**
+     * @var array
+     */
     public $linkingStrategies = array(
         self::ABSOLUTE_LINKING,
         self::RELATIVE_LINKING
     );
+
     /**
      * Returns the Title of the Task
+     *
      * @return string
      */
     public function getName()
@@ -35,11 +58,11 @@ class LinkSharedFilesTask extends AbstractTask implements IsReleaseAware
      */
     public function run()
     {
-        $linkedFiles    = $this->getParameter('linked_files', []);
-        $linkedFolders  = $this->getParameter(self::LINKED_FOLDERS, []);
+        $linkedFiles = $this->getParameter('linked_files', []);
+        $linkedFolders = $this->getParameter(self::LINKED_FOLDERS, []);
         $linkingStrategy = $this->getParameter(self::LINKED_STRATEGY, self::ABSOLUTE_LINKING);
 
-        $linkedEntities = array_merge($linkedFiles,$linkedFolders);
+        $linkedEntities = array_merge($linkedFiles, $linkedFolders);
 
         if (sizeof($linkedFiles) == 0 && sizeof($linkedFolders) == 0) {
             throw new SkipException('No files and folders configured for sym-linking.');
@@ -51,20 +74,23 @@ class LinkSharedFilesTask extends AbstractTask implements IsReleaseAware
         $releasesDirectoryPath = rtrim($this->getConfig()->deployment('to'), '/') . '/' . $releasesDirectory;
 
         $currentCopy = $releasesDirectoryPath . '/' . $this->getConfig()->getReleaseId();
-        $relativeDiffPath = str_replace($this->getConfig()->deployment('to'),'',$currentCopy) . '/';
+        $relativeDiffPath = str_replace($this->getConfig()->deployment('to'), '', $currentCopy) . '/';
 
         foreach ($linkedEntities as $ePath) {
-            if(is_array($ePath) && in_array($strategy = reset($ePath), $this->linkingStrategies ) ) {
+            if (is_array($ePath) && in_array($strategy = reset($ePath), $this->linkingStrategies)) {
                 $entityPath = key($ePath);
             } else {
                 $strategy = $linkingStrategy;
                 $entityPath = $ePath;
             }
             $sharedEntityLinkedPath = "$sharedFolderPath/$entityPath";
-            if($strategy==self::RELATIVE_LINKING) {
+            if ($strategy == self::RELATIVE_LINKING) {
                 $parentFolderPath = dirname($entityPath);
-                $relativePath = $parentFolderPath=='.'?$relativeDiffPath:$relativeDiffPath.$parentFolderPath.'/';
-                $sharedEntityLinkedPath = ltrim(preg_replace('/(\w+\/)/', '../', $relativePath),'/').$sharedFolderName .'/'. $entityPath;
+                $relativePath = $parentFolderPath == '.' ? $relativeDiffPath : $relativeDiffPath . $parentFolderPath . '/';
+                $sharedEntityLinkedPath = ltrim(
+                        preg_replace('/(\w+\/)/', '../', $relativePath),
+                        '/'
+                    ) . $sharedFolderName . '/' . $entityPath;
             }
             $command = "ln -nfs $sharedEntityLinkedPath $currentCopy/$entityPath";
             $this->runCommandRemote($command);
@@ -72,4 +98,4 @@ class LinkSharedFilesTask extends AbstractTask implements IsReleaseAware
 
         return true;
     }
-}
+}