Merge pull request #23 from pine3ree/patch-1

optionally disable layout rendering

Author: weierophinney

src/ZendViewRenderer.php

@@ -125,7 +125,10 @@ public function render($name, $params = [])
             ? $this->mergeViewModel($name, $params)
             : $this->createModel($name, $params);
 
-        $viewModel = $this->prepareLayout($viewModel);
+        $useLayout = false !== $viewModel->getVariable('layout', null);
+        if ($useLayout) {
+            $viewModel = $this->prepareLayout($viewModel);
+        }
 
         return $this->renderModel($viewModel, $this->renderer);
     }
@@ -317,16 +320,16 @@ private function mergeViewModel($name, ModelInterface $model)
      */
     private function prepareLayout(ModelInterface $viewModel)
     {
-        $layout = $this->layout ? clone $this->layout : null;
-
-        $providedLayout = $viewModel->getVariable('layout', false);
+        $providedLayout = $viewModel->getVariable('layout', null);
         if (is_string($providedLayout) && ! empty($providedLayout)) {
             $layout = new ViewModel();
             $layout->setTemplate($providedLayout);
             $viewModel->setVariable('layout', null);
         } elseif ($providedLayout instanceof ModelInterface) {
             $layout = $providedLayout;
             $viewModel->setVariable('layout', null);
+        } else {
+            $layout = $this->layout ? clone $this->layout : null;
         }
 
         if ($layout) {

test/ZendViewRendererTest.php

@@ -11,6 +11,7 @@
 use PHPUnit_Framework_TestCase as TestCase;
 use Zend\Expressive\Template\Exception\InvalidArgumentException;
 use Zend\Expressive\Template\TemplatePath;
+use Zend\Expressive\Template\TemplateRendererInterface;
 use Zend\Expressive\ZendView\ZendViewRenderer;
 use Zend\View\Model\ViewModel;
 use Zend\View\Renderer\PhpRenderer;
@@ -272,6 +273,51 @@ public function testCanPassViewModelForLayoutParameterWhenRendering()
         $this->assertContains('<title>ALTERNATE LAYOUT PAGE</title>', $result);
     }
 
+    /**
+     * @group layout
+     */
+    public function testDisableLayoutOnRender()
+    {
+        $layout = new ViewModel();
+        $layout->setTemplate('zendview-layout');
+
+        $renderer = new ZendViewRenderer(null, $layout);
+        $renderer->addPath(__DIR__ . '/TestAsset');
+
+        $name = 'zendview';
+        $rendered = $renderer->render('zendview', [
+            'layout' => false,
+            'name'   => $name,
+        ]);
+
+        $expected = file_get_contents(__DIR__ . '/TestAsset/zendview.phtml');
+        $expected = str_replace('<?php echo $name ?>', $name, $expected);
+
+        $this->assertEquals($rendered, $expected);
+    }
+
+    /**
+     * @group layout
+     */
+    public function testDisableLayoutViaDefaultParameter()
+    {
+        $layout = new ViewModel();
+        $layout->setTemplate('zendview-layout');
+
+        $renderer = new ZendViewRenderer(null, $layout);
+        $renderer->addPath(__DIR__ . '/TestAsset');
+        $renderer->addDefaultParam(TemplateRendererInterface::TEMPLATE_ALL, 'layout', false);
+
+
+        $name = 'zendview';
+        $rendered = $renderer->render('zendview', [ 'name' => $name ]);
+
+        $expected = file_get_contents(__DIR__ . '/TestAsset/zendview.phtml');
+        $expected = str_replace('<?php echo $name ?>', $name, $expected);
+
+        $this->assertEquals($rendered, $expected);
+    }
+
     /**
      * @group namespacing
      */