[TASK] Cleanup double-paranthesis in calls to trim() 72/42472/2
authorStefan Neufeind <typo3.neufeind@speedpartner.de>
Sun, 9 Aug 2015 22:32:16 +0000 (00:32 +0200)
committerAndreas Fernandez <typo3@scripting-base.de>
Mon, 10 Aug 2015 13:32:19 +0000 (15:32 +0200)
Change-Id: Ie8daf30c5bc621feb5707b2aecb1abb96b03bf60
Resolves: #68897
Releases: master
Reviewed-on: http://review.typo3.org/42472
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
12 files changed:
typo3/sysext/backend/Classes/Template/DocumentTemplate.php
typo3/sysext/backend/Classes/Utility/BackendUtility.php
typo3/sysext/core/Classes/Controller/CommandLineController.php
typo3/sysext/core/Classes/DataHandling/DataHandler.php
typo3/sysext/core/Classes/Database/QueryGenerator.php
typo3/sysext/core/Classes/Html/HtmlParser.php
typo3/sysext/core/Classes/Html/RteHtmlParser.php
typo3/sysext/core/Classes/Utility/GeneralUtility.php
typo3/sysext/dbal/Classes/Database/SqlCompilers/Adodb.php
typo3/sysext/dbal/Classes/Database/SqlCompilers/Mysql.php
typo3/sysext/frontend/Classes/ContentObject/ContentObjectRenderer.php
typo3/sysext/frontend/Classes/Plugin/AbstractPlugin.php

