[TASK] Remove superfluous parenthesis in sysexts.
authorArvid Jakobsson <arvid.jakobsson@gmail.com>
Sun, 16 Sep 2012 18:01:59 +0000 (20:01 +0200)
committerJigal van Hemert <jigal@xs4all.nl>
Sun, 16 Sep 2012 19:53:31 +0000 (21:53 +0200)
Removes superfluous parenthesis in sysexts "cli", "cms",
"context_help" and "cshmanual".

Change ("a" . "b") . "c" into "a" . "b" . "c" using the lex-pass tool.

Change-Id: I0ff0fc33d3c0733498d5821e77b224db20213705
Resolves: #40963
Related: #40095
Reviewed-on: http://review.typo3.org/14621
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert
typo3/sysext/cli/Classes/CliBootstrap.php
typo3/sysext/cli/Classes/Controller/CommandLineController.php
typo3/sysext/cms/ext_localconf.php
typo3/sysext/cms/layout/db_layout.php
typo3/sysext/cms/layout/db_new_content_el.php
typo3/sysext/cms/tbl_tt_content.php
typo3/sysext/cms/tslib/content/class.tslib_content_searchresult.php
typo3/sysext/cms/tslib/index_ts.php
typo3/sysext/context_help/Classes/ExtDirect/ContextHelpDataProvider.php
typo3/sysext/cshmanual/Classes/Controller/HelpModuleController.php

