Fixed bug #15073: inlineJS creates superfluous markup (Thanks to Georg Ringer)
authorSteffen Kamper <info@sk-typo3.de>
Tue, 13 Jul 2010 20:47:42 +0000 (20:47 +0000)
committerSteffen Kamper <info@sk-typo3.de>
Tue, 13 Jul 2010 20:47:42 +0000 (20:47 +0000)
git-svn-id: https://svn.typo3.org/TYPO3v4/Core/trunk@8182 709f56b5-9817-0410-a4d7-c38de5d9e867

ChangeLog
t3lib/class.t3lib_pagerenderer.php
typo3/sysext/cms/tslib/class.tslib_pagegen.php

index 5ac99e7..c7f39fc 100755 (executable)
--- a/ChangeLog
+++ b/ChangeLog
@@ -5,6 +5,7 @@
 
 2010-07-13  Steffen Kamper  <info@sk-typo3.de>
 
+       * Fixed bug #15073: inlineJS creates superfluous markup (Thanks to Georg Ringer)
        * Fixed bug #15103: TS Errors are hardly readable (white on yellow) (Thanks to Georg Ringer)
        * Fixed bug #15098: TS Errors should be shown with flash messages (Thanks to Georg Ringer)
        * Fixed bug #14855: Big Buttons in Page Module miss padding (Thanks to Georg Ringer)
index 27eceab..26e397b 100644 (file)
@@ -743,7 +743,7 @@ class t3lib_PageRenderer implements t3lib_Singleton {
         * @return void
         */
        public function addJsInlineCode($name, $block, $compress = TRUE, $forceOnTop = FALSE) {
-               if (!isset($this->jsInline[$name])) {
+               if (!isset($this->jsInline[$name]) && !empty($block)) {
                        $this->jsInline[$name] = array (
                                'code'        => $block . LF,
                                'section'     => self::PART_HEADER,
@@ -763,7 +763,7 @@ class t3lib_PageRenderer implements t3lib_Singleton {
         * @return void
         */
        public function addJsFooterInlineCode($name, $block, $compress = TRUE, $forceOnTop = FALSE) {
-               if (!isset($this->jsInline[$name])) {
+               if (!isset($this->jsInline[$name]) && !empty($block)) {
                        $this->jsInline[$name] = array (
                                'code'        => $block . LF,
                                'section'     => self::PART_FOOTER,
@@ -828,7 +828,7 @@ class t3lib_PageRenderer implements t3lib_Singleton {
         * @return void
         */
        public function addCssInlineBlock($name, $block, $compressed = FALSE, $forceOnTop = FALSE) {
-               if (!isset($this->cssInline[$name])) {
+               if (!isset($this->cssInline[$name]) && !empty($block)) {
                        $this->cssInline[$name] = array (
                                'code'       => $block,
                                'compress'   => $compress,
index 8636caa..bfc2b74 100644 (file)
@@ -945,12 +945,17 @@ See <a href="http://wiki.typo3.org/index.php/TYPO3_3.8.1" target="_blank">wiki.t
                        */
                        $inlineJSint = '';
                        self::stripIntObjectPlaceholder($inlineJS, $inlineJSint);
-                       $pageRenderer->addJsInlineCode('TS_inlineJSint', $inlineJSint, $GLOBALS['TSFE']->config['config']['minifyJS']);
+                       if ($inlineJSint) {
+                               $pageRenderer->addJsInlineCode('TS_inlineJSint', $inlineJSint, $GLOBALS['TSFE']->config['config']['minifyJS']);
+                       }
                        $pageRenderer->addJsFile(TSpagegen::inline2TempFile($scriptJsCode . $inlineJS, 'js'), 'text/javascript', $GLOBALS['TSFE']->config['config']['minifyJS']);
+
                        if ($inlineFooterJs) {
                                $inlineFooterJSint = '';
                                self::stripIntObjectPlaceholder($inlineFooterJs, $inlineFooterJSint);
-                               $pageRenderer->addJsFooterInlineCode('TS_inlineFooterJSint', $inlineFooterJSint, $GLOBALS['TSFE']->config['config']['minifyJS']);
+                               if ($inlineFooterJSint) {
+                                       $pageRenderer->addJsFooterInlineCode('TS_inlineFooterJSint', $inlineFooterJSint, $GLOBALS['TSFE']->config['config']['minifyJS']);
+                               }
                                $pageRenderer->addJsFooterFile(TSpagegen::inline2TempFile($inlineFooterJs, 'js'), 'text/javascript', $GLOBALS['TSFE']->config['config']['minifyJS']);
                        }
                } else {