[TASK] Remove superfluous parens in sysexts css_styled_content - feedit
authorArvid Jakobsson <arvid.jakobsson@gmail.com>
Sun, 16 Sep 2012 18:34:31 +0000 (20:34 +0200)
committerJigal van Hemert <jigal@xs4all.nl>
Mon, 17 Sep 2012 19:57:04 +0000 (21:57 +0200)
Removes superfluous parenthesis in sysexts "css_styled_content",
"extensionmanager", "extra_page_cm_options" and "feedit".

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

Change-Id: I9ec62d6b2782e349cef447c7a9c87f4a4fbc2585
Resolves: #40964
Related: #40095
Reviewed-on: http://review.typo3.org/14623
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert
34 files changed:
typo3/sysext/css_styled_content/Classes/Controller/CssStyledContentController.php
typo3/sysext/extensionmanager/Classes/Controller/ActionController.php
typo3/sysext/extensionmanager/Classes/Controller/DownloadController.php
typo3/sysext/extensionmanager/Classes/Controller/ListController.php
typo3/sysext/extensionmanager/Classes/Controller/UpdateFromTerController.php
typo3/sysext/extensionmanager/Classes/Domain/Model/Mirrors.php
typo3/sysext/extensionmanager/Classes/Domain/Repository/ConfigurationItemRepository.php
typo3/sysext/extensionmanager/Classes/Domain/Repository/ExtensionRepository.php
typo3/sysext/extensionmanager/Classes/Utility/ConfigurationUtility.php
typo3/sysext/extensionmanager/Classes/Utility/Connection/TerUtility.php
typo3/sysext/extensionmanager/Classes/Utility/DatabaseUtility.php
typo3/sysext/extensionmanager/Classes/Utility/DependencyUtility.php
typo3/sysext/extensionmanager/Classes/Utility/DownloadUtility.php
typo3/sysext/extensionmanager/Classes/Utility/EmConfUtility.php
typo3/sysext/extensionmanager/Classes/Utility/FileHandlingUtility.php
typo3/sysext/extensionmanager/Classes/Utility/InstallUtility.php
typo3/sysext/extensionmanager/Classes/Utility/ListUtility.php
typo3/sysext/extensionmanager/Classes/Utility/Parser/ExtensionXmlPullParser.php
typo3/sysext/extensionmanager/Classes/Utility/Parser/MirrorXmlPullParser.php
typo3/sysext/extensionmanager/Classes/Utility/Parser/XmlAbstractParser.php
typo3/sysext/extensionmanager/Classes/Utility/Parser/XmlParserFactory.php
typo3/sysext/extensionmanager/Classes/Utility/Repository/Helper.php
typo3/sysext/extensionmanager/Classes/ViewHelpers/Be/ContainerViewHelper.php
typo3/sysext/extensionmanager/Classes/ViewHelpers/ConfigureExtensionViewHelper.php
typo3/sysext/extensionmanager/Classes/ViewHelpers/DownloadExtensionViewHelper.php
typo3/sysext/extensionmanager/Classes/ViewHelpers/Form/TypoScriptConstantsViewHelper.php
typo3/sysext/extensionmanager/Classes/ViewHelpers/UpdateFromTerViewHelper.php
typo3/sysext/extensionmanager/Tests/Unit/Domain/Repository/ConfigurationItemRepositoryTest.php
typo3/sysext/extensionmanager/Tests/Unit/Utility/DependencyUtilityTest.php
typo3/sysext/extensionmanager/Tests/Unit/Utility/FileHandlingUtilityTest.php
typo3/sysext/extensionmanager/Tests/Unit/Utility/InstallUtilityTest.php
typo3/sysext/extensionmanager/ext_tables.php
typo3/sysext/extra_page_cm_options/Classes/ExtraPageContextMenuOptions.php
typo3/sysext/feedit/Classes/FrontendEditPanel.php

index a6566df..b14a454 100644 (file)
@@ -93,14 +93,14 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
                        // Split into single lines:
                        $lines = \TYPO3\CMS\Core\Utility\GeneralUtility::trimExplode(LF, $content);
                        foreach ($lines as &$val) {
-                               $val = ('<li>' . $this->cObj->stdWrap($val, $conf['innerStdWrap.'])) . '</li>';
+                               $val = '<li>' . $this->cObj->stdWrap($val, $conf['innerStdWrap.']) . '</li>';
                        }
                        unset($val);
                        // Set header type:
                        $type = intval($this->cObj->data['layout']);
                        // Compile list:
