[BUGFIX] Add a workaround for noPHPscriptInclude = TRUE (#37)

Resolves: #34

Author: InvisibleKind

Classes/Core/FrontendRenderingProvisioner.php

@@ -141,6 +141,8 @@ protected function configurePageRenderer(TypoScriptFrontendController $typoScrip
     protected function configureImageProcessing(TypoScriptFrontendController $typoScriptFrontendController)
     {
         $GLOBALS['TYPO3_CONF_VARS']['GFX']['im_noScaleUp'] = isset($typoScriptFrontendController->config['config']['noScaleUp']) ? '' . $typoScriptFrontendController->config['config']['noScaleUp'] : $GLOBALS['TYPO3_CONF_VARS']['GFX']['im_noScaleUp'];
-        $typoScriptFrontendController->TYPO3_CONF_VARS['GFX']['im_noScaleUp'] = $GLOBALS['TYPO3_CONF_VARS']['GFX']['im_noScaleUp'];
+        if (property_exists($typoScriptFrontendController, 'TYPO3_CONF_VARS')) {
+            $typoScriptFrontendController->TYPO3_CONF_VARS['GFX']['im_noScaleUp'] = $GLOBALS['TYPO3_CONF_VARS']['GFX']['im_noScaleUp'];
+        }
     }
 }

Classes/RenderingDispatcher.php

@@ -75,6 +75,12 @@ public function checkDataSubmission(TypoScriptFrontendController $typoScriptFron
 
             $typoScriptFrontendController->content = $response->getContent();
             $typoScriptFrontendController->config['config']['pageGenScript'] = 'EXT:typoscript_rendering/Scripts/DummyRendering.php';
+            // Force DummyRendering to be included, even if TYPO3 setup disallows inclusion of PHP scripts
+            if (property_exists($typoScriptFrontendController, 'TYPO3_CONF_VARS')) {
+                $typoScriptFrontendController->TYPO3_CONF_VARS['FE']['noPHPscriptInclude'] = false;
+            } else {
+                $GLOBALS['TYPO3_CONF_VARS']['FE']['noPHPscriptInclude'] = false;
+            }
         }
     }
 

Tests/Functional/FeNoPhpScriptIncludeRenderingTest.php

@@ -0,0 +1,39 @@
+<?php
+namespace Helhum\TyposcriptRendering\Tests\Functional;
+
+/*
+ * This file is part of the TypoScript Rendering TYPO3 extension.
+ *
+ * It is free software; you can redistribute it and/or modify it under
+ * the terms of the GNU General Public License, either version 2
+ * of the License, or any later version.
+ *
+ * For the full copyright and license information, please read
+ * LICENSE file that was distributed with this source code.
+ *
+ */
+
+/**
+ * Test case.
+ */
+class FeNoPhpScriptIncludeRenderingTest extends AbstractRenderingTestCase
+{
+    /**
+     * @var array
+     */
+    protected $configurationToUseInTestInstance = array(
+        'FE' => array(
+            'noPHPscriptInclude' => true,
+        ),
+    );
+
+    /**
+     * @test
+     */
+    public function contentIsRenderedWithNoPhpScriptInclusionEnabled()
+    {
+        $requestArguments = array('url' => $this->getRenderUrl(1, 1, 'lib.link'));
+        $expectedContent = '<a href="/index.php?id=1&amp;L=1" target="_blank">link</a>';
+        $this->assertSame($expectedContent, $this->fetchFrontendResponse($requestArguments)->getContent());
+    }
+}