Merge pull request #766 from odolbeau/avoid-array-to-string-conversion

Avoid "array to string conversion" error
nelmio · Dec 11, 2015 · 93b189b · 93b189b
2 parents 19d4e37 + d3bc49c
commit 93b189b
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 2 deletions.
diff --git a/Annotation/ApiDoc.php b/Annotation/ApiDoc.php
@@ -478,8 +478,10 @@ public function setRoute(Route $route)
             $this->host = $route->getHost() ? : null;
 
             //replace route placeholders
-             foreach ($route->getDefaults() as $key => $value) {
-                $this->host = str_replace('{' . $key . '}', $value, $this->host);
+            foreach ($route->getDefaults() as $key => $value) {
+                if (is_string($value)) {
+                    $this->host = str_replace('{' . $key . '}', $value, $this->host);
+                }
             }
         } else {
             $this->host = null;

diff --git a/Tests/Annotation/ApiDocTest.php b/Tests/Annotation/ApiDocTest.php
@@ -13,6 +13,7 @@
 
 use Nelmio\ApiDocBundle\Annotation\ApiDoc;
 use Nelmio\ApiDocBundle\Tests\TestCase;
+use Symfony\Component\Routing\Route;
 
 class ApiDocTest extends TestCase
 {
@@ -373,4 +374,28 @@ public function testAlignmentOfOutputAndResponseModels2()
         $this->assertArrayHasKey(400, $map);
         $this->assertEquals($apiDoc->getOutput(), $map[200]);
     }
+
+    public function testSetRoute()
+    {
+        $route = new Route(
+            '/path/{foo}',
+            [
+                'foo' => 'bar',
+                'nested' => [
+                    'key1' => 'value1',
+                    'key2' => 'value2',
+                ]
+            ],
+            [],
+            [],
+            '{foo}.awesome_host.com'
+        );
+
+        $apiDoc = new ApiDoc([]);
+        $apiDoc->setRoute($route);
+
+        $this->assertSame($route, $apiDoc->getRoute());
+        $this->assertEquals('bar.awesome_host.com', $apiDoc->getHost());
+        $this->assertEquals('ANY', $apiDoc->getMethod());
+    }
 }