[TASK] Remove superfluous parenthesis
authorArvid Jakobsson <arvid.jakobsson@gmail.com>
Sun, 9 Sep 2012 17:08:44 +0000 (19:08 +0200)
committerChristian Kuhn <lolli@schwarzbu.ch>
Sun, 9 Sep 2012 21:26:13 +0000 (23:26 +0200)
Change ("a" . "b") . "c" into "a" . "b" . "c" using the lex-pass tool.

Change-Id: Id4784f2c2d61a68e6e1238623f9450c8b593a663
Resolves: #40774
Related: #40095
Releases: 6.0
Reviewed-on: http://review.typo3.org/14481
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
14 files changed:
typo3/sysext/workspaces/Classes/Controller/AbstractController.php
typo3/sysext/workspaces/Classes/Controller/PreviewController.php
typo3/sysext/workspaces/Classes/ExtDirect/AbstractHandler.php
typo3/sysext/workspaces/Classes/ExtDirect/ActionHandler.php
typo3/sysext/workspaces/Classes/ExtDirect/ExtDirectServer.php
typo3/sysext/workspaces/Classes/ExtDirect/WorkspaceSelectorToolbarItem.php
typo3/sysext/workspaces/Classes/Service/AutoPublishService.php
typo3/sysext/workspaces/Classes/Service/GridDataService.php
typo3/sysext/workspaces/Classes/Service/HistoryService.php
typo3/sysext/workspaces/Classes/Service/IntegrityService.php
typo3/sysext/workspaces/Classes/Service/StagesService.php
typo3/sysext/workspaces/Classes/Service/WorkspaceService.php
typo3/sysext/workspaces/ext_localconf.php
typo3/sysext/workspaces/ext_tables.php

