[TASK] Remove unneeded parenthesis on array-access 12/44812/2
authorStefan Neufeind <typo3.neufeind@speedpartner.de>
Fri, 20 Nov 2015 10:00:57 +0000 (11:00 +0100)
committerStefan Neufeind <typo3.neufeind@speedpartner.de>
Fri, 20 Nov 2015 10:29:16 +0000 (11:29 +0100)
Change-Id: I450ef2985bf1086fb8cc247406a94cce8971f061
Resolves: #71714
Releases: master
Reviewed-on: https://review.typo3.org/44812
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
46 files changed:
typo3/sysext/backend/Classes/Backend/ToolbarItems/ShortcutToolbarItem.php
typo3/sysext/backend/Classes/Controller/Wizard/ColorpickerController.php
typo3/sysext/backend/Classes/Controller/Wizard/TableController.php
typo3/sysext/backend/Classes/Form/FormDataProvider/AbstractItemProvider.php
typo3/sysext/backend/Classes/Utility/BackendUtility.php
typo3/sysext/core/Classes/Authentication/AbstractUserAuthentication.php
typo3/sysext/core/Classes/Charset/CharsetConverter.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/RteHtmlParser.php
typo3/sysext/core/Classes/Imaging/GraphicalFunctions.php
typo3/sysext/core/Classes/Mail/Rfc822AddressesParser.php
typo3/sysext/core/Classes/TypoScript/ExtendedTemplateService.php
typo3/sysext/core/Classes/TypoScript/Parser/TypoScriptParser.php
typo3/sysext/core/Classes/Utility/File/ExtendedFileUtility.php
typo3/sysext/core/Classes/Utility/GeneralUtility.php
typo3/sysext/css_styled_content/Classes/Controller/CssStyledContentController.php
typo3/sysext/fluid/Classes/ViewHelpers/Form/AbstractFormFieldViewHelper.php
typo3/sysext/frontend/Classes/ContentObject/ContentContentObject.php
typo3/sysext/frontend/Classes/ContentObject/ContentObjectRenderer.php
typo3/sysext/frontend/Classes/ContentObject/Menu/TextMenuContentObject.php
typo3/sysext/frontend/Classes/ContentObject/RecordsContentObject.php
typo3/sysext/frontend/Classes/Imaging/GifBuilder.php
typo3/sysext/frontend/Classes/Page/PageGenerator.php
typo3/sysext/impexp/Classes/ImportExport.php
typo3/sysext/indexed_search/Classes/Controller/SearchController.php
typo3/sysext/indexed_search/Classes/Controller/SearchFormController.php
typo3/sysext/info_pagetsconfig/Classes/Controller/InfoPageTyposcriptConfigController.php
typo3/sysext/lowlevel/Classes/DoubleFilesCommand.php
typo3/sysext/rtehtmlarea/Classes/Controller/UserElementsController.php
typo3/sysext/rtehtmlarea/Classes/Extension/Abbreviation.php
typo3/sysext/rtehtmlarea/Classes/Extension/DefaultClean.php
typo3/sysext/rtehtmlarea/Classes/Extension/Language.php
typo3/sysext/rtehtmlarea/Classes/Extension/Plaintext.php
typo3/sysext/rtehtmlarea/Classes/Extension/SelectFont.php
typo3/sysext/rtehtmlarea/Classes/Extension/Spellchecker.php
typo3/sysext/rtehtmlarea/Classes/Extension/TableOperations.php
typo3/sysext/rtehtmlarea/Classes/Extension/Typo3HtmlParser.php
typo3/sysext/rtehtmlarea/Classes/Extension/Typo3Image.php
typo3/sysext/rtehtmlarea/Classes/Extension/Typo3Link.php
typo3/sysext/rtehtmlarea/Classes/Extension/UserElements.php
typo3/sysext/rtehtmlarea/Classes/RteHtmlAreaApi.php
typo3/sysext/saltedpasswords/Classes/Utility/ExtensionManagerConfigurationUtility.php
typo3/sysext/tstemplate/Classes/Controller/TypoScriptTemplateModuleController.php
typo3/sysext/tstemplate/Classes/Controller/TypoScriptTemplateObjectBrowserModuleFunctionController.php

