[BUGFIX] Streamline bool arguments throughout the core 40/53240/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:53:39 +0000 (16:53 +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/53240
Tested-by: TYPO3com <no-reply@typo3.com>
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/frontend/Classes/ContentObject/ContentObjectRenderer.php
typo3/sysext/frontend/Classes/ContentObject/Menu/AbstractMenuContentObject.php
typo3/sysext/lang/Classes/LanguageService.php

index 38dea74..c0cd524 100644 (file)
@@ -484,7 +484,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) . ' />';
                 }
             }
         }
@@ -522,7 +522,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 018d6a0..ae18e11 100644 (file)
@@ -4737,7 +4737,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 232485b..af35538 100644 (file)
@@ -2056,7 +2056,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>';
     }
 
index d75bb97..b03c8f5 100644 (file)
@@ -330,7 +330,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) {