[Bugfix] Add support for model bindings in route params (#506)

Check for url routable model when resolving model bindings in route parameters.

Author: benkingcode

src/Api/Url.php

@@ -103,7 +103,13 @@ public function replace(iterable $parameters): self
         $copy = clone $this;
 
         foreach ($parameters as $key => $value) {
-            $copy->namespace = \str_replace('{' . $key . '}', $value, $copy->namespace);
+            $routeParamValue = $value;
+
+            if ($value instanceof UrlRoutable) {
+              $routeParamValue = $value->getRouteKey();
+            }
+
+            $copy->namespace = \str_replace('{' . $key . '}', $routeParamValue, $copy->namespace);
         }
 
         return $copy;

tests/lib/Integration/Routing/RouteParameterTest.php

@@ -20,6 +20,7 @@
 use CloudCreativity\LaravelJsonApi\Routing\RouteRegistrar;
 use CloudCreativity\LaravelJsonApi\Tests\Integration\TestCase;
 use DummyApp\Post;
+use DummyApp\User;
 use Illuminate\Support\Arr;
 
 /**
@@ -77,4 +78,16 @@ public function testManual(): void
             Arr::get($data, 'data.links.self')
         );
     }
+
+    public function testModelBinding(): void
+    {
+        $post = factory(Post::class)->create();
+        $user = factory(User::class)->create();
+        $data = json_api(null, null, ['tenant' => $user])->encoder()->serializeData($post);
+
+        $this->assertSame(
+            url('/foo/2/api/posts', $post),
+            Arr::get($data, 'data.links.self')
+        );
+    }
 }