index 840b807..eeb7baf 100644 (file)
@@ -873,7 +873,7 @@ class ShortcutToolbarItem implements ToolbarItemInterface
             $splitModuleName = explode('_', $moduleName);
             $title = $languageService->moduleLabels['tabs'][$splitModuleName[0] . '_tab'];
             if (count($splitModuleName) > 1) {
-                $title .= '>' . $languageService->moduleLabels['tabs'][($moduleName . '_tab')];
+                $title .= '>' . $languageService->moduleLabels['tabs'][$moduleName . '_tab'];
             }
         }
         if ($parentModuleName) {
index 9f49b3a..0e4889b 100644 (file)
@@ -328,7 +328,7 @@ class ColorpickerController extends AbstractWizardController
         while (isset($color[$columns * $rows])) {
             $tCells = array();
             for ($i = 0; $i < $columns; $i++) {
-                $tCells[] = '<td bgcolor="' . $color[($columns * $rows + $i)] . '" class="t3js-colorpicker-value" data-color-value="' . htmlspecialchars($color[($columns * $rows + $i)]) . '" title="' . htmlspecialchars($color[($columns * $rows + $i)]) . '">&nbsp;&nbsp;</td>';
+                $tCells[] = '<td bgcolor="' . $color[$columns * $rows + $i] . '" class="t3js-colorpicker-value" data-color-value="' . htmlspecialchars($color[($columns * $rows + $i)]) . '" title="' . htmlspecialchars($color[($columns * $rows + $i)]) . '">&nbsp;&nbsp;</td>';
             }
             $tRows[] = '<tr>' . implode('', $tCells) . '</tr>';
             $rows++;
index 3198ebd..f0d33ab 100644 (file)
@@ -529,7 +529,7 @@ class TableController extends AbstractWizardController
                             // will be TRUE after one row is added while if rows are added in the bottom
                             // of the table there will be no existing rows to stop the addition of new rows
                             // which means it will add up to $this->numNewRows rows then.
-                            if (!isset($this->TABLECFG['c'][($kk + $a)])) {
+                            if (!isset($this->TABLECFG['c'][$kk + $a])) {
                                 $this->TABLECFG['c'][$kk + $a] = array();
                             } else {
                                 break;
index e725762..0b4e083 100644 (file)
@@ -324,7 +324,7 @@ abstract class AbstractItemProvider
                         // Add label for main module:
                         $pp = explode('_', $theMod);
                         if (count($pp) > 1) {
-                            $label .= $languageService->moduleLabels['tabs'][($pp[0] . '_tab')] . '>';
+                            $label .= $languageService->moduleLabels['tabs'][$pp[0] . '_tab'] . '>';
                         }
                         // Add modules own label now:
                         $label .= $languageService->moduleLabels['tabs'][$theMod . '_tab'];
index 5a17be2..44173d4 100755 (executable)
@@ -3663,7 +3663,7 @@ class BackendUtility
                     $userName,
                     self::calcAge($GLOBALS['EXEC_TIME'] - $row['tstamp'], $lang->sL('LLL:EXT:lang/locallang_core.xlf:labels.minutesHoursDaysYears'))
                 );
-                if ($row['record_pid'] && !isset($GLOBALS['LOCKED_RECORDS'][($row['record_table'] . ':' . $row['record_pid'])])) {
+                if ($row['record_pid'] && !isset($GLOBALS['LOCKED_RECORDS'][$row['record_table'] . ':' . $row['record_pid']])) {
                     $GLOBALS['LOCKED_RECORDS']['pages:' . $row['record_pid']]['msg'] = sprintf(
                         $lang->sL('LLL:EXT:lang/locallang_core.xlf:labels.lockedRecordUser_content'),
                         $userType,
index 6000fc8..323fa78 100644 (file)
@@ -653,7 +653,7 @@ abstract class AbstractUserAuthentication
                         GeneralUtility::devLog('User found: ' . GeneralUtility::arrayToLogString($row, array($this->userid_column, $this->username_column)), \TYPO3\CMS\Core\Authentication\AbstractUserAuthentication::class, 0);
                     }
                     // User found, just stop to search for more if not configured to go on
-                    if (!$this->svConfig['setup'][($this->loginType . '_fetchAllUsers')]) {
+                    if (!$this->svConfig['setup'][$this->loginType . '_fetchAllUsers']) {
                         break;
                     }
                 }
index 33693be..1827097 100644 (file)
@@ -1284,7 +1284,7 @@ class CharsetConverter
                     foreach (array_reverse($decomposition['U+' . $code_value]) as $cv) {
                         array_unshift($to, $cv);
                     }
-                } elseif (!isset($mark[('U+' . $code_value)])) {
+                } elseif (!isset($mark['U+' . $code_value])) {
                     // remove mark
                     array_push($code_decomp, $code_value);
                 }
index e9b046c..ca9f960 100644 (file)
@@ -155,7 +155,7 @@ class CommandLineController
                 foreach ($argSplit as $i => $v) {
                     $ii = $i;
                     if ($i > 0) {
-                        if (!isset($cli_args_copy[$argSplit[0]][($i - 1)]) && $v[0] != '[') {
+                        if (!isset($cli_args_copy[$argSplit[0]][$i - 1]) && $v[0] != '[') {
                             // Using "[]" around a parameter makes it optional
                             echo 'ERROR: Option "' . $argSplit[0] . '" requires a value ("' . $v . '") on position ' . $i . LF;
                             die;
index be55372..a4b0580 100644 (file)
@@ -1558,7 +1558,7 @@ class DataHandler
                 }
 
                 // Look for transformation flag:
-                if ((string)$incomingFieldArray[('_TRANSFORM_' . $vconf['field'])] === 'RTE') {
+                if ((string)$incomingFieldArray['_TRANSFORM_' . $vconf['field']] === 'RTE') {
                     if ($theTypeString === null) {
                         $theTypeString = BackendUtility::getTCAtypeValue($table, $currentRecord);
                     }
@@ -3131,7 +3131,7 @@ class DataHandler
                         // @deprecated: flexFormXMLincludeDiffBase is only enabled by ext:compatibility6 since TYPO3 CMS 7, vDEFbase can be unset / ignored with TYPO3 CMS 8
                         if ($this->clear_flexFormData_vDEFbase) {
                             $dataValues[$key][$vKey . '.vDEFbase'] = '';
-                        } elseif ($this->updateModeL10NdiffData && $GLOBALS['TYPO3_CONF_VARS']['BE']['flexFormXMLincludeDiffBase'] && $vKey !== 'vDEF' && ((string)$dataValues[$key][$vKey] !== (string)$dataValues_current[$key][$vKey] || !isset($dataValues_current[$key][($vKey . '.vDEFbase')]) || $this->updateModeL10NdiffData === 'FORCE_FFUPD')) {
+                        } elseif ($this->updateModeL10NdiffData && $GLOBALS['TYPO3_CONF_VARS']['BE']['flexFormXMLincludeDiffBase'] && $vKey !== 'vDEF' && ((string)$dataValues[$key][$vKey] !== (string)$dataValues_current[$key][$vKey] || !isset($dataValues_current[$key][$vKey . '.vDEFbase']) || $this->updateModeL10NdiffData === 'FORCE_FFUPD')) {
                             // Now, check if a vDEF value is submitted in the input data, if so we expect this has been processed prior to this operation (normally the case since those fields are higher in the form) and we can use that:
                             if (isset($dataValues[$key]['vDEF'])) {
                                 $diffValue = $dataValues[$key]['vDEF'];
@@ -6043,7 +6043,7 @@ class DataHandler
      */
     public function isRecordInWebMount($table, $id)
     {
-        if (!isset($this->isRecordInWebMount_Cache[($table . ':' . $id)])) {
+        if (!isset($this->isRecordInWebMount_Cache[$table . ':' . $id])) {
             $recP = $this->getRecordProperties($table, $id);
             $this->isRecordInWebMount_Cache[$table . ':' . $id] = $this->isInWebMount($recP['event_pid']);
         }
@@ -7038,12 +7038,12 @@ class DataHandler
                 if (!$fieldConfiguration['MM'] && $this->isSubmittedValueEqualToStoredValue($val, $currentRecord[$col], $cRecTypes[$col], $isNullField)) {
                     unset($fieldArray[$col]);
                 } else {
-                    if (!isset($this->mmHistoryRecords[($table . ':' . $id)]['oldRecord'][$col])) {
+                    if (!isset($this->mmHistoryRecords[$table . ':' . $id]['oldRecord'][$col])) {
                         $this->historyRecords[$table . ':' . $id]['oldRecord'][$col] = $currentRecord[$col];
                     } elseif ($this->mmHistoryRecords[$table . ':' . $id]['oldRecord'][$col] != $this->mmHistoryRecords[$table . ':' . $id]['newRecord'][$col]) {
                         $this->historyRecords[$table . ':' . $id]['oldRecord'][$col] = $this->mmHistoryRecords[$table . ':' . $id]['oldRecord'][$col];
                     }
-                    if (!isset($this->mmHistoryRecords[($table . ':' . $id)]['newRecord'][$col])) {
+                    if (!isset($this->mmHistoryRecords[$table . ':' . $id]['newRecord'][$col])) {
                         $this->historyRecords[$table . ':' . $id]['newRecord'][$col] = $fieldArray[$col];
                     } elseif ($this->mmHistoryRecords[$table . ':' . $id]['newRecord'][$col] != $this->mmHistoryRecords[$table . ':' . $id]['oldRecord'][$col]) {
                         $this->historyRecords[$table . ':' . $id]['newRecord'][$col] = $this->mmHistoryRecords[$table . ':' . $id]['newRecord'][$col];
index a7e0d3c..25c83fe 100644 (file)
@@ -1186,7 +1186,7 @@ class QueryGenerator
         $out[] = '<select class="form-control t3js-submit-change" name="' . $name . '">';
         for ($i = 32 * $compOffSet + $neg; $i < 32 * ($compOffSet + 1); $i += 2) {
             if ($this->lang['comparison'][$i . '_']) {
-                $out[] = '<option value="' . $i . '"' . ($i >> 1 === $comparison >> 1 ? ' selected' : '') . '>' . htmlspecialchars($this->lang['comparison'][($i . '_')]) . '</option>';
+                $out[] = '<option value="' . $i . '"' . ($i >> 1 === $comparison >> 1 ? ' selected' : '') . '>' . htmlspecialchars($this->lang['comparison'][$i . '_']) . '</option>';
             }
         }
         $out[] = '</select>';
index 7280637..2e2f776 100644 (file)
@@ -865,7 +865,7 @@ class RteHtmlParser extends \TYPO3\CMS\Core\Html\HtmlParser
         $cc = 0;
         $aC = count($blockSplit);
         // Avoid superfluous linebreaks by transform_db after ending headListTag
-        while ($aC && trim($blockSplit[($aC - 1)]) === '') {
+        while ($aC && trim($blockSplit[$aC - 1]) === '') {
             unset($blockSplit[$aC - 1]);
             $aC = count($blockSplit);
         }
index c9618a0..f5c03f6 100644 (file)
@@ -1293,7 +1293,7 @@ class GraphicalFunctions
         $wordsArray = preg_split('#([- .,!:]+)#', $string, -1, PREG_SPLIT_DELIM_CAPTURE);
         $wordsCount = count($wordsArray);
         for ($index = 0; $index < $wordsCount; $index += 2) {
-            $wordPairs[] = $wordsArray[$index] . $wordsArray[($index + 1)];
+            $wordPairs[] = $wordsArray[$index] . $wordsArray[$index + 1];
         }
         return $wordPairs;
     }
index fafe524..5ae3a4c 100644 (file)
@@ -302,7 +302,7 @@ class Rfc822AddressesParser
         for ($i = 0; $i < $partsCounter; $i++) {
             if ($this->_hasUnclosedQuotes($string) || $this->_hasUnclosedBrackets($string, '<>') || $this->_hasUnclosedBrackets($string, '[]') || $this->_hasUnclosedBrackets($string, '()') || substr($string, -1) == '\\') {
                 if (isset($parts[$i + 1])) {
-                    $string = $string . $char . $parts[($i + 1)];
+                    $string = $string . $char . $parts[$i + 1];
                 } else {
                     $this->error = 'Invalid address spec. Unclosed bracket or quotes';
                     return false;
@@ -385,7 +385,7 @@ class Rfc822AddressesParser
                 $num--;
             }
             if (isset($parts[$i + 1])) {
-                $parts[$i + 1] = $parts[$i] . $char . $parts[($i + 1)];
+                $parts[$i + 1] = $parts[$i] . $char . $parts[$i + 1];
             }
         }
         return $num;
index 8770bc3..02075a7 100644 (file)
@@ -473,7 +473,7 @@ class ExtendedTemplateService extends TemplateService
                         }
                         $aHref = BackendUtility::getModuleUrl('web_ts', $urlParameters);
                         if ($this->bType != 'const') {
-                            $ln = is_array($arr[$key . '.ln..']) ? 'Defined in: ' . $this->lineNumberToScript($arr[($key . '.ln..')]) : 'N/A';
+                            $ln = is_array($arr[$key . '.ln..']) ? 'Defined in: ' . $this->lineNumberToScript($arr[$key . '.ln..']) : 'N/A';
                         } else {
                             $ln = '';
                         }
index ab80353..d407088 100644 (file)
@@ -242,7 +242,7 @@ class TypoScriptParser
         $pre = '[GLOBAL]';
         while ($pre) {
             if ($this->breakPointLN && $pre === '[_BREAK]') {
-                $this->error('Breakpoint at ' . ($this->lineNumberOffset + $this->rawP - 2) . ': Line content was "' . $this->raw[($this->rawP - 2)] . '"', 1);
+                $this->error('Breakpoint at ' . ($this->lineNumberOffset + $this->rawP - 2) . ': Line content was "' . $this->raw[$this->rawP - 2] . '"', 1);
                 break;
             }
             $preUppercase = strtoupper($pre);
@@ -437,7 +437,7 @@ class TypoScriptParser
                                                     return $exitSig;
                                                 }
                                             } else {
-                                                if (!isset($setup[($objStrName . '.')])) {
+                                                if (!isset($setup[$objStrName . '.'])) {
                                                     $setup[$objStrName . '.'] = array();
                                                 }
                                                 $exitSig = $this->parseSub($setup[$objStrName . '.']);
index 798c8c2..9bc8a4d 100644 (file)
@@ -231,9 +231,9 @@ class ExtendedFileUtility extends BasicFileUtility
             if ($this->fileCmdMap['upload']) {
                 $uploads = $this->fileCmdMap['upload'];
                 foreach ($uploads as $upload) {
-                    if (empty($_FILES[('upload_' . $upload['data'])]['name'])
-                        || (is_array($_FILES[('upload_' . $upload['data'])]['name'])
-                            && empty($_FILES[('upload_' . $upload['data'])]['name'][0])
+                    if (empty($_FILES['upload_' . $upload['data']]['name'])
+                        || (is_array($_FILES['upload_' . $upload['data']]['name'])
+                            && empty($_FILES['upload_' . $upload['data']]['name'][0])
                         )
                     ) {
                         unset($this->fileCmdMap['upload'][$upload['data']]);
index d179d60..65ebca4 100755 (executable)
@@ -1206,23 +1206,23 @@ class GeneralUtility
         static $bytes = '';
         $bytesToGenerate = max(4096, $bytesToReturn);
         // if we have not enough random bytes cached, we generate new ones
-        if (!isset($bytes[($bytesToReturn - 1)])) {
+        if (!isset($bytes[$bytesToReturn - 1])) {
             if (TYPO3_OS === 'WIN') {
                 // Openssl seems to be deadly slow on Windows, so try to use mcrypt
                 $bytes .= self::generateRandomBytesMcrypt($bytesToGenerate);
             } else {
                 // Try to use native PHP functions first, precedence has openssl
                 $bytes .= self::generateRandomBytesOpenSsl($bytesToGenerate);
-                if (!isset($bytes[($bytesToReturn - 1)])) {
+                if (!isset($bytes[$bytesToReturn - 1])) {
                     $bytes .= self::generateRandomBytesMcrypt($bytesToGenerate);
                 }
                 // If openssl and mcrypt failed, try /dev/urandom
-                if (!isset($bytes[($bytesToReturn - 1)])) {
+                if (!isset($bytes[$bytesToReturn - 1])) {
                     $bytes .= self::generateRandomBytesUrandom($bytesToGenerate);
                 }
             }
             // Fall back if other random byte generation failed until now
-            if (!isset($bytes[($bytesToReturn - 1)])) {
+            if (!isset($bytes[$bytesToReturn - 1])) {
                 $bytes .= self::generateRandomBytesFallback($bytesToReturn);
             }
         }
@@ -1292,7 +1292,7 @@ class GeneralUtility
         $bytes = '';
         // We initialize with somewhat random.
         $randomState = $GLOBALS['TYPO3_CONF_VARS']['SYS']['encryptionKey'] . base_convert(memory_get_usage() % pow(10, 6), 10, 2) . microtime() . StringUtility::getUniqueId() . getmypid();
-        while (!isset($bytes[($bytesToReturn - 1)])) {
+        while (!isset($bytes[$bytesToReturn - 1])) {
             $randomState = sha1(microtime() . mt_rand() . $randomState);
             $bytes .= sha1(mt_rand() . $randomState, true);
         }
@@ -2102,15 +2102,15 @@ class GeneralUtility
             // Use tag based on grand-parent + parent tag name
             if (isset($options['grandParentTagMap'][$stackData['grandParentTagName'] . '/' . $stackData['parentTagName']])) {
                 $attr .= ' index="' . htmlspecialchars($tagName) . '"';
-                $tagName = (string)$options['grandParentTagMap'][($stackData['grandParentTagName'] . '/' . $stackData['parentTagName'])];
+                $tagName = (string)$options['grandParentTagMap'][$stackData['grandParentTagName'] . '/' . $stackData['parentTagName']];
             } elseif (isset($options['parentTagMap'][$stackData['parentTagName'] . ':_IS_NUM']) && MathUtility::canBeInterpretedAsInteger($tagName)) {
                 // Use tag based on parent tag name + if current tag is numeric
                 $attr .= ' index="' . htmlspecialchars($tagName) . '"';
-                $tagName = (string)$options['parentTagMap'][($stackData['parentTagName'] . ':_IS_NUM')];
+                $tagName = (string)$options['parentTagMap'][$stackData['parentTagName'] . ':_IS_NUM'];
             } elseif (isset($options['parentTagMap'][$stackData['parentTagName'] . ':' . $tagName])) {
                 // Use tag based on parent tag name + current tag
                 $attr .= ' index="' . htmlspecialchars($tagName) . '"';
-                $tagName = (string)$options['parentTagMap'][($stackData['parentTagName'] . ':' . $tagName)];
+                $tagName = (string)$options['parentTagMap'][$stackData['parentTagName'] . ':' . $tagName];
             } elseif (isset($options['parentTagMap'][$stackData['parentTagName']])) {
                 // Use tag based on parent tag name:
                 $attr .= ' index="' . htmlspecialchars($tagName) . '"';
index 4258a4c..e4dd959 100644 (file)
@@ -189,7 +189,7 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
             }
             if ($useTfoot) {
                 $tableContents .= '
-                                       <tfoot>' . $rows[($rCount - 1)] . '</tfoot>';
+                                       <tfoot>' . $rows[$rCount - 1] . '</tfoot>';
                 unset($rows[$rCount - 1]);
                 $addTbody = 1;
             }
@@ -861,7 +861,7 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
                                 $imageMarkers['caption'] = $this->cObj->substituteMarkerArray($imageMarkers['caption'], $captionMarkers, '###|###', 1, 1);
                             }
                             if ($addClassesImageConf[$imagesCounter - 1]['addClassesImage']) {
-                                $imageMarkers['classes'] = ' ' . $addClassesImageConf[($imagesCounter - 1)]['addClassesImage'];
+                                $imageMarkers['classes'] = ' ' . $addClassesImageConf[$imagesCounter - 1]['addClassesImage'];
                             }
                         }
                         $columnImages[] = $this->cObj->substituteMarkerArray($single, $imageMarkers, '###|###', 1, 1);
@@ -871,7 +871,7 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
                     // Start filling the markers for columnStdWrap
                     $columnMarkers = array();
                     if ($addClassesColConf[$columnCounter - 1]['addClassesCol']) {
-                        $columnMarkers['classes'] = ' ' . $addClassesColConf[($columnCounter - 1)]['addClassesCol'];
+                        $columnMarkers['classes'] = ' ' . $addClassesColConf[$columnCounter - 1]['addClassesCol'];
                     }
                     $rowColumns[] = $this->cObj->substituteMarkerArray($rowColumn, $columnMarkers, '###|###', 1, 1);
                 }
index abfe5bb..7e8203f 100644 (file)
@@ -301,7 +301,7 @@ abstract class AbstractFormFieldViewHelper extends AbstractFormViewHelper
         // If count == 2 -> we need to go through the for-loop exactly once
         for ($i = 1; $i < count($propertySegments); $i++) {
             $object = ObjectAccess::getPropertyPath($formObject, implode('.', array_slice($propertySegments, 0, $i)));
-            $objectName .= '[' . $propertySegments[($i - 1)] . ']';
+            $objectName .= '[' . $propertySegments[$i - 1] . ']';
             $hiddenIdentityField = $this->renderHiddenIdentityField($object, $objectName);
             // Add the hidden identity field to the ViewHelperVariableContainer
             $additionalIdentityProperties = $this->viewHelperVariableContainer->get(
index 7f1fe56..4307806 100644 (file)
@@ -88,7 +88,7 @@ class ContentContentObject extends AbstractContentObject
                             $_procObj->modifyDBRow($row, $conf['table']);
                         }
                     }
-                    if (!$frontendController->recordRegister[($conf['table'] . ':' . $row['uid'])]) {
+                    if (!$frontendController->recordRegister[$conf['table'] . ':' . $row['uid']]) {
                         $this->cObj->currentRecordNumber++;
                         $cObj->parentRecordNumber = $this->cObj->currentRecordNumber;
                         $frontendController->currentRecord = $conf['table'] . ':' . $row['uid'];
index 7b9c812..c9a68b8 100644 (file)
@@ -5182,7 +5182,7 @@ class ContentObjectRenderer
                                     $tag = $htmlParser->getFirstTag($vvv);
                                     $tagName = strtolower($htmlParser->getFirstTagName($vvv));
                                     $colParts[$kkk] = $htmlParser->removeFirstAndLastTag($vvv);
-                                    if ($cfg['HTMLtableCells.'][$cc . '.']['callRecursive'] || !isset($cfg['HTMLtableCells.'][($cc . '.')]['callRecursive']) && $cfg['HTMLtableCells.']['default.']['callRecursive']) {
+                                    if ($cfg['HTMLtableCells.'][$cc . '.']['callRecursive'] || !isset($cfg['HTMLtableCells.'][$cc . '.']['callRecursive']) && $cfg['HTMLtableCells.']['default.']['callRecursive']) {
                                         if ($cfg['HTMLtableCells.']['addChr10BetweenParagraphs']) {
                                             $colParts[$kkk] = str_replace('</p><p>', '</p>' . LF . '<p>', $colParts[$kkk]);
                                         }
@@ -5404,7 +5404,7 @@ class ContentObjectRenderer
                         $contentAccumP++;
                         // If the TypoTag section
                         if (!$breakOut) {
-                            $contentAccum[$contentAccumP - 2] .= $contentAccum[($contentAccumP - 1)] . $contentAccum[$contentAccumP];
+                            $contentAccum[$contentAccumP - 2] .= $contentAccum[$contentAccumP - 1] . $contentAccum[$contentAccumP];
                             unset($contentAccum[$contentAccumP]);
                             unset($contentAccum[$contentAccumP - 1]);
                             $contentAccumP -= 2;
index 1c9f532..52d8b1c 100644 (file)
@@ -181,7 +181,7 @@ class TextMenuContentObject extends AbstractMenuContentObject
             $theName = $this->imgNamePrefix . $this->I['uid'] . $this->I['INPfix'] . $pref;
             $name = ' ' . $this->nameAttribute . '="' . $theName . '"';
             $GLOBALS['TSFE']->imagesOnPage[] = $imgInfo[3];
-            $res = '<img' . ' src="' . $GLOBALS['TSFE']->absRefPrefix . $imgInfo[3] . '"' . ' width="' . $imgInfo[0] . '"' . ' height="' . $imgInfo[1] . '"' . $name . ($this->I['val'][$pref . 'ImgTagParams'] ? ' ' . $this->I['val'][($pref . 'ImgTagParams')] : '') . $this->parent_cObj->getBorderAttr(' border="0"');
+            $res = '<img' . ' src="' . $GLOBALS['TSFE']->absRefPrefix . $imgInfo[3] . '"' . ' width="' . $imgInfo[0] . '"' . ' height="' . $imgInfo[1] . '"' . $name . ($this->I['val'][$pref . 'ImgTagParams'] ? ' ' . $this->I['val'][$pref . 'ImgTagParams'] : '') . $this->parent_cObj->getBorderAttr(' border="0"');
             if (!strstr($res, 'alt="')) {
                 // Adding alt attribute if not set.
                 $res .= ' alt=""';
index 9e28d59..f622934 100644 (file)
@@ -107,7 +107,7 @@ class RecordsContentObject extends AbstractContentObject
                         if (!$dontCheckPid) {
                             $row = $this->cObj->checkPid($row['pid']) ? $row : '';
                         }
-                        if ($row && !$GLOBALS['TSFE']->recordRegister[($val['table'] . ':' . $val['id'])]) {
+                        if ($row && !$GLOBALS['TSFE']->recordRegister[$val['table'] . ':' . $val['id']]) {
                             $renderObjName = $conf['conf.'][$val['table']] ?: '<' . $val['table'];
                             $renderObjKey = $conf['conf.'][$val['table']] ? 'conf.' . $val['table'] : '';
                             $renderObjConf = $conf['conf.'][$val['table'] . '.'];
index 914a44b..da8ca3a 100644 (file)
@@ -656,7 +656,7 @@ class GifBuilder extends GraphicalFunctions
                         $travKeys = array('xSpaceBefore', 'xSpaceAfter', 'ySpaceBefore', 'ySpaceAfter');
                         foreach ($travKeys as $pxKey) {
                             if (isset($conf['splitRendering.'][$cfgK . '.'][$pxKey])) {
-                                $conf['splitRendering.'][$cfgK . '.'][$pxKey] = round($conf['splitRendering.'][($cfgK . '.')][$pxKey] * ($conf['fontSize'] / $this->charRangeMap[$fontBaseName]['pixelSpace']));
+                                $conf['splitRendering.'][$cfgK . '.'][$pxKey] = round($conf['splitRendering.'][$cfgK . '.'][$pxKey] * ($conf['fontSize'] / $this->charRangeMap[$fontBaseName]['pixelSpace']));
                             }
                         }
                     }
index 9346e82..a213277 100644 (file)
@@ -594,7 +594,7 @@ class PageGenerator
             unset($tsfe->pSetup['includeJSlibs.']);
             foreach ($tsfe->pSetup['includeJSLibs.'] as $key => $JSfile) {
                 if (!is_array($JSfile)) {
-                    if (isset($tsfe->pSetup['includeJSLibs.'][$key . '.']['if.']) && !$tsfe->cObj->checkIf($tsfe->pSetup['includeJSLibs.'][($key . '.')]['if.'])) {
+                    if (isset($tsfe->pSetup['includeJSLibs.'][$key . '.']['if.']) && !$tsfe->cObj->checkIf($tsfe->pSetup['includeJSLibs.'][$key . '.']['if.'])) {
                         continue;
                     }
                     $ss = $tsfe->pSetup['includeJSLibs.'][$key . '.']['external'] ? $JSfile : $tsfe->tmpl->getFileName($JSfile);
@@ -625,7 +625,7 @@ class PageGenerator
         if (is_array($tsfe->pSetup['includeJSFooterlibs.'])) {
             foreach ($tsfe->pSetup['includeJSFooterlibs.'] as $key => $JSfile) {
                 if (!is_array($JSfile)) {
-                    if (isset($tsfe->pSetup['includeJSFooterlibs.'][$key . '.']['if.']) && !$tsfe->cObj->checkIf($tsfe->pSetup['includeJSFooterlibs.'][($key . '.')]['if.'])) {
+                    if (isset($tsfe->pSetup['includeJSFooterlibs.'][$key . '.']['if.']) && !$tsfe->cObj->checkIf($tsfe->pSetup['includeJSFooterlibs.'][$key . '.']['if.'])) {
                         continue;
                     }
                     $ss = $tsfe->pSetup['includeJSFooterlibs.'][$key . '.']['external'] ? $JSfile : $tsfe->tmpl->getFileName($JSfile);
@@ -656,7 +656,7 @@ class PageGenerator
         if (is_array($tsfe->pSetup['includeJS.'])) {
             foreach ($tsfe->pSetup['includeJS.'] as $key => $JSfile) {
                 if (!is_array($JSfile)) {
-                    if (isset($tsfe->pSetup['includeJS.'][$key . '.']['if.']) && !$tsfe->cObj->checkIf($tsfe->pSetup['includeJS.'][($key . '.')]['if.'])) {
+                    if (isset($tsfe->pSetup['includeJS.'][$key . '.']['if.']) && !$tsfe->cObj->checkIf($tsfe->pSetup['includeJS.'][$key . '.']['if.'])) {
                         continue;
                     }
                     $ss = $tsfe->pSetup['includeJS.'][$key . '.']['external'] ? $JSfile : $tsfe->tmpl->getFileName($JSfile);
@@ -685,7 +685,7 @@ class PageGenerator
         if (is_array($tsfe->pSetup['includeJSFooter.'])) {
             foreach ($tsfe->pSetup['includeJSFooter.'] as $key => $JSfile) {
                 if (!is_array($JSfile)) {
-                    if (isset($tsfe->pSetup['includeJSFooter.'][$key . '.']['if.']) && !$tsfe->cObj->checkIf($tsfe->pSetup['includeJSFooter.'][($key . '.')]['if.'])) {
+                    if (isset($tsfe->pSetup['includeJSFooter.'][$key . '.']['if.']) && !$tsfe->cObj->checkIf($tsfe->pSetup['includeJSFooter.'][$key . '.']['if.'])) {
                         continue;
                     }
                     $ss = $tsfe->pSetup['includeJSFooter.'][$key . '.']['external'] ? $JSfile : $tsfe->tmpl->getFileName($JSfile);
index e7c05c5..761198c 100644 (file)
@@ -701,9 +701,9 @@ class ImportExport
     public function export_addRecord($table, $row, $relationLevel = 0)
     {
         BackendUtility::workspaceOL($table, $row);
-        if ((string)$table !== '' && is_array($row) && $row['uid'] > 0 && !$this->excludeMap[($table . ':' . $row['uid'])]) {
+        if ((string)$table !== '' && is_array($row) && $row['uid'] > 0 && !$this->excludeMap[$table . ':' . $row['uid']]) {
             if ($this->checkPID($table === 'pages' ? $row['uid'] : $row['pid'])) {
-                if (!isset($this->dat['records'][($table . ':' . $row['uid'])])) {
+                if (!isset($this->dat['records'][$table . ':' . $row['uid']])) {
                     // Prepare header info:
                     $row = $this->filterRecordFields($table, $row);
                     $headerInfo = array();
@@ -3731,7 +3731,7 @@ class ImportExport
                 }
             }
             $pInfo['preCode'] = $preCode . '<span title="' . htmlspecialchars($table . ':' . $uid) . '">'
-                . $this->iconFactory->getIconForRecord($table, (array)$this->dat['records'][($table . ':' . $uid)]['data'], Icon::SIZE_SMALL)->render()
+                . $this->iconFactory->getIconForRecord($table, (array)$this->dat['records'][$table . ':' . $uid]['data'], Icon::SIZE_SMALL)->render()
                 . '</span>';
             $pInfo['title'] = htmlspecialchars($record['title']);
             // View page:
index 0b618fd..53f7929 100644 (file)
@@ -716,7 +716,7 @@ class SearchController extends \TYPO3\CMS\Extbase\Mvc\Controller\ActionControlle
                     if ($strLen > $postPreLgd) {
                         $output[$k] = $divider . preg_replace('/^[^[:space:]]+[[:space:]]/', '', $GLOBALS['TSFE']->csConvObj->crop('utf-8', $parts[$k], -($postPreLgd - $postPreLgd_offset)));
                     }
-                } elseif ($summaryLgd > $summaryMax || !isset($parts[($k + 1)])) {
+                } elseif ($summaryLgd > $summaryMax || !isset($parts[$k + 1])) {
                     // In case summary length is exceed OR if there are no more entries at all:
                     if ($strLen > $postPreLgd) {
                         $output[$k] = preg_replace('/[[:space:]][^[:space:]]+$/', '', $GLOBALS['TSFE']->csConvObj->crop('utf-8', $parts[$k], ($postPreLgd - $postPreLgd_offset))) . $divider;
index e7fa7c4..acb66aa 100755 (executable)
@@ -2022,7 +2022,7 @@ class SearchFormController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlugin
                     if ($strLen > $postPreLgd) {
                         $output[$k] = $divider . preg_replace('/^[^[:space:]]+[[:space:]]/', '', $this->frontendController->csConvObj->crop('utf-8', $parts[$k], -($postPreLgd - $postPreLgd_offset)));
                     }
-                } elseif ($summaryLgd > $summaryMax || !isset($parts[($k + 1)])) {
+                } elseif ($summaryLgd > $summaryMax || !isset($parts[$k + 1])) {
                     // In case summary length is exceed OR if there are no more entries at all:
                     if ($strLen > $postPreLgd) {
                         $output[$k] = preg_replace('/[[:space:]][^[:space:]]+$/', '', $this->frontendController->csConvObj->crop('utf-8', $parts[$k], ($postPreLgd - $postPreLgd_offset))) . $divider;
index 4c78f31..0cdc2d0 100644 (file)
@@ -272,7 +272,7 @@ class InfoPageTyposcriptConfigController extends \TYPO3\CMS\Backend\Module\Abstr
         $hierarchicArray[$currentElement['uid']] = htmlspecialchars($currentElement['title']);
         array_shift($rootlineArray);
         if (!empty($rootlineArray)) {
-            if (!isset($hierarchicArray[($currentElement['uid'] . '.')])) {
+            if (!isset($hierarchicArray[$currentElement['uid'] . '.'])) {
                 $hierarchicArray[$currentElement['uid'] . '.'] = array();
             }
             $this->setInPageArray($hierarchicArray[$currentElement['uid'] . '.'], $rootlineArray, $row);
@@ -332,7 +332,7 @@ class InfoPageTyposcriptConfigController extends \TYPO3\CMS\Backend\Module\Abstr
                     . '" title="' . htmlspecialchars('ID: ' . $identifier) . '">'
                     . $this->iconFactory->getIconForRecord('pages', BackendUtility::getRecordWSOL('pages', $identifier), Icon::SIZE_SMALL)->render()
                     . GeneralUtility::fixed_lgd_cs($pageArray[$identifier], 30) . '</a></td>
-                                       <td>' . ($pageArray[($identifier . '_')]['includeLines'] === 0 ? '' : $pageArray[($identifier . '_')]['includeLines']) . '</td>
+                                       <td>' . ($pageArray[$identifier . '_']['includeLines'] === 0 ? '' : $pageArray[($identifier . '_')]['includeLines']) . '</td>
                                        <td>' . ($pageArray[$identifier . '_']['writtenLines'] === 0 ? '' : $pageArray[$identifier . '_']['writtenLines']) . '</td>
                                        </tr>';
             } else {
index ddb25d6..998cb9f 100644 (file)
@@ -88,7 +88,7 @@ This will check the system for double files relations.';
                 // Compile info string for location of reference:
                 $infoString = $this->infoStr($rec);
                 // Registering occurencies in directories:
-                $resultArray['dirname_registry'][dirname($rec['ref_string'])][($rec['tablename'] . ':' . $rec['field'])]++;
+                $resultArray['dirname_registry'][dirname($rec['ref_string'])][$rec['tablename'] . ':' . $rec['field']]++;
                 // Handle missing file:
                 if (!@is_file((PATH_site . $rec['ref_string']))) {
                     $resultArray['missingFiles'][$rec['ref_string']][$rec['hash']] = $infoString;
index 9194c26..2d7a77a 100644 (file)
@@ -249,13 +249,13 @@ class UserElementsController
                                 } else {
                                     $title = $GLOBALS['LANG']->sL($title, true);
                                 }
-                                $description = $GLOBALS['LANG']->sL($v[($k2i . '.')]['description'], true) . '<br />';
-                                if (!$v[($k2i . '.')]['dontInsertSiteUrl']) {
+                                $description = $GLOBALS['LANG']->sL($v[$k2i . '.']['description'], true) . '<br />';
+                                if (!$v[$k2i . '.']['dontInsertSiteUrl']) {
                                     $v[$k2i . '.']['content'] = str_replace('###_URL###', $this->siteUrl, $v[$k2i . '.']['content']);
                                 }
                                 $logo = $v[$k2i . '.']['_icon'] ?: '';
                                 $onClickEvent = '';
-                                switch ((string)$v[($k2i . '.')]['mode']) {
+                                switch ((string)$v[$k2i . '.']['mode']) {
                                     case 'wrap':
                                         $wrap = explode('|', $v[$k2i . '.']['content']);
                                         $onClickEvent = 'wrapHTML(' . GeneralUtility::quoteJSvalue($wrap[0]) . ',' . GeneralUtility::quoteJSvalue($wrap[1]) . ',false);';
@@ -272,7 +272,7 @@ class UserElementsController
                                     case 'insert':
 
                                     default:
-                                        $onClickEvent = 'insertHTML(' . GeneralUtility::quoteJSvalue($v[($k2i . '.')]['content']) . ');';
+                                        $onClickEvent = 'insertHTML(' . GeneralUtility::quoteJSvalue($v[$k2i . '.']['content']) . ');';
                                 }
                                 $A = array('<a href="#" onClick="' . $onClickEvent . 'return false;">', '</a>');
                                 $subcats[$k2i] = '<tr>
index 662d06c..de66fe6 100644 (file)
@@ -130,7 +130,7 @@ class Abbreviation extends RteHtmlAreaApi
         $button = 'abbreviation';
         $jsArray = array();
         if (in_array($button, $this->toolbar)) {
-            if (!is_array($this->configuration['thisConfig']['buttons.']) || !is_array($this->configuration['thisConfig']['buttons.'][($button . '.')])) {
+            if (!is_array($this->configuration['thisConfig']['buttons.']) || !is_array($this->configuration['thisConfig']['buttons.'][$button . '.'])) {
                 $jsArray[] = 'RTEarea[editornumber].buttons.' . $button . ' = new Object();';
             }
             $jsArray[] = 'RTEarea[editornumber].buttons.' . $button . '.abbreviationUrl = "' . $this->writeTemporaryFile('abbreviation_' . $this->configuration['contentLanguageUid'], 'js', $this->buildJSAbbreviationArray()) . '";';
index 151a9e8..0ebd615 100644 (file)
@@ -68,7 +68,7 @@ class DefaultClean extends RteHtmlAreaApi
         $jsArray = array();
         $button = 'cleanword';
         if (in_array($button, $this->toolbar)) {
-            if (!is_array($this->configuration['thisConfig']['buttons.']) || !is_array($this->configuration['thisConfig']['buttons.'][($button . '.')])) {
+            if (!is_array($this->configuration['thisConfig']['buttons.']) || !is_array($this->configuration['thisConfig']['buttons.'][$button . '.'])) {
                 $jsArray[] = 'RTEarea[editornumber].buttons.' . $button . ' = new Object();';
             }
             $jsArray[] = 'RTEarea[editornumber].buttons.' . $button . ' = {"hotKey" : "' . ($this->configuration['thisConfig']['enableWordClean.']['hotKey'] ? $this->configuration['thisConfig']['enableWordClean.']['hotKey'] : '0') . '"};';
index 5d2c0cd..a12213b 100644 (file)
@@ -75,7 +75,7 @@ class Language extends RteHtmlAreaApi
     {
         $button = 'language';
         $jsArray = array();
-        if (!is_array($this->configuration['thisConfig']['buttons.']) || !is_array($this->configuration['thisConfig']['buttons.'][($button . '.')])) {
+        if (!is_array($this->configuration['thisConfig']['buttons.']) || !is_array($this->configuration['thisConfig']['buttons.'][$button . '.'])) {
             $jsArray[] = 'RTEarea[editornumber].buttons.' . $button . ' = new Object();';
         }
         $languages = array(
index f7ae6df..b4b12f1 100644 (file)
@@ -69,7 +69,7 @@ class Plaintext extends RteHtmlAreaApi
         $button = 'pastebehaviour';
         // Get current TYPO3 User Setting, if available
         if (TYPO3_MODE === 'BE' && \TYPO3\CMS\Core\Utility\ExtensionManagementUtility::isLoaded('setup') && is_array($GLOBALS['TYPO3_USER_SETTINGS']) && is_object($GLOBALS['BE_USER'])) {
-            if (!is_array($this->configuration['thisConfig']['buttons.']) || !is_array($this->configuration['thisConfig']['buttons.'][($button . '.')])) {
+            if (!is_array($this->configuration['thisConfig']['buttons.']) || !is_array($this->configuration['thisConfig']['buttons.'][$button . '.'])) {
                 $jsArray[] = 'RTEarea[editornumber].buttons.' . $button . ' = new Object();';
             }
             $jsArray[] = 'RTEarea[editornumber].buttons.' . $button . '.current = "' . (isset($GLOBALS['BE_USER']->uc['rteCleanPasteBehaviour']) ? $GLOBALS['BE_USER']->uc['rteCleanPasteBehaviour'] : 'plainText') . '";';
index a6926f3..7c84491 100644 (file)
@@ -176,7 +176,7 @@ class SelectFont extends RteHtmlAreaApi
         }
         $itemsJSArray = json_encode(array('options' => $itemsJSArray));
         // Adding to button JS configuration
-        if (!is_array($this->configuration['thisConfig']['buttons.']) || !is_array($this->configuration['thisConfig']['buttons.'][($buttonId . '.')])) {
+        if (!is_array($this->configuration['thisConfig']['buttons.']) || !is_array($this->configuration['thisConfig']['buttons.'][$buttonId . '.'])) {
             $jsArray[] = 'RTEarea[editornumber].buttons.' . $buttonId . ' = new Object();';
         }
         $jsArray[] = 'RTEarea[editornumber].buttons.' . $buttonId . '.dataUrl = "' . $this->writeTemporaryFile($buttonId . '_' . $this->configuration['contentLanguageUid'], 'js', $itemsJSArray) . '";';
index 06447c0..73c1ccb 100644 (file)
@@ -87,7 +87,7 @@ class Spellchecker extends RteHtmlAreaApi
         // Set the use of personal dictionary
         $enablePersonalDicts = $this->configuration['thisConfig']['buttons.'][$button . '.']['enablePersonalDictionaries'] && !empty($GLOBALS['BE_USER']->userTS['options.']['enablePersonalDicts']);
         if (in_array($button, $this->toolbar)) {
-            if (!is_array($this->configuration['thisConfig']['buttons.']) || !is_array($this->configuration['thisConfig']['buttons.'][($button . '.')])) {
+            if (!is_array($this->configuration['thisConfig']['buttons.']) || !is_array($this->configuration['thisConfig']['buttons.'][$button . '.'])) {
                 $jsArray[] = 'RTEarea[editornumber].buttons.' . $button . ' = new Object();';
             }
             $jsArray[] = 'RTEarea[editornumber].buttons.' . $button . '.contentTypo3Language = "' . $this->configuration['contentTypo3Language'] . '";';
index 1ee3dbf..d8d2e52 100644 (file)
@@ -100,7 +100,7 @@ class TableOperations extends RteHtmlAreaApi
             // Combining fieldset disablers as a list
             $disabledFieldsets = array('Alignment', 'Borders', 'Color', 'Description', 'Layout', 'RowGroup', 'Spacing', 'Style');
             foreach ($disabledFieldsets as $index => $fieldset) {
-                if (!trim($this->configuration['thisConfig'][('disable' . $fieldset . 'FieldsetInTableOperations')])) {
+                if (!trim($this->configuration['thisConfig']['disable' . $fieldset . 'FieldsetInTableOperations'])) {
                     unset($disabledFieldsets[$index]);
                 }
             }
@@ -110,7 +110,7 @@ class TableOperations extends RteHtmlAreaApi
                 $dialogues = array('table', 'tableproperties', 'rowproperties', 'columnproperties', 'cellproperties');
                 foreach ($dialogues as $dialogue) {
                     if (in_array($dialogue, $this->toolbar)) {
-                        if (!is_array($this->configuration['thisConfig']['buttons.']) || !is_array($this->configuration['thisConfig']['buttons.'][($dialogue . '.')])) {
+                        if (!is_array($this->configuration['thisConfig']['buttons.']) || !is_array($this->configuration['thisConfig']['buttons.'][$dialogue . '.'])) {
                             $jsArray[] = 'RTEarea[editornumber].buttons.' . $dialogue . ' = new Object();';
                             $jsArray[] = 'RTEarea[editornumber].buttons.' . $dialogue . '.removeFieldsets = "' . $disabledFieldsets . '";';
                         } elseif ($this->configuration['thisConfig']['buttons.'][$dialogue . '.']['removeFieldsets']) {
index 9956c25..a7aa9b2 100644 (file)
@@ -69,7 +69,7 @@ class Typo3HtmlParser extends RteHtmlAreaApi
         $jsArray = array();
         $button = 'cleanword';
         if (in_array($button, $this->toolbar)) {
-            if (!is_array($this->configuration['thisConfig']['buttons.']) || !is_array($this->configuration['thisConfig']['buttons.'][($button . '.')])) {
+            if (!is_array($this->configuration['thisConfig']['buttons.']) || !is_array($this->configuration['thisConfig']['buttons.'][$button . '.'])) {
                 $jsArray[] = 'RTEarea[editornumber].buttons.' . $button . ' = new Object();';
             }
             $jsArray[] = 'RTEarea[editornumber].buttons.' . $button . '.pathParseHtmlModule = ' . GeneralUtility::quoteJSvalue(BackendUtility::getModuleUrl('rtehtmlarea_wizard_parse_html')) . ';';
index 74cb0e1..a509f6c 100644 (file)
@@ -74,7 +74,7 @@ class Typo3Image extends RteHtmlAreaApi
         $jsArray = array();
         $button = 'image';
         if (in_array($button, $this->toolbar)) {
-            if (!is_array($this->configuration['thisConfig']['buttons.']) || !is_array($this->configuration['thisConfig']['buttons.'][($button . '.')])) {
+            if (!is_array($this->configuration['thisConfig']['buttons.']) || !is_array($this->configuration['thisConfig']['buttons.'][$button . '.'])) {
                 $jsArray[] = 'RTEarea[editornumber]["buttons"]["' . $button . '"] = new Object();';
             }
             $jsArray[] = 'RTEarea[editornumber].buttons.' . $button . '.pathImageModule = ' . GeneralUtility::quoteJSvalue(BackendUtility::getModuleUrl('rtehtmlarea_wizard_select_image')) . ';';
index 592a55e..88f7a7f 100644 (file)
@@ -70,7 +70,7 @@ class Typo3Link extends RteHtmlAreaApi
         $jsArray = array();
         $button = 'link';
         if (in_array($button, $this->toolbar)) {
-            if (!is_array($this->configuration['thisConfig']['buttons.']) || !is_array($this->configuration['thisConfig']['buttons.'][($button . '.')])) {
+            if (!is_array($this->configuration['thisConfig']['buttons.']) || !is_array($this->configuration['thisConfig']['buttons.'][$button . '.'])) {
                 $jsArray[] = 'RTEarea[editornumber].buttons.' . $button . ' = new Object();';
             }
             $jsArray[] = 'RTEarea[editornumber].buttons.' . $button . '.pathLinkModule = ' . GeneralUtility::quoteJSvalue(BackendUtility::getModuleUrl('rtehtmlarea_wizard_browse_links')) . ';';
index a5b5e4f..184dad8 100644 (file)
@@ -56,7 +56,7 @@ class UserElements extends RteHtmlAreaApi
         $jsArray = array();
         $button = 'user';
         if (in_array($button, $this->toolbar)) {
-            if (!is_array($this->configuration['thisConfig']['buttons.']) || !is_array($this->configuration['thisConfig']['buttons.'][($button . '.')])) {
+            if (!is_array($this->configuration['thisConfig']['buttons.']) || !is_array($this->configuration['thisConfig']['buttons.'][$button . '.'])) {
                 $jsArray[] = 'RTEarea[editornumber].buttons.' . $button . ' = new Object();';
             }
             $jsArray[] = 'RTEarea[editornumber].buttons.' . $button . '.pathUserModule = ' . GeneralUtility::quoteJSvalue(BackendUtility::getModuleUrl('rtehtmlarea_wizard_user_elements')) . ';';
index 0676d3d..1de002f 100644 (file)
@@ -130,7 +130,7 @@ abstract class RteHtmlAreaApi
         $pluginButtons = GeneralUtility::trimExplode(',', $this->pluginButtons, true);
         foreach ($pluginButtons as $button) {
             if (in_array($button, $this->toolbar)) {
-                if (!is_array($this->configuration['thisConfig']['buttons.']) || !is_array($this->configuration['thisConfig']['buttons.'][($button . '.')])) {
+                if (!is_array($this->configuration['thisConfig']['buttons.']) || !is_array($this->configuration['thisConfig']['buttons.'][$button . '.'])) {
                     $jsArray[] = 'RTEarea[editornumber].buttons.' . $button . ' = new Object();';
                 }
             }
index 2cc5d81..5b6f19c 100755 (executable)
@@ -425,7 +425,7 @@ class ExtensionManagerConfigurationUtility
             if (count($parts) == 2) {
                 // @todo Explain
                 $value = $this->processPostData(array($parts[1] => $value));
-                $postArray[$parts[0] . '.'] = array_merge((array)$postArray[($parts[0] . '.')], $value);
+                $postArray[$parts[0] . '.'] = array_merge((array)$postArray[$parts[0] . '.'], $value);
             } else {
                 // @todo Explain
                 $postArray[$parts[0]] = $value;
index 9e6c98a..dfb8498 100755 (executable)
@@ -620,7 +620,7 @@ page.10.value = HELLO WORLD!
         $pArray[$cEl['uid']] = htmlspecialchars($cEl['title']);
         array_shift($rlArr);
         if (!empty($rlArr)) {
-            if (!isset($pArray[($cEl['uid'] . '.')])) {
+            if (!isset($pArray[$cEl['uid'] . '.'])) {
                 $pArray[$cEl['uid'] . '.'] = array();
             }
             $this->setInPageArray($pArray[$cEl['uid'] . '.'], $rlArr, $row);
@@ -651,7 +651,7 @@ page.10.value = HELLO WORLD!
                 if (isset($pArray[$k . '_'])) {
                     $lines[] = '<tr class="' . ($i++ % 2 == 0 ? 'bgColor4' : 'bgColor6') . '">
                                                <td nowrap><span style="width: 1px; height: 1px; display:inline-block; margin-left: ' . $c * 20 . 'px"></span>' . '<a href="' . htmlspecialchars(GeneralUtility::linkThisScript(array('id' => $k))) . '" title="' . htmlspecialchars('ID: ' . $k) . '">' . $this->moduleTemplate->getIconFactory()->getIconForRecord('pages', BackendUtility::getRecordWSOL('pages', $k), Icon::SIZE_SMALL)->render() . GeneralUtility::fixed_lgd_cs($pArray[$k], 30) . '</a></td>
-                                               <td>' . $pArray[($k . '_')]['count'] . '</td>
+                                               <td>' . $pArray[$k . '_']['count'] . '</td>
                                                <td>' . ($pArray[$k . '_']['root_max_val'] > 0 ? $statusCheckedIcon : '&nbsp;') . '</td>
                                                <td>' . ($pArray[$k . '_']['root_min_val'] == 0 ? $statusCheckedIcon : '&nbsp;') . '</td>
                                                </tr>';
index 6c172c9..452fe94 100644 (file)
@@ -97,7 +97,7 @@ class TypoScriptTemplateObjectBrowserModuleFunctionController extends AbstractFu
             }
             if (!empty($this->pObj->MOD_SETTINGS['ts_browser_TLKeys_' . $bType])) {
                 $modMenu['ts_browser_toplevel_' . $bType]['-'] = '---';
-                $modMenu['ts_browser_toplevel_' . $bType] = $modMenu[('ts_browser_toplevel_' . $bType)] + $this->pObj->MOD_SETTINGS[('ts_browser_TLKeys_' . $bType)];
+                $modMenu['ts_browser_toplevel_' . $bType] = $modMenu['ts_browser_toplevel_' . $bType] + $this->pObj->MOD_SETTINGS['ts_browser_TLKeys_' . $bType];
             }
         }
         return $modMenu;
@@ -389,7 +389,7 @@ class TypoScriptTemplateObjectBrowserModuleFunctionController extends AbstractFu
                 'id' => $this->pObj->id
             );
             $aHref = BackendUtility::getModuleUrl('web_ts', $urlParameters);
-            if (!$this->pObj->MOD_SETTINGS[('ts_browser_TLKeys_' . $bType)][$this->pObj->sObj]) {
+            if (!$this->pObj->MOD_SETTINGS['ts_browser_TLKeys_' . $bType][$this->pObj->sObj]) {
                 if (!empty($theSetup)) {
                     $out = '<a href="' . htmlspecialchars(($aHref . '&addKey[' . rawurlencode($this->pObj->sObj) . ']=1&SET[ts_browser_toplevel_' . $bType . ']=' . rawurlencode($this->pObj->sObj))) . '">';
                     $out .= sprintf($lang->getLL('addKey'), htmlspecialchars($this->pObj->sObj));