Make the max depth configurable in config and depth query parameter

Author: wouterj

src/Controller/ResourceController.php

@@ -11,6 +11,7 @@
 
 namespace Symfony\Cmf\Bundle\ResourceRestBundle\Controller;
 
+use Symfony\Cmf\Bundle\ResourceRestBundle\Serializer\Jms\Handler\ResourceHandler;
 use Symfony\Cmf\Component\Resource\Puli\Api\ResourceRepository;
 use Symfony\Cmf\Component\Resource\Puli\Api\ResourceNotFoundException;
 use Symfony\Cmf\Component\Resource\RepositoryRegistryInterface;
@@ -38,6 +39,11 @@ class ResourceController
      */
     private $serializer;
 
+    /**
+     * @var ResourceHandler
+     */
+    private $resourceHandler;
+
     /**
      * @var AuthorizationCheckerInterface|null
      */
@@ -47,11 +53,12 @@ class ResourceController
      * @param SerializerInterface         $serializer
      * @param RepositoryRegistryInterface $registry
      */
-    public function __construct(SerializerInterface $serializer, RepositoryRegistryInterface $registry, AuthorizationCheckerInterface $authorizationChecker = null)
+    public function __construct(SerializerInterface $serializer, RepositoryRegistryInterface $registry, ResourceHandler $resourceHandler, AuthorizationCheckerInterface $authorizationChecker = null)
     {
         $this->serializer = $serializer;
         $this->registry = $registry;
         $this->authorizationChecker = $authorizationChecker;
+        $this->resourceHandler = $resourceHandler;
     }
 
     /**
@@ -60,8 +67,12 @@ public function __construct(SerializerInterface $serializer, RepositoryRegistryI
      * @param string $repositoryName
      * @param string $path
      */
