Merge branch '2.0' into 2.1

Author: GuilhemN

.travis.yml

@@ -24,8 +24,6 @@ matrix:
       env: COMPOSER_FLAGS="--prefer-lowest" SYMFONY_DEPRECATIONS_HELPER=weak
     - php: 7.0
       env: SYMFONY_VERSION='2.8.*'
-    - php: 7.0
-      env: SYMFONY_VERSION='3.0.*'
     - php: 7.0
       env: SYMFONY_VERSION='3.1.*'
 

Controller/ExceptionController.php

@@ -114,10 +114,7 @@ protected function getStatusCode(\Exception $exception)
     }
 
     /**
-     * Determines the parameters to pass to the view layer.
-     *
-     * Overwrite it in a custom ExceptionController class to add additionally parameters
-     * that should be passed to the view layer.
+     * Determines the template parameters to pass to the view layer.
      *
      * @param string               $currentContent
      * @param int                  $code

DependencyInjection/FOSRestExtension.php

@@ -16,6 +16,8 @@
 use Symfony\Component\DependencyInjection\DefinitionDecorator;
 use Symfony\Component\DependencyInjection\Loader\XmlFileLoader;
 use Symfony\Component\DependencyInjection\Reference;
+use Symfony\Component\Form\AbstractType;
+use Symfony\Component\Form\Extension\Core\Type\FormType;
 use Symfony\Component\HttpKernel\DependencyInjection\Extension;
 use Symfony\Component\HttpFoundation\Response;
 
@@ -88,7 +90,16 @@ private function loadForm(array $config, XmlFileLoader $loader, ContainerBuilder
     {
         if (!empty($config['disable_csrf_role'])) {
             $loader->load('forms.xml');
-            $container->getDefinition('fos_rest.form.extension.csrf_disable')->replaceArgument(1, $config['disable_csrf_role']);
+
+            $definition = $container->getDefinition('fos_rest.form.extension.csrf_disable');
+            $definition->replaceArgument(1, $config['disable_csrf_role']);
+
+            // BC for Symfony < 2.8: the extended_type attribute is used on higher versions
+            if (!method_exists(AbstractType::class, 'getBlockPrefix')) {
+                $definition->addTag('form.type_extension', ['alias' => 'form']);
+            } else {
+                $definition->addTag('form.type_extension', ['extended_type' => FormType::class]);
+            }
         }
     }
 

README.md

@@ -17,6 +17,11 @@ applications with Symfony. Features include:
 [![Latest Stable Version](https://poser.pugx.org/FriendsOfSymfony/rest-bundle/v/stable.svg)](https://packagist.org/packages/FriendsOfSymfony/rest-bundle)
 [![SensioLabsInsight](https://insight.sensiolabs.com/projects/0be23389-2e85-49cf-b333-caaa36d11c62/mini.png)](https://insight.sensiolabs.com/projects/0be23389-2e85-49cf-b333-caaa36d11c62)
 
+Note
+----
+
+FOSRestBundle 1.x is no longer maintained, 1.8 only receives security fixes. Please upgrade to FOSRestBundle 2.x as soon as possible.
+
 Documentation
 -------------
 

Request/ParamFetcher.php

@@ -101,17 +101,19 @@ public function get($name, $strict = null)
         /** @var ParamInterface $param */
         $param = $params[$name];
         $default = $param->getDefault();
+        $default = $this->resolveValue($this->container, $default);
         $strict = (null !== $strict ? $strict : $param->isStrict());
 
         $paramValue = $param->getValue($this->getRequest(), $default);
 
-        return $this->cleanParamWithRequirements($param, $paramValue, $strict);
+        return $this->cleanParamWithRequirements($param, $paramValue, $strict, $default);
     }
 
     /**
      * @param ParamInterface $param
      * @param mixed          $paramValue
      * @param bool           $strict
+     * @param mixed          $default
      *
      * @throws BadRequestHttpException
      * @throws \RuntimeException
@@ -120,11 +122,8 @@ public function get($name, $strict = null)
      *
      * @internal
      */
