Fix filesystem path creation for nested dirs

src/Image.php

@@ -38,6 +38,9 @@ public function __construct(bool $friendly_url, string $data_dir, string $data_p
 		$this->identifier = $identifier;
 		$this->friendly_url = $friendly_url;
 
+		if (stripos($this->identifier, '/') === 0)
+			$this->identifier = substr($this->identifier, 1);
+
 		if ($props === null)
 			$props = [];
 
@@ -48,34 +51,29 @@ public function __construct(bool $friendly_url, string $data_dir, string $data_p
 		}
 	}
 
-
-	public function getPath(): Traversable
+	public function getPath(): string
 	{
-		return implode('/', [dirname($this->data_path), $this->createLink()]);
+		return implode('/', [$this->data_path, $this->identifier]);
 	}
 
-
 	public function __toString(): string
 	{
 		return $this->identifier;
 	}
 
-
 	public function getQuery(): string
 	{
 		return $this->script->toQuery();
 	}
 
-
-	public function createLink(): Traversable
+	public function createLink(): string
 	{
 		if ($this->friendly_url) {
 			return implode('/', [$this->data_dir, $this->getScript()->toQuery()]);
 		}
 		return implode('/', [$this->data_dir, $this->identifier]);
 	}
 
-
 	public function getScript(): ImageNameScript
 	{
 		return $this->script ?: ImageNameScript::fromIdentifier($this->identifier);

tests/cases/ImageTest.php

@@ -0,0 +1,40 @@
+<?php declare(strict_types = 1);
+
+namespace Tests\Cases;
+
+use Contributte\ImageStorage\Image;
+use Ninjify\Nunjuck\TestCase\BaseTestCase;
+use Tester\Assert;
+
+require __DIR__ . '/../bootstrap.php';
+
+final class ImageTest extends BaseTestCase
+{
+
+	public function testGetPath(): void
+	{
+		$image = new Image(false, '', '/data', '/namespace/47/img.jpg');
+		Assert::equal('/data/namespace/47/img.jpg', $image->getPath());
+	}
+
+	public function testGetPathNested(): void
+	{
+		$image = new Image(false, '', '/data/images', 'namespace/47/img.jpg');
+		Assert::equal('/data/images/namespace/47/img.jpg', $image->getPath());
+	}
+
+	public function testCreateLink(): void
+	{
+		$image = new Image(false, 'data', '', 'namespace/47/img.jpg');
+		Assert::equal('data/namespace/47/img.jpg', $image->createLink());
+	}
+
+	public function testCreateLinkNested(): void
+	{
+		$image = new Image(false, 'data/images', '', 'namespace/47/img.jpg');
+		Assert::equal('data/images/namespace/47/img.jpg', $image->createLink());
+	}
+
+}
+
+(new ImageTest())->run();