-                       $out = ((('
-                               <ul class="csc-bulletlist csc-bulletlist-' . $type) . '">') . implode('', $lines)) . '
+                       $out = '
+                               <ul class="csc-bulletlist csc-bulletlist-' . $type . '">' . implode('', $lines) . '
                                </ul>';
                        // Calling stdWrap:
                        if ($conf['stdWrap.']) {
@@ -153,7 +153,7 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
                        }
                        // Generate id prefix for accessible header
                        $headerScope = $headerPos == 'top' ? 'col' : 'row';
-                       $headerIdPrefix = ($headerScope . $this->cObj->data['uid']) . '-';
+                       $headerIdPrefix = $headerScope . $this->cObj->data['uid'] . '-';
                        // Split into single lines (will become table-rows):
                        $rows = \TYPO3\CMS\Core\Utility\GeneralUtility::trimExplode(LF, $content);
                        reset($rows);
@@ -166,57 +166,57 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
                                $newCells = array();
                                for ($a = 0; $a < $cols; $a++) {
                                        // Remove quotes if needed
-                                       if (($quotedInput && substr($cells[$a], 0, 1) == $quotedInput) && substr($cells[$a], -1, 1) == $quotedInput) {
+                                       if ($quotedInput && substr($cells[$a], 0, 1) == $quotedInput && substr($cells[$a], -1, 1) == $quotedInput) {
                                                $cells[$a] = substr($cells[$a], 1, -1);
                                        }
                                        if (!strcmp(trim($cells[$a]), '')) {
                                                $cells[$a] = '&nbsp;';
                                        }
-                                       $cellAttribs = $noStyles ? '' : ($a > 0 && $cols - 1 == $a ? (' class="td-last td-' . $a) . '"' : (' class="td-' . $a) . '"');
+                                       $cellAttribs = $noStyles ? '' : ($a > 0 && $cols - 1 == $a ? ' class="td-last td-' . $a . '"' : ' class="td-' . $a . '"');
                                        if ($headerPos == 'top' && !$k || $headerPos == 'left' && !$a) {
-                                               $scope = (' scope="' . $headerScope) . '"';
-                                               $scope .= ((' id="' . $headerIdPrefix) . ($headerScope == 'col' ? $a : $k)) . '"';
-                                               $newCells[$a] = (((('
-                                                       <th' . $cellAttribs) . $scope) . '>') . $this->cObj->stdWrap($cells[$a], $conf['innerStdWrap.'])) . '</th>';
+                                               $scope = ' scope="' . $headerScope . '"';
+                                               $scope .= ' id="' . $headerIdPrefix . ($headerScope == 'col' ? $a : $k) . '"';
+                                               $newCells[$a] = '
+                                                       <th' . $cellAttribs . $scope . '>' . $this->cObj->stdWrap($cells[$a], $conf['innerStdWrap.']) . '</th>';
                                        } else {
                                                if (empty($headerPos)) {
                                                        $accessibleHeader = '';
                                                } else {
-                                                       $accessibleHeader = ((' headers="' . $headerIdPrefix) . ($headerScope == 'col' ? $a : $k)) . '"';
+                                                       $accessibleHeader = ' headers="' . $headerIdPrefix . ($headerScope == 'col' ? $a : $k) . '"';
                                                }
-                                               $newCells[$a] = (((('
-                                                       <td' . $cellAttribs) . $accessibleHeader) . '>') . $this->cObj->stdWrap($cells[$a], $conf['innerStdWrap.'])) . '</td>';
+                                               $newCells[$a] = '
+                                                       <td' . $cellAttribs . $accessibleHeader . '>' . $this->cObj->stdWrap($cells[$a], $conf['innerStdWrap.']) . '</td>';
                                        }
                                }
                                if (!$noStyles) {
                                        $oddEven = $k % 2 ? 'tr-odd' : 'tr-even';
-                                       $rowAttribs = $k > 0 && $rCount - 1 == $k ? (' class="' . $oddEven) . ' tr-last"' : (((' class="' . $oddEven) . ' tr-') . $k) . '"';
+                                       $rowAttribs = $k > 0 && $rCount - 1 == $k ? ' class="' . $oddEven . ' tr-last"' : ' class="' . $oddEven . ' tr-' . $k . '"';
                                }
-                               $rows[$k] = ((('
-                                       <tr' . $rowAttribs) . '>') . implode('', $newCells)) . '
+                               $rows[$k] = '
+                                       <tr' . $rowAttribs . '>' . implode('', $newCells) . '
                                        </tr>';
                        }
                        $addTbody = 0;
                        $tableContents = '';
                        if ($caption) {
-                               $tableContents .= ('
-                                       <caption>' . $caption) . '</caption>';
+                               $tableContents .= '
+                                       <caption>' . $caption . '</caption>';
                        }
                        if ($headerPos == 'top' && $rows[0]) {
-                               $tableContents .= ('<thead>' . $rows[0]) . '
+                               $tableContents .= '<thead>' . $rows[0] . '
                                        </thead>';
                                unset($rows[0]);
                                $addTbody = 1;
                        }
                        if ($useTfoot) {
-                               $tableContents .= ('
-                                       <tfoot>' . $rows[($rCount - 1)]) . '</tfoot>';
+                               $tableContents .= '
+                                       <tfoot>' . $rows[($rCount - 1)] . '</tfoot>';
                                unset($rows[$rCount - 1]);
                                $addTbody = 1;
                        }
                        $tmpTable = implode('', $rows);
                        if ($addTbody) {
-                               $tmpTable = ('<tbody>' . $tmpTable) . '</tbody>';
+                               $tmpTable = '<tbody>' . $tmpTable . '</tbody>';
                        }
                        $tableContents .= $tmpTable;
                        // Set header type:
@@ -224,13 +224,13 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
                        // Table tag params.
                        $tableTagParams = $this->getTableAttributes($conf, $type);
                        if (!$noStyles) {
-                               $tableTagParams['class'] = (('contenttable contenttable-' . $type) . ($tableClass ? ' ' . $tableClass : '')) . $tableTagParams['class'];
+                               $tableTagParams['class'] = 'contenttable contenttable-' . $type . ($tableClass ? ' ' . $tableClass : '') . $tableTagParams['class'];
                        } elseif ($tableClass) {
                                $tableTagParams['class'] = $tableClass;
                        }
                        // Compile table output:
-                       $out = ((('
-                               <table ' . \TYPO3\CMS\Core\Utility\GeneralUtility::implodeAttributes($tableTagParams)) . '>') . $tableContents) . '
+                       $out = '
+                               <table ' . \TYPO3\CMS\Core\Utility\GeneralUtility::implodeAttributes($tableTagParams) . '>' . $tableContents . '
                                </table>';
                        // Calling stdWrap:
                        if ($conf['stdWrap.']) {
@@ -367,7 +367,7 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
                                        // Table tag params
                                        $tableTagParams = $this->getTableAttributes($conf, $type);
                                        $tableTagParams['class'] = 'csc-uploads csc-uploads-' . $type;
-                                       $outerWrap = ('<table ' . \TYPO3\CMS\Core\Utility\GeneralUtility::implodeAttributes($tableTagParams)) . '>|</table>';
+                                       $outerWrap = '<table ' . \TYPO3\CMS\Core\Utility\GeneralUtility::implodeAttributes($tableTagParams) . '>|</table>';
                                }
                                // Compile it all into table tags:
                                $out = $this->cObj->wrap(implode('', $outputEntries), $outerWrap);
@@ -439,7 +439,7 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
                        // This much width is available for the remaining images in this row (int)
                        $availableWidth = $netW - $accumWidth;
                        // Theoretical width of resized image. (float)
-                       $desiredWidth = ($netW / $relUnitCount) * $colRelations[$a];
+                       $desiredWidth = $netW / $relUnitCount * $colRelations[$a];
                        // Add this width. $accumDesiredWidth becomes the desired horizontal position
                        $accumDesiredWidth += $desiredWidth;
                        // Calculate width by comparing actual and desired horizontal position.
@@ -499,7 +499,7 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
                }
                $imgPath = $this->cObj->stdWrap($conf['imgPath'], $conf['imgPath.']);
                // Does we need to render a "global caption" (below the whole image block)?
-               $renderGlobalCaption = (!$conf['captionSplit'] && !$conf['imageTextSplit']) && is_array($conf['caption.']);
+               $renderGlobalCaption = !$conf['captionSplit'] && !$conf['imageTextSplit'] && is_array($conf['caption.']);
                if ($imgCount == 1) {
                        // If we just have one image, the caption relates to the image, so it is not "global"
                        $renderGlobalCaption = FALSE;
@@ -559,7 +559,7 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
                // Max Width
                $maxW = intval($this->cObj->stdWrap($conf['maxW'], $conf['maxW.']));
                $maxWInText = intval($this->cObj->stdWrap($conf['maxWInText'], $conf['maxWInText.']));
-               $fiftyPercentWidthInText = round(($maxW / 100) * 50);
+               $fiftyPercentWidthInText = round($maxW / 100 * 50);
                // in Text
                if ($contentPosition >= 16) {
                        if (!$maxWInText) {
@@ -570,7 +570,7 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
                        }
                }
                // max usuable width for images (without spacers and borders)
-               $netW = ($maxW - $colspacing * ($colCount - 1)) - (($colCount * $border) * ($borderThickness + $borderSpace)) * 2;
+               $netW = $maxW - $colspacing * ($colCount - 1) - $colCount * $border * ($borderThickness + $borderSpace) * 2;
                // Specify the maximum width for each column
                $columnWidths = $this->getImgColumnWidths($conf, $colCount, $netW);
                $image_compression = intval($this->cObj->stdWrap($conf['image_compression'], $conf['image_compression.']));
@@ -669,7 +669,7 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
                                $titleText = trim($this->cObj->stdWrap($imgConf['titleText'], $imgConf['titleText.']));
                                if ($titleText) {
                                        // This will be used by the IMAGE call later:
-                                       $GLOBALS['TSFE']->ATagParams .= (' title="' . $titleText) . '"';
+                                       $GLOBALS['TSFE']->ATagParams .= ' title="' . $titleText . '"';
                                }
                        }
                        if ($imgConf || $imgConf['file']) {
@@ -720,7 +720,7 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
                        }
                }
                // How much space will the image-block occupy?
-               $imageBlockWidth = (max($imageRowsFinalWidths) + $colspacing * ($colCount - 1)) + (($colCount * $border) * ($borderSpace + $borderThickness)) * 2;
+               $imageBlockWidth = max($imageRowsFinalWidths) + $colspacing * ($colCount - 1) + $colCount * $border * ($borderSpace + $borderThickness) * 2;
                $GLOBALS['TSFE']->register['rowwidth'] = $imageBlockWidth;
                $GLOBALS['TSFE']->register['rowWidthPlusTextMargin'] = $imageBlockWidth + $textMargin;
                // noRows is in fact just one ROW, with the amount of columns specified, where the images are placed in.
@@ -856,17 +856,17 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
                        }
                        // Set the margin for image + text, no wrap always to avoid multiple stylesheets
                        $noWrapMargin = (int) (($maxWInText ? $maxWInText : $fiftyPercentWidthInText) + intval($this->cObj->stdWrap($conf['textMargin'], $conf['textMargin.'])));
-                       $this->addPageStyle('.csc-textpic-intext-right-nowrap .csc-textpic-text', ('margin-right: ' . $noWrapMargin) . 'px;');
-                       $this->addPageStyle('.csc-textpic-intext-left-nowrap .csc-textpic-text', ('margin-left: ' . $noWrapMargin) . 'px;');
+                       $this->addPageStyle('.csc-textpic-intext-right-nowrap .csc-textpic-text', 'margin-right: ' . $noWrapMargin . 'px;');
+                       $this->addPageStyle('.csc-textpic-intext-left-nowrap .csc-textpic-text', 'margin-left: ' . $noWrapMargin . 'px;');
                        // Beside Text where the image block width is not equal to maxW
                        if ($contentPosition == 24 && $maxW != $imageBlockWidth) {
                                $noWrapMargin = $imageBlockWidth + $textMargin;
                                // Beside Text, Right
                                if ($imagePosition == 1) {
-                                       $this->addPageStyle(('.csc-textpic-intext-right-nowrap-' . $noWrapMargin) . ' .csc-textpic-text', ('margin-right: ' . $noWrapMargin) . 'px;');
+                                       $this->addPageStyle('.csc-textpic-intext-right-nowrap-' . $noWrapMargin . ' .csc-textpic-text', 'margin-right: ' . $noWrapMargin . 'px;');
                                        $classes[] = 'csc-textpic-intext-right-nowrap-' . $noWrapMargin;
                                } elseif ($imagePosition == 2) {
-                                       $this->addPageStyle(('.csc-textpic-intext-left-nowrap-' . $noWrapMargin) . ' .csc-textpic-text', ('margin-left: ' . $noWrapMargin) . 'px;');
+                                       $this->addPageStyle('.csc-textpic-intext-left-nowrap-' . $noWrapMargin . ' .csc-textpic-text', 'margin-left: ' . $noWrapMargin . 'px;');
                                        $classes[] = 'csc-textpic-intext-left-nowrap-' . $noWrapMargin;
                                }
                        }
@@ -913,9 +913,9 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
                                        }
                                        // Render one image
                                        if ($origImages[$imgKey][0] == 0) {
-                                               $imageSpace = $this->cObj->data['imagewidth'] + ($border * ($borderSpace + $borderThickness)) * 2;
+                                               $imageSpace = $this->cObj->data['imagewidth'] + $border * ($borderSpace + $borderThickness) * 2;
                                        } else {
-                                               $imageSpace = $origImages[$imgKey][0] + ($border * ($borderSpace + $borderThickness)) * 2;
+                                               $imageSpace = $origImages[$imgKey][0] + $border * ($borderSpace + $borderThickness) * 2;
                                        }
                                        $GLOBALS['TSFE']->register['IMAGE_NUM'] = $imgKey;
                                        $GLOBALS['TSFE']->register['IMAGE_NUM_CURRENT'] = $imgKey;
@@ -1048,10 +1048,10 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
                        $constant = (int) $configuration['constant'];
                        if ($configuration['space'] === 'before') {
                                $value = $constant + $this->cObj->data['spaceBefore'];
-                               $declaration = ('margin-top: ' . $value) . 'px;';
+                               $declaration = 'margin-top: ' . $value . 'px;';
                        } else {
                                $value = $constant + $this->cObj->data['spaceAfter'];
-                               $declaration = ('margin-bottom: ' . $value) . 'px;';
+                               $declaration = 'margin-bottom: ' . $value . 'px;';
                        }
                        if (!empty($value)) {
                                if ($configuration['stdWrap.']) {
@@ -1092,7 +1092,7 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
                        $pos = strrpos($linkText, '.');
                        $linkText = substr($linkText, 0, $pos);
                }
-               $links[1] = str_replace(('>' . $fileName) . '<', ('>' . htmlspecialchars($linkText)) . '<', $links[1]);
+               $links[1] = str_replace('>' . $fileName . '<', '>' . htmlspecialchars($linkText) . '<', $links[1]);
                return $links;
        }
 
@@ -1106,7 +1106,7 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
         */
        public function getTableAttributes($conf, $type) {
                // Initializing:
-               $tableTagParams_conf = $conf[('tableParams_' . $type) . '.'];
+               $tableTagParams_conf = $conf['tableParams_' . $type . '.'];
                $border = $this->cObj->data['table_border'] ? intval($this->cObj->data['table_border']) : $tableTagParams_conf['border'];
                $cellSpacing = $this->cObj->data['table_cellspacing'] ? intval($this->cObj->data['table_cellspacing']) : $tableTagParams_conf['cellspacing'];
                $cellPadding = $this->cObj->data['table_cellpadding'] ? intval($this->cObj->data['table_cellpadding']) : $tableTagParams_conf['cellpadding'];
@@ -1125,7 +1125,7 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
                        if ($border) {
                                // Border property has changed, now with class
                                $borderClass = 'contenttable-border-' . $border;
-                               $borderDeclaration = ('border-width: ' . $border) . 'px; border-style: solid;';
+                               $borderDeclaration = 'border-width: ' . $border . 'px; border-style: solid;';
                                $this->addPageStyle('.' . $borderClass, $borderDeclaration);
                                $classes[] = $borderClass;
                        }
@@ -1134,15 +1134,15 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
                                $tableTagParams['border'] = 1;
                                // Use CSS3 border-spacing in class to have cell spacing
                                $cellSpacingClass = 'contenttable-cellspacing-' . $cellSpacing;
-                               $cellSpacingDeclaration = ('border-spacing: ' . $cellSpacing) . 'px;';
+                               $cellSpacingDeclaration = 'border-spacing: ' . $cellSpacing . 'px;';
                                $this->addPageStyle('.' . $cellSpacingClass, $cellSpacingDeclaration);
                                $classes[] = $cellSpacingClass;
                        }
                        if ($cellPadding) {
                                // Cell padding property has changed, now with class
                                $cellPaddingClass = 'contenttable-cellpadding-' . $cellPadding;
-                               $cellSpacingSelector = ((('.' . $cellPaddingClass) . ' td, .') . $cellPaddingClass) . ' th';
-                               $cellPaddingDeclaration = ('padding: ' . $cellPadding) . 'px;';
+                               $cellSpacingSelector = '.' . $cellPaddingClass . ' td, .' . $cellPaddingClass . ' th';
+                               $cellPaddingDeclaration = 'padding: ' . $cellPadding . 'px;';
                                $this->addPageStyle($cellSpacingSelector, $cellPaddingDeclaration);
                                $classes[] = $cellPaddingClass;
                        }
@@ -1174,7 +1174,7 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
                        $GLOBALS['TSFE']->tmpl->setup['plugin.']['tx_cssstyledcontent.']['_CSS_PAGE_STYLE'] = array();
                }
                if (!isset($GLOBALS['TSFE']->tmpl->setup['plugin.']['tx_cssstyledcontent.']['_CSS_PAGE_STYLE'][$selector])) {
-                       $GLOBALS['TSFE']->tmpl->setup['plugin.']['tx_cssstyledcontent.']['_CSS_PAGE_STYLE'][$selector] = (((TAB . $selector) . ' { ') . $declaration) . ' }';
+                       $GLOBALS['TSFE']->tmpl->setup['plugin.']['tx_cssstyledcontent.']['_CSS_PAGE_STYLE'][$selector] = TAB . $selector . ' { ' . $declaration . ' }';
                }
        }
 
index 5f839f2..04a7f0c 100644 (file)
@@ -134,7 +134,7 @@ class ActionController extends \TYPO3\CMS\Extensionmanager\Controller\AbstractCo
                $sqlData = $this->installUtility->getExtensionSqlDataDump($extension);
                $dump = $sqlData['extTables'] . $sqlData['staticSql'];
                $fileName = $extension . '_sqlDump.sql';
-               $filePath = (PATH_site . 'typo3temp/') . $fileName;
+               $filePath = PATH_site . 'typo3temp/' . $fileName;
                $error = \TYPO3\CMS\Core\Utility\GeneralUtility::writeFileToTypo3tempDir($filePath, $dump);
                if (is_string($error)) {
                        throw new \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException($error, 1343048718);
index d6f288e..31cc8d9 100644 (file)
@@ -121,7 +121,7 @@ class DownloadController extends \TYPO3\CMS\Extensionmanager\Controller\Abstract
                        // @todo translate and beautify
                        $message = 'The following dependencies have to be resolved before installation:<br /><br />';
                        foreach ($dependencyTypes as $dependencyType => $dependencies) {
-                               $message .= ('<h3>Extensions marked for ' . $dependencyType) . ':</h3>';
+                               $message .= '<h3>Extensions marked for ' . $dependencyType . ':</h3>';
                                foreach ($dependencies as $extensionKey => $dependency) {
                                        $message .= $extensionKey . '<br />';
                                }
index 1131d29..f4b683b 100644 (file)
@@ -116,7 +116,7 @@ class ListController extends \TYPO3\CMS\Extensionmanager\Controller\AbstractCont
                $this->pageRenderer->addJsFile($this->backPath . '../t3lib/js/extjs/notifications.js');
                $extensions = array();
                $extensionKey = '';
-               if ((($this->request->hasArgument('allVersions') && $this->request->getArgument('allVersions') == 1) && $this->request->hasArgument('extensionKey')) && is_string($this->request->getArgument('extensionKey'))) {
+               if ($this->request->hasArgument('allVersions') && $this->request->getArgument('allVersions') == 1 && $this->request->hasArgument('extensionKey') && is_string($this->request->getArgument('extensionKey'))) {
                        $extensionKey = $this->request->getArgument('extensionKey');
                        $extensions = $this->extensionRepository->findByExtensionKeyOrderedByVersion($extensionKey);
                } else {
index bf5f7c0..562981e 100644 (file)
@@ -104,7 +104,7 @@ class UpdateFromTerController extends \TYPO3\CMS\Extensionmanager\Controller\Abs
                $errorMessage = '';
                /** @var $repository \TYPO3\CMS\Extensionmanager\Domain\Model\Repository */
                $repository = $this->repositoryRepository->findOneByUid((int)$this->settings['repositoryUid']);
-               if ($repository->getLastUpdate()->getTimestamp() < $GLOBALS['EXEC_TIME'] - (24 * 60) * 60 || $forceUpdateCheck) {
+               if ($repository->getLastUpdate()->getTimestamp() < $GLOBALS['EXEC_TIME'] - 24 * 60 * 60 || $forceUpdateCheck) {
                        try {
                                $updated = $this->repositoryHelper->updateExtList();
                        } catch (\TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException $e) {
index e5677c1..3d3bca5 100644 (file)
@@ -70,7 +70,7 @@ class Mirrors extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
                if (is_null($mirrorId)) {
                        $this->isRandomSelection = TRUE;
                } else {
-                       if ((is_int($mirrorId) && $mirrorId >= 1) && $mirrorId <= count($this->mirrors)) {
+                       if (is_int($mirrorId) && $mirrorId >= 1 && $mirrorId <= count($this->mirrors)) {
                                $this->currentMirror = $mirrorId - 1;
                        }
                }
index ed25dec..f8d7142 100644 (file)
@@ -54,7 +54,7 @@ class ConfigurationItemRepository {
         * @return null|SplObjectStorage
         */
        public function findByExtension(array $extension) {
-               $configRaw = \TYPO3\CMS\Core\Utility\GeneralUtility::getUrl((PATH_site . $extension['siteRelPath']) . '/ext_conf_template.txt');
+               $configRaw = \TYPO3\CMS\Core\Utility\GeneralUtility::getUrl(PATH_site . $extension['siteRelPath'] . '/ext_conf_template.txt');
                $configurationObjectStorage = NULL;
                if ($configRaw) {
                        $configurationObjectStorage = $this->convertRawConfigurationToObject($configRaw, $extension);
index 8387aac..00d9063 100644 (file)
@@ -106,24 +106,24 @@ class ExtensionRepository extends \TYPO3\CMS\Extbase\Persistence\Repository {
         */
        public function findByTitleOrAuthorNameOrExtensionKey($searchString) {
                $quotedSearchString = $GLOBALS['TYPO3_DB']->escapeStrForLike($GLOBALS['TYPO3_DB']->quoteStr($searchString, 'tx_extensionmanager_domain_model_extension'), 'tx_extensionmanager_domain_model_extension');
-               $quotedSearchStringForLike = ('\'%' . $quotedSearchString) . '%\'';
-               $quotedSearchString = ('\'' . $quotedSearchString) . '\'';
-               $select = ((((((('tx_extensionmanager_domain_model_extension.*,
+               $quotedSearchStringForLike = '\'%' . $quotedSearchString . '%\'';
+               $quotedSearchString = '\'' . $quotedSearchString . '\'';
+               $select = 'tx_extensionmanager_domain_model_extension.*,
                        (
-                               (extension_key like ' . $quotedSearchString) . ') * 8 +
-                               (extension_key like ') . $quotedSearchStringForLike) . ') * 4 +
-                               (title like ') . $quotedSearchStringForLike) . ') * 2 +
-                               (author_name like ') . $quotedSearchStringForLike) . ')
+                               (extension_key like ' . $quotedSearchString . ') * 8 +
+                               (extension_key like ' . $quotedSearchStringForLike . ') * 4 +
+                               (title like ' . $quotedSearchStringForLike . ') * 2 +
+                               (author_name like ' . $quotedSearchStringForLike . ')
                        ) as position';
                $from = 'tx_extensionmanager_domain_model_extension';
-               $where = ((((((('(
-                                       extension_key = ' . $quotedSearchString) . '
+               $where = '(
+                                       extension_key = ' . $quotedSearchString . '
                                        OR
-                                       extension_key LIKE ') . $quotedSearchStringForLike) . '
+                                       extension_key LIKE ' . $quotedSearchStringForLike . '
                                        OR
-                                       description LIKE ') . $quotedSearchStringForLike) . '
+                                       description LIKE ' . $quotedSearchStringForLike . '
                                        OR
-                                       title LIKE ') . $quotedSearchStringForLike) . '
+                                       title LIKE ' . $quotedSearchStringForLike . '
                                )
                                AND current_version=1
                                HAVING position > 0';
@@ -204,7 +204,7 @@ class ExtensionRepository extends \TYPO3\CMS\Extbase\Persistence\Repository {
                        $GLOBALS['TYPO3_DB']->exec_UPDATEquery('tx_extensionmanager_domain_model_extension', 'current_version=1 AND repository=' . intval($repositoryUid), array('current_version' => 0));
                        // Find latest version of extensions and set current_version to 1 for these
                        foreach ($groupedRows as $row) {
-                               $GLOBALS['TYPO3_DB']->exec_UPDATEquery('tx_extensionmanager_domain_model_extension', (((('extension_key=' . $GLOBALS['TYPO3_DB']->fullQuoteStr($row['extension_key'], 'tx_extensionmanager_domain_model_extension')) . ' AND integer_version=') . intval($row['maxintversion'])) . ' AND repository=') . intval($repositoryUid), array('current_version' => 1));
+                               $GLOBALS['TYPO3_DB']->exec_UPDATEquery('tx_extensionmanager_domain_model_extension', 'extension_key=' . $GLOBALS['TYPO3_DB']->fullQuoteStr($row['extension_key'], 'tx_extensionmanager_domain_model_extension') . ' AND integer_version=' . intval($row['maxintversion']) . ' AND repository=' . intval($repositoryUid), array('current_version' => 1));
                        }
                }
                return $extensions;
index c492543..1b51c67 100644 (file)
@@ -94,7 +94,7 @@ class ConfigurationUtility implements \TYPO3\CMS\Core\SingletonInterface {
         */
        public function getCurrentConfiguration($extensionKey) {
                $extension = $GLOBALS['TYPO3_LOADED_EXT'][$extensionKey];
-               $defaultConfig = $this->configurationItemRepository->createArrayFromConstants(\TYPO3\CMS\Core\Utility\GeneralUtility::getUrl((PATH_site . $extension['siteRelPath']) . '/ext_conf_template.txt'), $extension);
+               $defaultConfig = $this->configurationItemRepository->createArrayFromConstants(\TYPO3\CMS\Core\Utility\GeneralUtility::getUrl(PATH_site . $extension['siteRelPath'] . '/ext_conf_template.txt'), $extension);
                $currentExtensionConfig = unserialize($GLOBALS['TYPO3_CONF_VARS']['EXT']['extConf'][$extension['key']]);
                $currentExtensionConfig = is_array($currentExtensionConfig) ? $currentExtensionConfig : array();
                $currentFullConfiguration = \TYPO3\CMS\Core\Utility\GeneralUtility::array_merge_recursive_overrule($defaultConfig, $currentExtensionConfig);
index 76ac420..76edd8e 100644 (file)
@@ -56,7 +56,7 @@ class TerUtility {
         */
        public function fetchExtension($extensionKey, $version, $expectedMd5, $mirrorUrl) {
                $extensionPath = \TYPO3\CMS\Core\Utility\GeneralUtility::strtolower($extensionKey);
-               $mirrorUrl .= (((((($extensionPath[0] . '/') . $extensionPath[1]) . '/') . $extensionPath) . '_') . $version) . '.t3x';
+               $mirrorUrl .= $extensionPath[0] . '/' . $extensionPath[1] . '/' . $extensionPath . '_' . $version . '.t3x';
                $t3x = \TYPO3\CMS\Core\Utility\GeneralUtility::getUrl($mirrorUrl, 0, array(TYPO3_user_agent));
                $md5 = md5($t3x);
                if ($t3x === FALSE) {
@@ -66,7 +66,7 @@ class TerUtility {
                        // Fetch and return:
                        $extensionData = $this->decodeExchangeData($t3x);
                } else {
-                       throw new \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException((('Error: MD5 hash of downloaded file not as expected:<br />' . $md5) . ' != ') . $expectedMd5, 1334426098);
+                       throw new \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException('Error: MD5 hash of downloaded file not as expected:<br />' . $md5 . ' != ' . $expectedMd5, 1334426098);
                }
                return $extensionData;
        }
index 8bbcfcb..b4cc2a3 100644 (file)
@@ -56,7 +56,7 @@ class DatabaseUtility implements \TYPO3\CMS\Core\SingletonInterface {
                                $header = $this->dumpHeader();
                                $tableHeader = $this->dumpTableHeader($table, $dbFields[$table], TRUE);
                                $insertStatements = $this->dumpTableContent($table, $dbFields[$table]['fields']);
-                               $out .= (((($header . self::MULTI_LINEBREAKS) . $tableHeader) . self::MULTI_LINEBREAKS) . $insertStatements) . self::MULTI_LINEBREAKS;
+                               $out .= $header . self::MULTI_LINEBREAKS . $tableHeader . self::MULTI_LINEBREAKS . $insertStatements . self::MULTI_LINEBREAKS;
                        }
                }
                return $out;
@@ -68,10 +68,10 @@ class DatabaseUtility implements \TYPO3\CMS\Core\SingletonInterface {
         * @return string Table header
         */
        protected function dumpHeader() {
-               return trim(((('
+               return trim('
 # TYPO3 Extension Manager dump 1.1
 #
-# Host: ' . TYPO3_db_host) . '    Database: ') . TYPO3_db) . '
+# Host: ' . TYPO3_db_host . '    Database: ' . TYPO3_db . '
 #--------------------------------------------------------
 ');
        }
@@ -90,7 +90,7 @@ class DatabaseUtility implements \TYPO3\CMS\Core\SingletonInterface {
                // Create field definitions
                if (is_array($fieldKeyInfo['fields'])) {
                        foreach ($fieldKeyInfo['fields'] as $fieldN => $data) {
-                               $lines[] = (('  ' . $fieldN) . ' ') . $data;
+                               $lines[] = '  ' . $fieldN . ' ' . $data;
                        }
                }
                // Create index key definitions
@@ -101,13 +101,13 @@ class DatabaseUtility implements \TYPO3\CMS\Core\SingletonInterface {
                }
                // Compile final output:
                if (count($lines)) {
-                       $dump = trim(((((((('
+                       $dump = trim('
 #
-# Table structure for table "' . $table) . '"
+# Table structure for table "' . $table . '"
 #
-') . ($dropTableIfExists ? ('DROP TABLE IF EXISTS ' . $table) . ';
-' : '')) . 'CREATE TABLE ') . $table) . ' (
-') . implode((',' . LF), $lines)) . '
+' . ($dropTableIfExists ? 'DROP TABLE IF EXISTS ' . $table . ';
+' : '') . 'CREATE TABLE ' . $table . ' (
+' . implode((',' . LF), $lines) . '
 );');
                }
                return $dump;
@@ -135,9 +135,9 @@ class DatabaseUtility implements \TYPO3\CMS\Core\SingletonInterface {
                while ($row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($result)) {
                        $values = array();
                        foreach ($fieldStructure as $field => $structure) {
-                               $values[] = isset($row[$field]) ? ('\'' . str_replace($search, $replace, $row[$field])) . '\'' : 'NULL';
+                               $values[] = isset($row[$field]) ? '\'' . str_replace($search, $replace, $row[$field]) . '\'' : 'NULL';
                        }
-                       $lines[] = ((('INSERT INTO ' . $table) . ' VALUES (') . implode(', ', $values)) . ');';
+                       $lines[] = 'INSERT INTO ' . $table . ' VALUES (' . implode(', ', $values) . ');';
                }
                // Free DB result:
                $GLOBALS['TYPO3_DB']->sql_free_result($result);
index 19f3e60..a0bcd1d 100644 (file)
@@ -165,7 +165,7 @@ class DependencyUtility implements \TYPO3\CMS\Core\SingletonInterface {
                foreach ($dependencies as $dependency) {
                        $identifier = strtolower($dependency->getIdentifier());
                        if (in_array($identifier, \TYPO3\CMS\Extensionmanager\Domain\Model\Dependency::$specialDependencies)) {
-                               $methodname = ('check' . ucfirst($identifier)) . 'Dependency';
+                               $methodname = 'check' . ucfirst($identifier) . 'Dependency';
                                try {
                                        $this->{$methodname}($dependency);
                                } catch (\TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException $e) {
@@ -197,10 +197,10 @@ class DependencyUtility implements \TYPO3\CMS\Core\SingletonInterface {
                                throw new \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException('Your TYPO3 version is lower than necessary. You need at least TYPO3 version ' . $dependency->getLowestVersion());
                        }
                        if (!($dependency->getHighestVersion() === '') && version_compare($dependency->getHighestVersion(), \TYPO3\CMS\Core\Utility\VersionNumberUtility::getNumericTypo3Version()) === -1) {
-                               throw new \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException((('Your TYPO3 version is higher than allowed. You can use TYPO3 versions ' . $dependency->getLowestVersion()) . ' - ') . $dependency->getHighestVersion());
+                               throw new \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException('Your TYPO3 version is higher than allowed. You can use TYPO3 versions ' . $dependency->getLowestVersion() . ' - ' . $dependency->getHighestVersion());
                        }
                } else {
-                       throw new \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException(('checkTypo3Dependency can only check TYPO3 dependencies. Found dependency with identifier "' . $dependency->getIdentifier()) . '"');
+                       throw new \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException('checkTypo3Dependency can only check TYPO3 dependencies. Found dependency with identifier "' . $dependency->getIdentifier() . '"');
                }
                return TRUE;
        }
@@ -219,10 +219,10 @@ class DependencyUtility implements \TYPO3\CMS\Core\SingletonInterface {
                                throw new \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException('Your PHP version is lower than necessary. You need at least PHP version ' . $dependency->getLowestVersion());
                        }
                        if (!($dependency->getHighestVersion() === '') && version_compare($dependency->getHighestVersion(), PHP_VERSION) === -1) {
-                               throw new \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException((('Your PHP version is higher than allowed. You can use PHP versions ' . $dependency->getLowestVersion()) . ' - ') . $dependency->getHighestVersion());
+                               throw new \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException('Your PHP version is higher than allowed. You can use PHP versions ' . $dependency->getLowestVersion() . ' - ' . $dependency->getHighestVersion());
                        }
                } else {
-                       throw new \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException(('checkPhpDependency can only check PHP dependencies. Found dependency with identifier "' . $dependency->getIdentifier()) . '"');
+                       throw new \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException('checkPhpDependency can only check PHP dependencies. Found dependency with identifier "' . $dependency->getIdentifier() . '"');
                }
                return TRUE;
        }
@@ -293,7 +293,7 @@ class DependencyUtility implements \TYPO3\CMS\Core\SingletonInterface {
                                throw new \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException('No compatible version found for extension ' . $extensionKey);
                        }
                } else {
-                       throw new \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException(('The extension ' . $extensionKey) . ' is not available from TER.');
+                       throw new \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException('The extension ' . $extensionKey . ' is not available from TER.');
                }
        }
 
@@ -411,7 +411,7 @@ class DependencyUtility implements \TYPO3\CMS\Core\SingletonInterface {
                $dependentExtensions = array();
                foreach ($availableAndInstalledExtensions as $availableAndInstalledExtensionKey => $availableAndInstalledExtension) {
                        if (isset($availableAndInstalledExtension['installed']) && $availableAndInstalledExtension['installed'] === TRUE) {
-                               if ((is_array($availableAndInstalledExtension['constraints']) && is_array($availableAndInstalledExtension['constraints']['depends'])) && array_key_exists($extensionKey, $availableAndInstalledExtension['constraints']['depends'])) {
+                               if (is_array($availableAndInstalledExtension['constraints']) && is_array($availableAndInstalledExtension['constraints']['depends']) && array_key_exists($extensionKey, $availableAndInstalledExtension['constraints']['depends'])) {
                                        $dependentExtensions[] = $availableAndInstalledExtensionKey;
                                }
                        }
index fc2f2f1..3cc7ac8 100644 (file)
@@ -88,7 +88,7 @@ class DownloadUtility implements \TYPO3\CMS\Core\SingletonInterface {
        public function download(\TYPO3\CMS\Extensionmanager\Domain\Model\Extension $extension) {
                $mirrorUrl = $this->repositoryHelper->getMirrors()->getMirrorUrl();
                $fetchedExtension = $this->terUtility->fetchExtension($extension->getExtensionKey(), $extension->getVersion(), $extension->getMd5hash(), $mirrorUrl);
-               if ((isset($fetchedExtension['extKey']) && !empty($fetchedExtension['extKey'])) && is_string($fetchedExtension['extKey'])) {
+               if (isset($fetchedExtension['extKey']) && !empty($fetchedExtension['extKey']) && is_string($fetchedExtension['extKey'])) {
                        $this->fileHandlingUtility->unpackExtensionFromExtensionDataArray($fetchedExtension, $extension, $this->getDownloadPath());
                }
        }
index af87899..254538d 100644 (file)
@@ -43,7 +43,7 @@ class EmConfUtility implements \TYPO3\CMS\Core\SingletonInterface {
         */
        public function includeEmConf(array $extension) {
                $_EXTKEY = $extension['key'];
-               $path = (PATH_site . $extension['siteRelPath']) . '/ext_emconf.php';
+               $path = PATH_site . $extension['siteRelPath'] . '/ext_emconf.php';
                $EM_CONF = NULL;
                if (file_exists($path)) {
                        include $path;
@@ -69,19 +69,19 @@ class EmConfUtility implements \TYPO3\CMS\Core\SingletonInterface {
                }
                $emConf = $this->fixEmConf($extensionData['EM_CONF']);
                $emConf = var_export($emConf, TRUE);
-               $code = ((((('<?php
+               $code = '<?php
 
 /***************************************************************
-* Extension Manager/Repository config file for ext "' . $extensionData['extKey']) . '".
+* Extension Manager/Repository config file for ext "' . $extensionData['extKey'] . '".
 *
-* Auto generated ') . date('d-m-Y H:i')) . '
+* Auto generated ' . date('d-m-Y H:i') . '
 *
 * Manual updates:
 * Only the data in the array - everything else is removed by next
 * writing. "version" and "dependencies" must not be touched!
 ***************************************************************/
 
-$EM_CONF[$_EXTKEY] = ') . $emConf) . ';
+$EM_CONF[$_EXTKEY] = ' . $emConf . ';
 
 ?>';
                return str_replace('  ', TAB, $code);
@@ -94,7 +94,7 @@ $EM_CONF[$_EXTKEY] = ') . $emConf) . ';
         * @return array
         */
        public function fixEmConf(array $emConf) {
-               if (((!isset($emConf['constraints']) || !isset($emConf['constraints']['depends'])) || !isset($emConf['constraints']['conflicts'])) || !isset($emConf['constraints']['suggests'])) {
+               if (!isset($emConf['constraints']) || !isset($emConf['constraints']['depends']) || !isset($emConf['constraints']['conflicts']) || !isset($emConf['constraints']['suggests'])) {
                        if (!isset($emConf['constraints']) || !isset($emConf['constraints']['depends'])) {
                                $emConf['constraints']['depends'] = $this->stringToDependency($emConf['dependencies']);
                                if (strlen($emConf['PHP_version'])) {
index 5c46438..d36fa67 100644 (file)
@@ -146,10 +146,10 @@ class FileHandlingUtility implements \TYPO3\CMS\Core\SingletonInterface {
        protected function makeAndClearExtensionDir($extensionkey, $pathType = 'Local') {
                $paths = \TYPO3\CMS\Extensionmanager\Domain\Model\Extension::returnInstallPaths();
                $path = $paths[$pathType];
-               if ((!$path || !is_dir($path)) || !$extensionkey) {
+               if (!$path || !is_dir($path) || !$extensionkey) {
                        throw new \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException(sprintf('ERROR: The extension install path "%s" was no directory!', $path), 1337280417);
                } else {
-                       $extDirPath = ($path . $extensionkey) . '/';
+                       $extDirPath = $path . $extensionkey . '/';
                        if (is_dir($extDirPath)) {
                                $this->removeDirectory($extDirPath);
                        }
@@ -245,7 +245,7 @@ class FileHandlingUtility implements \TYPO3\CMS\Core\SingletonInterface {
         */
        public function createZipFileFromExtension($extension) {
                $extensionPath = $this->getAbsoluteExtensionPath($extension);
-               $fileName = ((PATH_site . 'typo3temp/') . $extension) . '.zip';
+               $fileName = PATH_site . 'typo3temp/' . $extension . '.zip';
                $zip = new \ZipArchive();
                $zip->open($fileName, \ZipArchive::CREATE);
                $iterator = new \RecursiveIteratorIterator(new \RecursiveDirectoryIterator($extensionPath));
@@ -283,7 +283,7 @@ class FileHandlingUtility implements \TYPO3\CMS\Core\SingletonInterface {
                                                \TYPO3\CMS\Core\Utility\GeneralUtility::mkdir_deep($extensionDir . $dir);
                                        }
                                        if (strlen(trim($file)) > 0) {
-                                               $return = \TYPO3\CMS\Core\Utility\GeneralUtility::writeFile((($extensionDir . $dir) . '/') . $file, zip_entry_read($zipEntry, zip_entry_filesize($zipEntry)));
+                                               $return = \TYPO3\CMS\Core\Utility\GeneralUtility::writeFile($extensionDir . $dir . '/' . $file, zip_entry_read($zipEntry, zip_entry_filesize($zipEntry)));
                                                if ($return === FALSE) {
                                                        throw new \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException('Could not write file ' . $file, 1344691048);
                                                }
@@ -310,7 +310,7 @@ class FileHandlingUtility implements \TYPO3\CMS\Core\SingletonInterface {
                }
                header('Content-Type: application/zip');
                header('Content-Length: ' . filesize($fileName));
-               header(('Content-Disposition: attachment; filename="' . $downloadName) . '.zip"');
+               header('Content-Disposition: attachment; filename="' . $downloadName . '.zip"');
                readfile($fileName);
                unlink($fileName);
                die;
@@ -331,7 +331,7 @@ class FileHandlingUtility implements \TYPO3\CMS\Core\SingletonInterface {
                }
                header('Content-Type: text');
                header('Content-Length: ' . filesize($fileName));
-               header(('Content-Disposition: attachment; filename="' . $downloadName) . '.sql"');
+               header('Content-Disposition: attachment; filename="' . $downloadName . '.sql"');
                readfile($fileName);
                unlink($fileName);
                die;
index 540651f..b6e9f7d 100644 (file)
@@ -132,7 +132,7 @@ class InstallUtility implements \TYPO3\CMS\Core\SingletonInterface {
        public function uninstall($extensionKey) {
                $dependentExtensions = $this->dependencyUtility->findInstalledExtensionsThatDependOnMe($extensionKey);
                if (is_array($dependentExtensions) && count($dependentExtensions) > 0) {
-                       throw new \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException(((('Cannot deactivate extension ' . $extensionKey) . ' - The extension(s) ') . implode(',', $dependentExtensions)) . ' depend on it', 1342554622);
+                       throw new \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException('Cannot deactivate extension ' . $extensionKey . ' - The extension(s) ' . implode(',', $dependentExtensions) . ' depend on it', 1342554622);
                } else {
                        $this->unloadExtension($extensionKey);
                }
@@ -190,7 +190,7 @@ class InstallUtility implements \TYPO3\CMS\Core\SingletonInterface {
                if (isset($availableExtensions[$extensionKey])) {
                        $extension = $availableExtensions[$extensionKey];
                } else {
-                       throw new \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException(('Extension ' . $extensionKey) . ' is not available', 1342864081);
+                       throw new \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException('Extension ' . $extensionKey . ' is not available', 1342864081);
                }
                $availableAndInstalledExtensions = $this->listUtility->enrichExtensionsWithEmConfAndTerInformation(array($extensionKey => $extension));
                return $availableAndInstalledExtensions[$extensionKey];
@@ -204,7 +204,7 @@ class InstallUtility implements \TYPO3\CMS\Core\SingletonInterface {
         * @return void
         */
        public function processDatabaseUpdates($extension) {
-               $extTablesSqlFile = (PATH_site . $extension['siteRelPath']) . '/ext_tables.sql';
+               $extTablesSqlFile = PATH_site . $extension['siteRelPath'] . '/ext_tables.sql';
                if (file_exists($extTablesSqlFile)) {
                        $extTablesSqlContent = \TYPO3\CMS\Core\Utility\GeneralUtility::getUrl($extTablesSqlFile);
                                // @TODO: This should probably moved to TYPO3\CMS\Core\Cache\Cache->getDatabaseTableDefinitions ?!
@@ -212,7 +212,7 @@ class InstallUtility implements \TYPO3\CMS\Core\SingletonInterface {
                        $extTablesSqlContent .= \TYPO3\CMS\Core\Cache\Cache::getDatabaseTableDefinitions();
                        $this->updateDbWithExtTablesSql($extTablesSqlContent);
                }
-               $extTablesStaticSqlFile = (PATH_site . $extension['siteRelPath']) . '/ext_tables_static+adt.sql';
+               $extTablesStaticSqlFile = PATH_site . $extension['siteRelPath'] . '/ext_tables_static+adt.sql';
                if (file_exists($extTablesStaticSqlFile)) {
                        $extTablesStaticSqlContent = \TYPO3\CMS\Core\Utility\GeneralUtility::getUrl($extTablesStaticSqlFile);
                        $this->importStaticSql($extTablesStaticSqlContent);
index 6444bf2..2b11b60 100644 (file)
@@ -110,7 +110,7 @@ class ListUtility implements \TYPO3\CMS\Core\SingletonInterface {
                                                                'siteRelPath' => str_replace(PATH_site, '', $path . $extKey),
                                                                'type' => $installationType,
                                                                'key' => $extKey,
-                                                               'ext_icon' => \TYPO3\CMS\Core\Extension\ExtensionManager::getExtensionIcon(($path . $extKey) . '/')
+                                                               'ext_icon' => \TYPO3\CMS\Core\Extension\ExtensionManager::getExtensionIcon($path . $extKey . '/')
                                                        );
                                                }
                                        }
index 2a1663e..4f19d54 100644 (file)
@@ -184,7 +184,7 @@ class ExtensionXmlPullParser extends \TYPO3\CMS\Extensionmanager\Utility\Parser\
                if (!$this->objXml->isEmptyElement) {
                        $value = '';
                        while ($this->objXml->read()) {
-                               if ((($this->objXml->nodeType == \XMLReader::TEXT || $this->objXml->nodeType == \XMLReader::CDATA) || $this->objXml->nodeType == \XMLReader::WHITESPACE) || $this->objXml->nodeType == \XMLReader::SIGNIFICANT_WHITESPACE) {
+                               if ($this->objXml->nodeType == \XMLReader::TEXT || $this->objXml->nodeType == \XMLReader::CDATA || $this->objXml->nodeType == \XMLReader::WHITESPACE || $this->objXml->nodeType == \XMLReader::SIGNIFICANT_WHITESPACE) {
                                        $value .= $this->objXml->value;
                                } else {
                                        if ($this->objXml->nodeType == \XMLReader::END_ELEMENT && $this->objXml->name === $elementName) {
index 7dfd321..e17ca01 100644 (file)
@@ -153,7 +153,7 @@ class MirrorXmlPullParser extends \TYPO3\CMS\Extensionmanager\Utility\Parser\Mir
                if (!$this->objXml->isEmptyElement) {
                        $value = '';
                        while ($this->objXml->read()) {
-                               if ((($this->objXml->nodeType == \XMLReader::TEXT || $this->objXml->nodeType == \XMLReader::CDATA) || $this->objXml->nodeType == \XMLReader::WHITESPACE) || $this->objXml->nodeType == \XMLReader::SIGNIFICANT_WHITESPACE) {
+                               if ($this->objXml->nodeType == \XMLReader::TEXT || $this->objXml->nodeType == \XMLReader::CDATA || $this->objXml->nodeType == \XMLReader::WHITESPACE || $this->objXml->nodeType == \XMLReader::SIGNIFICANT_WHITESPACE) {
                                        $value .= $this->objXml->value;
                                } else {
                                        if ($this->objXml->nodeType == \XMLReader::END_ELEMENT && $this->objXml->name === $elementName) {
index 913f388..1aedd2c 100644 (file)
@@ -62,7 +62,7 @@ abstract class XmlAbstractParser {
                $isAvailable = TRUE;
                if (!extension_loaded($this->requiredPhpExtensions)) {
                        $prefix = PHP_SHLIB_SUFFIX === 'dll' ? 'php_' : '';
-                       if (!((((bool) ini_get('enable_dl') && !(bool) ini_get('safe_mode')) && function_exists('dl')) && dl(($prefix . $this->requiredPhpExtensions) . PHP_SHLIB_SUFFIX))) {
+                       if (!(((bool) ini_get('enable_dl') && !(bool) ini_get('safe_mode')) && function_exists('dl') && dl($prefix . $this->requiredPhpExtensions . PHP_SHLIB_SUFFIX))) {
                                $isAvailable = FALSE;
                        }
                }
index 9f85490..2ae0c91 100644 (file)
@@ -75,7 +75,7 @@ class XmlParserFactory {
         * @return Tx_Extensionmanager_ExtensionXmlAbstractParser an instance of an extension.xml parser
         */
        static public function getParserInstance($parserType, $excludeClassNames = '') {
-               if ((!isset(self::$instance[$parserType]) || !is_object(self::$instance[$parserType])) || !empty($excludeClassNames)) {
+               if (!isset(self::$instance[$parserType]) || !is_object(self::$instance[$parserType]) || !empty($excludeClassNames)) {
                        // reset instance
                        self::$instance[$parserType] = ($objParser = NULL);
                        foreach (self::$parsers[$parserType] as $className => $file) {
index 829f798..8d6e7a1 100644 (file)
@@ -139,7 +139,7 @@ class Helper implements \TYPO3\CMS\Core\SingletonInterface {
         * @throws \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException
         */
        protected function fetchFile($remoteResource, $localResource) {
-               if (((is_string($remoteResource) && is_string($localResource)) && !empty($remoteResource)) && !empty($localResource)) {
+               if (is_string($remoteResource) && is_string($localResource) && !empty($remoteResource) && !empty($localResource)) {
                        $fileContent = \TYPO3\CMS\Core\Utility\GeneralUtility::getUrl($remoteResource, 0, array(TYPO3_user_agent));
                        if ($fileContent !== FALSE) {
                                if (\TYPO3\CMS\Core\Utility\GeneralUtility::writeFile($localResource, $fileContent) === FALSE) {
@@ -159,7 +159,7 @@ class Helper implements \TYPO3\CMS\Core\SingletonInterface {
         * @see getRemoteExtListFile()
         */
        public function getLocalExtListFile() {
-               $absFilePath = ((PATH_site . 'typo3temp/') . intval($this->repository->getUid())) . '.extensions.xml.gz';
+               $absFilePath = PATH_site . 'typo3temp/' . intval($this->repository->getUid()) . '.extensions.xml.gz';
                return $absFilePath;
        }
 
@@ -172,7 +172,7 @@ class Helper implements \TYPO3\CMS\Core\SingletonInterface {
         */
        public function getRemoteExtListFile() {
                $mirror = $this->getMirrors(TRUE)->getMirror();
-               $filePath = (('http://' . $mirror['host']) . $mirror['path']) . 'extensions.xml.gz';
+               $filePath = 'http://' . $mirror['host'] . $mirror['path'] . 'extensions.xml.gz';
                return $filePath;
        }
 
@@ -185,7 +185,7 @@ class Helper implements \TYPO3\CMS\Core\SingletonInterface {
         */
        public function getRemoteExtHashFile() {
                $mirror = $this->getMirrors(TRUE)->getMirror();
-               $filePath = (('http://' . $mirror['host']) . $mirror['path']) . 'extensions.md5';
+               $filePath = 'http://' . $mirror['host'] . $mirror['path'] . 'extensions.md5';
                return $filePath;
        }
 
@@ -197,7 +197,7 @@ class Helper implements \TYPO3\CMS\Core\SingletonInterface {
         * @see getRemoteMirrorListFile()
         */
        public function getLocalMirrorListFile() {
-               $absFilePath = ((PATH_site . 'typo3temp/') . intval($this->repository->getUid())) . '.mirrors.xml.gz';
+               $absFilePath = PATH_site . 'typo3temp/' . intval($this->repository->getUid()) . '.mirrors.xml.gz';
                return $absFilePath;
        }
 
@@ -226,7 +226,7 @@ class Helper implements \TYPO3\CMS\Core\SingletonInterface {
         */
        public function getMirrors($forcedUpdateFromRemote = TRUE) {
                $assignedMirror = $this->repository->getMirrors();
-               if (($forcedUpdateFromRemote || is_null($assignedMirror)) || !is_object($assignedMirror)) {
+               if ($forcedUpdateFromRemote || is_null($assignedMirror) || !is_object($assignedMirror)) {
                        if ($forcedUpdateFromRemote || !is_file($this->getLocalMirrorListFile())) {
                                $this->fetchMirrorListFile();
                        }
index 8eff8f9..52c4ed6 100644 (file)
@@ -73,13 +73,13 @@ class ContainerViewHelper extends \TYPO3\CMS\Fluid\ViewHelpers\Be\AbstractBacken
                $doc = $this->getDocInstance();
                $pageRenderer = $doc->getPageRenderer();
                if ($enableJumpToUrl) {
-                       $doc->JScode .= ('
+                       $doc->JScode .= '
                                <script language="javascript" type="text/javascript">
                                        script_ended = 0;
                                        function jumpToUrl(URL) {
                                                document.location = URL;
                                        }
-                                       ' . $doc->redirectUrls()) . '
+                                       ' . $doc->redirectUrls() . '
                                </script>
                        ';
                }
index 37b2188..4210025 100644 (file)
@@ -48,7 +48,7 @@ class ConfigureExtensionViewHelper extends \TYPO3\CMS\Fluid\ViewHelpers\Link\Act
         */
        public function render($extension) {
                $tag = '';
-               if ($extension['installed'] && file_exists((PATH_site . $extension['siteRelPath']) . '/ext_conf_template.txt')) {
+               if ($extension['installed'] && file_exists(PATH_site . $extension['siteRelPath'] . '/ext_conf_template.txt')) {
                        $uriBuilder = $this->controllerContext->getUriBuilder();
                        $action = 'showConfigurationForm';
                        $uri = $uriBuilder->reset()->uriFor($action, array(
index 538e01c..ecb6801 100644 (file)
@@ -50,9 +50,9 @@ class DownloadExtensionViewHelper extends \TYPO3\CMS\Fluid\ViewHelpers\FormViewH
                $installPaths = \TYPO3\CMS\Extensionmanager\Domain\Model\Extension::returnAllowedInstallPaths();
                $pathSelector = '<ul>';
                foreach ($installPaths as $installPathType => $installPath) {
-                       $pathSelector .= ((((((((((((((('<li>
-                               <input type="radio" id="' . $extension->getExtensionKey()) . '-downloadPath-') . $installPathType) . '" name="') . $this->getFieldNamePrefix('downloadPath')) . '[downloadPath]" class="downloadPath" value="') . $installPathType) . '"') . ($installPathType == 'Local' ? 'checked="checked"' : '')) . '/>
-                               <label for="') . $extension->getExtensionKey()) . '-downloadPath-') . $installPathType) . '">') . $installPathType) . '</label>
+                       $pathSelector .= '<li>
+                               <input type="radio" id="' . $extension->getExtensionKey() . '-downloadPath-' . $installPathType . '" name="' . $this->getFieldNamePrefix('downloadPath') . '[downloadPath]" class="downloadPath" value="' . $installPathType . '"' . ($installPathType == 'Local' ? 'checked="checked"' : '') . '/>
+                               <label for="' . $extension->getExtensionKey() . '-downloadPath-' . $installPathType . '">' . $installPathType . '</label>
                        </li>';
                }
                $pathSelector .= '</ul>';
@@ -67,7 +67,7 @@ class DownloadExtensionViewHelper extends \TYPO3\CMS\Fluid\ViewHelpers\FormViewH
                $label = '<input type="submit" value="Import and Install" />';
                $this->tag->setContent($label . $pathSelector);
                $this->tag->addAttribute('class', 'download');
-               return ((('<div id="' . $extension->getExtensionKey()) . '-downloadFromTer" class="downloadFromTer">') . $this->tag->render()) . '</div>';
+               return '<div id="' . $extension->getExtensionKey() . '-downloadFromTer" class="downloadFromTer">' . $this->tag->render() . '</div>';
        }
 
 }
index 2ff3d0b..2258e38 100644 (file)
@@ -92,9 +92,9 @@ class TypoScriptConstantsViewHelper extends \TYPO3\CMS\Fluid\Core\ViewHelper\Abs
                $pageRenderer = $doc->getPageRenderer();
                $pageRenderer->addCssFile('sysext/extensionmanager/Resources/Public/Contrib/Farbtastic/farbtastic.css');
                $pageRenderer->addJsFile('sysext/extensionmanager/Resources/Public/Contrib/Farbtastic/farbtastic.js');
-               $pageRenderer->addJsInlineCode('colorpicker', ('
+               $pageRenderer->addJsInlineCode('colorpicker', '
                        jQuery(document).ready(function() {
-                               jQuery(".colorPicker").farbtastic("#' . $configuration->getName()) . '");
+                               jQuery(".colorPicker").farbtastic("#' . $configuration->getName() . '");
                        });
                ');
                if ($configuration->getValue() !== NULL) {
@@ -152,11 +152,11 @@ class TypoScriptConstantsViewHelper extends \TYPO3\CMS\Fluid\Core\ViewHelper\Abs
                $optionValueArray = $configuration->getGeneric();
                $output = '';
                foreach ($optionValueArray as $label => $value) {
-                       $output .= ('<option value="' . htmlspecialchars($value)) . '"';
+                       $output .= '<option value="' . htmlspecialchars($value) . '"';
                        if ($configuration->getValue() == $value) {
                                $output .= ' selected="selected"';
                        }
-                       $output .= ('>' . htmlspecialchars($label)) . '</option>';
+                       $output .= '>' . htmlspecialchars($label) . '</option>';
                }
                $this->tag->setContent($output);
                return $this->tag->render();
@@ -266,7 +266,7 @@ class TypoScriptConstantsViewHelper extends \TYPO3\CMS\Fluid\Core\ViewHelper\Abs
         * @return string
         */
        protected function getName(\TYPO3\CMS\Extensionmanager\Domain\Model\ConfigurationItem $configuration) {
-               return ('tx_extensionmanager_tools_extensionmanagerextensionmanager[config][' . $configuration->getName()) . '][value]';
+               return 'tx_extensionmanager_tools_extensionmanagerextensionmanager[config][' . $configuration->getName() . '][value]';
        }
 
        /**
@@ -287,7 +287,7 @@ class TypoScriptConstantsViewHelper extends \TYPO3\CMS\Fluid\Core\ViewHelper\Abs
                if (!in_array($fieldName, $hiddenFieldNames)) {
                        $hiddenFieldNames[] = $fieldName;
                        $this->viewHelperVariableContainer->addOrUpdate('TYPO3\\CMS\\Fluid\\ViewHelpers\\FormViewHelper', 'renderedHiddenFields', $hiddenFieldNames);
-                       return ('<input type="hidden" name="' . htmlspecialchars($fieldName)) . '" value="0" />';
+                       return '<input type="hidden" name="' . htmlspecialchars($fieldName) . '" value="0" />';
                }
                return '';
        }
index c3eded9..b483df3 100644 (file)
@@ -55,7 +55,7 @@ class UpdateFromTerViewHelper extends \TYPO3\CMS\Fluid\ViewHelpers\Link\ActionVi
                $label = 'Update Extension List';
                $this->tag->setContent($label);
                $this->tag->addAttribute('class', 'text');
-               return ('<div class="updateFromTer">' . $this->tag->render()) . '<span class="spinner"></span></div>';
+               return '<div class="updateFromTer">' . $this->tag->render() . '<span class="spinner"></span></div>';
        }
 
 }
index b4bb7b6..82c2340 100644 (file)
@@ -25,6 +25,8 @@ namespace TYPO3\CMS\Extensionmanager\Tests\Unit\Domain\Repository;
  ***************************************************************/
 
 /**
+ * Tests for ConfigurationItemRepository
+ *
  * @package Extension Manager
  * @subpackage Tests
  */
@@ -49,7 +51,7 @@ class ConfigurationItemRepositoryTest extends \TYPO3\CMS\Extbase\Tests\Unit\Base
        public function getConfigurationItemRepositoryMock() {
                $className = 'Tx_Extensionmanager_Repository_ConfigurationItemRepositoryMock';
                if (!class_exists($className, FALSE)) {
-                       eval((((((((((((((('class ' . $className) . ' extends TYPO3\\CMS\\Extensionmanager\\Domain\\Repository\\ConfigurationItemRepository {') . '  public function addMetaInformation(&$configuration) {') . '    return parent::addMetaInformation($configuration);') . '  }') . '  public function extractInformationForConfigFieldsOfTypeUser($configurationOption) {') . '    return parent::extractInformationForConfigFieldsOfTypeUser($configurationOption);') . '  }') . '  public function extractInformationForConfigFieldsOfTypeOptions($configurationOption) {') . '    return parent::extractInformationForConfigFieldsOfTypeOptions($configurationOption);') . '  }') . '  public function mergeWithExistingConfiguration(array $configuration, array $extension) {') . '    return parent::mergeWithExistingConfiguration($configuration, $extension);') . '  }') . '}');
+                       eval('class ' . $className . ' extends TYPO3\\CMS\\Extensionmanager\\Domain\\Repository\\ConfigurationItemRepository {' . '  public function addMetaInformation(&$configuration) {' . '    return parent::addMetaInformation($configuration);' . '  }' . '  public function extractInformationForConfigFieldsOfTypeUser($configurationOption) {' . '    return parent::extractInformationForConfigFieldsOfTypeUser($configurationOption);' . '  }' . '  public function extractInformationForConfigFieldsOfTypeOptions($configurationOption) {' . '    return parent::extractInformationForConfigFieldsOfTypeOptions($configurationOption);' . '  }' . '  public function mergeWithExistingConfiguration(array $configuration, array $extension) {' . '    return parent::mergeWithExistingConfiguration($configuration, $extension);' . '  }' . '}');
                }
                return $className;
        }
index c09d770..00af3cc 100644 (file)
@@ -25,6 +25,8 @@ namespace TYPO3\CMS\Extensionmanager\Tests\Unit\Utility;
  ***************************************************************/
 
 /**
+ * Test for DependencyUtility
+ *
  * @package Extension Manager
  * @subpackage Tests
  */
@@ -495,7 +497,7 @@ class DependencyUtilityTest extends \TYPO3\CMS\Extbase\Tests\Unit\BaseTestCase {
                $extension2->setExtensionKey('bar');
                $extension2->setVersion('1.0.42');
                $className = uniqid('objectStorage');
-               eval((((((('class ' . $className) . ' {') . 'public $extensions = array();') . 'public function getFirst() {') . '  return $this->extensions[0];') . '}') . '}');
+               eval('class ' . $className . ' {' . 'public $extensions = array();' . 'public function getFirst() {' . '  return $this->extensions[0];' . '}' . '}');
                $myStorage = new $className();
                $myStorage->extensions[] = $extension1;
                $myStorage->extensions[] = $extension2;
index 7d2cf3c..8f73402 100644 (file)
@@ -42,7 +42,7 @@ class FileHandlingUtilityTest extends \TYPO3\CMS\Extbase\Tests\Unit\BaseTestCase
         */
        public function tearDown() {
                foreach ($this->fakedExtensions as $extension => $dummy) {
-                       \TYPO3\CMS\Core\Utility\GeneralUtility::rmdir((PATH_site . 'typo3conf/ext/') . $extension, TRUE);
+                       \TYPO3\CMS\Core\Utility\GeneralUtility::rmdir(PATH_site . 'typo3conf/ext/' . $extension, TRUE);
                }
        }
 
@@ -55,8 +55,8 @@ class FileHandlingUtilityTest extends \TYPO3\CMS\Extbase\Tests\Unit\BaseTestCase
         */
        protected function createFakeExtension($extkeyOnly = FALSE) {
                $extKey = strtolower(uniqid('testing'));
-               $absExtPath = ((PATH_site . 'typo3conf/ext/') . $extKey) . '/';
-               $relPath = ('typo3conf/ext/' . $extKey) . '/';
+               $absExtPath = PATH_site . 'typo3conf/ext/' . $extKey . '/';
+               $relPath = 'typo3conf/ext/' . $extKey . '/';
                $this->fakedExtensions[$extKey] = array(
                        'siteRelPath' => $relPath,
                        'siteAbsPath' => $absExtPath
@@ -75,7 +75,7 @@ class FileHandlingUtilityTest extends \TYPO3\CMS\Extbase\Tests\Unit\BaseTestCase
        public function makeAndClearExtensionDirRemovesExtensionDirIfAlreadyExists() {
                $extKey = $this->createFakeExtension();
                $fileHandlerMock = $this->getAccessibleMock('TYPO3\\CMS\\Extensionmanager\\Utility\\FileHandlingUtility', array('removeDirectory', 'addDirectory'));
-               $fileHandlerMock->expects($this->once())->method('removeDirectory')->with(((PATH_site . 'typo3conf/ext/') . $extKey) . '/');
+               $fileHandlerMock->expects($this->once())->method('removeDirectory')->with(PATH_site . 'typo3conf/ext/' . $extKey . '/');
                $fileHandlerMock->_call('makeAndClearExtensionDir', $extKey);
        }
 
@@ -86,7 +86,7 @@ class FileHandlingUtilityTest extends \TYPO3\CMS\Extbase\Tests\Unit\BaseTestCase
        public function makeAndClearExtensionDirAddsDir() {
                $extKey = $this->createFakeExtension();
                $fileHandlerMock = $this->getAccessibleMock('TYPO3\\CMS\\Extensionmanager\\Utility\\FileHandlingUtility', array('removeDirectory', 'addDirectory'));
-               $fileHandlerMock->expects($this->once())->method('addDirectory')->with(((PATH_site . 'typo3conf/ext/') . $extKey) . '/');
+               $fileHandlerMock->expects($this->once())->method('addDirectory')->with(PATH_site . 'typo3conf/ext/' . $extKey . '/');
                $fileHandlerMock->_call('makeAndClearExtensionDir', $extKey);
        }
 
index 6f9030f..b9298be 100644 (file)
@@ -70,7 +70,7 @@ class InstallUtilityTest extends \TYPO3\CMS\Extbase\Tests\Unit\BaseTestCase {
         */
        public function tearDown() {
                foreach ($this->fakedExtensions as $extension => $dummy) {
-                       \TYPO3\CMS\Core\Utility\GeneralUtility::rmdir((PATH_site . 'typo3temp/') . $extension, TRUE);
+                       \TYPO3\CMS\Core\Utility\GeneralUtility::rmdir(PATH_site . 'typo3temp/' . $extension, TRUE);
                }
        }
 
@@ -82,8 +82,8 @@ class InstallUtilityTest extends \TYPO3\CMS\Extbase\Tests\Unit\BaseTestCase {
         */
        protected function createFakeExtension() {
                $extKey = strtolower(uniqid('testing'));
-               $absExtPath = ((PATH_site . 'typo3temp/') . $extKey) . '/';
-               $relPath = ('typo3temp/' . $extKey) . '/';
+               $absExtPath = PATH_site . 'typo3temp/' . $extKey . '/';
+               $relPath = 'typo3temp/' . $extKey . '/';
                \TYPO3\CMS\Core\Utility\GeneralUtility::mkdir($absExtPath);
                $this->fakedExtensions[$extKey] = array(
                        'siteRelPath' => $relPath
@@ -159,7 +159,7 @@ class InstallUtilityTest extends \TYPO3\CMS\Extbase\Tests\Unit\BaseTestCase {
         */
        public function processDatabaseUpdatesCallsUpdateDbWithExtTablesSql() {
                $extKey = $this->createFakeExtension();
-               $extPath = ((PATH_site . 'typo3temp/') . $extKey) . '/';
+               $extPath = PATH_site . 'typo3temp/' . $extKey . '/';
                $extTablesFile = $extPath . 'ext_tables.sql';
                $fileContent = 'DUMMY TEXT TO COMPARE';
                file_put_contents($extTablesFile, $fileContent);
@@ -174,7 +174,7 @@ class InstallUtilityTest extends \TYPO3\CMS\Extbase\Tests\Unit\BaseTestCase {
         */
        public function processDatabaseUpdatesCallsUpdateDbWithExtTablesSqlIncludingCachingFrameworkTables() {
                $extKey = $this->createFakeExtension();
-               $extPath = ((PATH_site . 'typo3temp/') . $extKey) . '/';
+               $extPath = PATH_site . 'typo3temp/' . $extKey . '/';
                $extTablesFile = $extPath . 'ext_tables.sql';
                $fileContent = 'DUMMY TEXT TO COMPARE';
                file_put_contents($extTablesFile, $fileContent);
@@ -189,7 +189,7 @@ class InstallUtilityTest extends \TYPO3\CMS\Extbase\Tests\Unit\BaseTestCase {
         */
        public function processDatabaseUpdatesCallsImportStaticSql() {
                $extKey = $this->createFakeExtension();
-               $extPath = ((PATH_site . 'typo3temp/') . $extKey) . '/';
+               $extPath = PATH_site . 'typo3temp/' . $extKey . '/';
                $extTablesFile = $extPath . 'ext_tables_static+adt.sql';
                $fileContent = 'DUMMY TEXT TO COMPARE';
                file_put_contents($extTablesFile, $fileContent);
index b4a7923..18c0410 100644 (file)
@@ -37,8 +37,8 @@ if (TYPO3_MODE === 'BE') {
                ),
                array(
                        'access' => 'user,group',
-                       'icon' => ('EXT:' . $_EXTKEY) . '/ext_icon.gif',
-                       'labels' => ('LLL:EXT:' . $_EXTKEY) . '/Resources/Private/Language/locallang_mod.xml'
+                       'icon' => 'EXT:' . $_EXTKEY . '/ext_icon.gif',
+                       'labels' => 'LLL:EXT:' . $_EXTKEY . '/Resources/Private/Language/locallang_mod.xml'
                )
        );
 }
index 2d09ed7..97dda42 100644 (file)
@@ -64,14 +64,14 @@ class ExtraPageContextMenuOptions {
                        if ($backRef->editOK) {
                                $LL = $this->includeLL();
                                $localItems[] = 'spacer';
-                               $localItems['moreoptions'] = $backRef->linkItem($GLOBALS['LANG']->makeEntities($GLOBALS['LANG']->getLLL('label', $LL)), $backRef->excludeIcon(''), ('top.loadTopMenu(\'' . \TYPO3\CMS\Core\Utility\GeneralUtility::linkThisScript()) . '&cmLevel=1&subname=moreoptions\');return false;', 0, 1);
-                               if ((!in_array('hide', $backRef->disabledItems) && is_array($GLOBALS['TCA'][$table]['ctrl']['enablecolumns'])) && $GLOBALS['TCA'][$table]['ctrl']['enablecolumns']['disabled']) {
+                               $localItems['moreoptions'] = $backRef->linkItem($GLOBALS['LANG']->makeEntities($GLOBALS['LANG']->getLLL('label', $LL)), $backRef->excludeIcon(''), 'top.loadTopMenu(\'' . \TYPO3\CMS\Core\Utility\GeneralUtility::linkThisScript() . '&cmLevel=1&subname=moreoptions\');return false;', 0, 1);
+                               if (!in_array('hide', $backRef->disabledItems) && is_array($GLOBALS['TCA'][$table]['ctrl']['enablecolumns']) && $GLOBALS['TCA'][$table]['ctrl']['enablecolumns']['disabled']) {
                                        $localItems['hide'] = $backRef->DB_hideUnhide($table, $backRef->rec, $GLOBALS['TCA'][$table]['ctrl']['enablecolumns']['disabled']);
                                }
                                if (!in_array('edit_access', $backRef->disabledItems) && is_array($GLOBALS['TCA'][$table]['ctrl']['enablecolumns'])) {
                                        $localItems['edit_access'] = $backRef->DB_editAccess($table, $uid);
                                }
-                               if ((!in_array('edit_pageproperties', $backRef->disabledItems) && $table == 'pages') && $backRef->editPageIconSet) {
+                               if (!in_array('edit_pageproperties', $backRef->disabledItems) && $table == 'pages' && $backRef->editPageIconSet) {
                                        $localItems['edit_pageproperties'] = $backRef->DB_editPageProperties($uid);
                                }
                        }
@@ -102,7 +102,7 @@ class ExtraPageContextMenuOptions {
                                if (!in_array('new_wizard', $backRef->disabledItems) && ($table == 'pages' || $table == 'tt_content')) {
                                        $localItems['new_wizard'] = $backRef->DB_newWizard($table, $uid, $backRef->rec);
                                }
-                               if ((!in_array('perms', $backRef->disabledItems) && $table == 'pages') && $GLOBALS['BE_USER']->check('modules', 'web_perm')) {
+                               if (!in_array('perms', $backRef->disabledItems) && $table == 'pages' && $GLOBALS['BE_USER']->check('modules', 'web_perm')) {
                                        $localItems['perms'] = $backRef->DB_perms($table, $uid, $backRef->rec);
                                }
                                if (!in_array('db_list', $backRef->disabledItems) && $GLOBALS['BE_USER']->check('modules', 'web_list')) {
index 6470414..a6ea008 100644 (file)
@@ -73,14 +73,14 @@ class FrontendEditPanel {
                // Special content is about to be shown, so the cache must be disabled.
                $GLOBALS['TSFE']->set_no_cache();
                $formName = 'TSFE_EDIT_FORM_' . substr($GLOBALS['TSFE']->uniqueHash(), 0, 4);
-               $formTag = ((((((('<form name="' . $formName) . '" id ="') . $formName) . '" action="') . htmlspecialchars(\TYPO3\CMS\Core\Utility\GeneralUtility::getIndpEnv('REQUEST_URI'))) . '" method="post" enctype="') . $GLOBALS['TYPO3_CONF_VARS']['SYS']['form_enctype']) . '" onsubmit="return TBE_EDITOR.checkSubmit(1);" style="margin: 0 0 0 0;">';
+               $formTag = '<form name="' . $formName . '" id ="' . $formName . '" action="' . htmlspecialchars(\TYPO3\CMS\Core\Utility\GeneralUtility::getIndpEnv('REQUEST_URI')) . '" method="post" enctype="' . $GLOBALS['TYPO3_CONF_VARS']['SYS']['form_enctype'] . '" onsubmit="return TBE_EDITOR.checkSubmit(1);" style="margin: 0 0 0 0;">';
                $sortField = $GLOBALS['TCA'][$table]['ctrl']['sortby'];
                $labelField = $GLOBALS['TCA'][$table]['ctrl']['label'];
                $hideField = $GLOBALS['TCA'][$table]['ctrl']['enablecolumns']['disabled'];
-               $blackLine = $conf['line'] ? ('<img src="clear.gif" width="1" height="' . intval($conf['line'])) . '" alt="" title="" /><br /><table border="0" cellpadding="0" cellspacing="0" width="100%" bgcolor="black" style="border: 0px;" summary=""><tr style="border: 0px;"><td style="border: 0px;"><img src="clear.gif" width="1" height="1" alt="" title="" /></td></tr></table><br />' : '';
+               $blackLine = $conf['line'] ? '<img src="clear.gif" width="1" height="' . intval($conf['line']) . '" alt="" title="" /><br /><table border="0" cellpadding="0" cellspacing="0" width="100%" bgcolor="black" style="border: 0px;" summary=""><tr style="border: 0px;"><td style="border: 0px;"><img src="clear.gif" width="1" height="1" alt="" title="" /></td></tr></table><br />' : '';
                $theCmd = '';
                $TSFE_EDIT = $GLOBALS['BE_USER']->frontendEdit->TSFE_EDIT;
-               if ((is_array($TSFE_EDIT) && $TSFE_EDIT['record'] == $currentRecord) && !$TSFE_EDIT['update_close']) {
+               if (is_array($TSFE_EDIT) && $TSFE_EDIT['record'] == $currentRecord && !$TSFE_EDIT['update_close']) {
                        $theCmd = $TSFE_EDIT['cmd'];
                }
                switch ($theCmd) {
@@ -95,46 +95,46 @@ class FrontendEditPanel {
                                $panel .= $GLOBALS['BE_USER']->adminPanel->ext_makeToolBar() . '<img src="clear.gif" width="2" height="1" alt="" title="" />';
                        }
                        if (isset($allow['edit'])) {
-                               $panel .= $this->editPanelLinkWrap(((('<img ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(TYPO3_mainDir, 'gfx/edit2.gif', 'width="11" height="12" hspace="2" border="0"')) . ' title="') . $GLOBALS['BE_USER']->extGetLL('p_editRecord')) . '" align="top" alt="" />', $formName, 'edit', $dataArr['_LOCALIZED_UID'] ? ($table . ':') . $dataArr['_LOCALIZED_UID'] : $currentRecord);
+                               $panel .= $this->editPanelLinkWrap('<img ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(TYPO3_mainDir, 'gfx/edit2.gif', 'width="11" height="12" hspace="2" border="0"') . ' title="' . $GLOBALS['BE_USER']->extGetLL('p_editRecord') . '" align="top" alt="" />', $formName, 'edit', $dataArr['_LOCALIZED_UID'] ? $table . ':' . $dataArr['_LOCALIZED_UID'] : $currentRecord);
                        }
                        // Hiding in workspaces because implementation is incomplete
-                       if ((isset($allow['move']) && $sortField) && $GLOBALS['BE_USER']->workspace === 0) {
-                               $panel .= $this->editPanelLinkWrap(((('<img  ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(TYPO3_mainDir, 'gfx/button_up.gif', 'width="11" height="10" vspace="1" hspace="2" border="0" ')) . ' title="') . $GLOBALS['BE_USER']->extGetLL('p_moveUp')) . '" align="top" alt="" />', $formName, 'up');
-                               $panel .= $this->editPanelLinkWrap(((('<img  ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(TYPO3_mainDir, 'gfx/button_down.gif', 'width="11" height="10" vspace="1" hspace="2" border="0" ')) . ' title="') . $GLOBALS['BE_USER']->extGetLL('p_moveDown')) . '" align="top" alt="" />', $formName, 'down');
+                       if (isset($allow['move']) && $sortField && $GLOBALS['BE_USER']->workspace === 0) {
+                               $panel .= $this->editPanelLinkWrap('<img  ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(TYPO3_mainDir, 'gfx/button_up.gif', 'width="11" height="10" vspace="1" hspace="2" border="0" ') . ' title="' . $GLOBALS['BE_USER']->extGetLL('p_moveUp') . '" align="top" alt="" />', $formName, 'up');
+                               $panel .= $this->editPanelLinkWrap('<img  ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(TYPO3_mainDir, 'gfx/button_down.gif', 'width="11" height="10" vspace="1" hspace="2" border="0" ') . ' title="' . $GLOBALS['BE_USER']->extGetLL('p_moveDown') . '" align="top" alt="" />', $formName, 'down');
                        }
                        // Hiding in workspaces because implementation is incomplete, Hiding for localizations because it is unknown what should be the function in that case
-                       if (((isset($allow['hide']) && $hideField) && $GLOBALS['BE_USER']->workspace === 0) && !$dataArr['_LOCALIZED_UID']) {
+                       if (isset($allow['hide']) && $hideField && $GLOBALS['BE_USER']->workspace === 0 && !$dataArr['_LOCALIZED_UID']) {
                                if ($dataArr[$hideField]) {
-                                       $panel .= $this->editPanelLinkWrap(((('<img  ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(TYPO3_mainDir, 'gfx/button_unhide.gif', 'width="11" height="10" vspace="1" hspace="2" border="0" ')) . ' title="') . $GLOBALS['BE_USER']->extGetLL('p_unhide')) . '" align="top" alt="" />', $formName, 'unhide');
+                                       $panel .= $this->editPanelLinkWrap('<img  ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(TYPO3_mainDir, 'gfx/button_unhide.gif', 'width="11" height="10" vspace="1" hspace="2" border="0" ') . ' title="' . $GLOBALS['BE_USER']->extGetLL('p_unhide') . '" align="top" alt="" />', $formName, 'unhide');
                                } else {
-                                       $panel .= $this->editPanelLinkWrap(((('<img  ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(TYPO3_mainDir, 'gfx/button_hide.gif', 'width="11" height="10" vspace="1" hspace="2" border="0" ')) . ' title="') . $GLOBALS['BE_USER']->extGetLL('p_hide')) . '" align="top" alt="" />', $formName, 'hide', '', $GLOBALS['BE_USER']->extGetLL('p_hideConfirm'));
+                                       $panel .= $this->editPanelLinkWrap('<img  ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(TYPO3_mainDir, 'gfx/button_hide.gif', 'width="11" height="10" vspace="1" hspace="2" border="0" ') . ' title="' . $GLOBALS['BE_USER']->extGetLL('p_hide') . '" align="top" alt="" />', $formName, 'hide', '', $GLOBALS['BE_USER']->extGetLL('p_hideConfirm'));
                                }
                        }
                        if (isset($allow['new'])) {
                                if ($table == 'pages') {
-                                       $panel .= $this->editPanelLinkWrap(((('<img  ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(TYPO3_mainDir, 'gfx/new_page.gif', 'width="13" height="12" vspace="1" hspace="2" border="0" ')) . ' title="') . $GLOBALS['BE_USER']->extGetLL('p_newSubpage')) . '" align="top" alt="" />', $formName, 'new', $currentRecord, '');
+                                       $panel .= $this->editPanelLinkWrap('<img  ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(TYPO3_mainDir, 'gfx/new_page.gif', 'width="13" height="12" vspace="1" hspace="2" border="0" ') . ' title="' . $GLOBALS['BE_USER']->extGetLL('p_newSubpage') . '" align="top" alt="" />', $formName, 'new', $currentRecord, '');
                                } else {
-                                       $panel .= $this->editPanelLinkWrap(((('<img  ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(TYPO3_mainDir, 'gfx/new_record.gif', 'width="16" height="12" vspace="1" hspace="2" border="0" ')) . ' title="') . $GLOBALS['BE_USER']->extGetLL('p_newRecordAfter')) . '" align="top" alt="" />', $formName, 'new', $currentRecord, '', $newUID);
+                                       $panel .= $this->editPanelLinkWrap('<img  ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(TYPO3_mainDir, 'gfx/new_record.gif', 'width="16" height="12" vspace="1" hspace="2" border="0" ') . ' title="' . $GLOBALS['BE_USER']->extGetLL('p_newRecordAfter') . '" align="top" alt="" />', $formName, 'new', $currentRecord, '', $newUID);
                                }
                        }
                        // Hiding in workspaces because implementation is incomplete, Hiding for localizations because it is unknown what should be the function in that case
-                       if ((isset($allow['delete']) && $GLOBALS['BE_USER']->workspace === 0) && !$dataArr['_LOCALIZED_UID']) {
-                               $panel .= $this->editPanelLinkWrap(((('<img  ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(TYPO3_mainDir, 'gfx/delete_record.gif', 'width="12" height="12" vspace="1" hspace="2" border="0" ')) . ' title="') . $GLOBALS['BE_USER']->extGetLL('p_delete')) . '" align="top" alt="" />', $formName, 'delete', '', $GLOBALS['BE_USER']->extGetLL('p_deleteConfirm'));
+                       if (isset($allow['delete']) && $GLOBALS['BE_USER']->workspace === 0 && !$dataArr['_LOCALIZED_UID']) {
+                               $panel .= $this->editPanelLinkWrap('<img  ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(TYPO3_mainDir, 'gfx/delete_record.gif', 'width="12" height="12" vspace="1" hspace="2" border="0" ') . ' title="' . $GLOBALS['BE_USER']->extGetLL('p_delete') . '" align="top" alt="" />', $formName, 'delete', '', $GLOBALS['BE_USER']->extGetLL('p_deleteConfirm'));
                        }
                        //      Final
                        $labelTxt = $this->cObj->stdWrap($conf['label'], $conf['label.']);
                        foreach ((array) $hiddenFields as $name => $value) {
-                               $hiddenFieldString .= (((('<input type="hidden" name="TSFE_EDIT[' . $name) . ']" value="') . $value) . '"/>') . LF;
+                               $hiddenFieldString .= '<input type="hidden" name="TSFE_EDIT[' . $name . ']" value="' . $value . '"/>' . LF;
                        }
-                       $panel = (((((((('
+                       $panel = '
 
                                                        <!-- BE_USER Edit Panel: -->
-                                                       ' . $formTag) . $hiddenFieldString) . '
+                                                       ' . $formTag . $hiddenFieldString . '
                                                                <input type="hidden" name="TSFE_EDIT[cmd]" value="" />
-                                                               <input type="hidden" name="TSFE_EDIT[record]" value="') . $currentRecord) . '" />
+                                                               <input type="hidden" name="TSFE_EDIT[record]" value="' . $currentRecord . '" />
                                                                <table border="0" cellpadding="0" cellspacing="0" class="typo3-editPanel" summary="">
                                                                        <tr>
-                                                                               <td nowrap="nowrap" bgcolor="#ABBBB4" class="typo3-editPanel-controls">') . $panel) . '</td>') . ($labelTxt ? ('<td nowrap="nowrap" bgcolor="#F6F2E6" class="typo3-editPanel-label"><font face="verdana" size="1" color="black">&nbsp;' . sprintf($labelTxt, htmlspecialchars(\TYPO3\CMS\Core\Utility\GeneralUtility::fixed_lgd_cs($dataArr[$labelField], 50)))) . '&nbsp;</font></td>' : '')) . '
+                                                                               <td nowrap="nowrap" bgcolor="#ABBBB4" class="typo3-editPanel-controls">' . $panel . '</td>' . ($labelTxt ? '<td nowrap="nowrap" bgcolor="#F6F2E6" class="typo3-editPanel-label"><font face="verdana" size="1" color="black">&nbsp;' . sprintf($labelTxt, htmlspecialchars(\TYPO3\CMS\Core\Utility\GeneralUtility::fixed_lgd_cs($dataArr[$labelField], 50))) . '&nbsp;</font></td>' : '') . '
                                                                        </tr>
                                                                </table>
                                                        </form>';
@@ -188,22 +188,22 @@ class FrontendEditPanel {
        public function editIcons($content, $params, array $conf = array(), $currentRecord = '', array $dataArr = array(), $addUrlParamStr = '', $table, $editUid, $fieldList) {
                        // Special content is about to be shown, so the cache must be disabled.
                $GLOBALS['TSFE']->set_no_cache();
-               $style = $conf['styleAttribute'] ? (' style="' . htmlspecialchars($conf['styleAttribute'])) . '"' : '';
+               $style = $conf['styleAttribute'] ? ' style="' . htmlspecialchars($conf['styleAttribute']) . '"' : '';
                $iconTitle = $this->cObj->stdWrap($conf['iconTitle'], $conf['iconTitle.']);
-               $iconImg = $conf['iconImg'] ? $conf['iconImg'] : ((((('<img  ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(TYPO3_mainDir, 'gfx/edit_fe.gif', 'width="11" height="12" border="0" align="top" ')) . ' title="') . \TYPO3\CMS\Core\Utility\GeneralUtility::deHSCentities(htmlspecialchars($iconTitle))) . '"') . $style) . ' class="frontEndEditIcons" alt="" />';
+               $iconImg = $conf['iconImg'] ? $conf['iconImg'] : '<img  ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(TYPO3_mainDir, 'gfx/edit_fe.gif', 'width="11" height="12" border="0" align="top" ') . ' title="' . \TYPO3\CMS\Core\Utility\GeneralUtility::deHSCentities(htmlspecialchars($iconTitle)) . '"' . $style . ' class="frontEndEditIcons" alt="" />';
                $nV = \TYPO3\CMS\Core\Utility\GeneralUtility::_GP('ADMCMD_view') ? 1 : 0;
                $adminURL = \TYPO3\CMS\Core\Utility\GeneralUtility::getIndpEnv('TYPO3_SITE_URL') . TYPO3_mainDir;
-               $icon = $this->editPanelLinkWrap_doWrap($iconImg, (((((((($adminURL . 'alt_doc.php?edit[') . $table) . '][') . $editUid) . ']=edit&columnsOnly=') . rawurlencode($fieldList)) . '&noView=') . $nV) . $addUrlParamStr, $currentRecord);
+               $icon = $this->editPanelLinkWrap_doWrap($iconImg, $adminURL . 'alt_doc.php?edit[' . $table . '][' . $editUid . ']=edit&columnsOnly=' . rawurlencode($fieldList) . '&noView=' . $nV . $addUrlParamStr, $currentRecord);
                if ($conf['beforeLastTag'] < 0) {
                        $content = $icon . $content;
                } elseif ($conf['beforeLastTag'] > 0) {
                        $cBuf = rtrim($content);
                        $securCount = 30;
-                       while (($securCount && substr($cBuf, -1) == '>') && substr($cBuf, -4) != '</a>') {
+                       while ($securCount && substr($cBuf, -1) == '>' && substr($cBuf, -4) != '</a>') {
                                $cBuf = rtrim(preg_replace('/<[^<]*>$/', '', $cBuf));
                                $securCount--;
                        }
-                       $content = strlen($cBuf) && $securCount ? (substr($content, 0, strlen($cBuf)) . $icon) . substr($content, strlen($cBuf)) : ($content = $icon . $content);
+                       $content = strlen($cBuf) && $securCount ? substr($content, 0, strlen($cBuf)) . $icon . substr($content, strlen($cBuf)) : ($content = $icon . $content);
                } else {
                        $content .= $icon;
                }
@@ -230,26 +230,26 @@ class FrontendEditPanel {
                $adminURL = \TYPO3\CMS\Core\Utility\GeneralUtility::getIndpEnv('TYPO3_SITE_URL') . TYPO3_mainDir;
                if ($cmd == 'edit' && !$editFormsOnPage) {
                        $rParts = explode(':', $currentRecord);
-                       $out = $this->editPanelLinkWrap_doWrap($string, ((((($adminURL . 'alt_doc.php?edit[') . $rParts[0]) . '][') . $rParts[1]) . ']=edit&noView=') . $nV, $currentRecord);
+                       $out = $this->editPanelLinkWrap_doWrap($string, $adminURL . 'alt_doc.php?edit[' . $rParts[0] . '][' . $rParts[1] . ']=edit&noView=' . $nV, $currentRecord);
                } elseif ($cmd == 'new' && !$editFormsOnPage) {
                        $rParts = explode(':', $currentRecord);
                        if ($rParts[0] == 'pages') {
-                               $out = $this->editPanelLinkWrap_doWrap($string, (($adminURL . 'db_new.php?id=') . $rParts[1]) . '&pagesOnly=1', $currentRecord);
+                               $out = $this->editPanelLinkWrap_doWrap($string, $adminURL . 'db_new.php?id=' . $rParts[1] . '&pagesOnly=1', $currentRecord);
                        } else {
                                if (!intval($nPid)) {
                                        $nPid = \TYPO3\CMS\Core\Utility\MathUtility::canBeInterpretedAsInteger($rParts[1]) ? -$rParts[1] : $GLOBALS['TSFE']->id;
                                }
-                               $out = $this->editPanelLinkWrap_doWrap($string, ((((($adminURL . 'alt_doc.php?edit[') . $rParts[0]) . '][') . $nPid) . ']=new&noView=') . $nV, $currentRecord);
+                               $out = $this->editPanelLinkWrap_doWrap($string, $adminURL . 'alt_doc.php?edit[' . $rParts[0] . '][' . $nPid . ']=new&noView=' . $nV, $currentRecord);
                        }
                } else {
                        if ($confirm && $GLOBALS['BE_USER']->jsConfirmation(8)) {
                                // Gets htmlspecialchared later
-                               $cf1 = ('if (confirm(' . \TYPO3\CMS\Core\Utility\GeneralUtility::quoteJSvalue($confirm, TRUE)) . ')) {';
+                               $cf1 = 'if (confirm(' . \TYPO3\CMS\Core\Utility\GeneralUtility::quoteJSvalue($confirm, TRUE) . ')) {';
                                $cf2 = '}';
                        } else {
                                $cf1 = ($cf2 = '');
                        }
-                       $out = ((('<a href="#" onclick="' . htmlspecialchars(((((((((($cf1 . 'document.') . $formName) . '[\'TSFE_EDIT[cmd]\'].value=\'') . $cmd) . '\'; document.') . $formName) . '.submit();') . $cf2) . ' return false;'))) . '">') . $string) . '</a>';
+                       $out = '<a href="#" onclick="' . htmlspecialchars(($cf1 . 'document.' . $formName . '[\'TSFE_EDIT[cmd]\'].value=\'' . $cmd . '\'; document.' . $formName . '.submit();' . $cf2 . ' return false;')) . '">' . $string . '</a>';
                }
                return $out;
        }
@@ -271,9 +271,9 @@ class FrontendEditPanel {
                        if ($rParts[0] == 'tt_content' && $this->parentRecordNumber > 2) {
                                $retUrl .= '#' . $rParts[1];
                        }
-                       return ((('<a href="' . htmlspecialchars((($url . '&returnUrl=') . rawurlencode($retUrl)))) . '" class="frontEndEditIconLinks">') . $string) . '</a>';
+                       return '<a href="' . htmlspecialchars(($url . '&returnUrl=' . rawurlencode($retUrl))) . '" class="frontEndEditIconLinks">' . $string . '</a>';
                } else {
-                       return ((('<a href="#" onclick="' . htmlspecialchars((((('vHWin=window.open(\'' . $url) . '&returnUrl=close.html\',\'FEquickEditWindow\',\'') . ($GLOBALS['BE_USER']->uc['edit_wideDocument'] ? 'width=690,height=500' : 'width=540,height=400')) . ',status=0,menubar=0,scrollbars=1,resizable=1\');vHWin.focus();return false;'))) . '" class="frontEndEditIconLinks">') . $string) . '</a>';
+                       return '<a href="#" onclick="' . htmlspecialchars(('vHWin=window.open(\'' . $url . '&returnUrl=close.html\',\'FEquickEditWindow\',\'' . ($GLOBALS['BE_USER']->uc['edit_wideDocument'] ? 'width=690,height=500' : 'width=540,height=400') . ',status=0,menubar=0,scrollbars=1,resizable=1\');vHWin.focus();return false;')) . '" class="frontEndEditIconLinks">' . $string . '</a>';
                }
        }
 
@@ -299,7 +299,7 @@ class FrontendEditPanel {
                        if ($conf['innerWrap.']) {
                                $content = $this->stdWrap($content, $conf['innerWrap.']);
                        }
-                       $content = ((((('<table class="typo3-editPanel-previewBorder" border="' . $thick) . '" cellpadding="0" cellspacing="0" bordercolor="') . $color) . '" width="100%" summary=""><tr><td>') . $content) . '</td></tr></table>';
+                       $content = '<table class="typo3-editPanel-previewBorder" border="' . $thick . '" cellpadding="0" cellspacing="0" bordercolor="' . $color . '" width="100%" summary=""><tr><td>' . $content . '</td></tr></table>';
                        if ($conf['outerWrap']) {
                                $content = $this->wrap($content, $conf['outerWrap']);
                        }
@@ -319,7 +319,7 @@ class FrontendEditPanel {
         * @see         editPanelPreviewBorder()
         */
        protected function isDisabled($table, $row) {
-               if ((($GLOBALS['TCA'][$table]['ctrl']['enablecolumns']['disabled'] && $row[$GLOBALS['TCA'][$table]['ctrl']['enablecolumns']['disabled']] || ($GLOBALS['TCA'][$table]['ctrl']['enablecolumns']['fe_group'] && $GLOBALS['TSFE']->simUserGroup) && $row[$GLOBALS['TCA'][$table]['ctrl']['enablecolumns']['fe_group']] == $GLOBALS['TSFE']->simUserGroup) || $GLOBALS['TCA'][$table]['ctrl']['enablecolumns']['starttime'] && $row[$GLOBALS['TCA'][$table]['ctrl']['enablecolumns']['starttime']] > $GLOBALS['EXEC_TIME']) || ($GLOBALS['TCA'][$table]['ctrl']['enablecolumns']['endtime'] && $row[$GLOBALS['TCA'][$table]['ctrl']['enablecolumns']['endtime']]) && $row[$GLOBALS['TCA'][$table]['ctrl']['enablecolumns']['endtime']] < $GLOBALS['EXEC_TIME']) {
+               if ($GLOBALS['TCA'][$table]['ctrl']['enablecolumns']['disabled'] && $row[$GLOBALS['TCA'][$table]['ctrl']['enablecolumns']['disabled']] || $GLOBALS['TCA'][$table]['ctrl']['enablecolumns']['fe_group'] && $GLOBALS['TSFE']->simUserGroup && $row[$GLOBALS['TCA'][$table]['ctrl']['enablecolumns']['fe_group']] == $GLOBALS['TSFE']->simUserGroup || $GLOBALS['TCA'][$table]['ctrl']['enablecolumns']['starttime'] && $row[$GLOBALS['TCA'][$table]['ctrl']['enablecolumns']['starttime']] > $GLOBALS['EXEC_TIME'] || $GLOBALS['TCA'][$table]['ctrl']['enablecolumns']['endtime'] && $row[$GLOBALS['TCA'][$table]['ctrl']['enablecolumns']['endtime']] && $row[$GLOBALS['TCA'][$table]['ctrl']['enablecolumns']['endtime']] < $GLOBALS['EXEC_TIME']) {
                        return TRUE;
                }
        }
@@ -365,28 +365,28 @@ class FrontendEditPanel {
                $processedDataArr['uid'] = $theCmd == 'new' ? 'NEW' : $dataArray['uid'];
                $processedDataArr['pid'] = $theCmd == 'new' ? $newUID : $dataArray['pid'];
                $panel = '';
-               $buttons = ((('<input type="image" border="0" name="TSFE_EDIT[update]" src="' . $tceforms->backPath) . 'gfx/savedok.gif" hspace="2" width="21" height="16" title="') . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_core.php:rm.saveDoc', 1)) . '" />';
-               $buttons .= ((('<input type="image" border="0" name="TSFE_EDIT[update_close]" src="' . $tceforms->backPath) . 'gfx/saveandclosedok.gif" hspace="2" width="21" height="16" title="') . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_core.php:rm.saveCloseDoc', 1)) . '" />';
-               $buttons .= ((((('<input type="image" border="0" name="TSFE_EDIT[cancel]" onclick="' . htmlspecialchars((('window.location.href=\'' . \TYPO3\CMS\Core\Utility\GeneralUtility::getIndpEnv('REQUEST_URI')) . '\';return false;'))) . '" src="') . $tceforms->backPath) . 'gfx/closedok.gif" hspace="2" width="21" height="16" title="') . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_core.php:rm.closeDoc', 1)) . '" />';
+               $buttons = '<input type="image" border="0" name="TSFE_EDIT[update]" src="' . $tceforms->backPath . 'gfx/savedok.gif" hspace="2" width="21" height="16" title="' . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_core.php:rm.saveDoc', 1) . '" />';
+               $buttons .= '<input type="image" border="0" name="TSFE_EDIT[update_close]" src="' . $tceforms->backPath . 'gfx/saveandclosedok.gif" hspace="2" width="21" height="16" title="' . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_core.php:rm.saveCloseDoc', 1) . '" />';
+               $buttons .= '<input type="image" border="0" name="TSFE_EDIT[cancel]" onclick="' . htmlspecialchars(('window.location.href=\'' . \TYPO3\CMS\Core\Utility\GeneralUtility::getIndpEnv('REQUEST_URI') . '\';return false;')) . '" src="' . $tceforms->backPath . 'gfx/closedok.gif" hspace="2" width="21" height="16" title="' . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_core.php:rm.closeDoc', 1) . '" />';
                // Buttons top
                $panel .= $tceforms->intoTemplate(array('ITEM' => $buttons));
                $panel .= $tceforms->getMainFields($table, $processedDataArr);
                $hiddenF = '';
                if ($theCmd == 'new') {
-                       $hiddenF .= ((('<input type="hidden" name="TSFE_EDIT[data][' . $table) . '][NEW][pid]" value="') . $newUID) . '" />';
+                       $hiddenF .= '<input type="hidden" name="TSFE_EDIT[data][' . $table . '][NEW][pid]" value="' . $newUID . '" />';
                        if ($table == 'pages') {
                                // If a new page is created in front-end, then show it by default!
-                               $hiddenF .= ('<input type="hidden" name="TSFE_EDIT[data][' . $table) . '][NEW][hidden]" value="0" />';
+                               $hiddenF .= '<input type="hidden" name="TSFE_EDIT[data][' . $table . '][NEW][hidden]" value="0" />';
                        } else {
-                               $hiddenF .= ('<input type="hidden" name="TSFE_EDIT[record]" value="' . $currentRecord) . '" />';
+                               $hiddenF .= '<input type="hidden" name="TSFE_EDIT[record]" value="' . $currentRecord . '" />';
                                $hiddenF .= '<input type="hidden" name="TSFE_EDIT[cmd]" value="edit" />';
                        }
                }
                $hiddenF .= '<input type="hidden" name="TSFE_EDIT[doSave]" value="0" />';
                // Buttons AND hidden fields bottom.
                $panel .= $tceforms->intoTemplate(array('ITEM' => $buttons . $hiddenF));
-               $panel = (($formTag . $tceforms->wrapTotal($panel, $dataArray, $table)) . '</form>') . ($theCmd != 'new' ? $blackLine : '');
-               $finalOut = ((($tceforms->printNeededJSFunctions_top() . ($conf['edit.']['displayRecord'] ? $content : '')) . $panel) . ($theCmd == 'new' ? $blackLine : '')) . $tceforms->printNeededJSFunctions();
+               $panel = $formTag . $tceforms->wrapTotal($panel, $dataArray, $table) . '</form>' . ($theCmd != 'new' ? $blackLine : '');
+               $finalOut = $tceforms->printNeededJSFunctions_top() . ($conf['edit.']['displayRecord'] ? $content : '') . $panel . ($theCmd == 'new' ? $blackLine : '') . $tceforms->printNeededJSFunctions();
                $GLOBALS['SOBE']->doc->insertHeaderData();
                return $finalOut;
        }