index 4c0a7ab..ea7b0d3 100644 (file)
@@ -68,7 +68,7 @@ class CliBootstrap {
                        $cliKeys = array_keys($GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['GLOBAL']['cliKeys']);
                        asort($cliKeys);
                        foreach ($cliKeys as $key => $value) {
-                               $message .= ('  ' . $value) . LF;
+                               $message .= '  ' . $value . LF;
                        }
                        fwrite(STDERR, $message . LF);
                        die(1);
@@ -95,7 +95,7 @@ class CliBootstrap {
                        if (array_key_exists($var, $_SERVER)) {
                                echo 'SECURITY CHECK FAILED! This script cannot be used within your browser!' . chr(10);
                                echo 'If you are sure that we run in a shell or cronjob, please unset' . chr(10);
-                               echo (((('environment variable ' . $var) . ' (usually using \'unset ') . $var) . '\')') . chr(10);
+                               echo 'environment variable ' . $var . ' (usually using \'unset ' . $var . '\')' . chr(10);
                                echo 'before starting this script.' . chr(10);
                                die;
                        }
index 4d9871f..42eb84d 100644 (file)
@@ -143,7 +143,7 @@ class CommandLineController {
                        if ($token[0] === '-' && !\TYPO3\CMS\Core\Utility\MathUtility::canBeInterpretedAsInteger($token[1])) {
                                list($index, $opt) = explode('=', $token, 2);
                                if (isset($cli_options[$index])) {
-                                       echo (('ERROR: Option ' . $index) . ' was used twice!') . LF;
+                                       echo 'ERROR: Option ' . $index . ' was used twice!' . LF;
                                        die;
                                }
                                $cli_options[$index] = array();
@@ -176,21 +176,21 @@ class CommandLineController {
                                        if ($i > 0) {
                                                if (!isset($cli_args_copy[$argSplit[0]][($i - 1)]) && $v[0] != '[') {
                                                        // Using "[]" around a paramter makes it optional
-                                                       echo ((((('ERROR: Option "' . $argSplit[0]) . '" requires a value ("') . $v) . '") on position ') . $i) . LF;
+                                                       echo 'ERROR: Option "' . $argSplit[0] . '" requires a value ("' . $v . '") on position ' . $i . LF;
                                                        die;
                                                }
                                        }
                                }
                                $ii++;
                                if (isset($cli_args_copy[$argSplit[0]][$ii - 1])) {
-                                       echo ((('ERROR: Option "' . $argSplit[0]) . '" does not support a value on position ') . $ii) . LF;
+                                       echo 'ERROR: Option "' . $argSplit[0] . '" does not support a value on position ' . $ii . LF;
                                        die;
                                }
                                unset($cli_args_copy[$argSplit[0]]);
                        }
                }
                if (count($cli_args_copy)) {
-                       echo wordwrap((((((('ERROR: Option ' . implode(',', array_keys($cli_args_copy))) . ' was unknown to this script!') . LF) . '(Options are: ') . implode(', ', $allOptions)) . ')') . LF);
+                       echo wordwrap('ERROR: Option ' . implode(',', array_keys($cli_args_copy)) . ' was unknown to this script!' . LF . '(Options are: ' . implode(', ', $allOptions) . ')' . LF);
                        die;
                }
        }
@@ -262,7 +262,7 @@ class CommandLineController {
                        case 'synopsis':
                                $optStr = '';
                                foreach ($this->cli_options as $v) {
-                                       $optStr .= (' [' . $v[0]) . ']';
+                                       $optStr .= ' [' . $v[0] . ']';
                                }
                                $this->cli_echo($this->cli_indent(str_replace('###OPTIONS###', trim($optStr), $value), 4) . '
 
@@ -277,7 +277,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 c683545..2e71861 100644 (file)
@@ -174,7 +174,7 @@ $TYPO3_CONF_VARS['FE']['eID_include']['tx_cms_showpic'] = 'EXT:cms/tslib/showpic
 $TYPO3_CONF_VARS['SC_OPTIONS']['ext/install']['compat_version']['cms'] = array(
        'title' => 'CMS Frontend',
        'version' => 4000000,
-       'description' => ((('<ul>' . '<li><p>The extention simluatestatic has been removed in TYPO3 6.0</p></li>') . '<li><p>CSS Stylesheets and JavaScript are put into an external file by default.</p>') . '<p>Technically, that means that the default value of "config.inlineStyle2TempFile" is now set to "1" and that of "config.removeDefaultJS" to "external"</p></li>') . '</ul>'
+       'description' => '<ul>' . '<li><p>The extention simluatestatic has been removed in TYPO3 6.0</p></li>' . '<li><p>CSS Stylesheets and JavaScript are put into an external file by default.</p>' . '<p>Technically, that means that the default value of "config.inlineStyle2TempFile" is now set to "1" and that of "config.removeDefaultJS" to "external"</p></li>' . '</ul>'
 );
 // Registering hooks for the treelist cache
 $TYPO3_CONF_VARS['SC_OPTIONS']['t3lib/class.t3lib_tcemain.php']['processDatamapClass'][] = 'EXT:cms/tslib/hooks/class.tx_cms_treelistcacheupdate.php:&TYPO3\\CMS\\Frontend\\Hooks\\TreelistCacheUpdateHooks';
index 63a9619..b1631fb 100755 (executable)
@@ -72,8 +72,8 @@ class ext_posMap extends \TYPO3\CMS\Backend\Tree\View\PagePositionMap {
         * @todo Define visibility
         */
        public function wrapRecordTitle($str, $row) {
-               $aOnClick = ('jumpToUrl(\'' . $GLOBALS['SOBE']->local_linkThisScript(array('edit_record' => ('tt_content:' . $row['uid'])))) . '\');return false;';
-               return ((('<a href="#" onclick="' . htmlspecialchars($aOnClick)) . '">') . $str) . '</a>';
+               $aOnClick = 'jumpToUrl(\'' . $GLOBALS['SOBE']->local_linkThisScript(array('edit_record' => ('tt_content:' . $row['uid']))) . '\');return false;';
+               return '<a href="#" onclick="' . htmlspecialchars($aOnClick) . '">' . $str . '</a>';
        }
 
        /**
@@ -86,8 +86,8 @@ class ext_posMap extends \TYPO3\CMS\Backend\Tree\View\PagePositionMap {
         * @todo Define visibility
         */
        public function wrapColumnHeader($str, $vv) {
-               $aOnClick = ('jumpToUrl(\'' . $GLOBALS['SOBE']->local_linkThisScript(array('edit_record' => ('_EDIT_COL:' . $vv)))) . '\');return false;';
-               return ((('<a href="#" onclick="' . htmlspecialchars($aOnClick)) . '">') . $str) . '</a>';
+               $aOnClick = 'jumpToUrl(\'' . $GLOBALS['SOBE']->local_linkThisScript(array('edit_record' => ('_EDIT_COL:' . $vv))) . '\');return false;';
+               return '<a href="#" onclick="' . htmlspecialchars($aOnClick) . '">' . $str . '</a>';
        }
 
        /**
@@ -103,11 +103,11 @@ class ext_posMap extends \TYPO3\CMS\Backend\Tree\View\PagePositionMap {
         */
        public function onClickInsertRecord($row, $vv, $moveUid, $pid, $sys_lang = 0) {
                if (is_array($row)) {
-                       $location = $GLOBALS['SOBE']->local_linkThisScript(array('edit_record' => (('tt_content:new/-' . $row['uid']) . '/') . $row['colPos']));
+                       $location = $GLOBALS['SOBE']->local_linkThisScript(array('edit_record' => 'tt_content:new/-' . $row['uid'] . '/' . $row['colPos']));
                } else {
-                       $location = $GLOBALS['SOBE']->local_linkThisScript(array('edit_record' => (('tt_content:new/' . $pid) . '/') . $vv));
+                       $location = $GLOBALS['SOBE']->local_linkThisScript(array('edit_record' => 'tt_content:new/' . $pid . '/' . $vv));
                }
-               return ('jumpToUrl(\'' . $location) . '\');return false;';
+               return 'jumpToUrl(\'' . $location . '\');return false;';
        }
 
        /**
@@ -120,7 +120,7 @@ class ext_posMap extends \TYPO3\CMS\Backend\Tree\View\PagePositionMap {
         */
        public function wrapRecordHeader($str, $row) {
                if ($row['uid'] == $this->moveUid) {
-                       return (('<img' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg($GLOBALS['BACK_PATH'], 'gfx/content_client.gif', 'width="7" height="10"')) . ' alt="" />') . $str;
+                       return '<img' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg($GLOBALS['BACK_PATH'], 'gfx/content_client.gif', 'width="7" height="10"') . ' alt="" />' . $str;
                } else {
                        return $str;
                }
index 9c820da..44736c5 100644 (file)
@@ -85,8 +85,8 @@ class ext_posMap extends \TYPO3\CMS\Backend\Tree\View\PagePositionMap {
         */
        public function onClickInsertRecord($row, $vv, $moveUid, $pid, $sys_lang = 0) {
                $table = 'tt_content';
-               $location = ((((((($this->backPath . 'alt_doc.php?edit[tt_content][') . (is_array($row) ? -$row['uid'] : $pid)) . ']=new&defVals[tt_content][colPos]=') . $vv) . '&defVals[tt_content][sys_language_uid]=') . $sys_lang) . '&returnUrl=') . rawurlencode($GLOBALS['SOBE']->R_URI);
-               return ('window.location.href=\'' . $location) . '\'+document.editForm.defValues.value; return false;';
+               $location = $this->backPath . 'alt_doc.php?edit[tt_content][' . (is_array($row) ? -$row['uid'] : $pid) . ']=new&defVals[tt_content][colPos]=' . $vv . '&defVals[tt_content][sys_language_uid]=' . $sys_lang . '&returnUrl=' . rawurlencode($GLOBALS['SOBE']->R_URI);
+               return 'window.location.href=\'' . $location . '\'+document.editForm.defValues.value; return false;';
        }
 
 }
index e89f79f..3e8265d 100755 (executable)
@@ -1784,12 +1784,12 @@ $TCA['tt_content'] = array(
                                --div--;LLL:EXT:cms/locallang_ttc.xml:tabs.extended'
                ),
                'textpic' => array(
-                       'showitem' => ('--palette--;LLL:EXT:cms/locallang_ttc.xml:palette.general;general,
+                       'showitem' => '--palette--;LLL:EXT:cms/locallang_ttc.xml:palette.general;general,
                                        --palette--;LLL:EXT:cms/locallang_ttc.xml:palette.header;header,
                                        bodytext;Text;;richtext:rte_transform[flag=rte_enabled|mode=ts_css],
                                        rte_enabled;LLL:EXT:cms/locallang_ttc.xml:rte_enabled_formlabel,' . '--div--;LLL:EXT:cms/locallang_ttc.xml:tabs.images,
                                        image,
-                                       --palette--;LLL:EXT:cms/locallang_ttc.xml:palette.imagelinks;imagelinks,') . '--div--;LLL:EXT:cms/locallang_ttc.xml:tabs.appearance,
+                                       --palette--;LLL:EXT:cms/locallang_ttc.xml:palette.imagelinks;imagelinks,' . '--div--;LLL:EXT:cms/locallang_ttc.xml:tabs.appearance,
                                        --palette--;LLL:EXT:cms/locallang_ttc.xml:palette.frames;frames,
                                        --palette--;LLL:EXT:cms/locallang_ttc.xml:palette.image_settings;image_settings,
                                        --palette--;LLL:EXT:cms/locallang_ttc.xml:palette.imageblock;imageblock,
index 664c0ca..24f1257 100644 (file)
@@ -95,13 +95,13 @@ class SearchResultContentObject extends \TYPO3\CMS\Frontend\ContentObject\Abstra
                        }
                        // Generate page-tree
                        $search->pageIdList .= $this->cObj->getTreeList(-1 * $theStartId, $depth);
-                       $endClause = (((((('pages.uid IN (' . $search->pageIdList) . ')
-                               AND pages.doktype in (') . $GLOBALS['TYPO3_CONF_VARS']['FE']['content_doktypes']) . ($conf['addExtUrlsAndShortCuts'] ? ',3,4' : '')) . ')
-                               AND pages.no_search=0') . $this->cObj->enableFields($search->fTable)) . $this->cObj->enableFields('pages');
+                       $endClause = 'pages.uid IN (' . $search->pageIdList . ')
+                               AND pages.doktype in (' . $GLOBALS['TYPO3_CONF_VARS']['FE']['content_doktypes'] . ($conf['addExtUrlsAndShortCuts'] ? ',3,4' : '') . ')
+                               AND pages.no_search=0' . $this->cObj->enableFields($search->fTable) . $this->cObj->enableFields('pages');
                        if ($conf['languageField.'][$search->fTable]) {
                                // (using sys_language_uid which is the ACTUAL language of the page.
                                // sys_language_content is only for selecting DISPLAY content!)
-                               $endClause .= ((((' AND ' . $search->fTable) . '.') . $conf['languageField.'][$search->fTable]) . ' = ') . intval($GLOBALS['TSFE']->sys_language_uid);
+                               $endClause .= ' AND ' . $search->fTable . '.' . $conf['languageField.'][$search->fTable] . ' = ' . intval($GLOBALS['TSFE']->sys_language_uid);
                        }
                        // Build query
                        $search->build_search_query($endClause);
@@ -124,7 +124,7 @@ class SearchResultContentObject extends \TYPO3\CMS\Frontend\ContentObject\Abstra
                        if (!$noOrderBy) {
                                $search->queryParts['ORDERBY'] = 'pages.lastUpdated, pages.tstamp';
                        }
-                       $search->queryParts['LIMIT'] = ($spointer . ',') . $theRange;
+                       $search->queryParts['LIMIT'] = $spointer . ',' . $theRange;
                        // Search...
                        $search->execute_query();
                        if ($GLOBALS['TYPO3_DB']->sql_num_rows($search->result)) {
@@ -135,8 +135,8 @@ class SearchResultContentObject extends \TYPO3\CMS\Frontend\ContentObject\Abstra
                                // prev/next url:
                                $target = isset($conf['target.']) ? $this->cObj->stdWrap($conf['target'], $conf['target.']) : $conf['target'];
                                $LD = $GLOBALS['TSFE']->tmpl->linkData($GLOBALS['TSFE']->page, $target, 1, '', '', $this->cObj->getClosestMPvalueForPage($GLOBALS['TSFE']->page['uid']));
-                               $targetPart = $LD['target'] ? (' target="' . htmlspecialchars($LD['target'])) . '"' : '';
-                               $urlParams = $this->cObj->URLqMark($LD['totalURL'], (((((('&sword=' . rawurlencode(\TYPO3\CMS\Core\Utility\GeneralUtility::_GP('sword'))) . '&scols=') . rawurlencode(\TYPO3\CMS\Core\Utility\GeneralUtility::_GP('scols'))) . '&stype=') . rawurlencode(\TYPO3\CMS\Core\Utility\GeneralUtility::_GP('stype'))) . '&scount=') . $total);
+                               $targetPart = $LD['target'] ? ' target="' . htmlspecialchars($LD['target']) . '"' : '';
+                               $urlParams = $this->cObj->URLqMark($LD['totalURL'], '&sword=' . rawurlencode(\TYPO3\CMS\Core\Utility\GeneralUtility::_GP('sword')) . '&scols=' . rawurlencode(\TYPO3\CMS\Core\Utility\GeneralUtility::_GP('scols')) . '&stype=' . rawurlencode(\TYPO3\CMS\Core\Utility\GeneralUtility::_GP('stype')) . '&scount=' . $total);
                                // substitution:
                                $result = $this->cObj->cObjGetSingle($conf['layout'], $conf['layout.'], 'layout');
                                $result = str_replace('###RANGELOW###', $rangeLow, $result);
@@ -144,14 +144,14 @@ class SearchResultContentObject extends \TYPO3\CMS\Frontend\ContentObject\Abstra
                                $result = str_replace('###TOTAL###', $total, $result);
                                if ($rangeHigh < $total) {
                                        $next = $this->cObj->cObjGetSingle($conf['next'], $conf['next.'], 'next');
-                                       $next = (((((('<a href="' . htmlspecialchars((($urlParams . '&spointer=') . ($spointer + $theRange)))) . '"') . $targetPart) . $GLOBALS['TSFE']->ATagParams) . '>') . $next) . '</a>';
+                                       $next = '<a href="' . htmlspecialchars(($urlParams . '&spointer=' . ($spointer + $theRange))) . '"' . $targetPart . $GLOBALS['TSFE']->ATagParams . '>' . $next . '</a>';
                                } else {
                                        $next = '';
                                }
                                $result = str_replace('###NEXT###', $next, $result);
                                if ($rangeLow > 1) {
                                        $prev = $this->cObj->cObjGetSingle($conf['prev'], $conf['prev.'], 'prev');
-                                       $prev = (((((('<a href="' . htmlspecialchars((($urlParams . '&spointer=') . ($spointer - $theRange)))) . '"') . $targetPart) . $GLOBALS['TSFE']->ATagParams) . '>') . $prev) . '</a>';
+                                       $prev = '<a href="' . htmlspecialchars(($urlParams . '&spointer=' . ($spointer - $theRange))) . '"' . $targetPart . $GLOBALS['TSFE']->ATagParams . '>' . $prev . '</a>';
                                } else {
                                        $prev = '';
                                }
index 5a7c487..3a257b8 100644 (file)
@@ -235,7 +235,7 @@ $TSFE->storeSessionData();
 $TYPO3_MISC['microtime_end'] = microtime(TRUE);
 $TSFE->setParseTime();
 if ($TSFE->isOutputting() && (!empty($TSFE->TYPO3_CONF_VARS['FE']['debug']) || !empty($TSFE->config['config']['debug']))) {
-       $TSFE->content .= ((LF . '<!-- Parsetime: ') . $TSFE->scriptParseTime) . 'ms -->';
+       $TSFE->content .= LF . '<!-- Parsetime: ' . $TSFE->scriptParseTime . 'ms -->';
 }
 // Check JumpUrl
 $TSFE->jumpurl();
@@ -250,7 +250,7 @@ $TT->pull();
 // beLoginLinkIPList
 echo $TSFE->beLoginLinkIPList();
 // Admin panel
-if ((is_object($BE_USER) && $BE_USER->isAdminPanelVisible()) && $TSFE->isBackendUserLoggedIn()) {
+if (is_object($BE_USER) && $BE_USER->isAdminPanelVisible() && $TSFE->isBackendUserLoggedIn()) {
        $TSFE->content = str_ireplace('</head>', $BE_USER->adminPanel->getAdminPanelHeaderData() . '</head>', $TSFE->content);
        $TSFE->content = str_ireplace('</body>', $BE_USER->displayAdminPanel() . '</body>', $TSFE->content);
 }
@@ -258,7 +258,7 @@ if ($sendTSFEContent) {
        echo $TSFE->content;
 }
 // Debugging Output
-if ((isset($error) && is_object($error)) && @is_callable(array($error, 'debugOutput'))) {
+if (isset($error) && is_object($error) && @is_callable(array($error, 'debugOutput'))) {
        $error->debugOutput();
 }
 if (TYPO3_DLOG) {
index 4d86270..241e6bc 100644 (file)
@@ -47,8 +47,8 @@ class ContextHelpDataProvider {
                $moreIcon = $helpTextArray['moreInfo'] ? \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('actions-view-go-forward') : '';
                return array(
                        'title' => $helpTextArray['title'],
-                       'description' => (((('<p class="t3-help-short' . ($moreIcon ? ' tipIsLinked' : '')) . '">') . $helpTextArray['description']) . $moreIcon) . '</p>',
-                       'id' => ($table . '.') . $field,
+                       'description' => '<p class="t3-help-short' . ($moreIcon ? ' tipIsLinked' : '') . '">' . $helpTextArray['description'] . $moreIcon . '</p>',
+                       'id' => $table . '.' . $field,
                        'moreInfo' => $helpTextArray['moreInfo']
                );
        }
@@ -71,14 +71,14 @@ class ContextHelpDataProvider {
                                        'description' => NULL,
                                        'title' => NULL,
                                        'moreInfo' => FALSE,
-                                       'id' => ($table . '.') . $field
+                                       'id' => $table . '.' . $field
                                );
                                // Add alternative title, if defined
                                if ($data['alttitle']) {
                                        $output[$field]['title'] = $data['alttitle'];
                                }
                                // If we have more information to show
-                               if ((($data['image_descr'] || $data['seeAlso']) || $data['details']) || $data['syntax']) {
+                               if ($data['image_descr'] || $data['seeAlso'] || $data['details'] || $data['syntax']) {
                                        $output[$field]['moreInfo'] = TRUE;
                                }
                                // Add description
index 58511c0..1d8cdb3 100644 (file)
@@ -192,7 +192,7 @@ class HelpModuleController {
                        }
                        if (is_array($info['sub'])) {
                                foreach ($info['sub'] as $subMod => $subInfo) {
-                                       $cshKey = (('_MOD_' . $mainMod) . '_') . $subMod;
+                                       $cshKey = '_MOD_' . $mainMod . '_' . $subMod;
                                        if ($CSHkeys[$cshKey]) {
                                                $GLOBALS['LANG']->loadSingleTableDescription($cshKey);
                                                $this->render_TOC_el($cshKey, 'modules', $outputSections, $tocArray, $CSHkeys);
@@ -231,31 +231,31 @@ class HelpModuleController {
                }
                // COMPILE output:
                $output = '';
-               $output .= ('
+               $output .= '
 
-                       <h1>' . $GLOBALS['LANG']->getLL('manual_title', 1)) . '</h1>';
-               $output .= ((('
+                       <h1>' . $GLOBALS['LANG']->getLL('manual_title', 1) . '</h1>';
+               $output .= '
 
-                       <h2>' . $GLOBALS['LANG']->getLL('introduction', 1)) . '</h2>
-                       <p>') . $GLOBALS['LANG']->getLL('description', 1)) . '</p>';
-               $output .= (('
+                       <h2>' . $GLOBALS['LANG']->getLL('introduction', 1) . '</h2>
+                       <p>' . $GLOBALS['LANG']->getLL('description', 1) . '</p>';
+               $output .= '
 
-                       <h2>' . $GLOBALS['LANG']->getLL('TOC', 1)) . '</h2>') . $this->render_TOC_makeTocList($tocArray);
+                       <h2>' . $GLOBALS['LANG']->getLL('TOC', 1) . '</h2>' . $this->render_TOC_makeTocList($tocArray);
                if (!$this->renderALL) {
-                       $output .= ('
+                       $output .= '
                                <br/>
-                               <p class="c-nav"><a href="mod.php?M=help_cshmanual&renderALL=1">' . $GLOBALS['LANG']->getLL('full_manual', 1)) . '</a></p>';
+                               <p class="c-nav"><a href="mod.php?M=help_cshmanual&renderALL=1">' . $GLOBALS['LANG']->getLL('full_manual', 1) . '</a></p>';
                }
                if ($this->renderALL) {
-                       $output .= (('
+                       $output .= '
 
-                               <h2>' . $GLOBALS['LANG']->getLL('full_manual_chapters', 1)) . '</h2>') . implode('
+                               <h2>' . $GLOBALS['LANG']->getLL('full_manual_chapters', 1) . '</h2>' . implode('
 
 
                                <!-- NEW SECTION: -->
                                ', $outputSections);
                }
-               $output .= ('<hr /><p class="manual-title">' . \TYPO3\CMS\Backend\Utility\BackendUtility::TYPO3_copyRightNotice()) . '</p>';
+               $output .= '<hr /><p class="manual-title">' . \TYPO3\CMS\Backend\Utility\BackendUtility::TYPO3_copyRightNotice() . '</p>';
                return $output;
        }
 
@@ -275,20 +275,20 @@ class HelpModuleController {
                if ($this->renderALL) {
                        $outputSections[$table] = $this->render_Table($table);
                        if ($outputSections[$table]) {
-                               $outputSections[$table] = (((((((('
+                               $outputSections[$table] = '
 
-               <!-- New CSHkey/Table: ' . $table) . ' -->
-               <p class="c-nav"><a name="ANCHOR_') . $table) . '" href="#">') . $GLOBALS['LANG']->getLL('to_top', 1)) . '</a></p>
-               <h2>') . $this->getTableFieldLabel($table)) . '</h2>
+               <!-- New CSHkey/Table: ' . $table . ' -->
+               <p class="c-nav"><a name="ANCHOR_' . $table . '" href="#">' . $GLOBALS['LANG']->getLL('to_top', 1) . '</a></p>
+               <h2>' . $this->getTableFieldLabel($table) . '</h2>
 
-               ') . $outputSections[$table];
-                               $tocArray[$tocCat][$table] = ((('<a href="#ANCHOR_' . $table) . '">') . $this->getTableFieldLabel($table)) . '</a>';
+               ' . $outputSections[$table];
+                               $tocArray[$tocCat][$table] = '<a href="#ANCHOR_' . $table . '">' . $this->getTableFieldLabel($table) . '</a>';
                        } else {
                                unset($outputSections[$table]);
                        }
                } else {
                        // Only TOC:
-                       $tocArray[$tocCat][$table] = ((('<p><a href="mod.php?M=help_cshmanual&tfID=' . rawurlencode(($table . '.*'))) . '">') . $this->getTableFieldLabel($table)) . '</a></p>';
+                       $tocArray[$tocCat][$table] = '<p><a href="mod.php?M=help_cshmanual&tfID=' . rawurlencode(($table . '.*')) . '">' . $this->getTableFieldLabel($table) . '</a></p>';
                }
                // Unset CSH key:
                unset($CSHkeys[$table]);
@@ -308,22 +308,22 @@ class HelpModuleController {
                $output = '';
                foreach ($keys as $tocKey) {
                        if (is_array($tocArray[$tocKey])) {
-                               $output .= ((('
-                                       <li>' . $GLOBALS['LANG']->getLL(('TOC_' . $tocKey), 1)) . '
+                               $output .= '
+                                       <li>' . $GLOBALS['LANG']->getLL(('TOC_' . $tocKey), 1) . '
                                                <ul>
-                                                       <li>') . implode('</li>
-                                                       <li>', $tocArray[$tocKey])) . '</li>
+                                                       <li>' . implode('</li>
+                                                       <li>', $tocArray[$tocKey]) . '</li>
                                                </ul>
                                        </li>';
                        }
                }
                // Compile TOC:
-               $output = ('
+               $output = '
 
                        <!-- TOC: -->
                        <div class="c-toc">
                                <ul>
-                               ' . $output) . '
+                               ' . $output . '
                                </ul>
                        </div>';
                return $output;
@@ -355,7 +355,7 @@ class HelpModuleController {
                        // Traverse table columns as listed in TCA_DESCR
                        foreach ($GLOBALS['TCA_DESCR'][$key]['columns'] as $field => $_) {
                                $fieldValue = isset($GLOBALS['TCA'][$key]) && strcmp($field, '') ? $GLOBALS['TCA'][$key]['columns'][$field] : array();
-                               if (is_array($fieldValue) && ((!$this->limitAccess || !$fieldValue['exclude']) || $GLOBALS['BE_USER']->check('non_exclude_fields', ($table . ':') . $field))) {
+                               if (is_array($fieldValue) && (!$this->limitAccess || !$fieldValue['exclude'] || $GLOBALS['BE_USER']->check('non_exclude_fields', $table . ':' . $field))) {
                                        if (!$field) {
                                                // Header
                                                $parts[0] = $this->printItem($key, '', 1);
@@ -374,10 +374,10 @@ class HelpModuleController {
                $output = $this->substituteGlossaryWords($output);
                // TOC link:
                if (!$this->renderALL) {
-                       $tocLink = ('<p class="c-nav"><a href="mod.php?M=help_cshmanual">' . $GLOBALS['LANG']->getLL('goToToc', 1)) . '</a></p>';
-                       $output = ((($tocLink . '
-                               <br/>') . $output) . '
-                               <br />') . $tocLink;
+                       $tocLink = '<p class="c-nav"><a href="mod.php?M=help_cshmanual">' . $GLOBALS['LANG']->getLL('goToToc', 1) . '</a></p>';
+                       $output = $tocLink . '
+                               <br/>' . $output . '
+                               <br />' . $tocLink;
                }
                return $output;
        }
@@ -400,9 +400,9 @@ class HelpModuleController {
                $output = $this->substituteGlossaryWords($output);
                // Link to Full table description and TOC:
                $getLLKey = $this->limitAccess ? 'fullDescription' : 'fullDescription_module';
-               $output .= ((((('<br />
-                       <p class="c-nav"><a href="mod.php?M=help_cshmanual&tfID=' . rawurlencode(($key . '.*'))) . '">') . $GLOBALS['LANG']->getLL($getLLKey, 1)) . '</a></p>
-                       <p class="c-nav"><a href="mod.php?M=help_cshmanual">') . $GLOBALS['LANG']->getLL('goToToc', 1)) . '</a></p>';
+               $output .= '<br />
+                       <p class="c-nav"><a href="mod.php?M=help_cshmanual&tfID=' . rawurlencode(($key . '.*')) . '">' . $GLOBALS['LANG']->getLL($getLLKey, 1) . '</a></p>
+                       <p class="c-nav"><a href="mod.php?M=help_cshmanual">' . $GLOBALS['LANG']->getLL('goToToc', 1) . '</a></p>';
                return $output;
        }
 
@@ -421,7 +421,7 @@ class HelpModuleController {
         */
        public function make_seeAlso($value, $anchorTable = '') {
                // Split references by comma or linebreak
-               $items = preg_split(('/[,' . LF) . ']/', $value);
+               $items = preg_split('/[,' . LF . ']/', $value);
                $lines = array();
                foreach ($items as $val) {
                        $val = trim($val);
@@ -430,23 +430,23 @@ class HelpModuleController {
                                $iPUrl = \TYPO3\CMS\Core\Utility\GeneralUtility::trimExplode('|', $val);
                                // URL reference:
                                if (substr($iPUrl[1], 0, 4) == 'http') {
-                                       $lines[] = ((('<a href="' . htmlspecialchars($iPUrl[1])) . '" target="_blank"><em>') . htmlspecialchars($iPUrl[0])) . '</em></a>';
+                                       $lines[] = '<a href="' . htmlspecialchars($iPUrl[1]) . '" target="_blank"><em>' . htmlspecialchars($iPUrl[0]) . '</em></a>';
                                } elseif (substr($iPUrl[1], 0, 5) == 'FILE:') {
                                        $fileName = \TYPO3\CMS\Core\Utility\GeneralUtility::getFileAbsFileName(substr($iPUrl[1], 5), 1, 1);
                                        if ($fileName && @is_file($fileName)) {
                                                $fileName = '../' . substr($fileName, strlen(PATH_site));
-                                               $lines[] = ((('<a href="' . htmlspecialchars($fileName)) . '" target="_blank"><em>') . htmlspecialchars($iPUrl[0])) . '</em></a>';
+                                               $lines[] = '<a href="' . htmlspecialchars($fileName) . '" target="_blank"><em>' . htmlspecialchars($iPUrl[0]) . '</em></a>';
                                        }
                                } else {
                                        // "table" reference
                                        \TYPO3\CMS\Core\Utility\GeneralUtility::loadTCA($iP[0]);
-                                       if (!isset($GLOBALS['TCA'][$iP[0]]) || (!$iP[1] || is_array($GLOBALS['TCA'][$iP[0]]['columns'][$iP[1]])) && (!$this->limitAccess || $GLOBALS['BE_USER']->check('tables_select', $iP[0]) && ((!$iP[1] || !$GLOBALS['TCA'][$iP[0]]['columns'][$iP[1]]['exclude']) || $GLOBALS['BE_USER']->check('non_exclude_fields', ($iP[0] . ':') . $iP[1])))) {
+                                       if (!isset($GLOBALS['TCA'][$iP[0]]) || (!$iP[1] || is_array($GLOBALS['TCA'][$iP[0]]['columns'][$iP[1]])) && (!$this->limitAccess || $GLOBALS['BE_USER']->check('tables_select', $iP[0]) && (!$iP[1] || !$GLOBALS['TCA'][$iP[0]]['columns'][$iP[1]]['exclude'] || $GLOBALS['BE_USER']->check('non_exclude_fields', $iP[0] . ':' . $iP[1])))) {
                                                // Checking read access:
                                                if (isset($GLOBALS['TCA_DESCR'][$iP[0]])) {
                                                        // Make see-also link:
-                                                       $href = $this->renderALL || $anchorTable && $iP[0] == $anchorTable ? '#' . implode('.', $iP) : (('mod.php?M=help_cshmanual&tfID=' . rawurlencode(implode('.', $iP))) . '&back=') . $this->tfID;
+                                                       $href = $this->renderALL || $anchorTable && $iP[0] == $anchorTable ? '#' . implode('.', $iP) : 'mod.php?M=help_cshmanual&tfID=' . rawurlencode(implode('.', $iP)) . '&back=' . $this->tfID;
                                                        $label = $this->getTableFieldLabel($iP[0], $iP[1], ' / ');
-                                                       $lines[] = ((('<a href="' . htmlspecialchars($href)) . '">') . htmlspecialchars($label)) . '</a>';
+                                                       $lines[] = '<a href="' . htmlspecialchars($href) . '">' . htmlspecialchars($label) . '</a>';
                                                }
                                        }
                                }
@@ -477,15 +477,15 @@ class HelpModuleController {
                                        $imgInfo = @getimagesize($absImagePath);
                                        if (is_array($imgInfo)) {
                                                $imgFile = '../' . $imgFile;
-                                               $code .= ((('<br /><img src="' . $imgFile) . '" ') . $imgInfo[3]) . ' class="c-inlineimg" alt="" /><br />
+                                               $code .= '<br /><img src="' . $imgFile . '" ' . $imgInfo[3] . ' class="c-inlineimg" alt="" /><br />
                                                ';
-                                               $code .= ('<p><em>' . htmlspecialchars($descr)) . '</em></p>
+                                               $code .= '<p><em>' . htmlspecialchars($descr) . '</em></p>
                                                ';
                                        } else {
-                                               $code .= ('<div style="background-color: red; border: 1px solid black; color: white;">NOT AN IMAGE: ' . $imgFile) . '</div>';
+                                               $code .= '<div style="background-color: red; border: 1px solid black; color: white;">NOT AN IMAGE: ' . $imgFile . '</div>';
                                        }
                                } else {
-                                       $code .= ('<div style="background-color: red; border: 1px solid black; color: white;">IMAGE FILE NOT FOUND: ' . $image) . '</div>';
+                                       $code .= '<div style="background-color: red; border: 1px solid black; color: white;">IMAGE FILE NOT FOUND: ' . $image . '</div>';
                                }
                        }
                }
@@ -503,11 +503,11 @@ class HelpModuleController {
        public function headerLine($str, $type = 0) {
                switch ($type) {
                case 1:
-                       $str = ('<h2 class="t3-row-header">' . htmlspecialchars($str)) . '</h2>
+                       $str = '<h2 class="t3-row-header">' . htmlspecialchars($str) . '</h2>
                                ';
                        break;
                case 0:
-                       $str = ('<h3 class="divider">' . htmlspecialchars($str)) . '</h3>
+                       $str = '<h3 class="divider">' . htmlspecialchars($str) . '</h3>
                                ';
                        break;
                }
@@ -522,7 +522,7 @@ class HelpModuleController {
         * @todo Define visibility
         */
        public function prepareContent($str) {
-               return ('<p>' . nl2br(trim(strip_tags($str, $this->allowedHTML)))) . '</p>
+               return '<p>' . nl2br(trim(strip_tags($str, $this->allowedHTML))) . '</p>
                ';
        }
 
@@ -544,7 +544,7 @@ class HelpModuleController {
                        // Make seeAlso references.
                        $seeAlsoRes = $this->make_seeAlso($GLOBALS['TCA_DESCR'][$key]['columns'][$field]['seeAlso'], $anchors ? $key : '');
                        // Making item:
-                       $out = ((((((((((('<a name="' . $key) . '.') . $field) . '"></a>') . $this->headerLine($this->getTableFieldLabel($key, $field), 1)) . $this->prepareContent($GLOBALS['TCA_DESCR'][$key]['columns'][$field]['description'])) . ($GLOBALS['TCA_DESCR'][$key]['columns'][$field]['details'] ? $this->headerLine(($GLOBALS['LANG']->getLL('details') . ':')) . $this->prepareContent($GLOBALS['TCA_DESCR'][$key]['columns'][$field]['details']) : '')) . ($GLOBALS['TCA_DESCR'][$key]['columns'][$field]['syntax'] ? $this->headerLine(($GLOBALS['LANG']->getLL('syntax') . ':')) . $this->prepareContent($GLOBALS['TCA_DESCR'][$key]['columns'][$field]['syntax']) : '')) . ($GLOBALS['TCA_DESCR'][$key]['columns'][$field]['image'] ? $this->printImage($GLOBALS['TCA_DESCR'][$key]['columns'][$field]['image'], $GLOBALS['TCA_DESCR'][$key]['columns'][$field]['image_descr']) : '')) . ($GLOBALS['TCA_DESCR'][$key]['columns'][$field]['seeAlso'] && $seeAlsoRes ? (($this->headerLine(($GLOBALS['LANG']->getLL('seeAlso') . ':')) . '<p>') . $seeAlsoRes) . '</p>' : '')) . ($this->back ? ((('<br /><p><a href="' . htmlspecialchars(('mod.php?M=help_cshmanual&tfID=' . rawurlencode($this->back)))) . '" class="typo3-goBack">') . htmlspecialchars($GLOBALS['LANG']->getLL('goBack'))) . '</a></p>' : '')) . '<br />';
+                       $out = '<a name="' . $key . '.' . $field . '"></a>' . $this->headerLine($this->getTableFieldLabel($key, $field), 1) . $this->prepareContent($GLOBALS['TCA_DESCR'][$key]['columns'][$field]['description']) . ($GLOBALS['TCA_DESCR'][$key]['columns'][$field]['details'] ? $this->headerLine(($GLOBALS['LANG']->getLL('details') . ':')) . $this->prepareContent($GLOBALS['TCA_DESCR'][$key]['columns'][$field]['details']) : '') . ($GLOBALS['TCA_DESCR'][$key]['columns'][$field]['syntax'] ? $this->headerLine(($GLOBALS['LANG']->getLL('syntax') . ':')) . $this->prepareContent($GLOBALS['TCA_DESCR'][$key]['columns'][$field]['syntax']) : '') . ($GLOBALS['TCA_DESCR'][$key]['columns'][$field]['image'] ? $this->printImage($GLOBALS['TCA_DESCR'][$key]['columns'][$field]['image'], $GLOBALS['TCA_DESCR'][$key]['columns'][$field]['image_descr']) : '') . ($GLOBALS['TCA_DESCR'][$key]['columns'][$field]['seeAlso'] && $seeAlsoRes ? $this->headerLine(($GLOBALS['LANG']->getLL('seeAlso') . ':')) . '<p>' . $seeAlsoRes . '</p>' : '') . ($this->back ? '<br /><p><a href="' . htmlspecialchars(('mod.php?M=help_cshmanual&tfID=' . rawurlencode($this->back))) . '" class="typo3-goBack">' . htmlspecialchars($GLOBALS['LANG']->getLL('goBack')) . '</a></p>' : '') . '<br />';
                }
                return $out;
        }
@@ -633,7 +633,7 @@ class HelpModuleController {
                                                // Traverse table columns as listed in TCA_DESCR
                                                foreach ($GLOBALS['TCA_DESCR'][$cshKey]['columns'] as $field => $data) {
                                                        if ($field) {
-                                                               $this->glossaryWords[($cshKey . '.') . $field] = array(
+                                                               $this->glossaryWords[$cshKey . '.' . $field] = array(
                                                                        'title' => trim($data['alttitle'] ? $data['alttitle'] : $cshKey),
                                                                        'description' => str_replace('%22', '%23%23%23', rawurlencode($data['description']))
                                                                );
@@ -681,13 +681,13 @@ class HelpModuleController {
         * @todo Define visibility
         */
        public function substituteGlossaryWords_htmlcleaner_callback($code) {
-               if ((is_array($this->substWords) && count($this->substWords)) && strlen(trim($code))) {
+               if (is_array($this->substWords) && count($this->substWords) && strlen(trim($code))) {
                        // Substitute words:
                        foreach ($this->substWords as $wordKey => $wordSet) {
                                // quoteMeta used so special chars (which should not occur though) in words will not break the regex. Seemed to work (- kasper)
-                               $parts = preg_split(('/( |[\\(])(' . quoteMeta($wordSet['title'])) . ')([\\.\\!\\)\\?\\:\\,]+| )/i', (' ' . $code) . ' ', 2, PREG_SPLIT_DELIM_CAPTURE);
+                               $parts = preg_split('/( |[\\(])(' . quoteMeta($wordSet['title']) . ')([\\.\\!\\)\\?\\:\\,]+| )/i', ' ' . $code . ' ', 2, PREG_SPLIT_DELIM_CAPTURE);
                                if (count($parts) == 5) {
-                                       $parts[2] = ((((('<a class="glossary-term" href="' . htmlspecialchars(((('mod.php?M=help_cshmanual&tfID=' . rawurlencode($wordSet['key'])) . '&back=') . $this->tfID))) . '" title="') . rawurlencode(htmlspecialchars(\TYPO3\CMS\Core\Utility\GeneralUtility::fixed_lgd_cs(rawurldecode($wordSet['description']), 80)))) . '">') . htmlspecialchars($parts[2])) . '</a>';
+                                       $parts[2] = '<a class="glossary-term" href="' . htmlspecialchars(('mod.php?M=help_cshmanual&tfID=' . rawurlencode($wordSet['key']) . '&back=' . $this->tfID)) . '" title="' . rawurlencode(htmlspecialchars(\TYPO3\CMS\Core\Utility\GeneralUtility::fixed_lgd_cs(rawurldecode($wordSet['description']), 80))) . '">' . htmlspecialchars($parts[2]) . '</a>';
                                        $code = substr(implode('', $parts), 1, -1);
                                        // Disable entry so it doesn't get used next time:
                                        unset($this->substWords[$wordKey]);