-    protected function cleanParamWithRequirements(ParamInterface $param, $paramValue, $strict)
+    protected function cleanParamWithRequirements(ParamInterface $param, $paramValue, $strict, $default)
     {
-        $default = $param->getDefault();
-        $default = $this->resolveValue($this->container, $default);
-
         $this->checkNotIncompatibleParams($param);
         if ($default !== null && $default === $paramValue) {
             return $paramValue;

Resources/config/forms.xml

@@ -6,8 +6,6 @@
 
     <services>
         <service id="fos_rest.form.extension.csrf_disable" class="FOS\RestBundle\Form\Extension\DisableCSRFExtension">
-            <!-- "alias" option for SF <2.8 -->
-            <tag name="form.type_extension" alias="form" extended-type="Symfony\Component\Form\Extension\Core\Type\FormType" />
             <argument type="service" id="security.token_storage" />
             <argument /> <!-- disable CSRF role -->
             <argument type="service" id="security.authorization_checker" />

Resources/doc/2-the-view-layer.rst

@@ -185,7 +185,7 @@ Then:
   leads to a "validation failed" response.
 - In a rendered template, the form is passed as 'form' and ``createView()``
   is called automatically.
-- ``$form->getData()`` is passed into the view as template as ``'data'`` if the
+- ``$form->getData()`` is passed into the view template as ``'data'`` if the
   form is the only view data.
 - An invalid form will be wrapped into an exception.
 

Routing/Loader/Reader/RestActionReader.php

@@ -16,6 +16,7 @@
 use FOS\RestBundle\Inflector\InflectorInterface;
 use FOS\RestBundle\Request\ParamReaderInterface;
 use FOS\RestBundle\Routing\RestRouteCollection;
+use Psr\Http\Message\MessageInterface;
 use Symfony\Component\Routing\Route;
 
 /**
@@ -476,6 +477,7 @@ private function getMethodArguments(\ReflectionMethod $method)
             \FOS\RestBundle\Request\ParamFetcherInterface::class,
             \Symfony\Component\Validator\ConstraintViolationListInterface::class,
             \Sensio\Bundle\FrameworkExtraBundle\Configuration\ParamConverter::class,
+            MessageInterface::class,
         ];
 
         $arguments = [];

Tests/Fixtures/Controller/TypeHintedController.php

@@ -0,0 +1,41 @@
+<?php
+
+/*
+ * This file is part of the FOSRestBundle package.
+ *
+ * (c) FriendsOfSymfony <http://friendsofsymfony.github.com/>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace FOS\RestBundle\Tests\Fixtures\Controller;
+
+use FOS\RestBundle\Controller\Annotations as Rest;
+use FOS\RestBundle\Controller\FOSRestController;
+use FOS\RestBundle\Routing\ClassResourceInterface;
+use Psr\Http\Message\MessageInterface;
+use Psr\Http\Message\ServerRequestInterface;
+use Symfony\Component\HttpFoundation\Request;
+
+/**
+ * @Rest\RouteResource("Article")
+ */
+class TypeHintedController implements ClassResourceInterface
+{
+    public function cgetAction(Request $request)
+    {
+    }
+
+    public function cpostAction(MessageInterface $request)
+    {
+    }
+
+    public function getAction(Request $request, $id)
+    {
+    }
+
+    public function postAction(ServerRequestInterface $request, $id)
+    {
+    }
+}

Tests/Functional/Bundle/TestBundle/Controller/ParamFetcherController.php

@@ -25,18 +25,18 @@
 class ParamFetcherController extends FOSRestController
 {
     /**
-     * @RequestParam(name="raw", requirements=@IdenticalTo({"foo"="raw", "bar"="foo"}), default="invalid")
-     * @RequestParam(name="map", map=true, requirements=@IdenticalTo({"foo"="map", "foobar"="foo"}), default="%invalid% %%")
-     * @RequestParam(name="bar", map=true, requirements="%foo% foo", strict=true)
+     * @RequestParam(name="raw", requirements=@IdenticalTo({"foo"="raw", "bar"="foo"}), default="invalid", strict=false)
+     * @RequestParam(name="map", map=true, requirements=@IdenticalTo({"foo"="map", "foobar"="foo"}), default="%invalid2% %%", strict=false)
+     * @RequestParam(name="bar", nullable=true, requirements="%bar%\ foo")
      */
     public function paramsAction(ParamFetcherInterface $fetcher)
     {
-        return new JsonResponse($fetcher->all(false));
+        return new JsonResponse($fetcher->all());
     }
 
     /**
      * @QueryParam(name="foo", default="invalid")
-     * @RequestParam(name="bar", default="foo")
+     * @RequestParam(name="bar", default="%foo%")
      */
     public function testAction(Request $request, ParamFetcherInterface $fetcher)
     {