[BUGFIX] Streamline bool arguments throughout the core 36/53236/2
authorBenni Mack <benni@typo3.org>
Fri, 16 Jun 2017 11:10:03 +0000 (13:10 +0200)
committerFrank Naegler <frank.naegler@typo3.org>
Sat, 17 Jun 2017 14:37:22 +0000 (16:37 +0200)
There are still places left in TYPO3 where methods are called with
integers as parameters, although they are marked as boolean.

These places are changed to the correct and necessary value.

Resolves: #81595
Releases: master, 8.7
Change-Id: I1664e7d5e3962f0d2cf59ba6148812291d0b803b
Reviewed-on: https://review.typo3.org/53236
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Frank Naegler <frank.naegler@typo3.org>
Tested-by: Frank Naegler <frank.naegler@typo3.org>
typo3/sysext/core/Classes/Html/RteHtmlParser.php
typo3/sysext/core/Classes/Localization/LanguageService.php
typo3/sysext/frontend/Classes/ContentObject/ContentObjectRenderer.php
typo3/sysext/frontend/Classes/ContentObject/Menu/AbstractMenuContentObject.php

index 200eb53..9c30b85 100644 (file)
@@ -473,7 +473,7 @@ class RteHtmlParser extends HtmlParser
                     if (GeneralUtility::isFirstPartOfStr($attribArray['src'], $siteUrl)) {
                         $attribArray['src'] = substr($attribArray['src'], strlen($siteUrl));
                     }
-                    $imgSplit[$k] = '<img ' . GeneralUtility::implodeAttributes($attribArray, 1, 1) . ' />';
+                    $imgSplit[$k] = '<img ' . GeneralUtility::implodeAttributes($attribArray, true, true) . ' />';
                 }
             }
         }
@@ -511,7 +511,7 @@ class RteHtmlParser extends HtmlParser
                     if (!isset($attribArray['alt'])) {
                         $attribArray['alt'] = '';
                     }
-                    $imgSplit[$k] = '<img ' . GeneralUtility::implodeAttributes($attribArray, 1, 1) . ' />';
+                    $imgSplit[$k] = '<img ' . GeneralUtility::implodeAttributes($attribArray, true, true) . ' />';
                 }
             }
         }
index 90e3a1c..1500fcc 100644 (file)
@@ -207,7 +207,7 @@ class LanguageService
             $GLOBALS['TCA_DESCR'][$table]['columns'] = [];
                 // Get local-lang for each file in $TCA_DESCR[$table]['refs'] as they are ordered.
             foreach ($GLOBALS['TCA_DESCR'][$table]['refs'] as $llfile) {
-                $localLanguage = $this->includeLLFile($llfile, 0, 1);
+                $localLanguage = $this->includeLLFile($llfile, false, true);
                     // Traverse all keys
                 if (is_array($localLanguage['default'])) {
                     foreach ($localLanguage['default'] as $lkey => $lVal) {
index 65b400e..c790daf 100644 (file)
@@ -4437,7 +4437,7 @@ class ContentObjectRenderer
                 if (!$attrib['align'] && $defaultAlign) {
                     $attrib['align'] = $defaultAlign;
                 }
-                $params = GeneralUtility::implodeAttributes($attrib, 1);
+                $params = GeneralUtility::implodeAttributes($attrib, true);
                 if (!($conf['removeWrapping'] && !($emptyTag && $conf['removeWrapping.']['keepSingleTag']))) {
                     if ($emptyTag) {
                         $str_content = '<' . strtolower($uTagName) . (trim($params) ? ' ' . trim($params) : '') . ' />';
index 7635ed5..ae66e18 100644 (file)
@@ -2047,7 +2047,7 @@ abstract class AbstractMenuContentObject
     {
         $params = trim($this->I['val']['ATagParams']) . $this->I['accessKey']['code'];
         $params = $params !== '' ? ' ' . $params : '';
-        $this->I['A1'] = '<a ' . GeneralUtility::implodeAttributes($this->I['linkHREF'], 1) . $params . '>';
+        $this->I['A1'] = '<a ' . GeneralUtility::implodeAttributes($this->I['linkHREF'], true) . $params . '>';
         $this->I['A2'] = '</a>';
     }