Merge pull request #701 from cakephp/fix-paths

Fix template override paths
cakephp · Aug 3, 2020 · 16d2816 · 16d2816
2 parents 4f723a5 + 649ae7d
commit 16d2816
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 7 deletions.
diff --git a/docs/en/development.rst b/docs/en/development.rst
@@ -194,7 +194,7 @@ If you wish to modify the default output produced by the "bake" command, you can
 create your own bake templates in your application. This way does not use the
 ``--theme`` option in the command line when baking. The best way to do this is:
 
-#. Create a new directory **/templates/bake/**.
+#. Create a new directory **/templates/plugin/Bake/**.
 #. Copy any templates you want to override from
    **vendor/cakephp/bake/templates/bake/** to matching files in your
    application.

diff --git a/src/View/BakeView.php b/src/View/BakeView.php
@@ -88,11 +88,7 @@ public function initialize(): void
     public function render(?string $view = null, $layout = null): string
     {
         $viewFileName = $this->_getTemplateFileName($view);
-        $templateEventName = str_replace(
-            ['.twig', DS],
-            ['', '.'],
-            explode('templates' . DS . static::BAKE_TEMPLATE_FOLDER . DS, $viewFileName)[1]
-        );
+        $templateEventName = str_replace(DS, '.', $view);
 
         $this->_currentType = static::TYPE_TEMPLATE;
         $this->dispatchEvent('View.beforeRender', [$viewFileName]);
@@ -142,7 +138,10 @@ protected function _paths(?string $plugin = null, bool $cached = true): array
     {
         $paths = parent::_paths($plugin, false);
         foreach ($paths as &$path) {
-            $path .= static::BAKE_TEMPLATE_FOLDER . DS;
+            // Append 'bake' to all directories that aren't the application override directory.
+            if (strpos($path, 'plugin' . DS . 'Bake') === false) {
+                $path .= static::BAKE_TEMPLATE_FOLDER . DS;
+            }
         }
 
         return $paths;

diff --git a/tests/TestCase/View/BakeViewTest.php b/tests/TestCase/View/BakeViewTest.php
@@ -170,4 +170,15 @@ public function testCustomRenderEvents()
             $result
         );
     }
+
+    /**
+     * Ensure that application override templates don't have a double path in them.
+     *
+     * @return void
+     */
+    public function testApplicationOverride()
+    {
+        $result = $this->View->render('Bake.override');
+        $this->assertSame("Application override.\n", $result);
+    }
 }
diff --git a/tests/test_app/templates/plugin/Bake/override.twig b/tests/test_app/templates/plugin/Bake/override.twig
@@ -0,0 +1 @@
+Application override.