index 3937d5e..43808b2 100644 (file)
@@ -1110,7 +1110,7 @@ function jumpToUrl(URL) {
         * @return string HTML body tag
         */
        public function docBodyTagBegin() {
-               return '<body ' . trim(($this->bodyTagAdditions . ($this->bodyTagId ? ' id="' . $this->bodyTagId . '"' : ''))) . '>';
+               return '<body ' . trim($this->bodyTagAdditions . ($this->bodyTagId ? ' id="' . $this->bodyTagId . '"' : '')) . '>';
        }
 
        /**
index 8a9c71d..a6697e8 100755 (executable)
@@ -539,7 +539,7 @@ class BackendUtility {
                                                        }
                                                        $fieldLabel = !empty($field['TCEforms']['label']) ? $lang->sl($field['TCEforms']['label']) : $fieldName;
                                                        $fieldIdent = $table . ':' . $tableField . ';' . $extIdent . ';' . $sheetName . ';' . $fieldName;
-                                                       $excludeArrayTable[] = array(trim(($labelPrefix . ' ' . $extTitle), ': ') . ': ' . $fieldLabel, $fieldIdent);
+                                                       $excludeArrayTable[] = array(trim($labelPrefix . ' ' . $extTitle, ': ') . ': ' . $fieldLabel, $fieldIdent);
                                                }
                                        }
                                }
index 5076828..2aca8a1 100644 (file)
@@ -258,7 +258,7 @@ class CommandLineController {
                                                }
                                        }
                                        foreach ($this->cli_options as $v) {
-                                               $this->cli_echo($v[0] . substr($this->cli_indent(rtrim(($v[1] . LF . $v[2])), ($maxLen + 4)), strlen($v[0])) . LF);
+                                               $this->cli_echo($v[0] . substr($this->cli_indent(rtrim($v[1] . LF . $v[2]), $maxLen + 4), strlen($v[0])) . LF);
                                        }
                                        $this->cli_echo(LF);
                                        break;
index 5d6612a..891bdef 100644 (file)
@@ -7269,7 +7269,7 @@ class DataHandler {
        public function getCopyHeader($table, $pid, $field, $value, $count, $prevTitle = '') {
                // Set title value to check for:
                if ($count) {
-                       $checkTitle = $value . rtrim((' ' . sprintf($this->prependLabel($table), $count)));
+                       $checkTitle = $value . rtrim(' ' . sprintf($this->prependLabel($table), $count));
                } else {
                        $checkTitle = $value;
                }
index ac804e8..e1c6f0f 100644 (file)
@@ -1266,7 +1266,7 @@ class QueryGenerator {
                $prefix = $this->enablePrefix ? $this->table . '.' : '';
                if (!$first) {
                        // Is it OK to insert the AND operator if none is set?
-                       $qs .= trim(($conf['operator'] ?: 'AND')) . ' ';
+                       $qs .= trim($conf['operator'] ?: 'AND') . ' ';
                }
                $qsTmp = str_replace('#FIELD#', $prefix . trim(substr($conf['type'], 6)), $this->compSQL[$conf['comparison']]);
                $inputVal = $this->cleanInputVal($conf);
index f325488..770b81f 100644 (file)
@@ -952,7 +952,7 @@ class HtmlParser {
                                                                }
                                                                if ($setTag) {
                                                                        // Setting the tag
-                                                                       $newContent[$c++] = $this->processTag($lt . ($endTag ? '/' : '') . trim(($tagParts[0] . ' ' . $tagParts[1])) . ($emptyTag ? ' /' : '') . $gt, $addConfig, $endTag, $lt == '&lt;');
+                                                                       $newContent[$c++] = $this->processTag($lt . ($endTag ? '/' : '') . trim($tagParts[0] . ' ' . $tagParts[1]) . ($emptyTag ? ' /' : '') . $gt, $addConfig, $endTag, $lt == '&lt;');
                                                                }
                                                        }
                                                } else {
@@ -1083,7 +1083,7 @@ class HtmlParser {
                                if ($somethingDone) {
                                        $tagParts = preg_split('/\\s+/s', $v, 2);
                                        $tagParts[1] = $this->compileTagAttribs($params[0], $params[1]);
-                                       $parts[$k] = '<' . trim((strtolower($firstTagName) . ' ' . $tagParts[1])) . $tagEnd;
+                                       $parts[$k] = '<' . trim(strtolower($firstTagName) . ' ' . $tagParts[1]) . $tagEnd;
                                }
                        }
                }
@@ -1514,7 +1514,7 @@ class HtmlParser {
                                        // Set attributes: lowercase, always in quotes, with htmlspecialchars converted.
                                        $outA[] = $attrib_name . '="' . $this->bidir_htmlspecialchars($attrib_value, 2) . '"';
                                }
-                               $newTag = '<' . trim(($tagName . ' ' . implode(' ', $outA)));
+                               $newTag = '<' . trim($tagName . ' ' . implode(' ', $outA));
                                // All tags that are standalone (not wrapping, not having endtags) should be ended with '/>'
                                if (GeneralUtility::inList('img,br,hr,meta,link,base,area,input,param,col', $tagName) || substr($value, -2) == '/>') {
                                        $newTag .= ' />';
index dcd8b41..c49ef73 100644 (file)
@@ -1309,7 +1309,7 @@ class RteHtmlParser extends \TYPO3\CMS\Core\Html\HtmlParser {
                                                        if ($remapParagraphTag === 'DIV') {
                                                                $tagName = 'div';
                                                        }
-                                                       $subLines[$sk] = '<' . trim(($tagName . ' ' . $this->compileTagAttribs($newAttribs))) . '>' . $subLines[$sk] . '</' . $tagName . '>';
+                                                       $subLines[$sk] = '<' . trim($tagName . ' ' . $this->compileTagAttribs($newAttribs)) . '>' . $subLines[$sk] . '</' . $tagName . '>';
                                                }
                                        }
                                }
index dc79438..a8933df 100755 (executable)
@@ -5050,7 +5050,7 @@ Connection: close
                $valListCnt = count($valueList);
                foreach ($arr as $key => $value) {
                        if (!$valListCnt || in_array($key, $valueList)) {
-                               $str .= ((string)$key . trim((': ' . self::fixed_lgd_cs(str_replace(LF, '|', (string)$value), $valueLength)))) . '; ';
+                               $str .= (string)$key . trim(': ' . self::fixed_lgd_cs(str_replace(LF, '|', (string)$value), $valueLength)) . '; ';
                        }
                }
                return $str;
index 7bb9bda..8f551bc 100644 (file)
@@ -436,14 +436,14 @@ class Adodb extends AbstractCompiler {
                                                        switch (TRUE) {
                                                                case $this->databaseConnection->runningADOdbDriver('oci8'):
                                                                        // Oracle only knows BITAND(x,y) - sigh
-                                                                       $output .= 'BITAND(' . trim((($v['table'] ? $v['table'] . '.' : '') . $v['field'])) . ',' . $v['calc_value'][1] . $this->compileAddslashes($v['calc_value'][0]) . $v['calc_value'][1] . ')';
+                                                                       $output .= 'BITAND(' . trim(($v['table'] ? $v['table'] . '.' : '') . $v['field']) . ',' . $v['calc_value'][1] . $this->compileAddslashes($v['calc_value'][0]) . $v['calc_value'][1] . ')';
                                                                        break;
                                                                default:
                                                                        // MySQL, MS SQL Server, PostgreSQL support the &-syntax
-                                                                       $output .= trim((($v['table'] ? $v['table'] . '.' : '') . $v['field'])) . $v['calc'] . $v['calc_value'][1] . $this->compileAddslashes($v['calc_value'][0]) . $v['calc_value'][1];
+                                                                       $output .= trim(($v['table'] ? $v['table'] . '.' : '') . $v['field']) . $v['calc'] . $v['calc_value'][1] . $this->compileAddslashes($v['calc_value'][0]) . $v['calc_value'][1];
                                                        }
                                                } elseif ($v['calc']) {
-                                                       $output .= trim((($v['table'] ? $v['table'] . '.' : '') . $v['field'])) . $v['calc'];
+                                                       $output .= trim(($v['table'] ? $v['table'] . '.' : '') . $v['field']) . $v['calc'];
                                                        if (isset($v['calc_table'])) {
                                                                $output .= trim(($v['calc_table'] ? $v['calc_table'] . '.' : '') . $v['calc_field']);
                                                        } else {
index 08dbfb3..103720f 100644 (file)
@@ -261,7 +261,7 @@ class Mysql extends AbstractCompiler {
                                                $output .= ')';
                                        } else {
                                                // Set field/table with modifying prefix if any:
-                                               $output .= ' ' . trim(($v['modifier'] . ' ' . ($v['table'] ? $v['table'] . '.' : '') . $v['field']));
+                                               $output .= ' ' . trim($v['modifier'] . ' ' . ($v['table'] ? $v['table'] . '.' : '') . $v['field']);
                                                // Set calculation, if any:
                                                if ($v['calc']) {
                                                        $output .= $v['calc'] . $v['calc_value'][1] . $this->compileAddslashes($v['calc_value'][0]) . $v['calc_value'][1];
index e905a07..2411c7c 100755 (executable)
@@ -1900,7 +1900,7 @@ class ContentObjectRenderer {
                        $aTagParams = ' ' . $conf['ATagParams'];
                }
                if ($addGlobal) {
-                       $aTagParams = ' ' . trim(($GLOBALS['TSFE']->ATagParams . $aTagParams));
+                       $aTagParams = ' ' . trim($GLOBALS['TSFE']->ATagParams . $aTagParams);
                }
                // Extend params
                if (isset($GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['tslib/class.tslib_content.php']['getATagParamsPostProc']) && is_array($GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['tslib/class.tslib_content.php']['getATagParamsPostProc'])) {
index 7af826b..b4a9cff 100644 (file)
@@ -550,7 +550,7 @@ class AbstractPlugin {
                $wrapper['disabledLinkWrap'] = '<td nowrap="nowrap"><p>|</p></td>';
                $wrapper['inactiveLinkWrap'] = '<td nowrap="nowrap"><p>|</p></td>';
                $wrapper['activeLinkWrap'] = '<td' . $this->pi_classParam('browsebox-SCell') . ' nowrap="nowrap"><p>|</p></td>';
-               $wrapper['browseLinksWrap'] = trim(('<table ' . $tableParams)) . '><tr>|</tr></table>';
+               $wrapper['browseLinksWrap'] = rtrim('<table ' . $tableParams) . '><tr>|</tr></table>';
                $wrapper['showResultsWrap'] = '<p>|</p>';
                $wrapper['browseBoxWrap'] = '
                <!--
@@ -670,7 +670,7 @@ class AbstractPlugin {
                -->
                <div' . $this->pi_classParam('searchbox') . '>
                        <form action="' . htmlspecialchars(GeneralUtility::getIndpEnv('REQUEST_URI')) . '" method="post" style="margin: 0 0 0 0;">
-                       <' . trim(('table ' . $tableParams)) . '>
+                       <' . rtrim('table ' . $tableParams) . '>
                                <tr>
                                        <td><input type="text" name="' . $this->prefixId . '[sword]" value="' . htmlspecialchars($this->piVars['sword']) . '"' . $this->pi_classParam('searchbox-sword') . ' /></td>
                                        <td><input type="submit" value="' . $this->pi_getLL('pi_list_searchBox_search', 'Search', TRUE) . '"' . $this->pi_classParam('searchbox-button') . ' />' . '<input type="hidden" name="no_cache" value="1" />' . '<input type="hidden" name="' . $this->prefixId . '[pointer]" value="" />' . '</td>
@@ -700,7 +700,7 @@ class AbstractPlugin {
                        Mode selector (menu for list):
                -->
                <div' . $this->pi_classParam('modeSelector') . '>
-                       <' . trim(('table ' . $tableParams)) . '>
+                       <' . rtrim('table ' . $tableParams) . '>
                                <tr>
                                        ' . implode('', $cells) . '
                                </tr>
@@ -738,7 +738,7 @@ class AbstractPlugin {
                        Record list:
                -->
                <div' . $this->pi_classParam('listrow') . '>
-                       <' . trim(('table ' . $tableParams)) . '>
+                       <' . rtrim('table ' . $tableParams) . '>
                                ' . implode('', $tRows) . '
                        </table>
                </div>';