Merge pull request #544 from kimausloos/master

[NodeBundle] Allow more flexible entity controller handling
Kunstmaan · Jul 13, 2015 · 3d72165 · 3d72165
2 parents b232ed5 + ba77d4b
commit 3d72165
Showing 1 changed file with 16 additions and 4 deletions.
diff --git a/src/Kunstmaan/NodeBundle/EventListener/RenderContextListener.php b/src/Kunstmaan/NodeBundle/EventListener/RenderContextListener.php
@@ -3,6 +3,7 @@
 namespace Kunstmaan\NodeBundle\EventListener;
 
 use Doctrine\ORM\EntityManager;
+use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\HttpKernel\Event\GetResponseForControllerResultEvent;
 use Symfony\Component\Templating\EngineInterface;
 
@@ -29,8 +30,13 @@ public function __construct(EngineInterface $templating, EntityManager $em)
      */
     public function onKernelView(GetResponseForControllerResultEvent $event)
     {
-        $request    = $event->getRequest();
-        $nodeTranslation = $request->attributes->get('_nodeTranslation');
+        $response = $event->getControllerResult();
+        if ($response instanceof Response) { //if its a response, just continue
+            return;
+        }
+
+        $request            = $event->getRequest();
+        $nodeTranslation    = $request->attributes->get('_nodeTranslation');
 
         if ($nodeTranslation) {
             $entity          = $request->attributes->get('_entity');
@@ -62,8 +68,14 @@ public function onKernelView(GetResponseForControllerResultEvent $event)
                 $parameters = $renderContext;
             }
 
-            // Sent the response here, another option is to let the symfony kernel.view listener handle it
-            $event->setResponse($this->templating->renderResponse($entity->getDefaultView(), $parameters));
+            if (is_array($response)) { //if the response is an array, merge with rendercontext
+                $parameters = array_merge($parameters, $response);
+            }
+
+            //set the rendercontext with all params as response, plus the template in the request attribs
+            //the SensioFrameworkExtraBundle kernel.view will handle everything else
+            $event->setControllerResult((array)$parameters);
+            $request->attributes->set('_template', $entity->getDefaultView());
         }
     }
 }