Merge pull request #796 from zhukovra/master-array-requirements

Fix Array to string conversion error on array requirements

Extractor/Handler/FosRestHandler.php

@@ -87,6 +87,34 @@ class FosRestHandler implements HandlerInterface
             return (string) $requirements['rule'];
         }
 
+        if (is_array($requirements) && array_key_exists(0, $requirements)) {
+
+            $output = array();
+
+            foreach ($requirements as $req) {
+
+                if (is_object($req) && $req instanceof Constraint) {
+                    if ($req instanceof Regex) {
+                        $output[] = $req->getHtmlPattern();
+                    } else {
+                        $class = get_class($req);
+                        $output[] = substr($class, strrpos($class, '\\')+1);
+                    }
+
+                }
+
+                if (is_array($req)) {
+                    if (array_key_exists('_format', $req)) {
+                        $output[] = 'Format: '.$req['_format'];
+                    } else if (isset($req['rule'])) {
+                        $output[] = $req['rule'];
+                    }
+                }
+            }
+
+            return implode(', ', $output);
+        }
+
         return (string) $requirements;
     }
 

Tests/Extractor/Handler/FosRestHandlerTest.php

@@ -189,4 +189,18 @@ class FosRestHandlerTest extends WebTestCase
         $this->assertArrayHasKey('requirement', $filters['param1']);
         $this->assertEquals('regexp', $filters['param1']['requirement']);
     }
+
+    public function testWithRequestParamPlainArrayRequirements()
+    {
+        $container  = $this->getContainer();
+        $extractor  = $container->get('nelmio_api_doc.extractor.api_doc_extractor');
+        $annotation = $extractor->get('Nelmio\ApiDocBundle\Tests\Fixtures\Controller\TestController::routeWithQueryParamPlainArrayRequirementsAction', 'test_route_30');
+
+        $this->assertNotNull($annotation);
+        $filters = $annotation->getFilters();
+
+        $this->assertArrayHasKey('param1', $filters);
+        $this->assertArrayHasKey('requirement', $filters['param1']);
+        $this->assertEquals('NotNull, NotBlank', $filters['param1']['requirement']);
+    }
 }

Tests/Fixtures/Controller/TestController.php

@@ -18,7 +18,7 @@ use Nelmio\ApiDocBundle\Tests\Fixtures\DependencyTypePath;
 use Nelmio\ApiDocBundle\Tests\Fixtures\RequestParamHelper;
 use Nelmio\ApiDocBundle\Util\LegacyFormHelper;
 use Symfony\Component\HttpFoundation\Response;
-use Symfony\Component\Validator\Constraints\Email;
+use Symfony\Component\Validator\Constraints as Assert;
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Cache;
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Security;
 
@@ -150,7 +150,7 @@ class TestController
 
     /**
      * @ApiDoc()
-     * @QueryParam(name="mail", requirements=@Email, description="Email of someone.")
+     * @QueryParam(name="mail", requirements=@Assert\Email, description="Email of someone.")
      */
     public function zActionWithConstraintAsRequirements()
     {
@@ -406,4 +406,12 @@ class TestController
     public function routeWithQueryParamArrayRequirementsAction()
     {
     }
+
+    /**
+     * @ApiDoc()
+     * @QueryParam(name="param1", requirements={@Assert\NotNull(), @Assert\NotBlank()}, description="Param1 description.")
+     */
+    public function routeWithQueryParamPlainArrayRequirementsAction()
+    {
+    }
 }

Tests/Fixtures/app/config/routing.yml

@@ -243,3 +243,8 @@ test_route_29:
     path: /z-query-param-array-requirements
     methods: [GET]
     defaults: { _controller: NelmioApiDocTestBundle:Test:routeWithQueryParamArrayRequirementsAction }
+
+test_route_30:
+    path: /z-query-param-plain-array-requirements
+    methods: [GET]
+    defaults: { _controller: NelmioApiDocTestBundle:Test:routeWithQueryParamPlainArrayRequirementsAction }