BUGFIX: Adjust path for deep symlinks

Author: simonschaufi

src/Task/Generic/CreateSymlinksTask.php

@@ -55,7 +55,9 @@ public function execute(Node $node, Application $application, Deployment $deploy
         foreach ($options['symlinks'] as $linkPath => $sourcePath) {
             // creates empty directory if path does not exist
             if ($options['createNonExistingSharedDirectories'] === true) {
-                $commands[] = sprintf('test -e %s || mkdir -p %s', $sourcePath, $sourcePath);
+                $folderDepth = substr_count($linkPath, '/');
+                $changedSourceDirectoryAccordingToDeepnessOfLinkPath = $this->changeDeepnessOfPath($sourcePath, $folderDepth);
+                $commands[] = sprintf('test -e %s || mkdir -p %s', $changedSourceDirectoryAccordingToDeepnessOfLinkPath, $changedSourceDirectoryAccordingToDeepnessOfLinkPath);
             }
 
             $commands[] = sprintf('ln -s %s %s', $sourcePath, $linkPath);
@@ -64,6 +66,15 @@ public function execute(Node $node, Application $application, Deployment $deploy
         $this->shell->executeOrSimulate($commands, $node, $deployment);
     }
 
+    protected function changeDeepnessOfPath(string $path, int $level = 0): string
+    {
+        if ($level === 0 || substr($path, 0, 1) === '/') {
+            return $path;
+        }
+        $directoryParts = explode(DIRECTORY_SEPARATOR, $path);
+        return implode(DIRECTORY_SEPARATOR, array_splice($directoryParts, $level));
+    }
+
     /**
      * @codeCoverageIgnore
      */

tests/Unit/Task/Generic/CreateSymlinksTaskTest.php

@@ -55,7 +55,7 @@ public function createsSymlinkInConfiguredBasePath(): void
     {
         $options = [
             'symlinks' => ['media' => '../media'],
-            'genericSymlinksBaseDir' => '/home/foobar/data'
+            'genericSymlinksBaseDir' => '/home/foobar/data',
         ];
         $this->task->execute($this->node, $this->application, $this->deployment, $options);
 
@@ -70,7 +70,7 @@ public function createsSymlinkInConfiguredBasePath(): void
     public function doNotTryToCreateSymlinksIfOptionIsSetToFalse(): void
     {
         $options = [
-            'createNonExistingSharedDirectories' => false
+            'createNonExistingSharedDirectories' => false,
         ];
         $this->task->execute($this->node, $this->application, $this->deployment, $options);
 
@@ -98,4 +98,45 @@ public function createsMultipleSymlinks(): void
         $this->assertCommandExecuted('ln -s ../log log');
         $this->assertCommandExecuted('ln -s ../var var');
     }
+
+    /**
+     * @test
+     */
+    public function createsMultipleDeepSymlinks(): void
+    {
+        $options = [
+            'symlinks' => [
+                'var/log' => '../../../shared/var/log',
+                'var/temp' => '/tmp',
+            ],
+        ];
+        $this->task->execute($this->node, $this->application, $this->deployment, $options);
+
+        $this->assertCommandExecuted("cd {$this->deployment->getApplicationReleasePath($this->node)}");
+        $this->assertCommandExecuted('test -e ../../shared/var/log || mkdir -p ../../shared/var/log');
+        $this->assertCommandExecuted('ln -s ../../../shared/var/log var/log');
+        $this->assertCommandExecuted('test -e /tmp || mkdir -p /tmp');
+        $this->assertCommandExecuted('ln -s /tmp var/temp');
+    }
+
+    /**
+     * @test
+     */
+    public function createsMultipleDeepSymlinksWithDifferentBaseDir(): void
+    {
+        $options = [
+            'symlinks' => [
+                'var/log' => '../../../shared/var/log',
+                'var/temp' => '/tmp',
+            ],
+            'genericSymlinksBaseDir' => '/home/foobar/data',
+        ];
+        $this->task->execute($this->node, $this->application, $this->deployment, $options);
+
+        $this->assertCommandExecuted('cd /home/foobar/data');
+        $this->assertCommandExecuted('test -e ../../shared/var/log || mkdir -p ../../shared/var/log');
+        $this->assertCommandExecuted('ln -s ../../../shared/var/log var/log');
+        $this->assertCommandExecuted('test -e /tmp || mkdir -p /tmp');
+        $this->assertCommandExecuted('ln -s /tmp var/temp');
+    }
 }