Merge pull request #13 from victorwelling/master

Fix for inherited variables

src/Twig/PatchingCompiler.php

@@ -49,6 +49,6 @@ private function patch($string)
 
         return !is_string($string)
             ? $string
-            : preg_replace($patterns, '$1array_merge($originalContext ?? [], $context)$2', $string);
+            : preg_replace($patterns, '$1array_merge($context, $originalContext ?? [])$2', $string);
     }
 }

tests/Integration/Embedding/EmbeddingTest.php

@@ -47,6 +47,7 @@ public function testOverriddenBlocksShouldReceiveVariablesFromParent()
     {
         $output = $this->renderTemplate('page.twig');
 
+        $this->assertContains('<title>page_title</title>', $output);
         $this->assertContains('<h1>page_title</h1>', $output);
         $this->assertContains('<p>page_content</p>', $output);
     }

tests/Integration/Embedding/Models/Base.php

@@ -3,10 +3,22 @@
 
 namespace Shoot\Shoot\Tests\Integration\Embedding\Models;
 
+use Shoot\Shoot\HasPresenterInterface;
 use Shoot\Shoot\PresentationModel;
+use Shoot\Shoot\Tests\Integration\Embedding\Presenters\BasePresenter;
 
-final class Base extends PresentationModel
+final class Base extends PresentationModel implements HasPresenterInterface
 {
     /** @var string */
-    protected $title = 'base_title';
+    protected $title = '';
+
+    /**
+     * Returns the name by which to resolve the presenter through the DI container.
+     *
+     * @return string
+     */
+    public function getPresenterName(): string
+    {
+        return BasePresenter::class;
+    }
 }

tests/Integration/Embedding/Presenters/BasePresenter.php

@@ -0,0 +1,27 @@
+<?php
+declare(strict_types=1);
+
+namespace Shoot\Shoot\Tests\Integration\Embedding\Presenters;
+
+use Psr\Http\Message\ServerRequestInterface;
+use Shoot\Shoot\PresentationModel;
+use Shoot\Shoot\PresenterInterface;
+
+final class BasePresenter implements PresenterInterface
+{
+    /**
+     * Receives the current HTTP request context and the presentation model assigned to the view. If necessary,
+     * populates the presentation model with data and returns it.
+     *
+     * @param ServerRequestInterface $request
+     * @param PresentationModel      $presentationModel
+     *
+     * @return PresentationModel
+     */
+    public function present(ServerRequestInterface $request, PresentationModel $presentationModel): PresentationModel
+    {
+        return $presentationModel->withVariables([
+            'title' => 'base_title'
+        ]);
+    }
+}