feat: Handle origin to internal link

Author: marienfressinaud

src/models/Origin.php

@@ -12,7 +12,7 @@ class Origin
 {
     public readonly string $value;
 
-    public readonly User|Collection|null $model;
+    public readonly User|Link|Collection|null $model;
 
     public readonly string $label;
 
@@ -29,12 +29,14 @@ public function __construct(string $value)
     /**
      * Return the model (User or Collection) matching with the value if any.
      */
-    private function modelFromValue(): User|Collection|null
+    private function modelFromValue(): User|Link|Collection|null
     {
         list($origin_type, $origin_id) = utils\OriginHelper::extractFromPath($this->value);
 
         if ($origin_type === 'user' && $origin_id) {
             return User::find($origin_id);
+        } elseif ($origin_type === 'link' && $origin_id) {
+            return Link::find($origin_id);
         } elseif ($origin_type === 'collection' && $origin_id) {
             return Collection::find($origin_id);
         } else {
@@ -59,6 +61,8 @@ private function labelFromValue(): string
 
         if ($this->model instanceof User) {
             return $this->model->username;
+        } elseif ($this->model instanceof Link) {
+            return $this->model->title;
         } elseif ($this->model instanceof Collection) {
             return $this->model->name();
         } else {
@@ -77,6 +81,8 @@ private function urlFromValue(): string
 
         if ($this->model instanceof User) {
             return \Minz\Url::absoluteFor('profile', ['id' => $this->model->id]);
+        } elseif ($this->model instanceof Link) {
+            return \Minz\Url::absoluteFor('link', ['id' => $this->model->id]);
         } elseif ($this->model instanceof Collection) {
             return \Minz\Url::absoluteFor('collection', ['id' => $this->model->id]);
         } else {

src/utils/OriginHelper.php

@@ -17,13 +17,15 @@ class OriginHelper
      *
      * - For the path `/collections/1234567890`, ['collection', '1234567890']
      *   will be returned (if the collection exists in db)
-     * - For the path `/p/1234567890`, ['user', '1234567890'] will be
-     *   returned (if the user exists in db)
+     * - For the path `/links/1234567890`, ['link', '1234567890'] will be
+     *   returned (if the link exists in db)
+     * - For the path `/p/1234567890`, ['user', '1234567890'] will be returned
+     *   (if the user exists in db)
      * - For other paths, ['', null] will be returned
      *
      * The method also handles URLs starting with the base URL of the application.
      *
-     * @return array{'collection'|'user', string}|array{'', null}
+     * @return array{'collection'|'link'|'user', string}|array{'', null}
      * }
      */
     public static function extractFromPath(string $url_or_path): array
@@ -54,6 +56,17 @@ public static function extractFromPath(string $url_or_path): array
             return ['collection', $collection_id];
         }
 
+        $result = preg_match('#^/links/(?P<id>\d+)$#', $path, $matches);
+        if (isset($matches['id'])) {
+            $link_id = $matches['id'];
+
+            if (!models\Link::exists($link_id)) {
+                return ['', null];
+            }
+
+            return ['link', $link_id];
+        }
+
         $result = preg_match('#^/p/(?P<id>\d+)/?#', $path, $matches);
         if (isset($matches['id'])) {
             $user_id = $matches['id'];

tests/utils/OriginHelperTest.php

@@ -3,6 +3,7 @@
 namespace App\utils;
 
 use tests\factories\CollectionFactory;
+use tests\factories\LinkFactory;
 use tests\factories\UserFactory;
 
 class OriginHelperTest extends \PHPUnit\Framework\TestCase
@@ -43,6 +44,17 @@ public function testExtractOriginFromPathWithExistingUser(): void
         $this->assertSame($user->id, $origin_id);
     }
 
+    public function testExtractOriginFromPathWithExistingLink(): void
+    {
+        $link = LinkFactory::create();
+        $path = \Minz\Url::for('link', ['id' => $link->id]);
+
+        list($origin_type, $origin_id) = OriginHelper::extractFromPath($path);
+
+        $this->assertSame('link', $origin_type);
+        $this->assertSame($link->id, $origin_id);
+    }
+
     public function testExtractOriginFromPathWithNonExistingCollection(): void
     {
         $path = \Minz\Url::for('collection', ['id' => '12345']);
@@ -63,6 +75,16 @@ public function testExtractOriginFromPathWithNonExistingUser(): void
         $this->assertNull($origin_id);
     }
 
+    public function testExtractOriginFromPathWithNonExistingLink(): void
+    {
+        $path = \Minz\Url::for('link', ['id' => '12345']);
+
+        list($origin_type, $origin_id) = OriginHelper::extractFromPath($path);
+
+        $this->assertSame('', $origin_type);
+        $this->assertNull($origin_id);
+    }
+
     public function testExtractOriginFromPathWithUnsupportedPath(): void
     {
         $path = \Minz\Url::for('bookmarks');