-    public function getResourceAction($repositoryName, $path)
+    public function getResourceAction(Request $request, $repositoryName, $path)
     {
+        if ($request->query->has('depth')) {
+            $this->resourceHandler->setMaxDepth($request->query->getInt('depth'));
+        }
+
         $path = '/'.ltrim($path, '/');
 
         try {

src/DependencyInjection/CmfResourceRestExtension.php

@@ -35,6 +35,8 @@ public function load(array $configs, ContainerBuilder $container)
         $configuration = new Configuration();
         $config = $processor->processConfiguration($configuration, $configs);
 
+        $container->setParameter('cmf_resource_rest.max_depth', $config['max_depth']);
+
         $loader->load('serializer.xml');
         $loader->load('resource-rest.xml');
 

src/DependencyInjection/Configuration.php

@@ -28,6 +28,7 @@ public function getConfigTreeBuilder()
             ->fixXmlConfig('payload_alias', 'payload_alias_map')
             ->fixXmlConfig('enhance', 'enhancer_map')
             ->children()
+                ->integerNode('max_depth')->defaultValue(2)->end()
                 ->arrayNode('payload_alias_map')
                     ->useAttributeAsKey('name')
                     ->prototype('array')

src/Resources/config/resource-rest.xml

@@ -9,6 +9,7 @@
         <service id="cmf_resource_rest.controller.resource" class="Symfony\Cmf\Bundle\ResourceRestBundle\Controller\ResourceController">
             <argument type="service" id="serializer" />
             <argument type="service" id="cmf_resource.registry" />
+            <argument type="service" id="cmf_resource_rest.serializer.jms.handler.resource" />
             <argument type="service" id="security.authorization_checker" on-invalid="ignore" />
         </service>
 

src/Resources/config/serializer.xml

@@ -9,6 +9,8 @@
             <argument type="service" id="cmf_resource.registry" />
             <argument type="service" id="cmf_resource_rest.registry.payload_alias" />
             <argument type="service" id="cmf_resource_rest.registry.enhancer" />
+            <argument>%cmf_resource_rest.max_depth%</argument>
+
             <tag name="jms_serializer.subscribing_handler"/>
         </service>
 

src/Serializer/Jms/Handler/ResourceHandler.php

@@ -33,15 +33,18 @@ class ResourceHandler implements SubscribingHandlerInterface
     private $registry;
     private $payloadAliasRegistry;
     private $enhancerRegistry;
+    private $maxDepth;
 
     public function __construct(
         RepositoryRegistryInterface $registry,
         PayloadAliasRegistry $payloadAliasRegistry,
-        EnhancerRegistry $enhancerRegistry
+        EnhancerRegistry $enhancerRegistry,
+        $maxDepth = 2
     ) {
         $this->registry = $registry;
         $this->payloadAliasRegistry = $payloadAliasRegistry;
         $this->enhancerRegistry = $enhancerRegistry;
+        $this->maxDepth = $maxDepth;
     }
 
     public static function getSubscribingMethods()
@@ -72,6 +75,11 @@ public function serializeResource(
         $context->accept($data);
     }
 
+    public function setMaxDepth($maxDepth)
+    {
+        $this->maxDepth = $maxDepth;
+    }
+
     private function doSerializeResource(PuliResource $resource, $depth = 0)
     {
         $data = array();
@@ -96,7 +104,7 @@ private function doSerializeResource(PuliResource $resource, $depth = 0)
         foreach ($resource->listChildren() as $name => $childResource) {
             $children[$name] = array();
 
-            if ($depth < 2) {
+            if ($depth < $this->maxDepth) {
                 $children[$name] = $this->doSerializeResource($childResource, $depth + 1);
             }
         }

tests/Features/nesting.feature

@@ -0,0 +1,68 @@
+Feature: Nesting resources
+    In order to retrieve a tree of data
+    As a webservice user
+    I need to be able to get nested resources
+
+    Background:
+        Given the test application has the following configuration:
+            """
+            cmf_resource:
+                repositories:
+                    default:
+                        type: doctrine_phpcr_odm
+                        basepath: /tests/cmf/articles
+            """
+        And there exists an "Article" document at "/cmf/articles/foo":
+            | title | Article 1          |
+            | body  | This is my article |
+        And there exists an "Article" document at "/cmf/articles/foo/sub":
+            | title | Sub-article 1      |
+            | body  | This is my article |
+
+    Scenario: Retrieving nested resources
+        When I send a GET request to "/api/default/foo"
+        Then the response should contain json:
+            """
+            {
+                "repository_alias": "default",
+                "repository_type": "doctrine_phpcr_odm",
+                "payload_alias": null,
+                "payload_type": "Symfony\\Cmf\\Bundle\\ResourceRestBundle\\Tests\\Resources\\TestBundle\\Document\\Article",
+                "path": "\/foo",
+                "node_name": "foo",
+                "label": "foo",
+                "repository_path": "\/foo",
+                "children": {
+                    "sub": {
+                        "repository_alias": "default",
+                        "repository_type": "doctrine_phpcr_odm",
+                        "payload_alias": null,
+                        "payload_type": "Symfony\\Cmf\\Bundle\\ResourceRestBundle\\Tests\\Resources\\TestBundle\\Document\\Article",
+                        "path": "\/foo\/sub",
+                        "node_name": "sub",
+                        "label": "sub",
+                        "repository_path": "\/foo\/sub",
+                        "children": []
+                    }
+                }
+            }
+            """
+
+    Scenario: Specifying a depth
+        When I send a GET request to "/api/default/foo?depth=0"
+        Then the response should contain json:
+            """
+            {
+                "repository_alias": "default",
+                "repository_type": "doctrine_phpcr_odm",
+                "payload_alias": null,
+                "payload_type": "Symfony\\Cmf\\Bundle\\ResourceRestBundle\\Tests\\Resources\\TestBundle\\Document\\Article",
+                "path": "\/foo",
+                "node_name": "foo",
+                "label": "foo",
+                "repository_path": "\/foo",
+                "children": {
+                    "sub": []
+                }
+            }
+            """

tests/Resources/app/config/routing.php

@@ -13,7 +13,7 @@
 
 $collection = new RouteCollection();
 $collection->addCollection(
-    $loader->import(__DIR__.'/../../../../Resources/config/routing.yml')
+    $loader->import('@CmfResourceRestBundle/Resources/config/routing.yml')
 );
 
 return $collection;