index ef59adb..e1fa7b2 100644 (file)
@@ -103,7 +103,7 @@ class AbstractController extends \TYPO3\CMS\Extbase\Mvc\Controller\ActionControl
                parent::processRequest($request, $response);
                $pageHeader = $this->template->startpage($GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang.xml:module.title'));
                $pageEnd = $this->template->endPage();
-               $response->setContent(($pageHeader . $response->getContent()) . $pageEnd);
+               $response->setContent($pageHeader . $response->getContent() . $pageEnd);
        }
 
        /**
index 9081586..b54fa88 100644 (file)
@@ -116,19 +116,19 @@ class PreviewController extends \TYPO3\CMS\Workspaces\Controller\AbstractControl
                $wsSettingsPath = \TYPO3\CMS\Core\Utility\GeneralUtility::getIndpEnv('TYPO3_SITE_URL') . 'typo3/';
                $wsSettingsUri = $uriBuilder->uriFor('singleIndex', array(), 'TYPO3\\CMS\\Workspaces\\Controller\\ReviewController', 'workspaces', 'web_workspacesworkspaces');
                $wsSettingsParams = '&tx_workspaces_web_workspacesworkspaces[controller]=Review';
-               $wsSettingsUrl = ($wsSettingsPath . $wsSettingsUri) . $wsSettingsParams;
+               $wsSettingsUrl = $wsSettingsPath . $wsSettingsUri . $wsSettingsParams;
                $viewDomain = \TYPO3\CMS\Backend\Utility\BackendUtility::getViewDomain($this->pageId);
-               $wsBaseUrl = ((($viewDomain . '/index.php?id=') . $this->pageId) . '&L=') . $language;
+               $wsBaseUrl = $viewDomain . '/index.php?id=' . $this->pageId . '&L=' . $language;
                // @todo - handle new pages here
                // branchpoints are not handled anymore because this feature is not supposed anymore
                if (\TYPO3\CMS\Workspaces\Service\WorkspaceService::isNewPage($this->pageId)) {
                        $wsNewPageUri = $uriBuilder->uriFor('newPage', array(), 'TYPO3\\CMS\\Workspaces\\Controller\\PreviewController', 'workspaces', 'web_workspacesworkspaces');
                        $wsNewPageParams = '&tx_workspaces_web_workspacesworkspaces[controller]=Preview';
-                       $this->view->assign('liveUrl', ($wsSettingsPath . $wsNewPageUri) . $wsNewPageParams);
+                       $this->view->assign('liveUrl', $wsSettingsPath . $wsNewPageUri . $wsNewPageParams);
                } else {
                        $this->view->assign('liveUrl', $wsBaseUrl . '&ADMCMD_noBeUser=1');
                }
-               $this->view->assign('wsUrl', ($wsBaseUrl . '&ADMCMD_view=1&ADMCMD_editIcons=1&ADMCMD_previewWS=') . $GLOBALS['BE_USER']->workspace);
+               $this->view->assign('wsUrl', $wsBaseUrl . '&ADMCMD_view=1&ADMCMD_editIcons=1&ADMCMD_previewWS=' . $GLOBALS['BE_USER']->workspace);
                $this->view->assign('wsSettingsUrl', $wsSettingsUrl);
                $this->view->assign('backendDomain', \TYPO3\CMS\Core\Utility\GeneralUtility::getIndpEnv('TYPO3_HOST_ONLY'));
                $splitPreviewTsConfig = \TYPO3\CMS\Backend\Utility\BackendUtility::getModTSconfig($this->pageId, 'workspaces.splitPreviewModes');
@@ -144,20 +144,20 @@ class PreviewController extends \TYPO3\CMS\Workspaces\Controller\AbstractControl
                $this->pageRenderer->addInlineSetting('Workspaces', 'disableDiscardStageButton', $this->isInvalidStage($nextStage) && $this->isInvalidStage($previousStage));
                $resourcePath = \TYPO3\CMS\Core\Extension\ExtensionManager::extRelPath('lang') . 'res/js/be/';
                $this->pageRenderer->addJsFile($resourcePath . 'typo3lang.js');
-               $this->pageRenderer->addJsInlineCode('workspaces.preview.lll', ((((((((((((((((((((((('
+               $this->pageRenderer->addJsInlineCode('workspaces.preview.lll', '
                TYPO3.lang = {
-                       visualPreview: \'' . $GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang.xml:preview.visualPreview', TRUE)) . '\',
-                       listView: \'') . $GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang.xml:preview.listView', TRUE)) . '\',
-                       livePreview: \'') . $GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang.xml:preview.livePreview', TRUE)) . '\',
-                       livePreviewDetail: \'') . $GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang.xml:preview.livePreviewDetail', TRUE)) . '\',
-                       workspacePreview: \'') . $GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang.xml:preview.workspacePreview', TRUE)) . '\',
-                       workspacePreviewDetail: \'') . $GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang.xml:preview.workspacePreviewDetail', TRUE)) . '\',
-                       modeSlider: \'') . $GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang.xml:preview.modeSlider', TRUE)) . '\',
-                       modeVbox: \'') . $GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang.xml:preview.modeVbox', TRUE)) . '\',
-                       modeHbox: \'') . $GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang.xml:preview.modeHbox', TRUE)) . '\',
-                       discard: \'') . $GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang.xml:label_doaction_discard', TRUE)) . '\',
-                       nextStage: \'') . $nextStage['title']) . '\',
-                       previousStage: \'') . $previousStage['title']) . '\'
+                       visualPreview: \'' . $GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang.xml:preview.visualPreview', TRUE) . '\',
+                       listView: \'' . $GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang.xml:preview.listView', TRUE) . '\',
+                       livePreview: \'' . $GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang.xml:preview.livePreview', TRUE) . '\',
+                       livePreviewDetail: \'' . $GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang.xml:preview.livePreviewDetail', TRUE) . '\',
+                       workspacePreview: \'' . $GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang.xml:preview.workspacePreview', TRUE) . '\',
+                       workspacePreviewDetail: \'' . $GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang.xml:preview.workspacePreviewDetail', TRUE) . '\',
+                       modeSlider: \'' . $GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang.xml:preview.modeSlider', TRUE) . '\',
+                       modeVbox: \'' . $GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang.xml:preview.modeVbox', TRUE) . '\',
+                       modeHbox: \'' . $GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang.xml:preview.modeHbox', TRUE) . '\',
+                       discard: \'' . $GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang.xml:label_doaction_discard', TRUE) . '\',
+                       nextStage: \'' . $nextStage['title'] . '\',
+                       previousStage: \'' . $previousStage['title'] . '\'
                };TYPO3.l10n.initialize();
 ');
                $resourcePath = \TYPO3\CMS\Core\Extension\ExtensionManager::extRelPath('workspaces') . 'Resources/Public/';
@@ -262,10 +262,10 @@ class PreviewController extends \TYPO3\CMS\Workspaces\Controller\AbstractControl
                        'donateWindow_button_disable' => $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_core.xml:donateWindow.button_disable'),
                        'donateWindow_button_postpone' => $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_core.xml:donateWindow.button_postpone')
                );
-               $js = ((('
-               TYPO3.configuration = ' . json_encode($t3Configuration)) . ';
+               $js = '
+               TYPO3.configuration = ' . json_encode($t3Configuration) . ';
                TYPO3.LLL = {
-                       core : ') . json_encode($t3LLLcore)) . '
+                       core : ' . json_encode($t3LLLcore) . '
                };
 
                /**
index 662d4b2..b6c94b5 100644 (file)
@@ -57,7 +57,7 @@ abstract class AbstractHandler {
                $response = array(
                        'error' => array(
                                'code' => $errorCode,
-                               'message' => $GLOBALS['LANG']->sL(($localLangFile . ':') . $errorLabel)
+                               'message' => $GLOBALS['LANG']->sL($localLangFile . ':' . $errorLabel)
                        ),
                        'success' => $successFlagValue
                );
index 3b6191d..b52aa88 100644 (file)
@@ -506,7 +506,7 @@ class ActionHandler extends \TYPO3\CMS\Workspaces\ExtDirect\AbstractHandler {
                                array(
                                        'xtype' => 'panel',
                                        'bodyStyle' => 'margin-bottom: 7px; border: none;',
-                                       'html' => ($GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang.xlf:window.sendToNextStageWindow.itemsWillBeSentTo') . ' ') . $stageTitle
+                                       'html' => $GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang.xlf:window.sendToNextStageWindow.itemsWillBeSentTo') . ' ' . $stageTitle
                                )
                        )
                );
@@ -640,7 +640,7 @@ class ActionHandler extends \TYPO3\CMS\Workspaces\ExtDirect\AbstractHandler {
                // get only the relevant items for processing
                $workspaceItemsArray = $workspaceService->selectVersionsInWorkspace($this->stageService->getWorkspaceId(), ($filter = 1), $currentStage['uid'], $id, ($recursionLevel = 0), ($selectionType = 'tables_modify'));
                return array(
-                       'title' => (('Status message: Page send to next stage - ID: ' . $id) . ' - Next stage title: ') . $previousStage['title'],
+                       'title' => 'Status message: Page send to next stage - ID: ' . $id . ' - Next stage title: ' . $previousStage['title'],
                        'items' => $this->getSentToStageWindow($previousStage['uid']),
                        'affects' => $workspaceItemsArray,
                        'stageId' => $previousStage['uid']
@@ -659,7 +659,7 @@ class ActionHandler extends \TYPO3\CMS\Workspaces\ExtDirect\AbstractHandler {
                // get only the relevant items for processing
                $workspaceItemsArray = $workspaceService->selectVersionsInWorkspace($this->stageService->getWorkspaceId(), ($filter = 1), $currentStage['uid'], $id, ($recursionLevel = 0), ($selectionType = 'tables_modify'));
                return array(
-                       'title' => (('Status message: Page send to next stage - ID: ' . $id) . ' - Next stage title: ') . $nextStage['title'],
+                       'title' => 'Status message: Page send to next stage - ID: ' . $id . ' - Next stage title: ' . $nextStage['title'],
                        'items' => $this->getSentToStageWindow($nextStage['uid']),
                        'affects' => $workspaceItemsArray,
                        'stageId' => $nextStage['uid']
index e5727f0..d7ea076 100644 (file)
@@ -141,7 +141,7 @@ class ExtDirectServer extends \TYPO3\CMS\Workspaces\ExtDirect\AbstractHandler {
                }
                foreach ($fieldsOfRecords as $fieldName) {
                        // check for exclude fields
-                       if (($GLOBALS['BE_USER']->isAdmin() || $GLOBALS['TCA'][$parameter->table]['columns'][$fieldName]['exclude'] == 0) || \TYPO3\CMS\Core\Utility\GeneralUtility::inList($GLOBALS['BE_USER']->groupData['non_exclude_fields'], ($parameter->table . ':') . $fieldName)) {
+                       if ($GLOBALS['BE_USER']->isAdmin() || $GLOBALS['TCA'][$parameter->table]['columns'][$fieldName]['exclude'] == 0 || \TYPO3\CMS\Core\Utility\GeneralUtility::inList($GLOBALS['BE_USER']->groupData['non_exclude_fields'], $parameter->table . ':' . $fieldName)) {
                                // call diff class only if there is a difference
                                if (strcmp($liveRecord[$fieldName], $versionRecord[$fieldName]) !== 0) {
                                        // Select the human readable values before diff
@@ -216,9 +216,9 @@ class ExtDirectServer extends \TYPO3\CMS\Workspaces\ExtDirect\AbstractHandler {
         */
        public function getCommentsForRecord($uid, $table) {
                $sysLogReturnArray = array();
-               $sysLogRows = $GLOBALS['TYPO3_DB']->exec_SELECTgetRows('log_data,tstamp,userid', 'sys_log', (('action=6 and details_nr=30
-                               AND tablename=' . $GLOBALS['TYPO3_DB']->fullQuoteStr($table, 'sys_log')) . '
-                               AND recuid=') . intval($uid), '', 'tstamp DESC');
+               $sysLogRows = $GLOBALS['TYPO3_DB']->exec_SELECTgetRows('log_data,tstamp,userid', 'sys_log', 'action=6 and details_nr=30
+                               AND tablename=' . $GLOBALS['TYPO3_DB']->fullQuoteStr($table, 'sys_log') . '
+                               AND recuid=' . intval($uid), '', 'tstamp DESC');
                foreach ($sysLogRows as $sysLogRow) {
                        $sysLogEntry = array();
                        $data = unserialize($sysLogRow['log_data']);
index 2e22e81..d1b8149 100644 (file)
@@ -64,7 +64,7 @@ class WorkspaceSelectorToolbarItem implements \TYPO3\CMS\Backend\Toolbar\Toolbar
                $this->changeWorkspace = \TYPO3\CMS\Core\Utility\GeneralUtility::_GP('changeWorkspace');
                $this->changeWorkspacePreview = \TYPO3\CMS\Core\Utility\GeneralUtility::_GP('changeWorkspacePreview');
                $pageRenderer = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance('t3lib_pageRenderer');
-               $this->backendReference->addJavaScript(('TYPO3.Workspaces = { workspaceTitle : \'' . addslashes(\Tx_Workspaces_Service_Workspaces::getWorkspaceTitle($GLOBALS['BE_USER']->workspace))) . '\'};
+               $this->backendReference->addJavaScript('TYPO3.Workspaces = { workspaceTitle : \'' . addslashes(\Tx_Workspaces_Service_Workspaces::getWorkspaceTitle($GLOBALS['BE_USER']->workspace)) . '\'};
 ');
        }
 
@@ -103,7 +103,7 @@ class WorkspaceSelectorToolbarItem implements \TYPO3\CMS\Backend\Toolbar\Toolbar
                $stateUncheckedIcon = \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('empty-empty', array(
                        'title' => $GLOBALS['LANG']->getLL('bookmark_inactive')
                ));
-               $workspaceMenu[] = ('<a href="#" class="toolbar-item">' . \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('apps-toolbar-menu-workspace', array('title' => $title))) . '</a>';
+               $workspaceMenu[] = '<a href="#" class="toolbar-item">' . \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('apps-toolbar-menu-workspace', array('title' => $title)) . '</a>';
                $workspaceMenu[] = '<ul class="toolbar-item-menu" style="display: none;">';
                if (count($availableWorkspaces)) {
                        foreach ($availableWorkspaces as $workspaceId => $label) {
@@ -113,14 +113,14 @@ class WorkspaceSelectorToolbarItem implements \TYPO3\CMS\Backend\Toolbar\Toolbar
                                        $selected = ' class="selected"';
                                        $icon = $stateCheckedIcon;
                                }
-                               $workspaceMenu[] = (((((((((('<li' . $selected) . '>') . '<a href="backend.php?changeWorkspace=') . intval($workspaceId)) . '" id="ws-') . intval($workspaceId)) . '" class="ws">') . $icon) . ' ') . htmlspecialchars($label)) . '</a></li>';
+                               $workspaceMenu[] = '<li' . $selected . '>' . '<a href="backend.php?changeWorkspace=' . intval($workspaceId) . '" id="ws-' . intval($workspaceId) . '" class="ws">' . $icon . ' ' . htmlspecialchars($label) . '</a></li>';
                        }
                } else {
-                       $workspaceMenu[] = ((('<li>' . $stateUncheckedIcon) . ' ') . $GLOBALS['LANG']->getLL('bookmark_noWSfound', TRUE)) . '</li>';
+                       $workspaceMenu[] = '<li>' . $stateUncheckedIcon . ' ' . $GLOBALS['LANG']->getLL('bookmark_noWSfound', TRUE) . '</li>';
                }
                if ($GLOBALS['BE_USER']->check('modules', 'web_WorkspacesWorkspaces')) {
                        // go to workspace module link
-                       $workspaceMenu[] = ((((('<li class="divider">' . $stateUncheckedIcon) . ' ') . '<a href="javascript:top.goToModule(\'web_WorkspacesWorkspaces\');" target="content" id="goToWsModule">') . ' ') . $GLOBALS['LANG']->getLL('bookmark_workspace', TRUE)) . '</a></li>';
+                       $workspaceMenu[] = '<li class="divider">' . $stateUncheckedIcon . ' ' . '<a href="javascript:top.goToModule(\'web_WorkspacesWorkspaces\');" target="content" id="goToWsModule">' . ' ' . $GLOBALS['LANG']->getLL('bookmark_workspace', TRUE) . '</a></li>';
                }
                $workspaceMenu[] = '</ul>';
                return implode(LF, $workspaceMenu);
index bbcdc56..d2e6914 100644 (file)
@@ -47,10 +47,10 @@ class AutoPublishService {
                $currentAdminStatus = $GLOBALS['BE_USER']->user['admin'];
                $GLOBALS['BE_USER']->user['admin'] = 1;
                // Select all workspaces that needs to be published / unpublished:
-               $workspaces = $GLOBALS['TYPO3_DB']->exec_SELECTgetRows('uid,swap_modes,publish_time,unpublish_time', 'sys_workspace', (((('pid=0
+               $workspaces = $GLOBALS['TYPO3_DB']->exec_SELECTgetRows('uid,swap_modes,publish_time,unpublish_time', 'sys_workspace', 'pid=0
                                AND
-                               ((publish_time!=0 AND publish_time<=' . intval($GLOBALS['EXEC_TIME'])) . ')
-                               OR (publish_time=0 AND unpublish_time!=0 AND unpublish_time<=') . intval($GLOBALS['EXEC_TIME'])) . '))') . \TYPO3\CMS\Backend\Utility\BackendUtility::deleteClause('sys_workspace'));
+                               ((publish_time!=0 AND publish_time<=' . intval($GLOBALS['EXEC_TIME']) . ')
+                               OR (publish_time=0 AND unpublish_time!=0 AND unpublish_time<=' . intval($GLOBALS['EXEC_TIME']) . '))' . \TYPO3\CMS\Backend\Utility\BackendUtility::deleteClause('sys_workspace'));
                $workspaceService = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance('TYPO3\\CMS\\Workspaces\\Service\\WorkspaceService');
                foreach ($workspaces as $rec) {
                        // First, clear start/end time so it doesn't get select once again:
index 9ebf148..7725986 100644 (file)
@@ -142,7 +142,7 @@ class GridDataService {
                                        }
                                        $isDeletedPage = $table == 'pages' && $recordState == 'deleted';
                                        $viewUrl = \TYPO3\CMS\Workspaces\Service\WorkspaceService::viewSingleRecord($table, $record['t3ver_oid'], $origRecord);
-                                       $versionArray['id'] = ($table . ':') . $record['uid'];
+                                       $versionArray['id'] = $table . ':' . $record['uid'];
                                        $versionArray['uid'] = $record['uid'];
                                        $versionArray['workspace'] = $versionRecord['t3ver_id'];
                                        $versionArray['label_Workspace'] = htmlspecialchars(\TYPO3\CMS\Backend\Utility\BackendUtility::getRecordTitle($table, $versionRecord));
@@ -170,7 +170,7 @@ class GridDataService {
                                        );
                                        $versionArray['allowedAction_nextStage'] = $isRecordTypeAllowedToModify && $stagesObj->isNextStageAllowedForUser($versionRecord['t3ver_stage']);
                                        $versionArray['allowedAction_prevStage'] = $isRecordTypeAllowedToModify && $stagesObj->isPrevStageAllowedForUser($versionRecord['t3ver_stage']);
-                                       if (($swapAccess && $swapStage != 0) && $versionRecord['t3ver_stage'] == $swapStage) {
+                                       if ($swapAccess && $swapStage != 0 && $versionRecord['t3ver_stage'] == $swapStage) {
                                                $versionArray['allowedAction_swap'] = $isRecordTypeAllowedToModify && $stagesObj->isNextStageAllowedForUser($swapStage);
                                        } elseif ($swapAccess && $swapStage == 0) {
                                                $versionArray['allowedAction_swap'] = $isRecordTypeAllowedToModify;
@@ -193,7 +193,7 @@ class GridDataService {
                        $this->emitSignal(self::SIGNAL_GenerateDataArray_BeforeCaching, $this->dataArray, $versions);
                        // Enrich elements after everything has been processed:
                        foreach ($this->dataArray as &$element) {
-                               $identifier = ($element['table'] . ':') . $element['t3ver_oid'];
+                               $identifier = $element['table'] . ':' . $element['t3ver_oid'];
                                $element['integrity'] = array(
                                        'status' => $this->getIntegrityService()->getStatusRepresentation($identifier),
                                        'messages' => htmlspecialchars($this->getIntegrityService()->getIssueMessages($identifier, TRUE))
@@ -323,7 +323,7 @@ class GridDataService {
                                break;
                        }
                } else {
-                       \TYPO3\CMS\Core\Utility\GeneralUtility::sysLog(('Try to sort "' . $this->sort) . '" in "TYPO3\\CMS\\Workspaces\\Service\\GridDataService::sortDataArray" but $this->dataArray is empty! This might be the Bug #26422 which could not reproduced yet.', 3);
+                       \TYPO3\CMS\Core\Utility\GeneralUtility::sysLog('Try to sort "' . $this->sort . '" in "TYPO3\\CMS\\Workspaces\\Service\\GridDataService::sortDataArray" but $this->dataArray is empty! This might be the Bug #26422 which could not reproduced yet.', 3);
                }
                // Suggested slot method:
                // methodName(Tx_Workspaces_Service_GridData $gridData, array &$dataArray, $sortColumn, $sortDirection)
@@ -394,7 +394,7 @@ class GridDataService {
        protected function isFilterTextInVisibleColumns($filterText, array $versionArray) {
                if (is_array($GLOBALS['BE_USER']->uc['moduleData']['Workspaces'][$GLOBALS['BE_USER']->workspace]['columns'])) {
                        foreach ($GLOBALS['BE_USER']->uc['moduleData']['Workspaces'][$GLOBALS['BE_USER']->workspace]['columns'] as $column => $value) {
-                               if ((isset($value['hidden']) && isset($column)) && isset($versionArray[$column])) {
+                               if (isset($value['hidden']) && isset($column) && isset($versionArray[$column])) {
                                        if ($value['hidden'] == 0) {
                                                switch ($column) {
                                                case 'workspace_Tstamp':
index 5f1fa51..195b8a5 100644 (file)
@@ -145,7 +145,7 @@ class HistoryService implements \TYPO3\CMS\Core\SingletonInterface {
                if (!isset($this->historyObjects[$table][$id])) {
                        /** @var $historyObject \TYPO3\CMS\Backend\History\RecordHistory */
                        $historyObject = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance('TYPO3\\CMS\\Backend\\History\\RecordHistory');
-                       $historyObject->element = ($table . ':') . $id;
+                       $historyObject->element = $table . ':' . $id;
                        $historyObject->createChangeLog();
                        $this->historyObjects[$table][$id] = $historyObject;
                }
index 2404430..1ee18d5 100644 (file)
@@ -144,7 +144,7 @@ class IntegrityService {
                                        // Add warning for current versionized record:
                                        $this->addIssue($element->getLiveRecord()->getIdentifier(), self::STATUS_Warning, sprintf(\TYPO3\CMS\Extbase\Utility\LocalizationUtility::translate('integrity.dependsOnDefaultLanguageRecord', 'workspaces'), $title));
                                        // Add info for related localization parent record:
-                                       $this->addIssue(($table . ':') . $languageParentRecord['uid'], self::STATUS_Info, sprintf(\TYPO3\CMS\Extbase\Utility\LocalizationUtility::translate('integrity.isDefaultLanguageRecord', 'workspaces'), $title));
+                                       $this->addIssue($table . ':' . $languageParentRecord['uid'], self::STATUS_Info, sprintf(\TYPO3\CMS\Extbase\Utility\LocalizationUtility::translate('integrity.isDefaultLanguageRecord', 'workspaces'), $title));
                                }
                        }
                }
index 0131e76..b5e121b 100644 (file)
@@ -214,20 +214,20 @@ class StagesService {
                } else {
                        $stages[] = array(
                                'uid' => self::STAGE_EDIT_ID,
-                               'title' => (($GLOBALS['LANG']->sL(($this->pathToLocallang . ':actionSendToStage')) . ' "') . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_mod_user_ws.xml:stage_editing')) . '"'
+                               'title' => $GLOBALS['LANG']->sL(($this->pathToLocallang . ':actionSendToStage')) . ' "' . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_mod_user_ws.xml:stage_editing') . '"'
                        );
                        $workspaceRec = \TYPO3\CMS\Backend\Utility\BackendUtility::getRecord('sys_workspace', $this->getWorkspaceId());
                        if ($workspaceRec['custom_stages'] > 0) {
                                // Get all stage records for this workspace
-                               $workspaceStageRecs = $GLOBALS['TYPO3_DB']->exec_SELECTgetRows('*', self::TABLE_STAGE, ((('parentid=' . $this->getWorkspaceId()) . ' AND parenttable=') . $GLOBALS['TYPO3_DB']->fullQuoteStr('sys_workspace', self::TABLE_STAGE)) . ' AND deleted=0', '', 'sorting', '', 'uid');
+                               $workspaceStageRecs = $GLOBALS['TYPO3_DB']->exec_SELECTgetRows('*', self::TABLE_STAGE, 'parentid=' . $this->getWorkspaceId() . ' AND parenttable=' . $GLOBALS['TYPO3_DB']->fullQuoteStr('sys_workspace', self::TABLE_STAGE) . ' AND deleted=0', '', 'sorting', '', 'uid');
                                foreach ($workspaceStageRecs as $stage) {
-                                       $stage['title'] = (($GLOBALS['LANG']->sL(($this->pathToLocallang . ':actionSendToStage')) . ' "') . $stage['title']) . '"';
+                                       $stage['title'] = $GLOBALS['LANG']->sL(($this->pathToLocallang . ':actionSendToStage')) . ' "' . $stage['title'] . '"';
                                        $stages[] = $stage;
                                }
                        }
                        $stages[] = array(
                                'uid' => self::STAGE_PUBLISH_ID,
-                               'title' => (($GLOBALS['LANG']->sL(($this->pathToLocallang . ':actionSendToStage')) . ' "') . $GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang_mod.xml:stage_ready_to_publish')) . '"'
+                               'title' => $GLOBALS['LANG']->sL(($this->pathToLocallang . ':actionSendToStage')) . ' "' . $GLOBALS['LANG']->sL('LLL:EXT:workspaces/Resources/Private/Language/locallang_mod.xml:stage_ready_to_publish') . '"'
                        );
                        $stages[] = array(
                                'uid' => self::STAGE_PUBLISH_EXECUTE_ID,
@@ -353,7 +353,7 @@ class StagesService {
                if ($nextStage === FALSE) {
                        $nextStage[] = array(
                                'uid' => self::STAGE_EDIT_ID,
-                               'title' => (($GLOBALS['LANG']->sL(($this->pathToLocallang . ':actionSendToStage')) . ' "') . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_mod_user_ws.xml:stage_editing')) . '"'
+                               'title' => $GLOBALS['LANG']->sL(($this->pathToLocallang . ':actionSendToStage')) . ' "' . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_mod_user_ws.xml:stage_editing') . '"'
                        );
                }
                return $nextStage;
@@ -486,7 +486,7 @@ class StagesService {
                        break;
                }
                if (!empty($userList)) {
-                       $userRecords = \TYPO3\CMS\Backend\Utility\BackendUtility::getUserNames('username, uid, email, realName', ('AND uid IN (' . $userList) . ')');
+                       $userRecords = \TYPO3\CMS\Backend\Utility\BackendUtility::getUserNames('username, uid, email, realName', 'AND uid IN (' . $userList . ')');
                }
                if (!empty($userRecords) && is_array($userRecords)) {
                        foreach ($userRecords as $userUid => $userRecord) {
@@ -562,7 +562,7 @@ class StagesService {
         * @return void
         */
        private function fetchGroupsFromDB(array $groups) {
-               $whereSQL = ('deleted=0 AND hidden=0 AND pid=0 AND uid IN (' . implode(',', $groups)) . ') ';
+               $whereSQL = 'deleted=0 AND hidden=0 AND pid=0 AND uid IN (' . implode(',', $groups) . ') ';
                $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('*', 'be_groups', $whereSQL);
                // The userGroups array is filled
                while ($row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res)) {
@@ -598,7 +598,7 @@ class StagesService {
                                                // Make integer list
                                                $theList = implode(',', \TYPO3\CMS\Core\Utility\GeneralUtility::intExplode(',', $row['subgroup']));
                                                // Get the subarray
-                                               $subbarray = $this->fetchGroups($theList, ($idList . ',') . $uid);
+                                               $subbarray = $this->fetchGroups($theList, $idList . ',' . $uid);
                                                list($subUid, $subArray) = each($subbarray);
                                                // Merge the subarray to the already existing userGroups array
                                                $this->userGroups[$subUid] = $subArray;
@@ -706,7 +706,7 @@ class StagesService {
         * @return bool
         */
        protected function isStageAllowedForUser($stageId) {
-               $cacheKey = ($this->getWorkspaceId() . '_') . $stageId;
+               $cacheKey = $this->getWorkspaceId() . '_' . $stageId;
                $isAllowed = FALSE;
                if (isset($this->workspaceStageAllowedCache[$cacheKey])) {
                        $isAllowed = $this->workspaceStageAllowedCache[$cacheKey];
index 9bd4ce9..c06f690 100644 (file)
@@ -244,22 +244,22 @@ class WorkspaceService implements \TYPO3\CMS\Core\SingletonInterface {
                if ($isTableLocalizable === FALSE && $language > 0) {
                        return array();
                } elseif ($isTableLocalizable) {
-                       $languageParentField = ('A.' . $GLOBALS['TCA'][$table]['ctrl']['transOrigPointerField']) . ', ';
+                       $languageParentField = 'A.' . $GLOBALS['TCA'][$table]['ctrl']['transOrigPointerField'] . ', ';
                }
-               $fields = ('A.uid, A.t3ver_oid, A.t3ver_stage, ' . $languageParentField) . 'B.pid AS wspid, B.pid AS livepid';
+               $fields = 'A.uid, A.t3ver_oid, A.t3ver_stage, ' . $languageParentField . 'B.pid AS wspid, B.pid AS livepid';
                if ($isTableLocalizable) {
                        $fields .= ', A.' . $GLOBALS['TCA'][$table]['ctrl']['languageField'];
                }
-               $from = (($table . ' A,') . $table) . ' B';
+               $from = $table . ' A,' . $table . ' B';
                // Table A is the offline version and pid=-1 defines offline
                $where = 'A.pid=-1 AND A.t3ver_state!=4';
                if ($pageList) {
                        $pidField = $table === 'pages' ? 'uid' : 'pid';
-                       $pidConstraint = strstr($pageList, ',') ? (' IN (' . $pageList) . ')' : '=' . $pageList;
-                       $where .= (' AND B.' . $pidField) . $pidConstraint;
+                       $pidConstraint = strstr($pageList, ',') ? ' IN (' . $pageList . ')' : '=' . $pageList;
+                       $where .= ' AND B.' . $pidField . $pidConstraint;
                }
                if ($isTableLocalizable && \TYPO3\CMS\Core\Utility\MathUtility::canBeInterpretedAsInteger($language)) {
-                       $where .= ((' AND A.' . $GLOBALS['TCA'][$table]['ctrl']['languageField']) . '=') . $language;
+                       $where .= ' AND A.' . $GLOBALS['TCA'][$table]['ctrl']['languageField'] . '=' . $language;
                }
                /** For "real" workspace numbers, select by that.
                If = -98, select all that are NOT online (zero).
@@ -307,10 +307,10 @@ class WorkspaceService implements \TYPO3\CMS\Core\SingletonInterface {
                B - online record
                C - moveFrom placeholder */
                $fields = 'A.pid AS wspid, B.uid AS t3ver_oid, C.uid AS uid, B.pid AS livepid';
-               $from = (((($table . ' A, ') . $table) . ' B,') . $table) . ' C';
+               $from = $table . ' A, ' . $table . ' B,' . $table . ' C';
                $where = 'A.t3ver_state=3 AND B.pid>0 AND B.t3ver_state=0 AND B.t3ver_wsid=0 AND C.pid=-1 AND C.t3ver_state=4';
                if ($wsid > self::SELECT_ALL_WORKSPACES) {
-                       $where .= ((' AND A.t3ver_wsid=' . $wsid) . ' AND C.t3ver_wsid=') . $wsid;
+                       $where .= ' AND A.t3ver_wsid=' . $wsid . ' AND C.t3ver_wsid=' . $wsid;
                } elseif ($wsid === self::SELECT_ALL_WORKSPACES) {
                        $where .= ' AND A.t3ver_wsid!=0 AND C.t3ver_wsid!=0 ';
                }
@@ -325,8 +325,8 @@ class WorkspaceService implements \TYPO3\CMS\Core\SingletonInterface {
                }
                if ($pageList) {
                        $pidField = $table === 'pages' ? 'B.uid' : 'A.pid';
-                       $pidConstraint = strstr($pageList, ',') ? (' IN (' . $pageList) . ')' : '=' . $pageList;
-                       $where .= (' AND ' . $pidField) . $pidConstraint;
+                       $pidConstraint = strstr($pageList, ',') ? ' IN (' . $pageList . ')' : '=' . $pageList;
+                       $where .= ' AND ' . $pidField . $pidConstraint;
                }
                $where .= ' AND A.t3ver_move_id = B.uid AND B.uid = C.t3ver_oid';
                $where .= \TYPO3\CMS\Backend\Utility\BackendUtility::deleteClause($table, 'A');
@@ -367,7 +367,7 @@ class WorkspaceService implements \TYPO3\CMS\Core\SingletonInterface {
                unset($searchObj);
                if (intval($GLOBALS['TCA']['pages']['ctrl']['versioningWS']) === 2 && $pageList) {
                        // Remove the "subbranch" if a page was moved away
-                       $movedAwayPages = $GLOBALS['TYPO3_DB']->exec_SELECTgetRows('uid, pid, t3ver_move_id', 'pages', ((('t3ver_move_id IN (' . $pageList) . ') AND t3ver_wsid=') . intval($wsid)) . \TYPO3\CMS\Backend\Utility\BackendUtility::deleteClause('pages'), '', 'uid', '', 't3ver_move_id');
+                       $movedAwayPages = $GLOBALS['TYPO3_DB']->exec_SELECTgetRows('uid, pid, t3ver_move_id', 'pages', 't3ver_move_id IN (' . $pageList . ') AND t3ver_wsid=' . intval($wsid) . \TYPO3\CMS\Backend\Utility\BackendUtility::deleteClause('pages'), '', 'uid', '', 't3ver_move_id');
                        $pageIds = \TYPO3\CMS\Core\Utility\GeneralUtility::intExplode(',', $pageList, TRUE);
                        // move all pages away
                        $newList = array_diff($pageIds, array_keys($movedAwayPages));
@@ -385,7 +385,7 @@ class WorkspaceService implements \TYPO3\CMS\Core\SingletonInterface {
                        } while ($changed);
                        $pageList = implode(',', $newList);
                        // In case moving pages is enabled we need to replace all move-to pointer with their origin
-                       $pages = $GLOBALS['TYPO3_DB']->exec_SELECTgetRows('uid, t3ver_move_id', 'pages', (('uid IN (' . $pageList) . ')') . \TYPO3\CMS\Backend\Utility\BackendUtility::deleteClause('pages'), '', 'uid', '', 'uid');
+                       $pages = $GLOBALS['TYPO3_DB']->exec_SELECTgetRows('uid, t3ver_move_id', 'pages', 'uid IN (' . $pageList . ')' . \TYPO3\CMS\Backend\Utility\BackendUtility::deleteClause('pages'), '', 'uid', '', 'uid');
                        $newList = array();
                        $pageIds = \TYPO3\CMS\Core\Utility\GeneralUtility::intExplode(',', $pageList, TRUE);
                        if (!in_array($pageId, $pageIds)) {
@@ -475,7 +475,7 @@ class WorkspaceService implements \TYPO3\CMS\Core\SingletonInterface {
                // If the language is not default, check state of overlay
                if ($language > 0) {
                        $whereClause = 'pid = ' . $id;
-                       $whereClause .= ((' AND ' . $GLOBALS['TCA']['sys_language_overlay']['ctrl']['languageField']) . ' = ') . $language;
+                       $whereClause .= ' AND ' . $GLOBALS['TCA']['sys_language_overlay']['ctrl']['languageField'] . ' = ' . $language;
                        $whereClause .= ' AND t3ver_wsid = ' . $GLOBALS['BE_USER']->workspace;
                        $whereClause .= \TYPO3\CMS\Backend\Utility\BackendUtility::deleteClause('sys_language_overlay');
                        $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('t3ver_state', 'sys_language_overlay', $whereClause);
@@ -553,7 +553,7 @@ class WorkspaceService implements \TYPO3\CMS\Core\SingletonInterface {
                        'ADMCMD_prev' => $previewKeyword,
                        'id' => $uid
                );
-               return (\TYPO3\CMS\Backend\Utility\BackendUtility::getViewDomain($uid) . '/index.php?') . \TYPO3\CMS\Core\Utility\GeneralUtility::implodeArrayForUrl('', $linkParams);
+               return \TYPO3\CMS\Backend\Utility\BackendUtility::getViewDomain($uid) . '/index.php?' . \TYPO3\CMS\Core\Utility\GeneralUtility::implodeArrayForUrl('', $linkParams);
        }
 
        /**
@@ -579,9 +579,9 @@ class WorkspaceService implements \TYPO3\CMS\Core\SingletonInterface {
                $redirect = $backPath . 'index.php?redirect_url=';
                // @todo why do we need these additional params? the URIBuilder should add the controller, but he doesn't :(
                $additionalParams = '&tx_workspaces_web_workspacesworkspaces%5Bcontroller%5D=Preview&M=web_WorkspacesWorkspaces&id=';
-               $viewScript = ($backPath . $uriBuilder->setArguments(array('tx_workspaces_web_workspacesworkspaces' => array('previewWS' => $GLOBALS['BE_USER']->workspace)))->uriFor('index', array(), 'TYPO3\\CMS\\Workspaces\\Controller\\PreviewController', 'workspaces', 'web_workspacesworkspaces')) . $additionalParams;
+               $viewScript = $backPath . $uriBuilder->setArguments(array('tx_workspaces_web_workspacesworkspaces' => array('previewWS' => $GLOBALS['BE_USER']->workspace)))->uriFor('index', array(), 'TYPO3\\CMS\\Workspaces\\Controller\\PreviewController', 'workspaces', 'web_workspacesworkspaces') . $additionalParams;
                if ($addDomain === TRUE) {
-                       return ((\TYPO3\CMS\Backend\Utility\BackendUtility::getViewDomain($uid) . $redirect) . urlencode($viewScript)) . $uid;
+                       return \TYPO3\CMS\Backend\Utility\BackendUtility::getViewDomain($uid) . $redirect . urlencode($viewScript) . $uid;
                } else {
                        return $viewScript;
                }
index d348349..7f39d0e 100644 (file)
@@ -9,14 +9,14 @@ if (TYPO3_MODE == 'BE') {
 // Register the autopublishing task
 $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['scheduler']['tasks']['TYPO3\\CMS\\Workspaces\\Service\\AutoPublishServiceTask'] = array(
        'extension' => $_EXTKEY,
-       'title' => ('LLL:EXT:' . $_EXTKEY) . '/Resources/Private/Language/locallang_mod.xml:autopublishTask.name',
-       'description' => ('LLL:EXT:' . $_EXTKEY) . '/Resources/Private/Language/locallang_mod.xml:autopublishTask.description'
+       'title' => 'LLL:EXT:' . $_EXTKEY . '/Resources/Private/Language/locallang_mod.xml:autopublishTask.name',
+       'description' => 'LLL:EXT:' . $_EXTKEY . '/Resources/Private/Language/locallang_mod.xml:autopublishTask.description'
 );
 // Register the cleanup preview links task
 $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['scheduler']['tasks']['TYPO3\\CMS\\Workspaces\\Task\\CleanupPreviewLinkTask'] = array(
        'extension' => $_EXTKEY,
-       'title' => ('LLL:EXT:' . $_EXTKEY) . '/Resources/Private/Language/locallang_mod.xml:cleanupPreviewLinkTask.name',
-       'description' => ('LLL:EXT:' . $_EXTKEY) . '/Resources/Private/Language/locallang_mod.xml:cleanupPreviewLinkTask.description'
+       'title' => 'LLL:EXT:' . $_EXTKEY . '/Resources/Private/Language/locallang_mod.xml:cleanupPreviewLinkTask.name',
+       'description' => 'LLL:EXT:' . $_EXTKEY . '/Resources/Private/Language/locallang_mod.xml:cleanupPreviewLinkTask.description'
 );
 $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['t3lib/class.t3lib_tcemain.php']['processCmdmapClass']['workspaces'] = 'EXT:workspaces/Classes/Service/Tcemain.php:TYPO3\\CMS\\Workspaces\\Hook\\DataHandlerHook';
 $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['t3lib/class.t3lib_befunc.php']['viewOnClickClass']['workspaces'] = 'EXT:workspaces/Classes/Service/Befunc.php:TYPO3\\CMS\\Workspaces\\Hook\\BackendUtilityHook';
index 4d375e7..7d710e0 100644 (file)
@@ -12,7 +12,7 @@ if (TYPO3_MODE == 'BE' && !(TYPO3_REQUESTTYPE & TYPO3_REQUESTTYPE_INSTALL)) {
        ), array(
                'access' => 'user,group',
                'icon' => 'EXT:workspaces/Resources/Public/Images/moduleicon.gif',
-               'labels' => ('LLL:EXT:' . $_EXTKEY) . '/Resources/Private/Language/locallang_mod.xml',
+               'labels' => 'LLL:EXT:' . $_EXTKEY . '/Resources/Private/Language/locallang_mod.xml',
                'navigationComponentId' => 'typo3-pagetree'
        ));
        // register ExtDirect