[TASK] Remove superfluous parenthesis in sysexts
authorArvid Jakobsson <arvid.jakobsson@gmail.com>
Sun, 16 Sep 2012 20:30:54 +0000 (22:30 +0200)
committerJigal van Hemert <jigal@xs4all.nl>
Mon, 17 Sep 2012 20:41:51 +0000 (22:41 +0200)
Removes superfluous parenthesis in sysexts "saltedpasswords",
"scheduler", "setup" and "statictemplates".

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

Change-Id: Ie1e7d052f5d8dfac95d01a70ed750e69cf4bb7f6
Resolves: #40970
Related: #40095
Reviewed-on: http://review.typo3.org/14629
Reviewed-by: Thomas Loeffler
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert
44 files changed:
typo3/sysext/saltedpasswords/Classes/Salt/AbstractSalt.php
typo3/sysext/saltedpasswords/Classes/Salt/BlowfishSalt.php
typo3/sysext/saltedpasswords/Classes/Salt/Md5Salt.php
typo3/sysext/saltedpasswords/Classes/Salt/PhpassSalt.php
typo3/sysext/saltedpasswords/Classes/Salt/SaltFactory.php
typo3/sysext/saltedpasswords/Classes/SaltedPasswordService.php
typo3/sysext/saltedpasswords/Classes/Task/BulkUpdateFieldProvider.php
typo3/sysext/saltedpasswords/Classes/Task/BulkUpdateTask.php
typo3/sysext/saltedpasswords/Classes/Utility/ExtensionManagerConfigurationUtility.php
typo3/sysext/saltedpasswords/Classes/Utility/SaltedPasswordsUtility.php
typo3/sysext/saltedpasswords/Tests/Unit/Salts/BlowfishSaltTest.php
typo3/sysext/saltedpasswords/Tests/Unit/Salts/Md5SaltTest.php
typo3/sysext/saltedpasswords/Tests/Unit/Salts/PhpassSaltTest.php
typo3/sysext/saltedpasswords/Tests/Unit/Salts/SaltFactoryTest.php
typo3/sysext/saltedpasswords/ext_localconf.php
typo3/sysext/saltedpasswords/ext_tables.php
typo3/sysext/scheduler/Classes/Controller/SchedulerModuleController.php
typo3/sysext/scheduler/Classes/CronCommand/CronCommand.php
typo3/sysext/scheduler/Classes/CronCommand/NormalizeCommand.php
typo3/sysext/scheduler/Classes/Example/SleepTask.php
typo3/sysext/scheduler/Classes/Example/SleepTaskAdditionalFieldProvider.php
typo3/sysext/scheduler/Classes/Example/TestTask.php
typo3/sysext/scheduler/Classes/Example/TestTaskAdditionalFieldProvider.php
typo3/sysext/scheduler/Classes/Execution.php
typo3/sysext/scheduler/Classes/Scheduler.php
typo3/sysext/scheduler/Classes/Task.php
typo3/sysext/scheduler/Classes/Task/CachingFrameworkGarbageCollectionAdditionalFieldProvider.php
typo3/sysext/scheduler/Classes/Task/RecyclerGarbageCollectionAdditionalFieldProvider.php
typo3/sysext/scheduler/Classes/Task/RecyclerGarbageCollectionTask.php
typo3/sysext/scheduler/Classes/Task/TableGarbageCollectionAdditionalFieldProvider.php
typo3/sysext/scheduler/Classes/Task/TableGarbageCollectionTask.php
typo3/sysext/scheduler/Tests/Unit/Controller/SchedulerModuleControllerTest.php
typo3/sysext/scheduler/Tests/Unit/CronCommand/CronCommandTest.php
typo3/sysext/scheduler/cli/scheduler_cli_dispatch.php
typo3/sysext/scheduler/ext_localconf.php
typo3/sysext/scheduler/ext_tables.php
typo3/sysext/setup/Classes/Controller/SetupModuleController.php
typo3/sysext/statictemplates/Classes/StaticTemplatesHook.php
typo3/sysext/statictemplates/media/scripts/example_callfunction.php
typo3/sysext/statictemplates/media/scripts/example_itemArrayProcFunc.php
typo3/sysext/statictemplates/media/scripts/example_keepRollover.php
typo3/sysext/statictemplates/media/scripts/example_languageMenu.php
typo3/sysext/statictemplates/media/scripts/example_typolinkpop.php
typo3/sysext/statictemplates/media/scripts/testscript_EXT.php

index e131435..5fad478 100644 (file)
@@ -113,7 +113,7 @@ abstract class AbstractSalt {
         */
        protected function getLengthBase64FromBytes($byteLength) {
                // Calculates bytes in bits in base64
-               return intval(ceil(($byteLength * 8) / 6));
+               return intval(ceil($byteLength * 8 / 6));
        }
 
 }
index 193887d..392b9c9 100644 (file)
@@ -110,7 +110,7 @@ class BlowfishSalt extends \TYPO3\CMS\Saltedpasswords\Salt\Md5Salt {
                $reqLenBase64 = $this->getLengthBase64FromBytes($this->getSaltLength());
                // salt without setting
                if (strlen($salt) == $reqLenBase64) {
-                       $saltWithSettings = (($this->getSetting() . sprintf('%02u', $this->getHashCount())) . '$') . $salt;
+                       $saltWithSettings = $this->getSetting() . sprintf('%02u', $this->getHashCount()) . '$' . $salt;
                }
                return $saltWithSettings;
        }
@@ -126,7 +126,7 @@ class BlowfishSalt extends \TYPO3\CMS\Saltedpasswords\Salt\Md5Salt {
                $setting = substr($setting, strlen($this->getSetting()));
                $firstSplitPos = strpos($setting, '$');
                // Hashcount existing
-               if (($firstSplitPos !== FALSE && $firstSplitPos <= 2) && is_numeric(substr($setting, 0, $firstSplitPos))) {
+               if ($firstSplitPos !== FALSE && $firstSplitPos <= 2 && is_numeric(substr($setting, 0, $firstSplitPos))) {
                        $countLog2 = intval(substr($setting, 0, $firstSplitPos));
                }
                return $countLog2;
@@ -246,7 +246,7 @@ class BlowfishSalt extends \TYPO3\CMS\Saltedpasswords\Salt\Md5Salt {
                        }
                        // Checking base64 characters
                        if (!$skip && strlen($salt) >= $reqLenBase64) {
-                               if (preg_match(((((('/^[' . preg_quote($this->getItoa64(), '/')) . ']{') . $reqLenBase64) . ',') . $reqLenBase64) . '}$/', substr($salt, 0, $reqLenBase64))) {
+                               if (preg_match('/^[' . preg_quote($this->getItoa64(), '/') . ']{' . $reqLenBase64 . ',' . $reqLenBase64 . '}$/', substr($salt, 0, $reqLenBase64))) {
                                        $isValid = TRUE;
                                }
                        }
@@ -278,7 +278,7 @@ class BlowfishSalt extends \TYPO3\CMS\Saltedpasswords\Salt\Md5Salt {
         * @see getHashCount()
         */
        public function setHashCount($hashCount = NULL) {
-               self::$hashCount = ((!is_NULL($hashCount) && is_int($hashCount)) && $hashCount >= $this->getMinHashCount()) && $hashCount <= $this->getMaxHashCount() ? $hashCount : self::HASH_COUNT;
+               self::$hashCount = !is_NULL($hashCount) && is_int($hashCount) && $hashCount >= $this->getMinHashCount() && $hashCount <= $this->getMaxHashCount() ? $hashCount : self::HASH_COUNT;
        }
 
        /**
index 4ce2fe2..dc53d23 100644 (file)
@@ -81,7 +81,7 @@ class Md5Salt extends \TYPO3\CMS\Saltedpasswords\Salt\AbstractSalt implements \T
                $reqLenBase64 = $this->getLengthBase64FromBytes($this->getSaltLength());
                // Salt without setting
                if (strlen($salt) == $reqLenBase64) {
-                       $saltWithSettings = ($this->getSetting() . $salt) . $this->getSaltSuffix();
+                       $saltWithSettings = $this->getSetting() . $salt . $this->getSaltSuffix();
                }
                return $saltWithSettings;
        }
@@ -218,7 +218,7 @@ class Md5Salt extends \TYPO3\CMS\Saltedpasswords\Salt\AbstractSalt implements \T
                        }
                        // Checking base64 characters
                        if (!$skip && strlen($salt) >= $reqLenBase64) {
-                               if (preg_match(((((('/^[' . preg_quote($this->getItoa64(), '/')) . ']{') . $reqLenBase64) . ',') . $reqLenBase64) . '}$/', substr($salt, 0, $reqLenBase64))) {
+                               if (preg_match('/^[' . preg_quote($this->getItoa64(), '/') . ']{' . $reqLenBase64 . ',' . $reqLenBase64 . '}$/', substr($salt, 0, $reqLenBase64))) {
                                        $isValid = TRUE;
                                }
                        }
index d0dd85b..3ba45b4 100644 (file)
@@ -166,7 +166,7 @@ class PhpassSalt extends \TYPO3\CMS\Saltedpasswords\Salt\AbstractSalt implements
                $saltedPW = NULL;
                $reqLenBase64 = $this->getLengthBase64FromBytes($this->getSaltLength());
                // Retrieving settings with salt
-               $setting = substr($setting, 0, (strlen($this->getSetting()) + 1) + $reqLenBase64);
+               $setting = substr($setting, 0, strlen($this->getSetting()) + 1 + $reqLenBase64);
                $count_log2 = $this->getCountLog2($setting);
                // Hashes may be imported from elsewhere, so we allow != HASH_COUNT
                if ($count_log2 >= $this->getMinHashCount() && $count_log2 <= $this->getMaxHashCount()) {
@@ -337,7 +337,7 @@ class PhpassSalt extends \TYPO3\CMS\Saltedpasswords\Salt\AbstractSalt implements
                        }
                        // Checking base64 characters
                        if (!$skip && strlen($salt) >= $reqLenBase64) {
-                               if (preg_match(((((('/^[' . preg_quote($this->getItoa64(), '/')) . ']{') . $reqLenBase64) . ',') . $reqLenBase64) . '}$/', substr($salt, 0, $reqLenBase64))) {
+                               if (preg_match('/^[' . preg_quote($this->getItoa64(), '/') . ']{' . $reqLenBase64 . ',' . $reqLenBase64 . '}$/', substr($salt, 0, $reqLenBase64))) {
                                        $isValid = TRUE;
                                }
                        }
@@ -369,7 +369,7 @@ class PhpassSalt extends \TYPO3\CMS\Saltedpasswords\Salt\AbstractSalt implements
         * @see getHashCount()
         */
        public function setHashCount($hashCount = NULL) {
-               self::$hashCount = ((!is_NULL($hashCount) && is_int($hashCount)) && $hashCount >= $this->getMinHashCount()) && $hashCount <= $this->getMaxHashCount() ? $hashCount : self::HASH_COUNT;
+               self::$hashCount = !is_NULL($hashCount) && is_int($hashCount) && $hashCount >= $this->getMinHashCount() && $hashCount <= $this->getMaxHashCount() ? $hashCount : self::HASH_COUNT;
        }
 
        /**
index 32266c0..05d0c3b 100644 (file)
@@ -66,7 +66,7 @@ class SaltFactory {
                // * no instance existing
                // * a salted hash given to determine salted hashing method from
                // * a NULL parameter given to reset instance back to default method
-               if ((!is_object(self::$instance) || !empty($saltedHash)) || is_NULL($saltedHash)) {
+               if (!is_object(self::$instance) || !empty($saltedHash) || is_NULL($saltedHash)) {
                        // Determine method by checking the given hash
                        if (!empty($saltedHash)) {
                                $result = self::determineSaltingHashingMethod($saltedHash);
index 5a9e811..11b3430 100644 (file)
@@ -144,7 +144,7 @@ class SaltedPasswordService extends \TYPO3\CMS\Sv\AbstractAuthenticationService
                                $this->objInstanceSaltedPW = \TYPO3\CMS\Saltedpasswords\Salt\SaltFactory::getSaltingInstance(NULL);
                                $this->updatePassword(intval($user['uid']), array('password' => $this->objInstanceSaltedPW->getHashedPassword($password)));
                        }
-                       if (($validPasswd && !$skip) && $this->objInstanceSaltedPW->isHashUpdateNeeded($user['password'])) {
+                       if ($validPasswd && !$skip && $this->objInstanceSaltedPW->isHashUpdateNeeded($user['password'])) {
                                $this->updatePassword(intval($user['uid']), array('password' => $this->objInstanceSaltedPW->getHashedPassword($password)));
                        }
                } elseif (!intval($this->extConf['forceSalted'])) {
@@ -217,7 +217,7 @@ class SaltedPasswordService extends \TYPO3\CMS\Sv\AbstractAuthenticationService
                                if (intval($this->extConf['onlyAuthService']) || $this->authenticationFailed) {
                                        $OK = 0;
                                }
-                       } elseif (($validPasswd && $user['lockToDomain']) && strcasecmp($user['lockToDomain'], $this->authInfo['HTTP_HOST'])) {
+                       } elseif ($validPasswd && $user['lockToDomain'] && strcasecmp($user['lockToDomain'], $this->authInfo['HTTP_HOST'])) {
                                // Lock domain didn't match, so error:
                                $errorMessage = 'Login-attempt from %s (%s), username \'%s\', locked domain \'%s\' did not match \'%s\'!';
                                $this->writeLogMessage($errorMessage, $this->authInfo['REMOTE_ADDR'], $this->authInfo['REMOTE_HOST'], $this->login['uname'], $user['lockToDomain'], $this->authInfo['HTTP_HOST']);
index d415253..14cbe8f 100644 (file)
@@ -77,7 +77,7 @@ class BulkUpdateFieldProvider implements \TYPO3\CMS\Scheduler\AdditionalFieldPro
                $fieldId = 'task_saltedpasswordsBulkUpdateCanDeactivateSelf';
                $fieldValue = 'IsChecked';
                $fieldChecked = (bool) $taskInfo['scheduler_saltedpasswordsBulkUpdateCanDeactivateSelf'];
-               $fieldHtml = (((((((((('<input type="checkbox"' . ' name="') . $fieldName) . '"') . ' id="') . $fieldId) . '"') . ' value="') . $fieldValue) . '"') . ($fieldChecked ? ' checked="checked"' : '')) . ' />';
+               $fieldHtml = '<input type="checkbox"' . ' name="' . $fieldName . '"' . ' id="' . $fieldId . '"' . ' value="' . $fieldValue . '"' . ($fieldChecked ? ' checked="checked"' : '') . ' />';
                $additionalFields[$fieldId] = array(
                        'code' => $fieldHtml,
                        'label' => 'LLL:EXT:saltedpasswords/locallang.xml:ext.saltedpasswords.tasks.bulkupdate.label.canDeactivateSelf',
@@ -88,7 +88,7 @@ class BulkUpdateFieldProvider implements \TYPO3\CMS\Scheduler\AdditionalFieldPro
                $fieldName = 'TYPO3\\CMS\\Scheduler\\Scheduler[scheduler_saltedpasswordsBulkUpdateNumberOfRecords]';
                $fieldId = 'task_saltedpasswordsBulkUpdateNumberOfRecords';
                $fieldValue = intval($taskInfo['scheduler_saltedpasswordsBulkUpdateNumberOfRecords']);
-               $fieldHtml = ((((('<input type="text" name="' . $fieldName) . '" id="') . $fieldId) . '" value="') . htmlspecialchars($fieldValue)) . '" />';
+               $fieldHtml = '<input type="text" name="' . $fieldName . '" id="' . $fieldId . '" value="' . htmlspecialchars($fieldValue) . '" />';
                $additionalFields[$fieldId] = array(
                        'code' => $fieldHtml,
                        'label' => 'LLL:EXT:saltedpasswords/locallang.xml:ext.saltedpasswords.tasks.bulkupdate.label.numberOfRecords',
index cc0ebf6..e41689a 100644 (file)
@@ -108,7 +108,7 @@ class BulkUpdateTask extends \TYPO3\CMS\Scheduler\Task {
         * @return string Additional information
         */
        public function getAdditionalInformation() {
-               $information = ((($GLOBALS['LANG']->sL('LLL:EXT:saltedpasswords/locallang.xml:ext.saltedpasswords.tasks.bulkupdate.label.additionalinformation.deactivateself') . $this->getCanDeactivateSelf()) . '; ') . $GLOBALS['LANG']->sL('LLL:EXT:saltedpasswords/locallang.xml:ext.saltedpasswords.tasks.bulkupdate.label.additionalinformation.numberofrecords')) . $this->getNumberOfRecords();
+               $information = $GLOBALS['LANG']->sL('LLL:EXT:saltedpasswords/locallang.xml:ext.saltedpasswords.tasks.bulkupdate.label.additionalinformation.deactivateself') . $this->getCanDeactivateSelf() . '; ' . $GLOBALS['LANG']->sL('LLL:EXT:saltedpasswords/locallang.xml:ext.saltedpasswords.tasks.bulkupdate.label.additionalinformation.numberofrecords') . $this->getNumberOfRecords();
                return $information;
        }
 
@@ -119,7 +119,7 @@ class BulkUpdateTask extends \TYPO3\CMS\Scheduler\Task {
         * @return array Rows with uid and password
         */
        protected function findUsersToUpdate($mode) {
-               $usersToUpdate = $GLOBALS['TYPO3_DB']->exec_SELECTgetRows('uid, password', strtolower($mode) . '_users', '1 = 1', '', 'uid ASC', ($this->userRecordPointer[$mode] . ', ') . $this->numberOfRecords);
+               $usersToUpdate = $GLOBALS['TYPO3_DB']->exec_SELECTgetRows('uid, password', strtolower($mode) . '_users', '1 = 1', '', 'uid ASC', $this->userRecordPointer[$mode] . ', ' . $this->numberOfRecords);
                return $usersToUpdate;
        }
 
index 5e44b81..cc723c7 100644 (file)
@@ -136,7 +136,7 @@ your TYPO3 installation and the usability of the backend.';
                                        } else {
                                                // This means that login would fail because rsaauth is not working properly
                                                $this->setErrorLevel('error');
-                                               $problems[] = ('<strong>Using the extension "rsaauth" is not possible, as no encryption backend ' . 'is available. Please install and configure the PHP extension "openssl". ') . 'See <a href="http://php.net/manual/en/openssl.installation.php" target="_blank">PHP.net</a></strong>.';
+                                               $problems[] = '<strong>Using the extension "rsaauth" is not possible, as no encryption backend ' . 'is available. Please install and configure the PHP extension "openssl". ' . 'See <a href="http://php.net/manual/en/openssl.installation.php" target="_blank">PHP.net</a></strong>.';
                                        }
                                } else {
                                        // This means that we are not using saltedpasswords
@@ -269,7 +269,7 @@ Make sure that the Login Security Level is not set to "" or "challenged"!';
                                } else {
                                        // This means that login would fail because rsaauth is not working properly
                                        $this->setErrorLevel('error');
-                                       $problems[] = ('<strong>Using the extension "rsaauth" is not possible, as no encryption backend ' . 'is available. Please install and configure the PHP extension "openssl". ') . 'See <a href="http://php.net/manual/en/openssl.installation.php" target="_blank">PHP.net</a></strong>.';
+                                       $problems[] = '<strong>Using the extension "rsaauth" is not possible, as no encryption backend ' . 'is available. Please install and configure the PHP extension "openssl". ' . 'See <a href="http://php.net/manual/en/openssl.installation.php" target="_blank">PHP.net</a></strong>.';
                                }
                        }
                        // Only saltedpasswords as authsservice
@@ -329,17 +329,17 @@ Please disable either one of them.';
        protected function buildHashMethodSelector(array $params, $pObj, $disposal) {
                $this->init();
                $propertyName = $params['propertyName'];
-               $unknownVariablePleaseRenameMe = ('\'' . substr(md5($propertyName), 0, 10)) . '\'';
+               $unknownVariablePleaseRenameMe = '\'' . substr(md5($propertyName), 0, 10) . '\'';
                $p_field = '';
                foreach ($GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['ext/saltedpasswords']['saltMethods'] as $class => $reference) {
                        $classInstance = \TYPO3\CMS\Core\Utility\GeneralUtility::getUserObj($reference, 'tx_');
                        if ($classInstance instanceof \TYPO3\CMS\Saltedpasswords\Salt\SaltInterface && $classInstance->isAvailable()) {
                                $sel = $this->extConf[$disposal]['saltedPWHashingMethod'] == $class ? ' selected="selected" ' : '';
                                $label = 'ext.saltedpasswords.title.' . $class;
-                               $p_field .= ((((('<option value="' . htmlspecialchars($class)) . '"') . $sel) . '>') . $GLOBALS['LANG']->getLL($label)) . '</option>';
+                               $p_field .= '<option value="' . htmlspecialchars($class) . '"' . $sel . '>' . $GLOBALS['LANG']->getLL($label) . '</option>';
                        }
                }
-               $p_field = ((((((('<select id="' . $propertyName) . '" name="') . $params['fieldName']) . '" onChange="uFormUrl(') . $unknownVariablePleaseRenameMe) . ')">') . $p_field) . '</select>';
+               $p_field = '<select id="' . $propertyName . '" name="' . $params['fieldName'] . '" onChange="uFormUrl(' . $unknownVariablePleaseRenameMe . ')">' . $p_field . '</select>';
                return $p_field;
        }
 
index 126b227..5da6c36 100644 (file)
@@ -53,7 +53,7 @@ class SaltedPasswordsUtility {
         * @return integer
         */
        static public function getNumberOfBackendUsersWithInsecurePassword() {
-               $userCount = $GLOBALS['TYPO3_DB']->exec_SELECTcountRows('*', 'be_users', (('password NOT LIKE ' . $GLOBALS['TYPO3_DB']->fullQuoteStr('$%', 'be_users')) . ' AND password NOT LIKE ') . $GLOBALS['TYPO3_DB']->fullQuoteStr('M$%', 'be_users'));
+               $userCount = $GLOBALS['TYPO3_DB']->exec_SELECTcountRows('*', 'be_users', 'password NOT LIKE ' . $GLOBALS['TYPO3_DB']->fullQuoteStr('$%', 'be_users') . ' AND password NOT LIKE ' . $GLOBALS['TYPO3_DB']->fullQuoteStr('M$%', 'be_users'));
                return $userCount;
        }
 
index 6e03bed..597aeaa 100644 (file)
@@ -281,7 +281,7 @@ class BlowfishSaltTest extends \tx_phpunit_testcase {
                                break;
                        }
                }
-               $this->assertTrue($criticalPwLength == 0 || $criticalPwLength > 32, ('Duplicates of hashed passwords with plaintext password of length ' . $criticalPwLength) . '+.');
+               $this->assertTrue($criticalPwLength == 0 || $criticalPwLength > 32, 'Duplicates of hashed passwords with plaintext password of length ' . $criticalPwLength . '+.');
        }
 
        /**
index 1827c52..62653c0 100644 (file)
@@ -244,7 +244,7 @@ class Md5SaltTest extends \tx_phpunit_testcase {
                                break;
                        }
                }
-               $this->assertTrue($criticalPwLength == 0 || $criticalPwLength > 32, (($this->getWarningWhenMethodUnavailable() . 'Duplicates of hashed passwords with plaintext password of length ') . $criticalPwLength) . '+.');
+               $this->assertTrue($criticalPwLength == 0 || $criticalPwLength > 32, $this->getWarningWhenMethodUnavailable() . 'Duplicates of hashed passwords with plaintext password of length ' . $criticalPwLength . '+.');
        }
 
        /**
index 4f83281..e43a50a 100644 (file)
@@ -258,7 +258,7 @@ class PhpassSaltTest extends \tx_phpunit_testcase {
                                break;
                        }
                }
-               $this->assertTrue($criticalPwLength == 0 || $criticalPwLength > 32, ('Duplicates of hashed passwords with plaintext password of length ' . $criticalPwLength) . '+.');
+               $this->assertTrue($criticalPwLength == 0 || $criticalPwLength > 32, 'Duplicates of hashed passwords with plaintext password of length ' . $criticalPwLength . '+.');
        }
 
        /**
index 4f05984..dccb0be 100644 (file)
@@ -98,13 +98,13 @@ class SaltFactoryTest extends \tx_phpunit_testcase {
                // 3 Bytes should result in a 6 char length base64 encoded string
                // used for MD5 and PHPass salted hashing
                $byteLength = 3;
-               $reqLengthBase64 = intval(ceil(($byteLength * 8) / 6));
+               $reqLengthBase64 = intval(ceil($byteLength * 8 / 6));
                $randomBytes = \TYPO3\CMS\Core\Utility\GeneralUtility::generateRandomBytes($byteLength);
                $this->assertTrue(strlen($this->objectInstance->base64Encode($randomBytes, $byteLength)) == $reqLengthBase64);
                // 16 Bytes should result in a 22 char length base64 encoded string
                // used for Blowfish salted hashing
                $byteLength = 16;
-               $reqLengthBase64 = intval(ceil(($byteLength * 8) / 6));
+               $reqLengthBase64 = intval(ceil($byteLength * 8 / 6));
                $randomBytes = \TYPO3\CMS\Core\Utility\GeneralUtility::generateRandomBytes($byteLength);
                $this->assertTrue(strlen($this->objectInstance->base64Encode($randomBytes, $byteLength)) == $reqLengthBase64);
        }
index 89a2053..07bac98 100644 (file)
@@ -32,8 +32,8 @@ $TYPO3_CONF_VARS['BE']['showRefreshLoginPopup'] = 1;
 // Register bulk update task
 $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['scheduler']['tasks']['TYPO3\\CMS\\Saltedpasswords\\Task\\BulkUpdateTask'] = array(
        'extension' => $_EXTKEY,
-       'title' => ('LLL:EXT:' . $_EXTKEY) . '/locallang.xml:ext.saltedpasswords.tasks.bulkupdate.name',
-       'description' => ('LLL:EXT:' . $_EXTKEY) . '/locallang.xml:ext.saltedpasswords.tasks.bulkupdate.description',
+       'title' => 'LLL:EXT:' . $_EXTKEY . '/locallang.xml:ext.saltedpasswords.tasks.bulkupdate.name',
+       'description' => 'LLL:EXT:' . $_EXTKEY . '/locallang.xml:ext.saltedpasswords.tasks.bulkupdate.description',
        'additionalFields' => 'TYPO3\\CMS\\Saltedpasswords\\Task\\BulkUpdateFieldProvider'
 );
 ?>
index 0e746c8..e0c1590 100644 (file)
@@ -32,5 +32,5 @@ if (\TYPO3\CMS\Saltedpasswords\Utility\SaltedPasswordsUtility::isUsageEnabled('B
        $GLOBALS['TYPO3_USER_SETTINGS']['columns']['password2']['eval'] = '';
 }
 // Add context sensitive help (csh) for scheduler task
-\TYPO3\CMS\Core\Extension\ExtensionManager::addLLrefForTCAdescr('_txsaltedpasswords', ('EXT:' . $_EXTKEY) . '/locallang_csh_saltedpasswords.xml');
+\TYPO3\CMS\Core\Extension\ExtensionManager::addLLrefForTCAdescr('_txsaltedpasswords', 'EXT:' . $_EXTKEY . '/locallang_csh_saltedpasswords.xml');
 ?>
\ No newline at end of file
index 4132bdb..738f66a 100644 (file)
@@ -221,7 +221,7 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                        break;
                }
                // Wrap the content in a section
-               return $this->doc->section($sectionTitle, ('<div class="tx_scheduler_mod1">' . $content) . '</div>', 0, 1);
+               return $this->doc->section($sectionTitle, '<div class="tx_scheduler_mod1">' . $content . '</div>', 0, 1);
        }
 
        /**
@@ -316,7 +316,7 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                        $message = $GLOBALS['LANG']->getLL('msg.noLastRun');
                        $severity = \TYPO3\CMS\Core\Messaging\FlashMessage::WARNING;
                } else {
-                       if ((empty($lastRun['end']) || empty($lastRun['start'])) || empty($lastRun['type'])) {
+                       if (empty($lastRun['end']) || empty($lastRun['start']) || empty($lastRun['type'])) {
                                $message = $GLOBALS['LANG']->getLL('msg.incompleteLastRun');
                                $severity = \TYPO3\CMS\Core\Messaging\FlashMessage::WARNING;
                        } else {
@@ -336,13 +336,13 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                /** @var $flashMessage \TYPO3\CMS\Core\Messaging\FlashMessage */
                $flashMessage = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance('TYPO3\\CMS\\Core\\Messaging\\FlashMessage', $message, '', $severity);
                $content .= '<div class="info-block">';
-               $content .= ('<h3>' . $GLOBALS['LANG']->getLL('hdg.lastRun')) . '</h3>';
+               $content .= '<h3>' . $GLOBALS['LANG']->getLL('hdg.lastRun') . '</h3>';
                $content .= $flashMessage->render();
                $content .= '</div>';
                // Check CLI user
                $content .= '<div class="info-block">';
-               $content .= ('<h3>' . $GLOBALS['LANG']->getLL('hdg.schedulerUser')) . '</h3>';
-               $content .= ('<p>' . $GLOBALS['LANG']->getLL('msg.schedulerUser')) . '</p>';
+               $content .= '<h3>' . $GLOBALS['LANG']->getLL('hdg.schedulerUser') . '</h3>';
+               $content .= '<p>' . $GLOBALS['LANG']->getLL('msg.schedulerUser') . '</p>';
                $checkUser = $this->checkSchedulerUser();
                if ($checkUser == -1) {
                        $link = $GLOBALS['MCONF']['_'] . '&SET[function]=check&CMD=user';
@@ -368,8 +368,8 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                        $isExecutable = is_executable($script);
                }
                $content .= '<div class="info-block">';
-               $content .= ('<h3>' . $GLOBALS['LANG']->getLL('hdg.cliScript')) . '</h3>';
-               $content .= ('<p>' . sprintf($GLOBALS['LANG']->getLL('msg.cliScript'), $script)) . '</p>';
+               $content .= '<h3>' . $GLOBALS['LANG']->getLL('hdg.cliScript') . '</h3>';
+               $content .= '<p>' . sprintf($GLOBALS['LANG']->getLL('msg.cliScript'), $script) . '</p>';
                if ($isExecutable) {
                        $message = $GLOBALS['LANG']->getLL('msg.cliScriptExecutable');
                        $severity = \TYPO3\CMS\Core\Messaging\FlashMessage::OK;
@@ -421,12 +421,12 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                                $table[$tr][] = $classInfo['title'];
                                $table[$tr][] = $classInfo['extension'];
                                $table[$tr][] = $classInfo['description'];
-                               $link = ($GLOBALS['MCONF']['_'] . '&SET[function]=list&CMD=add&tx_scheduler[class]=') . $class;
-                               $table[$tr][] = ((((('<a href="' . htmlspecialchars($link)) . '" title="') . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_common.xml:new', TRUE)) . '" class="icon">') . \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('actions-document-new')) . '</a>';
+                               $link = $GLOBALS['MCONF']['_'] . '&SET[function]=list&CMD=add&tx_scheduler[class]=' . $class;
+                               $table[$tr][] = '<a href="' . htmlspecialchars($link) . '" title="' . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_common.xml:new', TRUE) . '" class="icon">' . \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('actions-document-new') . '</a>';
                                $tr++;
                        }
                        // Render the table and return it
-                       $content = ('<div>' . $GLOBALS['LANG']->getLL('msg.infoScreenIntro')) . '</div>';
+                       $content = '<div>' . $GLOBALS['LANG']->getLL('msg.infoScreenIntro') . '</div>';
                        $content .= $this->doc->spacer(5);
                        $content .= $this->doc->table($table, $tableLayout);
                }
@@ -441,12 +441,12 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
         */
        protected function displayServerTime() {
                // Get the current time, formatted
-               $dateFormat = (($GLOBALS['TYPO3_CONF_VARS']['SYS']['ddmmyy'] . ' ') . $GLOBALS['TYPO3_CONF_VARS']['SYS']['hhmm']) . ' T (e';
-               $now = ((date($dateFormat) . ', GMT ') . date('P')) . ')';
+               $dateFormat = $GLOBALS['TYPO3_CONF_VARS']['SYS']['ddmmyy'] . ' ' . $GLOBALS['TYPO3_CONF_VARS']['SYS']['hhmm'] . ' T (e';
+               $now = date($dateFormat) . ', GMT ' . date('P') . ')';
                // Display the help text
-               $serverTime = ('<h4>' . $GLOBALS['LANG']->getLL('label.serverTime')) . '</h4>';
-               $serverTime .= ('<p>' . $GLOBALS['LANG']->getLL('msg.serverTimeHelp')) . '</p>';
-               $serverTime .= ('<p>' . sprintf($GLOBALS['LANG']->getLL('msg.serverTime'), $now)) . '</p>';
+               $serverTime = '<h4>' . $GLOBALS['LANG']->getLL('label.serverTime') . '</h4>';
+               $serverTime .= '<p>' . $GLOBALS['LANG']->getLL('msg.serverTimeHelp') . '</p>';
+               $serverTime .= '<p>' . sprintf($GLOBALS['LANG']->getLL('msg.serverTime'), $now) . '</p>';
                return $serverTime;
        }
 
@@ -457,8 +457,8 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
         * @return string Progress bar markup
         */
        protected function renderTaskProgressBar($progress) {
-               $progressText .= (($GLOBALS['LANG']->getLL('status.progress') . ':&nbsp;') . $progress) . '%';
-               $progressBar = ((('<div class="progress"> <div class="bar" style="width: ' . $progress) . '%;">') . $progressText) . '</div> </div>';
+               $progressText .= $GLOBALS['LANG']->getLL('status.progress') . ':&nbsp;' . $progress . '%';
+               $progressBar = '<div class="progress"> <div class="bar" style="width: ' . $progress . '%;">' . $progressText . '</div> </div>';
                return $progressBar;
        }
 
@@ -652,17 +652,17 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                        $style = ' style="display: none"';
                }
                // Start rendering the add/edit form
-               $content .= ('<input type="hidden" name="tx_scheduler[uid]" value="' . $this->submittedData['uid']) . '" />';
-               $content .= ('<input type="hidden" name="previousCMD" value="' . $this->CMD) . '" />';
+               $content .= '<input type="hidden" name="tx_scheduler[uid]" value="' . $this->submittedData['uid'] . '" />';
+               $content .= '<input type="hidden" name="previousCMD" value="' . $this->CMD . '" />';
                $content .= '<input type="hidden" name="CMD" value="save" />';
                $table = array();
                $tr = 0;
                $defaultCell = array('<td class="td-input">', '</td>');
                // Disable checkbox
-               $label = ('<label for="task_disable">' . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_common.xml:disable')) . '</label>';
+               $label = '<label for="task_disable">' . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_common.xml:disable') . '</label>';
                $table[$tr][] = \TYPO3\CMS\Backend\Utility\BackendUtility::wrapInHelp($this->cshKey, 'task_disable', $label);
-               $table[$tr][] = ('<input type="hidden"   name="tx_scheduler[disable]" value="0" />
-                        <input type="checkbox" name="tx_scheduler[disable]" value="1" id="task_disable"' . ($taskInfo['disable'] == 1 ? ' checked="checked"' : '')) . ' />';
+               $table[$tr][] = '<input type="hidden"   name="tx_scheduler[disable]" value="0" />
+                        <input type="checkbox" name="tx_scheduler[disable]" value="1" id="task_disable"' . ($taskInfo['disable'] == 1 ? ' checked="checked"' : '') . ' />';
                $tableLayout[$tr] = array(
                        'tr' => array('<tr id="task_disable_row">', '</tr>'),
                        'defCol' => $defaultCell,
@@ -670,12 +670,12 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                );
                $tr++;
                // Task class selector
-               $label = ('<label for="task_class">' . $GLOBALS['LANG']->getLL('label.class')) . '</label>';
+               $label = '<label for="task_class">' . $GLOBALS['LANG']->getLL('label.class') . '</label>';
                $table[$tr][] = \TYPO3\CMS\Backend\Utility\BackendUtility::wrapInHelp($this->cshKey, 'task_class', $label);
                // On editing, don't allow changing of the task class, unless it was not valid
                if ($this->submittedData['uid'] > 0 && !empty($taskInfo['class'])) {
-                       $cell = (($registeredClasses[$taskInfo['class']]['title'] . ' (') . $registeredClasses[$taskInfo['class']]['extension']) . ')';
-                       $cell .= ('<input type="hidden" name="tx_scheduler[class]" id="task_class" value="' . $taskInfo['class']) . '" />';
+                       $cell = $registeredClasses[$taskInfo['class']]['title'] . ' (' . $registeredClasses[$taskInfo['class']]['extension'] . ')';
+                       $cell .= '<input type="hidden" name="tx_scheduler[class]" id="task_class" value="' . $taskInfo['class'] . '" />';
                } else {
                        $cell = '<select name="tx_scheduler[class]" id="task_class" class="wide" onchange="actOnChangedTaskClass(this)">';
                        // Group registered classes by classname
@@ -686,10 +686,10 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                        ksort($groupedClasses);
                        // Loop on all grouped classes to display a selector
                        foreach ($groupedClasses as $extension => $class) {
-                               $cell .= ('<optgroup label="' . $extension) . '">';
+                               $cell .= '<optgroup label="' . $extension . '">';
                                foreach ($groupedClasses[$extension] as $class => $classInfo) {
                                        $selected = $class == $taskInfo['class'] ? ' selected="selected"' : '';
-                                       $cell .= (((((((('<option value="' . $class) . '"') . 'title="') . $classInfo['description']) . '"') . $selected) . '>') . $classInfo['title']) . '</option>';
+                                       $cell .= '<option value="' . $class . '"' . 'title="' . $classInfo['description'] . '"' . $selected . '>' . $classInfo['title'] . '</option>';
                                }
                                $cell .= '</optgroup>';
                        }
@@ -704,9 +704,9 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                );
                $tr++;
                // Task type selector
-               $label = ('<label for="task_type">' . $GLOBALS['LANG']->getLL('label.type')) . '</label>';
+               $label = '<label for="task_type">' . $GLOBALS['LANG']->getLL('label.type') . '</label>';
                $table[$tr][] = \TYPO3\CMS\Backend\Utility\BackendUtility::wrapInHelp($this->cshKey, 'task_type', $label);
-               $table[$tr][] = (((((((((('<select name="tx_scheduler[type]" id="task_type" onchange="actOnChangedTaskType(this)">' . '<option value="1"') . ($taskInfo['type'] == 1 ? ' selected="selected"' : '')) . '>') . $GLOBALS['LANG']->getLL('label.type.single')) . '</option>') . '<option value="2"') . ($taskInfo['type'] == 2 ? ' selected="selected"' : '')) . '>') . $GLOBALS['LANG']->getLL('label.type.recurring')) . '</option>') . '</select>';
+               $table[$tr][] = '<select name="tx_scheduler[type]" id="task_type" onchange="actOnChangedTaskType(this)">' . '<option value="1"' . ($taskInfo['type'] == 1 ? ' selected="selected"' : '') . '>' . $GLOBALS['LANG']->getLL('label.type.single') . '</option>' . '<option value="2"' . ($taskInfo['type'] == 2 ? ' selected="selected"' : '') . '>' . $GLOBALS['LANG']->getLL('label.type.recurring') . '</option>' . '</select>';
                $tableLayout[$tr] = array(
                        'tr' => array('<tr id="task_type_row">', '</tr>'),
                        'defCol' => $defaultCell,
@@ -715,9 +715,9 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                $tr++;
                // Start date/time field
                // NOTE: datetime fields need a special id naming scheme
-               $label = ('<label for="tceforms-datetimefield-task_start">' . $GLOBALS['LANG']->getLL('label.start')) . '</label>';
+               $label = '<label for="tceforms-datetimefield-task_start">' . $GLOBALS['LANG']->getLL('label.start') . '</label>';
                $table[$tr][] = \TYPO3\CMS\Backend\Utility\BackendUtility::wrapInHelp($this->cshKey, 'task_start', $label);
-               $table[$tr][] = (('<input name="tx_scheduler[start]" type="text" id="tceforms-datetimefield-task_start" value="' . (empty($taskInfo['start']) ? '' : strftime('%H:%M %d-%m-%Y', $taskInfo['start']))) . '" />') . \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('actions-edit-pick-date', array(
+               $table[$tr][] = '<input name="tx_scheduler[start]" type="text" id="tceforms-datetimefield-task_start" value="' . (empty($taskInfo['start']) ? '' : strftime('%H:%M %d-%m-%Y', $taskInfo['start'])) . '" />' . \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('actions-edit-pick-date', array(
                        'style' => 'cursor:pointer;',
                        'id' => 'picker-tceforms-datetimefield-task_start'
                ));
@@ -729,36 +729,36 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                $tr++;
                // End date/time field
                // NOTE: datetime fields need a special id naming scheme
-               $label = ('<label for="tceforms-datetimefield-task_end">' . $GLOBALS['LANG']->getLL('label.end')) . '</label>';
+               $label = '<label for="tceforms-datetimefield-task_end">' . $GLOBALS['LANG']->getLL('label.end') . '</label>';
                $table[$tr][] = \TYPO3\CMS\Backend\Utility\BackendUtility::wrapInHelp($this->cshKey, 'task_end', $label);
-               $table[$tr][] = (('<input name="tx_scheduler[end]" type="text" id="tceforms-datetimefield-task_end" value="' . (empty($taskInfo['end']) ? '' : strftime('%H:%M %d-%m-%Y', $taskInfo['end']))) . '" />') . \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('actions-edit-pick-date', array(
+               $table[$tr][] = '<input name="tx_scheduler[end]" type="text" id="tceforms-datetimefield-task_end" value="' . (empty($taskInfo['end']) ? '' : strftime('%H:%M %d-%m-%Y', $taskInfo['end'])) . '" />' . \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('actions-edit-pick-date', array(
                        'style' => 'cursor:pointer;',
                        'id' => 'picker-tceforms-datetimefield-task_end'
                ));
                $tableLayout[$tr] = array(
-                       'tr' => array(('<tr id="task_end_row"' . $style) . '>', '</tr>'),
+                       'tr' => array('<tr id="task_end_row"' . $style . '>', '</tr>'),
                        'defCol' => $defaultCell,
                        '0' => array('<td class="td-label">', '</td>')
                );
                $tr++;
                // Frequency input field
-               $label = ('<label for="task_frequency">' . $GLOBALS['LANG']->getLL('label.frequency.long')) . '</label>';
+               $label = '<label for="task_frequency">' . $GLOBALS['LANG']->getLL('label.frequency.long') . '</label>';
                $table[$tr][] = \TYPO3\CMS\Backend\Utility\BackendUtility::wrapInHelp($this->cshKey, 'task_frequency', $label);
-               $cell = ('<input type="text" name="tx_scheduler[frequency]" id="task_frequency" value="' . htmlspecialchars($taskInfo['frequency'])) . '" />';
+               $cell = '<input type="text" name="tx_scheduler[frequency]" id="task_frequency" value="' . htmlspecialchars($taskInfo['frequency']) . '" />';
                $table[$tr][] = $cell;
                $tableLayout[$tr] = array(
-                       'tr' => array(('<tr id="task_frequency_row"' . $style) . '>', '</tr>'),
+                       'tr' => array('<tr id="task_frequency_row"' . $style . '>', '</tr>'),
                        'defCol' => $defaultCell,
                        '0' => array('<td class="td-label">', '</td>')
                );
                $tr++;
                // Multiple execution selector
-               $label = ('<label for="task_multiple">' . $GLOBALS['LANG']->getLL('label.parallel.long')) . '</label>';
+               $label = '<label for="task_multiple">' . $GLOBALS['LANG']->getLL('label.parallel.long') . '</label>';
                $table[$tr][] = \TYPO3\CMS\Backend\Utility\BackendUtility::wrapInHelp($this->cshKey, 'task_multiple', $label);
-               $table[$tr][] = ('<input type="hidden"   name="tx_scheduler[multiple]" value="0" />
-                        <input type="checkbox" name="tx_scheduler[multiple]" value="1" id="task_multiple"' . ($taskInfo['multiple'] == 1 ? ' checked="checked"' : '')) . ' />';
+               $table[$tr][] = '<input type="hidden"   name="tx_scheduler[multiple]" value="0" />
+                        <input type="checkbox" name="tx_scheduler[multiple]" value="1" id="task_multiple"' . ($taskInfo['multiple'] == 1 ? ' checked="checked"' : '') . ' />';
                $tableLayout[$tr] = array(
-                       'tr' => array(('<tr id="task_multiple_row"' . $style) . '>', '</tr>'),
+                       'tr' => array('<tr id="task_multiple_row"' . $style . '>', '</tr>'),
                        'defCol' => $defaultCell,
                        '0' => array('<td class="td-label">', '</td>')
                );
@@ -773,11 +773,11 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                        // Add each field to the display, if there are indeed any
                        if (isset($fields) && is_array($fields)) {
                                foreach ($fields as $fieldID => $fieldInfo) {
-                                       $label = ((('<label for="' . $fieldID) . '">') . $GLOBALS['LANG']->sL($fieldInfo['label'])) . '</label>';
+                                       $label = '<label for="' . $fieldID . '">' . $GLOBALS['LANG']->sL($fieldInfo['label']) . '</label>';
                                        $table[$tr][] = \TYPO3\CMS\Backend\Utility\BackendUtility::wrapInHelp($fieldInfo['cshKey'], $fieldInfo['cshLabel'], $label);
                                        $table[$tr][] = $fieldInfo['code'];
                                        $tableLayout[$tr] = array(
-                                               'tr' => array(((((('<tr id="' . $fieldID) . '_row"') . $additionalFieldsStyle) . ' class="extraFields extra_fields_') . $class) . '">', '</tr>'),
+                                               'tr' => array('<tr id="' . $fieldID . '_row"' . $additionalFieldsStyle . ' class="extraFields extra_fields_' . $class . '">', '</tr>'),
                                                'defCol' => $defaultCell,
                                                '0' => array('<td class="td-label">', '</td>')
                                        );
@@ -815,7 +815,7 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                                        // Try fetching the task
                                        $task = $this->scheduler->fetchTask($uid);
                                        $class = get_class($task);
-                                       $name = (($registeredClasses[$class]['title'] . ' (') . $registeredClasses[$class]['extension']) . ')';
+                                       $name = $registeredClasses[$class]['title'] . ' (' . $registeredClasses[$class]['extension'] . ')';
                                        // Now try to execute it and report on outcome
                                        try {
                                                $result = $this->scheduler->executeTask($task);
@@ -848,7 +848,7 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
         */
        protected function listTasks() {
                // Define display format for dates
-               $dateFormat = ($GLOBALS['TYPO3_CONF_VARS']['SYS']['ddmmyy'] . ' ') . $GLOBALS['TYPO3_CONF_VARS']['SYS']['hhmm'];
+               $dateFormat = $GLOBALS['TYPO3_CONF_VARS']['SYS']['ddmmyy'] . ' ' . $GLOBALS['TYPO3_CONF_VARS']['SYS']['hhmm'];
                $content = '';
                // Get list of registered classes
                $registeredClasses = self::getRegisteredClasses();
@@ -907,7 +907,7 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                        $table = array();
                        $tr = 0;
                        // Header row
-                       $table[$tr][] = ((('<a href="#" onclick="toggleCheckboxes();" title="' . $GLOBALS['LANG']->getLL('label.checkAll', TRUE)) . '" class="icon">') . \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('actions-document-select')) . '</a>';
+                       $table[$tr][] = '<a href="#" onclick="toggleCheckboxes();" title="' . $GLOBALS['LANG']->getLL('label.checkAll', TRUE) . '" class="icon">' . \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('actions-document-select') . '</a>';
                        $table[$tr][] = '&nbsp;';
                        $table[$tr][] = $GLOBALS['LANG']->getLL('label.id');
                        $table[$tr][] = $GLOBALS['LANG']->getLL('task');
@@ -920,9 +920,9 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                        // Loop on all tasks
                        while ($schedulerRecord = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res)) {
                                // Define action icons
-                               $editAction = ((((((('<a href="' . $GLOBALS['MCONF']['_']) . '&CMD=edit&tx_scheduler[uid]=') . $schedulerRecord['uid']) . '" title="') . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_common.xml:edit', TRUE)) . '" class="icon">') . \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('actions-document-open')) . '</a>';
-                               $deleteAction = ((((((((('<a href="' . $GLOBALS['MCONF']['_']) . '&CMD=delete&tx_scheduler[uid]=') . $schedulerRecord['uid']) . '" onclick="return confirm(\'') . $GLOBALS['LANG']->getLL('msg.delete')) . '\');" title="') . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_common.xml:delete', TRUE)) . '" class="icon">') . \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('actions-edit-delete')) . '</a>';
-                               $stopAction = ((((((((((('<a href="' . $GLOBALS['MCONF']['_']) . '&CMD=stop&tx_scheduler[uid]=') . $schedulerRecord['uid']) . '" onclick="return confirm(\'') . $GLOBALS['LANG']->getLL('msg.stop')) . '\');" title="') . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_common.xml:stop', TRUE)) . '" class="icon"><img ') . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg($this->backPath, (\TYPO3\CMS\Core\Extension\ExtensionManager::extRelPath('scheduler') . '/res/gfx/stop.png'))) . ' alt="') . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_common.xml:stop')) . '" /></a>';
+                               $editAction = '<a href="' . $GLOBALS['MCONF']['_'] . '&CMD=edit&tx_scheduler[uid]=' . $schedulerRecord['uid'] . '" title="' . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_common.xml:edit', TRUE) . '" class="icon">' . \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('actions-document-open') . '</a>';
+                               $deleteAction = '<a href="' . $GLOBALS['MCONF']['_'] . '&CMD=delete&tx_scheduler[uid]=' . $schedulerRecord['uid'] . '" onclick="return confirm(\'' . $GLOBALS['LANG']->getLL('msg.delete') . '\');" title="' . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_common.xml:delete', TRUE) . '" class="icon">' . \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('actions-edit-delete') . '</a>';
+                               $stopAction = '<a href="' . $GLOBALS['MCONF']['_'] . '&CMD=stop&tx_scheduler[uid]=' . $schedulerRecord['uid'] . '" onclick="return confirm(\'' . $GLOBALS['LANG']->getLL('msg.stop') . '\');" title="' . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_common.xml:stop', TRUE) . '" class="icon"><img ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg($this->backPath, (\TYPO3\CMS\Core\Extension\ExtensionManager::extRelPath('scheduler') . '/res/gfx/stop.png')) . ' alt="' . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_common.xml:stop') . '" /></a>';
                                // Define some default values
                                $lastExecution = '-';
                                $isRunning = FALSE;
@@ -946,11 +946,11 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                                        } else {
                                                $context = $GLOBALS['LANG']->getLL('label.manual');
                                        }
-                                       $lastExecution .= (' (' . $context) . ')';
+                                       $lastExecution .= ' (' . $context . ')';
                                }
                                if ($this->scheduler->isValidTaskObject($task)) {
                                        // The task object is valid
-                                       $name = htmlspecialchars((($registeredClasses[$schedulerRecord['classname']]['title'] . ' (') . $registeredClasses[$schedulerRecord['classname']]['extension']) . ')');
+                                       $name = htmlspecialchars($registeredClasses[$schedulerRecord['classname']]['title'] . ' (' . $registeredClasses[$schedulerRecord['classname']]['extension'] . ')');
                                        $name .= '<br /> ';
                                        $additionalInformation = $task->getAdditionalInformation();
                                        if ($task instanceof \TYPO3\CMS\Scheduler\ProgressProviderInterface) {
@@ -958,7 +958,7 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                                                $name .= $this->renderTaskProgressBar($progress);
                                        }
                                        if (!empty($additionalInformation)) {
-                                               $name .= ('[' . htmlspecialchars($additionalInformation)) . ']';
+                                               $name .= '[' . htmlspecialchars($additionalInformation) . ']';
                                        }
                                        // Check if task currently has a running execution
                                        if (!empty($schedulerRecord['serialized_executions'])) {
@@ -976,7 +976,7 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                                                        $nextDate = $GLOBALS['LANG']->getLL('none');
                                                } elseif ($schedulerRecord['nextexecution'] < $GLOBALS['EXEC_TIME']) {
                                                        // Next execution is overdue, highlight date
-                                                       $nextDate = ((('<span class="late" title="' . $GLOBALS['LANG']->getLL('status.legend.scheduled')) . '">') . $nextDate) . '</span>';
+                                                       $nextDate = '<span class="late" title="' . $GLOBALS['LANG']->getLL('status.legend.scheduled') . '">' . $nextDate . '</span>';
                                                        $executionStatus = 'late';
                                                }
                                        }
@@ -999,7 +999,7 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                                                $multiple = $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_common.xml:no');
                                        }
                                        // Define checkbox
-                                       $startExecutionElement = ((('<input type="checkbox" name="TYPO3\\CMS\\Scheduler\\Scheduler[execute][]" value="' . $schedulerRecord['uid']) . '" id="task_') . $schedulerRecord['uid']) . '" class="checkboxes" />';
+                                       $startExecutionElement = '<input type="checkbox" name="TYPO3\\CMS\\Scheduler\\Scheduler[execute][]" value="' . $schedulerRecord['uid'] . '" id="task_' . $schedulerRecord['uid'] . '" class="checkboxes" />';
                                        // Show no action links (edit, delete) if task is running
                                        $actions = $editAction . $deleteAction;
                                        if ($isRunning) {
@@ -1023,11 +1023,11 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                                                } else {
                                                        $failureDetail = sprintf($GLOBALS['LANG']->getLL('msg.executionFailureReport'), $exception->getCode(), $exception->getMessage());
                                                }
-                                               $failureOutput = (((((' <img ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(\TYPO3\CMS\Core\Extension\ExtensionManager::extRelPath('scheduler'), 'res/gfx/status_failure.png')) . ' alt="') . htmlspecialchars($GLOBALS['LANG']->getLL('status.failure'))) . '" title="') . htmlspecialchars($failureDetail)) . '" />';
+                                               $failureOutput = ' <img ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(\TYPO3\CMS\Core\Extension\ExtensionManager::extRelPath('scheduler'), 'res/gfx/status_failure.png') . ' alt="' . htmlspecialchars($GLOBALS['LANG']->getLL('status.failure')) . '" title="' . htmlspecialchars($failureDetail) . '" />';
                                        }
                                        // Format the execution status,
                                        // including failure feedback, if any
-                                       $executionStatusOutput = (((((((('<img ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(\TYPO3\CMS\Core\Extension\ExtensionManager::extRelPath('scheduler'), (('res/gfx/status_' . $executionStatus) . '.png'))) . ' id="executionstatus_') . $schedulerRecord['uid']) . '" alt="') . htmlspecialchars($GLOBALS['LANG']->getLL(('status.' . $executionStatus)))) . '" title="') . htmlspecialchars($GLOBALS['LANG']->getLL(('status.legend.' . $executionStatus)))) . '" />') . $failureOutput;
+                                       $executionStatusOutput = '<img ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(\TYPO3\CMS\Core\Extension\ExtensionManager::extRelPath('scheduler'), ('res/gfx/status_' . $executionStatus . '.png')) . ' id="executionstatus_' . $schedulerRecord['uid'] . '" alt="' . htmlspecialchars($GLOBALS['LANG']->getLL(('status.' . $executionStatus))) . '" title="' . htmlspecialchars($GLOBALS['LANG']->getLL(('status.legend.' . $executionStatus))) . '" />' . $failureOutput;
                                        $table[$tr][] = $startExecutionElement;
                                        $table[$tr][] = $actions;
                                        $table[$tr][] = $schedulerRecord['uid'];
@@ -1054,7 +1054,7 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                        }
                        // Render table
                        $content .= $this->doc->table($table, $tableLayout);
-                       $content .= ('<input type="submit" class="button" name="go" id="scheduler_executeselected" value="' . $GLOBALS['LANG']->getLL('label.executeSelected')) . '" />';
+                       $content .= '<input type="submit" class="button" name="go" id="scheduler_executeselected" value="' . $GLOBALS['LANG']->getLL('label.executeSelected') . '" />';
                }
                if (!count($registeredClasses) > 0) {
                        /** @var \TYPO3\CMS\Core\Messaging\FlashMessage $flashMessage */
@@ -1065,13 +1065,13 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                // Also display information about the usage of server time
                if ($numRows > 0) {
                        $content .= $this->doc->spacer(20);
-                       $content .= ((((((((((((((((((((((((((((((((((((((((('<h4>' . $GLOBALS['LANG']->getLL('status.legend')) . '</h4>
+                       $content .= '<h4>' . $GLOBALS['LANG']->getLL('status.legend') . '</h4>
                        <ul>
-                               <li><img ') . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(\TYPO3\CMS\Core\Extension\ExtensionManager::extRelPath('scheduler'), 'res/gfx/status_failure.png')) . ' alt="') . htmlspecialchars($GLOBALS['LANG']->getLL('status.failure'))) . '" title="') . htmlspecialchars($GLOBALS['LANG']->getLL('status.failure'))) . '" /> ') . $GLOBALS['LANG']->getLL('status.legend.failure')) . '</li>
-                               <li><img ') . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(\TYPO3\CMS\Core\Extension\ExtensionManager::extRelPath('scheduler'), 'res/gfx/status_late.png')) . ' alt="') . htmlspecialchars($GLOBALS['LANG']->getLL('status.late'))) . '" title="') . htmlspecialchars($GLOBALS['LANG']->getLL('status.late'))) . '" /> ') . $GLOBALS['LANG']->getLL('status.legend.late')) . '</li>
-                               <li><img ') . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(\TYPO3\CMS\Core\Extension\ExtensionManager::extRelPath('scheduler'), 'res/gfx/status_running.png')) . ' alt="') . htmlspecialchars($GLOBALS['LANG']->getLL('status.running'))) . '" title="') . htmlspecialchars($GLOBALS['LANG']->getLL('status.running'))) . '" /> ') . $GLOBALS['LANG']->getLL('status.legend.running')) . '</li>
-                               <li><img ') . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(\TYPO3\CMS\Core\Extension\ExtensionManager::extRelPath('scheduler'), 'res/gfx/status_scheduled.png')) . ' alt="') . htmlspecialchars($GLOBALS['LANG']->getLL('status.scheduled'))) . '" title="') . htmlspecialchars($GLOBALS['LANG']->getLL('status.scheduled'))) . '" /> ') . $GLOBALS['LANG']->getLL('status.legend.scheduled')) . '</li>
-                               <li><img ') . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(\TYPO3\CMS\Core\Extension\ExtensionManager::extRelPath('scheduler'), 'res/gfx/status_disabled.png')) . ' alt="') . htmlspecialchars($GLOBALS['LANG']->getLL('status.disabled'))) . '" title="') . htmlspecialchars($GLOBALS['LANG']->getLL('status.disabled'))) . '" /> ') . $GLOBALS['LANG']->getLL('status.legend.disabled')) . '</li>
+                               <li><img ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(\TYPO3\CMS\Core\Extension\ExtensionManager::extRelPath('scheduler'), 'res/gfx/status_failure.png') . ' alt="' . htmlspecialchars($GLOBALS['LANG']->getLL('status.failure')) . '" title="' . htmlspecialchars($GLOBALS['LANG']->getLL('status.failure')) . '" /> ' . $GLOBALS['LANG']->getLL('status.legend.failure') . '</li>
+                               <li><img ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(\TYPO3\CMS\Core\Extension\ExtensionManager::extRelPath('scheduler'), 'res/gfx/status_late.png') . ' alt="' . htmlspecialchars($GLOBALS['LANG']->getLL('status.late')) . '" title="' . htmlspecialchars($GLOBALS['LANG']->getLL('status.late')) . '" /> ' . $GLOBALS['LANG']->getLL('status.legend.late') . '</li>
+                               <li><img ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(\TYPO3\CMS\Core\Extension\ExtensionManager::extRelPath('scheduler'), 'res/gfx/status_running.png') . ' alt="' . htmlspecialchars($GLOBALS['LANG']->getLL('status.running')) . '" title="' . htmlspecialchars($GLOBALS['LANG']->getLL('status.running')) . '" /> ' . $GLOBALS['LANG']->getLL('status.legend.running') . '</li>
+                               <li><img ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(\TYPO3\CMS\Core\Extension\ExtensionManager::extRelPath('scheduler'), 'res/gfx/status_scheduled.png') . ' alt="' . htmlspecialchars($GLOBALS['LANG']->getLL('status.scheduled')) . '" title="' . htmlspecialchars($GLOBALS['LANG']->getLL('status.scheduled')) . '" /> ' . $GLOBALS['LANG']->getLL('status.legend.scheduled') . '</li>
+                               <li><img ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg(\TYPO3\CMS\Core\Extension\ExtensionManager::extRelPath('scheduler'), 'res/gfx/status_disabled.png') . ' alt="' . htmlspecialchars($GLOBALS['LANG']->getLL('status.disabled')) . '" title="' . htmlspecialchars($GLOBALS['LANG']->getLL('status.disabled')) . '" /> ' . $GLOBALS['LANG']->getLL('status.legend.disabled') . '</li>
                        </ul>';
                        $content .= $this->doc->spacer(10);
                        $content .= $this->displayServerTime();
@@ -1285,7 +1285,7 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                        }
                        // If the timestamp is still FALSE, throw an exception
                        if ($timestamp === FALSE) {
-                               throw new \InvalidArgumentException(('"' . $string) . '" seems not to be a correct date.', 1294587694);
+                               throw new \InvalidArgumentException('"' . $string . '" seems not to be a correct date.', 1294587694);
                        }
                }
                return $timestamp;
@@ -1381,17 +1381,17 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                        'reload' => '',
                        'shortcut' => $this->getShortcutButton()
                );
-               if ((empty($this->CMD) || $this->CMD == 'list') || $this->CMD == 'delete') {
-                       $buttons['reload'] = ((((('<a href="' . $GLOBALS['MCONF']['_']) . '" title="') . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_core.php:labels.reload', TRUE)) . '">') . \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('actions-system-refresh')) . '</a>';
+               if (empty($this->CMD) || $this->CMD == 'list' || $this->CMD == 'delete') {
+                       $buttons['reload'] = '<a href="' . $GLOBALS['MCONF']['_'] . '" title="' . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_core.php:labels.reload', TRUE) . '">' . \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('actions-system-refresh') . '</a>';
                        if ($this->MOD_SETTINGS['function'] === 'scheduler' && count(self::getRegisteredClasses())) {
                                $link = $GLOBALS['MCONF']['_'] . '&CMD=add';
-                               $image = ((('<img ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg($this->backPath, 'gfx/new_el.gif')) . ' alt="') . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_common.xml:new', TRUE)) . '" />';
-                               $buttons['addtask'] = (((((('<a href="' . htmlspecialchars($link)) . '" ') . 'title="') . $GLOBALS['LANG']->getLL('action.add')) . '">') . $image) . '</a>';
+                               $image = '<img ' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg($this->backPath, 'gfx/new_el.gif') . ' alt="' . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_common.xml:new', TRUE) . '" />';
+                               $buttons['addtask'] = '<a href="' . htmlspecialchars($link) . '" ' . 'title="' . $GLOBALS['LANG']->getLL('action.add') . '">' . $image . '</a>';
                        }
                }
                if ($this->CMD === 'add' || $this->CMD === 'edit') {
-                       $buttons['close'] = ((((('<a href="#" onclick="document.location=\'' . $GLOBALS['MCONF']['_']) . '\'" title="') . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_common.xml:cancel', TRUE)) . '">') . \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('actions-document-close')) . '</a>';
-                       $buttons['save'] = \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('actions-document-save-close', array('html' => ('<input type="image" name="data[save]" class="c-inputButton" src="clear.gif" title="' . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_common.xml:saveAndClose', TRUE)) . '" />'));
+                       $buttons['close'] = '<a href="#" onclick="document.location=\'' . $GLOBALS['MCONF']['_'] . '\'" title="' . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_common.xml:cancel', TRUE) . '">' . \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('actions-document-close') . '</a>';
+                       $buttons['save'] = \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('actions-document-save-close', array('html' => '<input type="image" name="data[save]" class="c-inputButton" src="clear.gif" title="' . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_common.xml:saveAndClose', TRUE) . '" />'));
                }
                return $buttons;
        }
index 7fa9d97..e31c696 100644 (file)
@@ -177,7 +177,7 @@ class CronCommand {
                $isDayOfWeekRestricted = (string) $this->cronCommandSections[4] === '*' ? FALSE : TRUE;
                $commandMatch = FALSE;
                if ($isInMonth) {
-                       if (($isInDayOfMonth && $isDayOfMonthRestricted || $isInDayOfWeek && $isDayOfWeekRestricted) || (($isInDayOfMonth && !$isDayOfMonthRestricted) && $isInDayOfWeek) && !$isDayOfWeekRestricted) {
+                       if ($isInDayOfMonth && $isDayOfMonthRestricted || $isInDayOfWeek && $isDayOfWeekRestricted || $isInDayOfMonth && !$isDayOfMonthRestricted && $isInDayOfWeek && !$isDayOfWeekRestricted) {
                                $commandMatch = TRUE;
                        }
                }
@@ -217,7 +217,7 @@ class CronCommand {
        protected function numberOfSecondsInDay($timestamp) {
                $now = mktime(0, 0, 0, date('n', $timestamp), date('j', $timestamp), date('Y', $timestamp));
                // Make sure to be in next day, even if day has 25 hours
-               $nextDay = $now + (60 * 60) * 25;
+               $nextDay = $now + 60 * 60 * 25;
                $nextDay = mktime(0, 0, 0, date('n', $nextDay), date('j', $nextDay), date('Y', $nextDay));
                return $nextDay - $now;
        }
index a70e99f..4ad0f6b 100644 (file)
@@ -140,18 +140,18 @@ class NormalizeCommand {
                                                list($leftBound, $rightBound) = explode('-', $left);
                                                $leftBound = self::normalizeMonthAndWeekday($leftBound, $isMonthField);
                                                $rightBound = self::normalizeMonthAndWeekday($rightBound, $isMonthField);
-                                               $left = ($leftBound . '-') . $rightBound;
+                                               $left = $leftBound . '-' . $rightBound;
                                        } else {
                                                if ((string) $left !== '*') {
                                                        $left = self::normalizeMonthAndWeekday($left, $isMonthField);
                                                }
                                        }
-                                       $fieldArray[] = ($left . '/') . $right;
+                                       $fieldArray[] = $left . '/' . $right;
                                } elseif (strpos($listElement, '-') !== FALSE) {
                                        list($left, $right) = explode('-', $listElement);
                                        $left = self::normalizeMonthAndWeekday($left, $isMonthField);
                                        $right = self::normalizeMonthAndWeekday($right, $isMonthField);
-                                       $fieldArray[] = ($left . '-') . $right;
+                                       $fieldArray[] = $left . '-' . $right;
                                } else {
                                        $fieldArray[] = self::normalizeMonthAndWeekday($listElement, $isMonthField);
                                }
@@ -180,11 +180,11 @@ class NormalizeCommand {
                                if (strpos($listElement, '/') !== FALSE) {
                                        list($left, $right) = explode('/', $listElement);
                                        if ((string) $left === '*') {
-                                               $leftList = self::convertRangeToListOfValues(($lowerBound . '-') . $upperBound);
+                                               $leftList = self::convertRangeToListOfValues($lowerBound . '-' . $upperBound);
                                        } else {
                                                $leftList = self::convertRangeToListOfValues($left);
                                        }
-                                       $fieldArray[] = self::reduceListOfValuesByStepValue(($leftList . '/') . $right);
+                                       $fieldArray[] = self::reduceListOfValuesByStepValue($leftList . '/' . $right);
                                } elseif (strpos($listElement, '-') !== FALSE) {
                                        $fieldArray[] = self::convertRangeToListOfValues($listElement);
                                } elseif (strcmp(intval($listElement), $listElement) === 0) {
@@ -324,9 +324,9 @@ class NormalizeCommand {
         * @return integer month integer representation between 1 and 12
         */
        static protected function normalizeMonth($month) {
-               $timestamp = strtotime(('2010-' . $month) . '-01');
+               $timestamp = strtotime('2010-' . $month . '-01');
                // timestamp must be >= 2010-01-01 and <= 2010-12-01
-               if ((!$timestamp || $timestamp < strtotime('2010-01-01')) || $timestamp > strtotime('2010-12-01')) {
+               if (!$timestamp || $timestamp < strtotime('2010-01-01') || $timestamp > strtotime('2010-12-01')) {
                        throw new \InvalidArgumentException('Unable to convert given month name.', 1291083486);
                }
                return (int) date('n', $timestamp);
@@ -352,7 +352,7 @@ class NormalizeCommand {
                if (!$normalizedWeekday) {
                        // Convert string representation like 'sun' to integer
                        $timestamp = strtotime('next ' . $weekday, mktime(0, 0, 0, 1, 1, 2010));
-                       if ((!$timestamp || $timestamp < strtotime('2010-01-01')) || $timestamp > strtotime('2010-01-08')) {
+                       if (!$timestamp || $timestamp < strtotime('2010-01-01') || $timestamp > strtotime('2010-01-08')) {
                                throw new \InvalidArgumentException('Unable to convert given weekday name.', 1291163589);
                        }
                        $normalizedWeekday = (int) date('N', $timestamp);
index 404f855..d822f79 100644 (file)
@@ -61,7 +61,7 @@ class SleepTask extends \TYPO3\CMS\Scheduler\Task {
         * @return string Information to display
         */
        public function getAdditionalInformation() {
-               return ($GLOBALS['LANG']->sL('LLL:EXT:scheduler/mod1/locallang.xml:label.sleepTime') . ': ') . $this->sleepTime;
+               return $GLOBALS['LANG']->sL('LLL:EXT:scheduler/mod1/locallang.xml:label.sleepTime') . ': ' . $this->sleepTime;
        }
 
 }
index 1cc1ca0..79ac74c 100644 (file)
@@ -57,7 +57,7 @@ class SleepTaskAdditionalFieldProvider implements \TYPO3\CMS\Scheduler\Additiona
                }
                // Write the code for the field
                $fieldID = 'task_sleepTime';
-               $fieldCode = ((('<input type="text" name="TYPO3\\CMS\\Scheduler\\Scheduler[sleepTime]" id="' . $fieldID) . '" value="') . $taskInfo['sleepTime']) . '" size="10" />';
+               $fieldCode = '<input type="text" name="TYPO3\\CMS\\Scheduler\\Scheduler[sleepTime]" id="' . $fieldID . '" value="' . $taskInfo['sleepTime'] . '" size="10" />';
                $additionalFields = array();
                $additionalFields[$fieldID] = array(
                        'code' => $fieldCode,
index 4e40337..07c1d33 100644 (file)
@@ -52,7 +52,7 @@ class TestTask extends \TYPO3\CMS\Scheduler\Task {
                        // If an email address is defined, send a message to it
                        // NOTE: the TYPO3_DLOG constant is not used in this case, as this is a test task
                        // and debugging is its main purpose anyway
-                       \TYPO3\CMS\Core\Utility\GeneralUtility::devLog(('[TYPO3\\CMS\\Scheduler\\Example\\TestTask]: Test email sent to "' . $this->email) . '"', 'scheduler', 0);
+                       \TYPO3\CMS\Core\Utility\GeneralUtility::devLog('[TYPO3\\CMS\\Scheduler\\Example\\TestTask]: Test email sent to "' . $this->email . '"', 'scheduler', 0);
                        // Get execution information
                        $exec = $this->getExecution();
                        // Get call method
@@ -68,7 +68,7 @@ class TestTask extends \TYPO3\CMS\Scheduler\Task {
                        $interval = $exec->getInterval();
                        $multiple = $exec->getMultiple();
                        $cronCmd = $exec->getCronCmd();
-                       $mailBody = (((((((((((((((((((((((((((((((((((((((('SCHEDULER TEST-TASK' . LF) . '- - - - - - - - - - - - - - - -') . LF) . 'UID: ') . $this->taskUid) . LF) . 'Sitename: ') . $GLOBALS['TYPO3_CONF_VARS']['SYS']['sitename']) . LF) . 'Site: ') . $site) . LF) . 'Called by: ') . $calledBy) . LF) . 'tstamp: ') . date('Y-m-d H:i:s')) . ' [') . time()) . ']') . LF) . 'maxLifetime: ') . $this->scheduler->extConf['maxLifetime']) . LF) . 'start: ') . date('Y-m-d H:i:s', $start)) . ' [') . $start) . ']') . LF) . 'end: ') . (empty($end) ? '-' : ((date('Y-m-d H:i:s', $end) . ' [') . $end) . ']')) . LF) . 'interval: ') . $interval) . LF) . 'multiple: ') . ($multiple ? 'yes' : 'no')) . LF) . 'cronCmd: ') . ($cronCmd ? $cronCmd : 'not used');
+                       $mailBody = 'SCHEDULER TEST-TASK' . LF . '- - - - - - - - - - - - - - - -' . LF . 'UID: ' . $this->taskUid . LF . 'Sitename: ' . $GLOBALS['TYPO3_CONF_VARS']['SYS']['sitename'] . LF . 'Site: ' . $site . LF . 'Called by: ' . $calledBy . LF . 'tstamp: ' . date('Y-m-d H:i:s') . ' [' . time() . ']' . LF . 'maxLifetime: ' . $this->scheduler->extConf['maxLifetime'] . LF . 'start: ' . date('Y-m-d H:i:s', $start) . ' [' . $start . ']' . LF . 'end: ' . (empty($end) ? '-' : date('Y-m-d H:i:s', $end) . ' [' . $end . ']') . LF . 'interval: ' . $interval . LF . 'multiple: ' . ($multiple ? 'yes' : 'no') . LF . 'cronCmd: ' . ($cronCmd ? $cronCmd : 'not used');
                        // Prepare mailer and send the mail
                        try {
                                /** @var $mailer t3lib_mail_message */
@@ -96,7 +96,7 @@ class TestTask extends \TYPO3\CMS\Scheduler\Task {
         * @return string Information to display
         */
        public function getAdditionalInformation() {
-               return ($GLOBALS['LANG']->sL('LLL:EXT:scheduler/mod1/locallang.xml:label.email') . ': ') . $this->email;
+               return $GLOBALS['LANG']->sL('LLL:EXT:scheduler/mod1/locallang.xml:label.email') . ': ' . $this->email;
        }
 
 }
index 18573f3..28028e3 100644 (file)
@@ -57,7 +57,7 @@ class TestTaskAdditionalFieldProvider implements \TYPO3\CMS\Scheduler\Additional
                }
                // Write the code for the field
                $fieldID = 'task_email';
-               $fieldCode = ((('<input type="text" name="TYPO3\\CMS\\Scheduler\\Scheduler[email]" id="' . $fieldID) . '" value="') . htmlspecialchars($taskInfo['email'])) . '" size="30" />';
+               $fieldCode = '<input type="text" name="TYPO3\\CMS\\Scheduler\\Scheduler[email]" id="' . $fieldID . '" value="' . htmlspecialchars($taskInfo['email']) . '" size="30" />';
                $additionalFields = array();
                $additionalFields[$fieldID] = array(
                        'code' => $fieldCode,
index cfd3aed..1e0985e 100644 (file)
@@ -231,7 +231,7 @@ class Execution {
                                } else {
                                        // Otherwise calculate date based on interval
                                        $now = time();
-                                       $date = ($now + $this->interval) - ($now - $this->start) % $this->interval;
+                                       $date = $now + $this->interval - ($now - $this->start) % $this->interval;
                                }
                                // If date is in the future, throw an exception
                                if (!empty($this->end) && $date > $this->end) {
index d6939dc..05da6f5 100644 (file)
@@ -108,7 +108,7 @@ class Scheduler implements \TYPO3\CMS\Core\SingletonInterface {
                                        if ($tstamp - $task < $maxDuration) {
                                                $executions[] = $task;
                                        } else {
-                                               $logMessage = (((('Removing logged execution, assuming that the process is dead. Execution of \'' . $row['classname']) . '\' (UID: ') . $row['uid']) . ') was started at ') . date('Y-m-d H:i:s', $task);
+                                               $logMessage = 'Removing logged execution, assuming that the process is dead. Execution of \'' . $row['classname'] . '\' (UID: ' . $row['uid'] . ') was started at ' . date('Y-m-d H:i:s', $task);
                                                $this->log($logMessage);
                                        }
                                }
@@ -144,12 +144,12 @@ class Scheduler implements \TYPO3\CMS\Core\SingletonInterface {
                // Task is already running and multiple executions are not allowed
                if (!$task->areMultipleExecutionsAllowed() && $task->isExecutionRunning()) {
                        // Log multiple execution error
-                       $logMessage = (('Task is already running and multiple executions are not allowed, skipping! Class: ' . get_class($task)) . ', UID: ') . $task->getTaskUid();
+                       $logMessage = 'Task is already running and multiple executions are not allowed, skipping! Class: ' . get_class($task) . ', UID: ' . $task->getTaskUid();
                        $this->log($logMessage);
                        $result = FALSE;
                } else {
                        // Log scheduler invocation
-                       $logMessage = (('Start execution. Class: ' . get_class($task)) . ', UID: ') . $task->getTaskUid();
+                       $logMessage = 'Start execution. Class: ' . get_class($task) . ', UID: ' . $task->getTaskUid();
                        $this->log($logMessage);
                        // Register execution
                        $executionID = $task->markExecution();
@@ -158,7 +158,7 @@ class Scheduler implements \TYPO3\CMS\Core\SingletonInterface {
                                // Execute task
                                $successfullyExecuted = $task->execute();
                                if (!$successfullyExecuted) {
-                                       throw new \TYPO3\CMS\Scheduler\FailedExecutionException((('Task failed to execute successfully. Class: ' . get_class($task)) . ', UID: ') . $task->getTaskUid(), 1250596541);
+                                       throw new \TYPO3\CMS\Scheduler\FailedExecutionException('Task failed to execute successfully. Class: ' . get_class($task) . ', UID: ' . $task->getTaskUid(), 1250596541);
                                }
                        } catch (\Exception $e) {
                                // Store exception, so that it can be saved to database
@@ -167,7 +167,7 @@ class Scheduler implements \TYPO3\CMS\Core\SingletonInterface {
                        // Un-register execution
                        $task->unmarkExecution($executionID, $failure);
                        // Log completion of execution
-                       $logMessage = (('Task executed. Class: ' . get_class($task)) . ', UID: ') . $task->getTaskUid();
+                       $logMessage = 'Task executed. Class: ' . get_class($task) . ', UID: ' . $task->getTaskUid();
                        $this->log($logMessage);
                        // Now that the result of the task execution has been handled,
                        // throw the exception again, if any
@@ -397,7 +397,7 @@ class Scheduler implements \TYPO3\CMS\Core\SingletonInterface {
                // Get at job id from registry and remove at job
                $atJobId = $registry->get('tx_scheduler', 'atJobId');
                if (\TYPO3\CMS\Core\Utility\MathUtility::canBeInterpretedAsInteger($atJobId)) {
-                       shell_exec(('atrm ' . (int) $atJobId) . ' 2>&1');
+                       shell_exec('atrm ' . (int) $atJobId . ' 2>&1');
                }
                // Can not use fetchTask() here because if tasks have just executed
                // they are not in the list of next executions
@@ -421,7 +421,7 @@ class Scheduler implements \TYPO3\CMS\Core\SingletonInterface {
                                $currentLocale = setlocale(LC_CTYPE, 0);
                                setlocale(LC_CTYPE, $GLOBALS['TYPO3_CONF_VARS']['SYS']['systemLocale']);
                        }
-                       $cmd = ((('echo ' . escapeshellarg($cliDispatchPath)) . ' scheduler | at ') . escapeshellarg($startTime)) . ' 2>&1';
+                       $cmd = 'echo ' . escapeshellarg($cliDispatchPath) . ' scheduler | at ' . escapeshellarg($startTime) . ' 2>&1';
                        if ($GLOBALS['TYPO3_CONF_VARS']['SYS']['UTF8filesystem']) {
                                setlocale(LC_CTYPE, $currentLocale);
                        }
index ef8b10d..65eb535 100644 (file)
@@ -384,7 +384,7 @@ abstract class Task {
                                }
                                if ($failure instanceof \Exception) {
                                        // Log failed execution
-                                       $logMessage = (((('Task failed to execute successfully. Class: ' . get_class($this)) . ', UID: ') . $this->taskUid) . '. ') . $failure->getMessage();
+                                       $logMessage = 'Task failed to execute successfully. Class: ' . get_class($this) . ', UID: ' . $this->taskUid . '. ' . $failure->getMessage();
                                        $this->scheduler->log($logMessage, 1, $failure->getCode());
                                        $failure = serialize($failure);
                                } else {
index d831561..b8e77a7 100644 (file)
@@ -58,7 +58,7 @@ class CachingFrameworkGarbageCollectionAdditionalFieldProvider implements \TYPO3
                $fieldName = 'TYPO3\\CMS\\Scheduler\\Scheduler[scheduler_cachingFrameworkGarbageCollection_selectedBackends][]';
                $fieldId = 'task_cachingFrameworkGarbageCollection_selectedBackends';
                $fieldOptions = $this->getCacheBackendOptions($taskInfo['scheduler_cachingFrameworkGarbageCollection_selectedBackends']);
-               $fieldHtml = ((((('<select name="' . $fieldName) . '" id="') . $fieldId) . '" class="wide" size="10" multiple="multiple">') . $fieldOptions) . '</select>';
+               $fieldHtml = '<select name="' . $fieldName . '" id="' . $fieldId . '" class="wide" size="10" multiple="multiple">' . $fieldOptions . '</select>';
                $additionalFields[$fieldId] = array(
                        'code' => $fieldHtml,
                        'label' => 'LLL:EXT:scheduler/mod1/locallang.xml:label.cachingFrameworkGarbageCollection.selectBackends',
@@ -117,7 +117,7 @@ class CachingFrameworkGarbageCollectionAdditionalFieldProvider implements \TYPO3
                        } else {
                                $selected = '';
                        }
-                       $options[] = ((((('<option value="' . $backendName) . '"') . $selected) . '>') . $backendName) . '</option>';
+                       $options[] = '<option value="' . $backendName . '"' . $selected . '>' . $backendName . '</option>';
                }
                return implode('', $options);
        }
index 5be72f7..89c9927 100644 (file)
@@ -62,7 +62,7 @@ class RecyclerGarbageCollectionAdditionalFieldProvider implements \TYPO3\CMS\Sch
                $fieldName = 'tx_scheduler[scheduler_recyclerGarbageCollection_numberOfDays]';
                $fieldId = 'task_recyclerGarbageCollection_numberOfDays';
                $fieldValue = intval($taskInfo['scheduler_recyclerGarbageCollection_numberOfDays']);
-               $fieldHtml = ((((('<input type="text" name="' . $fieldName) . '" id="') . $fieldId) . '" value="') . htmlspecialchars($fieldValue)) . '" />';
+               $fieldHtml = '<input type="text" name="' . $fieldName . '" id="' . $fieldId . '" value="' . htmlspecialchars($fieldValue) . '" />';
                $additionalFields[$fieldId] = array(
                        'code' => $fieldHtml,
                        'label' => 'LLL:EXT:scheduler/mod1/locallang.xml:label.recyclerGarbageCollection.numberOfDays',
index 74e8b8e..a459ad9 100644 (file)
@@ -61,7 +61,7 @@ class RecyclerGarbageCollectionTask extends \TYPO3\CMS\Scheduler\Task {
                if (TYPO3_OS == 'WIN') {
                        throw new \BadMethodCallException('This task is not reliable for Windows OS', 1308270454);
                }
-               $seconds = ((60 * 60) * 24) * (int) $this->numberOfDays;
+               $seconds = 60 * 60 * 24 * (int) $this->numberOfDays;
                $timestamp = $GLOBALS['EXEC_TIME'] - $seconds;
                // Get fileadmin directory
                $directory = PATH_site . 'fileadmin/';
@@ -86,7 +86,7 @@ class RecyclerGarbageCollectionTask extends \TYPO3\CMS\Scheduler\Task {
                $timestamp = (int) $timestamp;
                // Check if given directory exists
                if (!@is_dir($directory)) {
-                       throw new \RuntimeException(('Given directory "' . $directory) . '" does not exist', 1301614535);
+                       throw new \RuntimeException('Given directory "' . $directory . '" does not exist', 1301614535);
                }
                // Find all _recycler_ directories
                $directoryContent = new \RecursiveIteratorIterator(new \RecursiveDirectoryIterator($directory));
@@ -99,7 +99,7 @@ class RecyclerGarbageCollectionTask extends \TYPO3\CMS\Scheduler\Task {
                        // Remove files from _recycler_ that where moved to this folder for more than 'number of days'
                        if ($file->isFile() && $timestamp > $file->getCTime()) {
                                if (!@unlink($fileName)) {
-                                       throw new \RuntimeException(('Could not remove file "' . $fileName) . '"', 1301614537);
+                                       throw new \RuntimeException('Could not remove file "' . $fileName . '"', 1301614537);
                                }
                        }
                }
index 75a6f37..5fd957f 100644 (file)
@@ -83,7 +83,7 @@ class TableGarbageCollectionAdditionalFieldProvider implements \TYPO3\CMS\Schedu
                }
                $fieldName = 'tx_scheduler[scheduler_tableGarbageCollection_allTables]';
                $fieldId = 'task_tableGarbageCollection_allTables';
-               $fieldHtml = ((((((('<input type="checkbox" ' . $checked) . 'onChange="actOnChangeSchedulerTableGarbageCollectionAllTables(this)" ') . 'name="') . $fieldName) . '" ') . 'id="') . $fieldId) . '" />';
+               $fieldHtml = '<input type="checkbox" ' . $checked . 'onChange="actOnChangeSchedulerTableGarbageCollectionAllTables(this)" ' . 'name="' . $fieldName . '" ' . 'id="' . $fieldId . '" />';
                $fieldConfiguration = array(
                        'code' => $fieldHtml,
                        'label' => 'LLL:EXT:scheduler/mod1/locallang.xml:label.tableGarbageCollection.allTables',
@@ -112,12 +112,12 @@ class TableGarbageCollectionAdditionalFieldProvider implements \TYPO3\CMS\Schedu
                foreach ($tableConfiguration as $tableName => $configuration) {
                        if ($parentObject->CMD === 'add' && count($options) === 0) {
                                // Select first table by default if adding a new task
-                               $options[] = ((('<option value="' . $tableName) . '" selected="selected">') . $tableName) . '</option>';
+                               $options[] = '<option value="' . $tableName . '" selected="selected">' . $tableName . '</option>';
                        } elseif ($task->table === $tableName) {
                                // Select currently selected table
-                               $options[] = ((('<option value="' . $tableName) . '" selected="selected">') . $tableName) . '</option>';
+                               $options[] = '<option value="' . $tableName . '" selected="selected">' . $tableName . '</option>';
                        } else {
-                               $options[] = ((('<option value="' . $tableName) . '">') . $tableName) . '</option>';
+                               $options[] = '<option value="' . $tableName . '">' . $tableName . '</option>';
                        }
                }
                $disabled = $task->allTables === TRUE ? ' disabled="disabled"' : '';
@@ -125,10 +125,10 @@ class TableGarbageCollectionAdditionalFieldProvider implements \TYPO3\CMS\Schedu
                $fieldId = 'task_tableGarbageCollection_table';
                $fieldHtml = array();
                // Add table drop down html
-               $fieldHtml[] = ((((((((('<select ' . 'name="') . $fieldName) . '" ') . $disabled) . 'onChange="actOnChangeSchedulerTableGarbageCollectionTable(this)"') . 'id="') . $fieldId) . '">') . implode(LF, $options)) . '</select>';
+               $fieldHtml[] = '<select ' . 'name="' . $fieldName . '" ' . $disabled . 'onChange="actOnChangeSchedulerTableGarbageCollectionTable(this)"' . 'id="' . $fieldId . '">' . implode(LF, $options) . '</select>';
                // Add js array for default 'number of days' values
                $fieldHtml[] = '<script type="text/javascript">/*<![CDATA[*/<!--';
-               $fieldHtml[] = ('var defaultNumberOfDays = ' . json_encode($this->defaultNumberOfDays)) . ';';
+               $fieldHtml[] = 'var defaultNumberOfDays = ' . json_encode($this->defaultNumberOfDays) . ';';
                $fieldHtml[] = '// -->/*]]>*/</script>';
                $fieldConfiguration = array(
                        'code' => implode(LF, $fieldHtml),
@@ -167,7 +167,7 @@ class TableGarbageCollectionAdditionalFieldProvider implements \TYPO3\CMS\Schedu
                }
                $fieldName = 'tx_scheduler[scheduler_tableGarbageCollection_numberOfDays]';
                $fieldId = 'task_tableGarbageCollection_numberOfDays';
-               $fieldHtml = (((((((((('<input type="text" ' . 'name="') . $fieldName) . '" ') . 'id="') . $fieldId) . '" ') . $disabled) . 'value="') . intval($taskInfo['scheduler_tableGarbageCollection_numberOfDays'])) . '" ') . 'size="4" />';
+               $fieldHtml = '<input type="text" ' . 'name="' . $fieldName . '" ' . 'id="' . $fieldId . '" ' . $disabled . 'value="' . intval($taskInfo['scheduler_tableGarbageCollection_numberOfDays']) . '" ' . 'size="4" />';
                $fieldConfiguration = array(
                        'code' => $fieldHtml,
                        'label' => 'LLL:EXT:scheduler/mod1/locallang.xml:label.tableGarbageCollection.numberOfDays',
index 3ce89a4..f18e3c6 100644 (file)
@@ -69,7 +69,7 @@ class TableGarbageCollectionTask extends \TYPO3\CMS\Scheduler\Task {
                        }
                }
                if (!$tableHandled) {
-                       throw new \RuntimeException(('TYPO3\\CMS\\Scheduler\\Scheduler_TableGarbageCollection misconfiguration: ' . $this->table) . ' does not exist in configuration', 1308354399);
+                       throw new \RuntimeException('TYPO3\\CMS\\Scheduler\\Scheduler_TableGarbageCollection misconfiguration: ' . $this->table . ' does not exist in configuration', 1308354399);
                }
                return TRUE;
        }
@@ -88,25 +88,25 @@ class TableGarbageCollectionTask extends \TYPO3\CMS\Scheduler\Task {
                        $dateLimit = $GLOBALS['EXEC_TIME'];
                        // If expire field value is 0, do not delete
                        // Expire field = 0 means no expiration
-                       $where = (((($field . ' <= \'') . $dateLimit) . '\' AND ') . $field) . ' > \'0\'';
+                       $where = $field . ' <= \'' . $dateLimit . '\' AND ' . $field . ' > \'0\'';
                } elseif (!empty($configuration['dateField'])) {
                        $field = $configuration['dateField'];
                        if (!$this->allTables) {
-                               $deleteTimestamp = strtotime(('-' . $this->numberOfDays) . 'days');
+                               $deleteTimestamp = strtotime('-' . $this->numberOfDays . 'days');
                        } else {
                                if (!isset($configuration['expirePeriod'])) {
                                        throw new \RuntimeException('TYPO3\\CMS\\Scheduler\\Scheduler_TableGarbageCollection misconfiguration: No expirePeriod defined for table ' . $table, 1308355095);
                                }
-                               $deleteTimestamp = strtotime(('-' . $configuration['expirePeriod']) . 'days');
+                               $deleteTimestamp = strtotime('-' . $configuration['expirePeriod'] . 'days');
                        }
-                       $where = ($configuration['dateField'] . ' < ') . $deleteTimestamp;
+                       $where = $configuration['dateField'] . ' < ' . $deleteTimestamp;
                } else {
                        throw new \RuntimeException('TYPO3\\CMS\\Scheduler\\Scheduler_TableGarbageCollection misconfiguration: Either expireField or dateField must be defined for table ' . $table, 1308355268);
                }
                $GLOBALS['TYPO3_DB']->exec_DELETEquery($table, $where);
                $error = $GLOBALS['TYPO3_DB']->sql_error();
                if ($error) {
-                       throw new \RuntimeException((('TYPO3\\CMS\\Scheduler\\Scheduler_TableGarbageCollection failed for table ' . $this->table) . ' with error: ') . $error, 1308255491);
+                       throw new \RuntimeException('TYPO3\\CMS\\Scheduler\\Scheduler_TableGarbageCollection failed for table ' . $this->table . ' with error: ' . $error, 1308255491);
                }
                return TRUE;
        }
index fdc7f0b..9b4e6c1 100644 (file)
@@ -103,8 +103,8 @@ class SchedulerModuleControllerTest extends \tx_phpunit_testcase {
                        // If this tests runs over 1 seconds the test will fail if we use assertSame / assertEquals
                        // With assertLessThan the tests could run 0 till 3 seconds ($delta = 4)
                $delta = 4;
-               $this->assertLessThan($delta, $checkDateResult - $expectedTimestamp, ('assertLessThan fails with value "' . $strToTimeValue) . '"');
-               $this->assertInternalType(\PHPUnit_Framework_Constraint_IsType::TYPE_INT, $checkDateResult, ('assertType fails with value "' . $strToTimeValue) . '"');
+               $this->assertLessThan($delta, $checkDateResult - $expectedTimestamp, 'assertLessThan fails with value "' . $strToTimeValue . '"');
+               $this->assertInternalType(\PHPUnit_Framework_Constraint_IsType::TYPE_INT, $checkDateResult, 'assertType fails with value "' . $strToTimeValue . '"');
        }
 
        /**
@@ -161,7 +161,7 @@ class SchedulerModuleControllerTest extends \tx_phpunit_testcase {
         * @param integer $expectedTimestamp Expected value to compare with result from operation
         */
        public function checkDateWithTypo3DateSyntax($typo3DateValue, $expectedTimestamp) {
-               $this->assertSame($expectedTimestamp, $this->testObject->checkDate($typo3DateValue), ('Fails with value "' . $typo3DateValue) . '"');
+               $this->assertSame($expectedTimestamp, $this->testObject->checkDate($typo3DateValue), 'Fails with value "' . $typo3DateValue . '"');
        }
 
        /**
index f24e6b9..349fccb 100644 (file)
@@ -114,55 +114,55 @@ class CronCommandTest extends \tx_phpunit_testcase {
                                '1 * * * *',
                                self::TIMESTAMP,
                                self::TIMESTAMP + 60,
-                               (self::TIMESTAMP + 60) + 60 * 60
+                               self::TIMESTAMP + 60 + 60 * 60
                        ),
                        'once an hour at 0' => array(
                                '0 * * * *',
                                self::TIMESTAMP,
                                self::TIMESTAMP + 60 * 60,
-                               (self::TIMESTAMP + 60 * 60) + 60 * 60
+                               self::TIMESTAMP + 60 * 60 + 60 * 60
                        ),
                        'once a day at 1:00' => array(
                                '0 1 * * *',
                                self::TIMESTAMP,
                                self::TIMESTAMP + 60 * 60,
-                               (self::TIMESTAMP + 60 * 60) + (60 * 60) * 24
+                               self::TIMESTAMP + 60 * 60 + 60 * 60 * 24
                        ),
                        'once a day at 0:00' => array(
                                '0 0 * * *',
                                self::TIMESTAMP,
-                               self::TIMESTAMP + (60 * 60) * 24,
-                               self::TIMESTAMP + ((60 * 60) * 24) * 2
+                               self::TIMESTAMP + 60 * 60 * 24,
+                               self::TIMESTAMP + 60 * 60 * 24 * 2
                        ),
                        'once a month' => array(
                                '0 0 4 * *',
                                self::TIMESTAMP,
-                               self::TIMESTAMP + ((60 * 60) * 24) * 3,
-                               (self::TIMESTAMP + ((60 * 60) * 24) * 3) + ((60 * 60) * 24) * 31
+                               self::TIMESTAMP + 60 * 60 * 24 * 3,
+                               self::TIMESTAMP + 60 * 60 * 24 * 3 + 60 * 60 * 24 * 31
                        ),
                        'once every Saturday' => array(
                                '0 0 * * sat',
                                self::TIMESTAMP,
-                               self::TIMESTAMP + (60 * 60) * 24,
-                               (self::TIMESTAMP + (60 * 60) * 24) + ((60 * 60) * 24) * 7
+                               self::TIMESTAMP + 60 * 60 * 24,
+                               self::TIMESTAMP + 60 * 60 * 24 + 60 * 60 * 24 * 7
                        ),
                        'once every day in February' => array(
                                '0 0 * feb *',
                                self::TIMESTAMP,
-                               self::TIMESTAMP + ((60 * 60) * 24) * 31,
-                               (self::TIMESTAMP + ((60 * 60) * 24) * 31) + (60 * 60) * 24
+                               self::TIMESTAMP + 60 * 60 * 24 * 31,
+                               self::TIMESTAMP + 60 * 60 * 24 * 31 + 60 * 60 * 24
                        ),
                        'day of week and day of month restricted, next match in day of month field' => array(
                                '0 0 2 * sun',
                                self::TIMESTAMP,
-                               self::TIMESTAMP + (60 * 60) * 24,
-                               (self::TIMESTAMP + (60 * 60) * 24) + (60 * 60) * 24
+                               self::TIMESTAMP + 60 * 60 * 24,
+                               self::TIMESTAMP + 60 * 60 * 24 + 60 * 60 * 24
                        ),
                        'day of week and day of month restricted, next match in day of week field' => array(
                                '0 0 3 * sat',
                                self::TIMESTAMP,
-                               self::TIMESTAMP + (60 * 60) * 24,
-                               (self::TIMESTAMP + (60 * 60) * 24) + (60 * 60) * 24
+                               self::TIMESTAMP + 60 * 60 * 24,
+                               self::TIMESTAMP + 60 * 60 * 24 + 60 * 60 * 24
                        ),
                        'list of minutes' => array(
                                '2,4 * * * *',
@@ -173,8 +173,8 @@ class CronCommandTest extends \tx_phpunit_testcase {
                        'list of hours' => array(
                                '0 2,4 * * *',
                                self::TIMESTAMP,
-                               self::TIMESTAMP + (60 * 60) * 2,
-                               self::TIMESTAMP + (60 * 60) * 4
+                               self::TIMESTAMP + 60 * 60 * 2,
+                               self::TIMESTAMP + 60 * 60 * 4
                        ),
                );
        }
index e2f08c6..08ecfea 100644 (file)
@@ -44,7 +44,7 @@ if (TYPO3_REQUESTTYPE & TYPO3_REQUESTTYPE_CLI && basename(PATH_thisScript) == 'c
                        if ($cli->cli_isArg('-f')) {
                                $task = $scheduler->fetchTask($taskId);
                        } else {
-                               $whereClause = (('uid = ' . $taskId) . ' AND nextexecution != 0 AND nextexecution <= ') . $GLOBALS['EXEC_TIME'];
+                               $whereClause = 'uid = ' . $taskId . ' AND nextexecution != 0 AND nextexecution <= ' . $GLOBALS['EXEC_TIME'];
                                list($task) = $scheduler->fetchTasksWithCondition($whereClause);
                        }
                        if ($scheduler->isValidTaskObject($task)) {
index edc6601..c006278 100644 (file)
@@ -4,7 +4,7 @@ if (!defined('TYPO3_MODE')) {
 }
 // Register the Scheduler as a possible key for CLI calls
 $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['GLOBAL']['cliKeys'][$_EXTKEY] = array(
-       ('EXT:' . $_EXTKEY) . '/cli/scheduler_cli_dispatch.php',
+       'EXT:' . $_EXTKEY . '/cli/scheduler_cli_dispatch.php',
        '_CLI_scheduler'
 );
 // Get the extensions's configuration
@@ -14,45 +14,45 @@ $extConf = unserialize($GLOBALS['TYPO3_CONF_VARS']['EXT']['extConf']['scheduler'
 if (!empty($extConf['showSampleTasks'])) {
        $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['scheduler']['tasks']['TYPO3\\CMS\\Scheduler\\Example\\TestTask'] = array(
                'extension' => $_EXTKEY,
-               'title' => ('LLL:EXT:' . $_EXTKEY) . '/locallang.xml:testTask.name',
-               'description' => ('LLL:EXT:' . $_EXTKEY) . '/locallang.xml:testTask.description',
+               'title' => 'LLL:EXT:' . $_EXTKEY . '/locallang.xml:testTask.name',
+               'description' => 'LLL:EXT:' . $_EXTKEY . '/locallang.xml:testTask.description',
                'additionalFields' => 'TYPO3\\CMS\\Scheduler\\Example\\TestTaskAdditionalFieldProvider'
        );
        $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['scheduler']['tasks']['TYPO3\\CMS\\Scheduler\\Example\\SleepTask'] = array(
                'extension' => $_EXTKEY,
-               'title' => ('LLL:EXT:' . $_EXTKEY) . '/locallang.xml:sleepTask.name',
-               'description' => ('LLL:EXT:' . $_EXTKEY) . '/locallang.xml:sleepTask.description',
+               'title' => 'LLL:EXT:' . $_EXTKEY . '/locallang.xml:sleepTask.name',
+               'description' => 'LLL:EXT:' . $_EXTKEY . '/locallang.xml:sleepTask.description',
                'additionalFields' => 'TYPO3\\CMS\\Scheduler\\Example\\SleepTaskAdditionalFieldProvider'
        );
 }
 // Add caching framework garbage collection task
 $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['scheduler']['tasks']['TYPO3\\CMS\\Scheduler\\Scheduler\\Task\\CachingFrameworkGarbageCollectionTask'] = array(
        'extension' => $_EXTKEY,
-       'title' => ('LLL:EXT:' . $_EXTKEY) . '/locallang.xml:cachingFrameworkGarbageCollection.name',
-       'description' => ('LLL:EXT:' . $_EXTKEY) . '/locallang.xml:cachingFrameworkGarbageCollection.description',
+       'title' => 'LLL:EXT:' . $_EXTKEY . '/locallang.xml:cachingFrameworkGarbageCollection.name',
+       'description' => 'LLL:EXT:' . $_EXTKEY . '/locallang.xml:cachingFrameworkGarbageCollection.description',
        'additionalFields' => 'TYPO3\\CMS\\Scheduler\\Task\\CachingFrameworkGarbageCollectionAdditionalFieldProvider'
 );
 // Add file indexing task
 $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['scheduler']['tasks']['TYPO3\\CMS\\Scheduler\\Task\\FileIndexingTask'] = array(
        'extension' => $_EXTKEY,
-       'title' => ('LLL:EXT:' . $_EXTKEY) . '/locallang.xml:fileIndexing.name',
-       'description' => ('LLL:EXT:' . $_EXTKEY) . '/locallang.xml:fileIndexing.description'
+       'title' => 'LLL:EXT:' . $_EXTKEY . '/locallang.xml:fileIndexing.name',
+       'description' => 'LLL:EXT:' . $_EXTKEY . '/locallang.xml:fileIndexing.description'
 );
 // Add recycler directory cleanup task. Windows is not supported
 // because "filectime" does not change after moving a file
 if (TYPO3_OS != 'WIN') {
        $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['scheduler']['tasks']['TYPO3\\CMS\\Scheduler\\Task\\RecyclerGarbageCollectionTask'] = array(
                'extension' => $_EXTKEY,
-               'title' => ('LLL:EXT:' . $_EXTKEY) . '/locallang.xml:recyclerGarbageCollection.name',
-               'description' => ('LLL:EXT:' . $_EXTKEY) . '/locallang.xml:recyclerGarbageCollection.description',
+               'title' => 'LLL:EXT:' . $_EXTKEY . '/locallang.xml:recyclerGarbageCollection.name',
+               'description' => 'LLL:EXT:' . $_EXTKEY . '/locallang.xml:recyclerGarbageCollection.description',
                'additionalFields' => 'TYPO3\\CMS\\Scheduler\\Task\\RecyclerGarbageCollectionAdditionalFieldProvider'
        );
 }
 // Add table garbage collection task
 $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['scheduler']['tasks']['TYPO3\\CMS\\Scheduler\\Task\\TableGarbageCollectionTask'] = array(
        'extension' => $_EXTKEY,
-       'title' => ('LLL:EXT:' . $_EXTKEY) . '/locallang.xml:tableGarbageCollection.name',
-       'description' => ('LLL:EXT:' . $_EXTKEY) . '/locallang.xml:tableGarbageCollection.description',
+       'title' => 'LLL:EXT:' . $_EXTKEY . '/locallang.xml:tableGarbageCollection.name',
+       'description' => 'LLL:EXT:' . $_EXTKEY . '/locallang.xml:tableGarbageCollection.description',
        'additionalFields' => 'TYPO3\\CMS\\Scheduler\\Task\\TableGarbageCollectionAdditionalFieldProvider'
 );
 // Initialize option array of table garbage collection task if not already done by some other extension or localconf.php
index 750d697..8e84141 100755 (executable)
@@ -6,6 +6,6 @@ if (TYPO3_MODE == 'BE') {
        // Add module
        \TYPO3\CMS\Core\Extension\ExtensionManager::addModule('tools', 'txschedulerM1', '', \TYPO3\CMS\Core\Extension\ExtensionManager::extPath($_EXTKEY) . 'mod1/');
        // Add context sensitive help (csh) to the backend module
-       \TYPO3\CMS\Core\Extension\ExtensionManager::addLLrefForTCAdescr('_MOD_tools_txschedulerM1', ('EXT:' . $_EXTKEY) . '/mod1/locallang_csh_scheduler.xml');
+       \TYPO3\CMS\Core\Extension\ExtensionManager::addLLrefForTCAdescr('_MOD_tools_txschedulerM1', 'EXT:' . $_EXTKEY . '/mod1/locallang_csh_scheduler.xml');
 }
 ?>
\ No newline at end of file
index 134de30..b4136ae 100644 (file)
@@ -281,7 +281,7 @@ class SetupModuleController {
                $this->doc = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance('TYPO3\\CMS\\Backend\\Template\\DocumentTemplate');
                $this->doc->backPath = $GLOBALS['BACK_PATH'];
                $this->doc->setModuleTemplate('templates/setup.html');
-               $this->doc->form = ('<form action="' . \TYPO3\CMS\Backend\Utility\BackendUtility::getModuleUrl('user_setup')) . '" method="post" name="usersetup" enctype="application/x-www-form-urlencoded">';
+               $this->doc->form = '<form action="' . \TYPO3\CMS\Backend\Utility\BackendUtility::getModuleUrl('user_setup') . '" method="post" name="usersetup" enctype="application/x-www-form-urlencoded">';
                $this->doc->tableLayout = array(
                        'defRow' => array(
                                '0' => array('<td class="td-label">', '</td>'),
@@ -339,7 +339,7 @@ class SetupModuleController {
                $this->loadModules->load($GLOBALS['TBE_MODULES']);
                $this->content .= $this->doc->header($LANG->getLL('UserSettings'));
                // Show if setup was saved
-               if (($this->setupIsUpdated && !$this->tempDataIsCleared) && !$this->settingsAreResetToDefault) {
+               if ($this->setupIsUpdated && !$this->tempDataIsCleared && !$this->settingsAreResetToDefault) {
                        $flashMessage = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance('TYPO3\\CMS\\Core\\Messaging\\FlashMessage', $LANG->getLL('setupWasUpdated'), $LANG->getLL('UserSettings'));
                        $this->content .= $flashMessage->render();
                }
@@ -371,8 +371,8 @@ class SetupModuleController {
                $menuItems = $this->renderUserSetup();
                $this->content .= $this->doc->getDynTabMenu($menuItems, 'user-setup', FALSE, FALSE, 1, FALSE, 1, $this->dividers2tabs);
                $formToken = $this->formProtection->generateToken('BE user setup', 'edit');
-               $this->content .= $this->doc->section('', ((('<input type="hidden" name="simUser" value="' . $this->simUser) . '" />
-                       <input type="hidden" name="formToken" value="') . $formToken) . '" />
+               $this->content .= $this->doc->section('', '<input type="hidden" name="simUser" value="' . $this->simUser . '" />
+                       <input type="hidden" name="formToken" value="' . $formToken . '" />
                        <input type="hidden" value="1" name="data[save]" />
                        <input type="hidden" name="data[setValuesToDefault]" value="0" id="setValuesToDefault" />
                        <input type="hidden" name="data[clearSessionVars]" value="0" id="clearSessionVars" />');
@@ -410,7 +410,7 @@ class SetupModuleController {
                        'shortcut' => ''
                );
                $buttons['csh'] = \TYPO3\CMS\Backend\Utility\BackendUtility::cshItem('_MOD_user_setup', '', $GLOBALS['BACK_PATH'], '|', TRUE);
-               $buttons['save'] = \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('actions-document-save', array('html' => ('<input type="image" name="data[save]" class="c-inputButton" src="clear.gif" title="' . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_core.php:rm.saveDoc', 1)) . '" />'));
+               $buttons['save'] = \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('actions-document-save', array('html' => '<input type="image" name="data[save]" class="c-inputButton" src="clear.gif" title="' . $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_core.php:rm.saveDoc', 1) . '" />'));
                if ($GLOBALS['BE_USER']->mayMakeShortcut()) {
                        $buttons['shortcut'] = $this->doc->makeShortcutIcon('', '', $this->MCONF['name']);
                }
@@ -471,10 +471,10 @@ class SetupModuleController {
                        $class = $config['class'];
                        $style = $config['style'];
                        if ($class) {
-                               $more .= (' class="' . $class) . '"';
+                               $more .= ' class="' . $class . '"';
                        }
                        if ($style) {
-                               $more .= (' style="' . $style) . '"';
+                               $more .= ' style="' . $style . '"';
                        }
                        if ($this->overrideConf[$fieldName]) {
                                $more .= ' disabled="disabled"';
@@ -498,17 +498,17 @@ class SetupModuleController {
                                        $value = '';
                                }
                                $noAutocomplete = $type == 'password' ? 'autocomplete="off" ' : '';
-                               $html = (((((((((((((('<input id="field_' . $fieldName) . '"
-                                                       type="') . $type) . '"
-                                                       name="data') . $dataAdd) . '[') . $fieldName) . ']" ') . $noAutocomplete) . 'value="') . htmlspecialchars($value)) . '" ') . $GLOBALS['TBE_TEMPLATE']->formWidth(20)) . $more) . ' />';
+                               $html = '<input id="field_' . $fieldName . '"
+                                                       type="' . $type . '"
+                                                       name="data' . $dataAdd . '[' . $fieldName . ']" ' . $noAutocomplete . 'value="' . htmlspecialchars($value) . '" ' . $GLOBALS['TBE_TEMPLATE']->formWidth(20) . $more . ' />';
                                break;
                        case 'check':
                                if (!$class) {
                                        $more .= ' class="check"';
                                }
-                               $html = (((((('<input id="field_' . $fieldName) . '"
+                               $html = '<input id="field_' . $fieldName . '"
                                                                        type="checkbox"
-                                                                       name="data[') . $fieldName) . ']"') . ($value ? ' checked="checked"' : '')) . $more) . ' />';
+                                                                       name="data[' . $fieldName . ']"' . ($value ? ' checked="checked"' : '') . $more . ' />';
                                break;
                        case 'select':
                                if (!$class) {
@@ -517,9 +517,9 @@ class SetupModuleController {
                                if ($config['itemsProcFunc']) {
                                        $html = \TYPO3\CMS\Core\Utility\GeneralUtility::callUserFunction($config['itemsProcFunc'], $config, $this, '');
                                } else {
-                                       $html = (((((((('<select ' . $GLOBALS['TBE_TEMPLATE']->formWidth(20)) . ' id="field_') . $fieldName) . '" name="data[') . $fieldName) . ']"') . $more) . '>') . LF;
+                                       $html = '<select ' . $GLOBALS['TBE_TEMPLATE']->formWidth(20) . ' id="field_' . $fieldName . '" name="data[' . $fieldName . ']"' . $more . '>' . LF;
                                        foreach ($config['items'] as $key => $optionLabel) {
-                                               $html .= (((((('<option value="' . $key) . '"') . ($value == $key ? ' selected="selected"' : '')) . '>') . $this->getLabel($optionLabel, '', FALSE)) . '</option>') . LF;
+                                               $html .= '<option value="' . $key . '"' . ($value == $key ? ' selected="selected"' : '') . '>' . $this->getLabel($optionLabel, '', FALSE) . '</option>' . LF;
                                        }
                                        $html .= '</select>';
                                }
@@ -536,7 +536,7 @@ class SetupModuleController {
                                                }
                                                $onClick = vsprintf($onClick, $config['onClickLabels']);
                                        }
-                                       $html = ((((('<input ' . $GLOBALS['TBE_TEMPLATE']->formWidth(20)) . ' type="button" value="') . $this->getLabel($config['buttonlabel'], '', FALSE)) . '" onclick="') . $onClick) . '" />';
+                                       $html = '<input ' . $GLOBALS['TBE_TEMPLATE']->formWidth(20) . ' type="button" value="' . $this->getLabel($config['buttonlabel'], '', FALSE) . '" onclick="' . $onClick . '" />';
                                }
                                break;
                        default:
@@ -579,7 +579,7 @@ class SetupModuleController {
                $languageOptions = array();
                // Compile the languages dropdown
                $langDefault = $GLOBALS['LANG']->getLL('lang_default', 1);
-               $languageOptions[$langDefault] = ((('<option value=""' . ($GLOBALS['BE_USER']->uc['lang'] === '' ? ' selected="selected"' : '')) . '>') . $langDefault) . '</option>';
+               $languageOptions[$langDefault] = '<option value=""' . ($GLOBALS['BE_USER']->uc['lang'] === '' ? ' selected="selected"' : '') . '>' . $langDefault . '</option>';
                // Traverse the number of languages
                /** @var $locales \TYPO3\CMS\Core\Localization\Locales */
                $locales = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance('TYPO3\\CMS\\Core\\Localization\\Locales');
@@ -591,19 +591,19 @@ class SetupModuleController {
                                if ($localizedName === '') {
                                        $localizedName = htmlspecialchars($name);
                                }
-                               $localLabel = ('  -  [' . htmlspecialchars($defaultName)) . ']';
-                               $available = is_dir((PATH_typo3conf . 'l10n/') . $locale) ? TRUE : FALSE;
+                               $localLabel = '  -  [' . htmlspecialchars($defaultName) . ']';
+                               $available = is_dir(PATH_typo3conf . 'l10n/' . $locale) ? TRUE : FALSE;
                                if ($available) {
-                                       $languageOptions[$defaultName] = (((((('<option value="' . $locale) . '"') . ($GLOBALS['BE_USER']->uc['lang'] === $locale ? ' selected="selected"' : '')) . '>') . $localizedName) . $localLabel) . '</option>';
+                                       $languageOptions[$defaultName] = '<option value="' . $locale . '"' . ($GLOBALS['BE_USER']->uc['lang'] === $locale ? ' selected="selected"' : '') . '>' . $localizedName . $localLabel . '</option>';
                                }
                        }
                }
                ksort($languageOptions);
-               $languageCode = ((('
-                               <select ' . $GLOBALS['TBE_TEMPLATE']->formWidth(20)) . ' id="field_lang" name="data[lang]" class="select">') . implode('', $languageOptions)) . '
+               $languageCode = '
+                               <select ' . $GLOBALS['TBE_TEMPLATE']->formWidth(20) . ' id="field_lang" name="data[lang]" class="select">' . implode('', $languageOptions) . '
                                </select>';
-               if ($GLOBALS['BE_USER']->uc['lang'] && !@is_dir(((PATH_typo3conf . 'l10n/') . $GLOBALS['BE_USER']->uc['lang']))) {
-                       $languageUnavailableWarning = (('The selected language "' . $GLOBALS['LANG']->getLL(('lang_' . $GLOBALS['BE_USER']->uc['lang']), 1)) . '" is not available before the language pack is installed.<br />') . ($GLOBALS['BE_USER']->isAdmin() ? 'You can use the Extension Manager to easily download and install new language packs.' : 'Please ask your system administrator to do this.');
+               if ($GLOBALS['BE_USER']->uc['lang'] && !@is_dir((PATH_typo3conf . 'l10n/' . $GLOBALS['BE_USER']->uc['lang']))) {
+                       $languageUnavailableWarning = 'The selected language "' . $GLOBALS['LANG']->getLL(('lang_' . $GLOBALS['BE_USER']->uc['lang']), 1) . '" is not available before the language pack is installed.<br />' . ($GLOBALS['BE_USER']->isAdmin() ? 'You can use the Extension Manager to easily download and install new language packs.' : 'Please ask your system administrator to do this.');
                        $languageUnavailableMessage = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance('TYPO3\\CMS\\Core\\Messaging\\FlashMessage', $languageUnavailableWarning, '', \TYPO3\CMS\Core\Messaging\FlashMessage::WARNING);
                        $languageCode = $languageUnavailableMessage->render() . $languageCode;
                }
@@ -623,15 +623,15 @@ class SetupModuleController {
                $startModuleSelect = '<option value=""></option>';
                foreach ($pObj->loadModules->modules as $mainMod => $modData) {
                        if (isset($modData['sub']) && is_array($modData['sub'])) {
-                               $startModuleSelect .= ('<option disabled="disabled">' . $GLOBALS['LANG']->moduleLabels['tabs'][($mainMod . '_tab')]) . '</option>';
+                               $startModuleSelect .= '<option disabled="disabled">' . $GLOBALS['LANG']->moduleLabels['tabs'][($mainMod . '_tab')] . '</option>';
                                foreach ($modData['sub'] as $subKey => $subData) {
                                        $modName = $subData['name'];
-                                       $startModuleSelect .= ((('<option value="' . $modName) . '"') . ($GLOBALS['BE_USER']->uc['startModule'] == $modName ? ' selected="selected"' : '')) . '>';
-                                       $startModuleSelect .= (' - ' . $GLOBALS['LANG']->moduleLabels['tabs'][($modName . '_tab')]) . '</option>';
+                                       $startModuleSelect .= '<option value="' . $modName . '"' . ($GLOBALS['BE_USER']->uc['startModule'] == $modName ? ' selected="selected"' : '') . '>';
+                                       $startModuleSelect .= ' - ' . $GLOBALS['LANG']->moduleLabels['tabs'][($modName . '_tab')] . '</option>';
                                }
                        }
                }
-               return ((('<select ' . $GLOBALS['TBE_TEMPLATE']->formWidth(20)) . 'id="field_startModule" name="data[startModule]" class="select">') . $startModuleSelect) . '</select>';
+               return '<select ' . $GLOBALS['TBE_TEMPLATE']->formWidth(20) . 'id="field_startModule" name="data[startModule]" class="select">' . $startModuleSelect . '</select>';
        }
 
        /**
@@ -652,11 +652,11 @@ class SetupModuleController {
                        $opt = array();
                        foreach ($users as $rr) {
                                if ($rr['uid'] != $GLOBALS['BE_USER']->user['uid']) {
-                                       $opt[] = ((((('<option value="' . $rr['uid']) . '"') . ($this->simUser == $rr['uid'] ? ' selected="selected"' : '')) . '>') . htmlspecialchars(((($rr['username'] . ' (') . $rr['realName']) . ')'))) . '</option>';
+                                       $opt[] = '<option value="' . $rr['uid'] . '"' . ($this->simUser == $rr['uid'] ? ' selected="selected"' : '') . '>' . htmlspecialchars(($rr['username'] . ' (' . $rr['realName'] . ')')) . '</option>';
                                }
                        }
                        if (count($opt)) {
-                               $this->simulateSelector = ((((('<select ' . $GLOBALS['TBE_TEMPLATE']->formWidth(20)) . ' id="field_simulate" name="simulateUser" onchange="window.location.href=\'') . \TYPO3\CMS\Backend\Utility\BackendUtility::getModuleUrl('user_setup')) . '&simUser=\'+this.options[this.selectedIndex].value;"><option></option>') . implode('', $opt)) . '</select>';
+                               $this->simulateSelector = '<select ' . $GLOBALS['TBE_TEMPLATE']->formWidth(20) . ' id="field_simulate" name="simulateUser" onchange="window.location.href=\'' . \TYPO3\CMS\Backend\Utility\BackendUtility::getModuleUrl('user_setup') . '&simUser=\'+this.options[this.selectedIndex].value;"><option></option>' . implode('', $opt) . '</select>';
                        }
                }
                // This can only be set if the previous code was executed.
@@ -694,7 +694,7 @@ class SetupModuleController {
        protected function checkAccess(array $config) {
                $access = $config['access'];
                // Check for hook
-               $accessObject = \TYPO3\CMS\Core\Utility\GeneralUtility::getUserObj(($GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['setup']['accessLevelCheck'][$access] . ':&') . $access);
+               $accessObject = \TYPO3\CMS\Core\Utility\GeneralUtility::getUserObj($GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['setup']['accessLevelCheck'][$access] . ':&' . $access);
                if (is_object($accessObject) && method_exists($accessObject, 'accessLevelCheck')) {
                        // Initialize vars. If method fails, $set will be set to FALSE
                        return $accessObject->accessLevelCheck($config);
@@ -719,10 +719,10 @@ class SetupModuleController {
                        $out = htmlspecialchars($str);
                }
                if (isset($this->overrideConf[$key ? $key : $str])) {
-                       $out = ('<span style="color:#999999">' . $out) . '</span>';
+                       $out = '<span style="color:#999999">' . $out . '</span>';
                }
                if ($addLabelTag) {
-                       $out = ((('<label for="' . ($altLabelTagId ? $altLabelTagId : 'field_' . $key)) . '">') . $out) . '</label>';
+                       $out = '<label for="' . ($altLabelTagId ? $altLabelTagId : 'field_' . $key) . '">' . $out . '</label>';
                }
                return $out;
        }
index 398cf1d..fc8d5bb 100644 (file)
@@ -54,7 +54,7 @@ class StaticTemplatesHook {
                                                // ... there was a template, then we fetch that
                                        if ($subrow = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res)) {
                                                $subrow = $pObj->prependStaticExtra($subrow);
-                                               $pObj->processTemplate($subrow, ($params['idList'] . ',static_') . $id, $params['pid'], 'static_' . $id, $params['templateId']);
+                                               $pObj->processTemplate($subrow, $params['idList'] . ',static_' . $id, $params['pid'], 'static_' . $id, $params['templateId']);
                                        }
                                        $GLOBALS['TYPO3_DB']->sql_free_result($res);
                                }
index 35235dc..b64df40 100644 (file)
@@ -83,7 +83,7 @@ function user_reverseString($content, $conf)
  */
 function user_printTime($content, $conf)
 {
-       return ('<font color="red">Dynamic time: ' . date('H:i:s')) . '</font><br />';
+       return '<font color="red">Dynamic time: ' . date('H:i:s') . '</font><br />';
 }
 /**
  * Example of calling a method in a PHP class from TypoScript
@@ -136,8 +136,8 @@ class user_various {
         * @todo Define visibility
         */
        public function listContentRecordsOnPage($content, $conf) {
-               $query = $GLOBALS['TYPO3_DB']->SELECTquery('header', 'tt_content', ('pid=' . intval($GLOBALS['TSFE']->id)) . $this->cObj->enableFields('tt_content'), '', 'sorting' . ($conf['reverseOrder'] ? ' DESC' : ''));
-               $output = ('This is the query: <strong>' . $query) . '</strong><br /><br />';
+               $query = $GLOBALS['TYPO3_DB']->SELECTquery('header', 'tt_content', 'pid=' . intval($GLOBALS['TSFE']->id) . $this->cObj->enableFields('tt_content'), '', 'sorting' . ($conf['reverseOrder'] ? ' DESC' : ''));
+               $output = 'This is the query: <strong>' . $query . '</strong><br /><br />';
                return $output . $this->selectThem($query);
        }
 
index 7650cf9..efbe784 100644 (file)
@@ -68,7 +68,7 @@ function user_itemArrayProcFuncTest($menuArr, $conf)
                $teststates = explode(',', 'NO,ACT,IFSUB,CUR,USR,SPC,USERDEF1,USERDEF2');
                foreach ($menuArr as $k => $v) {
                        $menuArr[$k]['ITEM_STATE'] = $teststates[$c];
-                       $menuArr[$k]['title'] .= $teststates[$c] ? (' [' . $teststates[$c]) . ']' : '';
+                       $menuArr[$k]['title'] .= $teststates[$c] ? ' [' . $teststates[$c] . ']' : '';
                        $c++;
                }
        } else {
@@ -98,7 +98,7 @@ function user_IProcFuncTest($I, $conf)
 {
        $itemRow = $conf['parentObj']->menuArr[$I['key']];
        // Setting the document status content to the value of the page title on mouse over
-       $I['linkHREF']['onMouseover'] .= ('extraRollover(\'' . rawurlencode($itemRow['title'])) . '\');';
+       $I['linkHREF']['onMouseover'] .= 'extraRollover(\'' . rawurlencode($itemRow['title']) . '\');';
        $conf['parentObj']->I = $I;
        $conf['parentObj']->setATagParts();
        $I = $conf['parentObj']->I;
@@ -107,7 +107,7 @@ function user_IProcFuncTest($I, $conf)
        }
        if ($conf['debug']) {
                // Outputting for debug example:
-               echo ('ITEM: <h2>' . htmlspecialchars((($itemRow['uid'] . ': ') . $itemRow['title']))) . '</h2>';
+               echo 'ITEM: <h2>' . htmlspecialchars(($itemRow['uid'] . ': ' . $itemRow['title'])) . '</h2>';
                t3lib_utility_Debug::debug($itemRow);
                t3lib_utility_Debug::debug($I);
                echo '<hr />';
index abaeafe..de0a526 100644 (file)
@@ -48,9 +48,9 @@ function user_keepRolloverAtOnClick($I, $conf)
        // on mouse over
        if (!$I['linkHREF']['TARGET']) {
                $I['linkHREF']['HREF'] = '#';
-               $I['linkHREF']['onClick'] .= ((((('ARO_setLocation' . $conf['setLocation']) . '(') . $itemRow['uid']) . ',\'') . $I['theName']) . '\'); return false;';
+               $I['linkHREF']['onClick'] .= 'ARO_setLocation' . $conf['setLocation'] . '(' . $itemRow['uid'] . ',\'' . $I['theName'] . '\'); return false;';
        } else {
-               $I['linkHREF']['onClick'] .= (('ARO_setActiveImg' . '(\'') . $I['theName']) . '\');';
+               $I['linkHREF']['onClick'] .= 'ARO_setActiveImg' . '(\'' . $I['theName'] . '\');';
        }
        if ($I['linkHREF']['onMouseover']) {
                $I['linkHREF']['onMouseover'] = 'ARO_' . $I['linkHREF']['onMouseover'];
@@ -59,11 +59,11 @@ function user_keepRolloverAtOnClick($I, $conf)
                $I['linkHREF']['onMouseout'] = 'ARO_' . $I['linkHREF']['onMouseout'];
        }
        if ($conf['parentObj']->isActive($itemRow['uid'])) {
-               $conf['parentObj']->WMextraScript .= ((('
+               $conf['parentObj']->WMextraScript .= '
 <script type="text/javascript">
        /*<![CDATA[*/
- ARO_Image = "' . $I['theName']) . '";
- ') . $I['linkHREF']['onMouseover']) . '
+ ARO_Image = "' . $I['theName'] . '";
+ ' . $I['linkHREF']['onMouseover'] . '
        /*]]>*/
 </script>
                ';
@@ -80,7 +80,7 @@ function user_keepRolloverAtOnClick($I, $conf)
        // Debug:
        if ($conf['debug']) {
                // Outputting for debug example:
-               echo ('ITEM: <h2>' . htmlspecialchars((($itemRow['uid'] . ': ') . $itemRow['title']))) . '</h2>';
+               echo 'ITEM: <h2>' . htmlspecialchars(($itemRow['uid'] . ': ' . $itemRow['title'])) . '</h2>';
                t3lib_utility_Debug::debug($itemRow);
                t3lib_utility_Debug::debug($I);
                echo '<hr />';
index 59ba256..5bbc793 100644 (file)
@@ -42,7 +42,7 @@ if (!is_object($this)) {
 }
 // First, select all pages_language_overlay records on the current page.
 // Each represents a possibility for a language.
-$res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('*', 'pages_language_overlay', ('pid=' . intval($GLOBALS['TSFE']->id)) . $GLOBALS['TSFE']->sys_page->enableFields('pages_language_overlay'), 'sys_language_uid');
+$res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('*', 'pages_language_overlay', 'pid=' . intval($GLOBALS['TSFE']->id) . $GLOBALS['TSFE']->sys_page->enableFields('pages_language_overlay'), 'sys_language_uid');
 $langArr = array();
 while ($row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res)) {
        $langArr[$row['sys_language_uid']] = $row['title'];
@@ -51,16 +51,16 @@ while ($row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res)) {
 // if the TSFE->sys_language_uid equals the language uid (notice that
 // 0=english, 1=danish and 2=german is SPECIFIC to this database,
 // because these numbers refer to uid's of the table sys_language)
-$pointer = ('<img src="' . t3lib_extMgm::siteRelPath('statictemplates')) . 'media/icons_misc/content_client.gif" width="7" height="10" align="middle" alt="" />';
+$pointer = '<img src="' . t3lib_extMgm::siteRelPath('statictemplates') . 'media/icons_misc/content_client.gif" width="7" height="10" align="middle" alt="" />';
 // Set each icon. If the language is the current, red arrow is printed
 // to the left. If the language is NOT found (represented by a
 // pages_language_overlay record on this page), the icon is dimmed.
 $flags = array();
-$flags[] = ((($GLOBALS['TSFE']->sys_language_uid == 0 ? $pointer : '') . '<a href="') . htmlspecialchars((('index.php?id=' . $GLOBALS['TSFE']->id) . '&L=0'))) . '" target="_top"><img src="typo3/sysext/statictemplates/media/uploads/flag_uk.gif" width="21" height="13" hspace="5" border="0" alt="" /></a>';
-$flags[] = ((((($GLOBALS['TSFE']->sys_language_uid == 1 ? $pointer : '') . '<a href="') . htmlspecialchars((('index.php?id=' . $GLOBALS['TSFE']->id) . '&L=1'))) . '" target="_top"><img src="typo3/sysext/statictemplates/media/uploads/flag_dk') . ($langArr[1] ? '' : '_d')) . '.gif" width="21" height="13" hspace="5" border="0" alt="" /></a>';
-$flags[] = ((((($GLOBALS['TSFE']->sys_language_uid == 2 ? $pointer : '') . '<a href="') . htmlspecialchars((('index.php?id=' . $GLOBALS['TSFE']->id) . '&L=2'))) . '" target="_top"><img src="typo3/sysext/statictemplates/media/uploads/flag_de') . ($langArr[2] ? '' : '_d')) . '.gif" width="21" height="13" hspace="5" border="0" alt="" /></a>';
+$flags[] = ($GLOBALS['TSFE']->sys_language_uid == 0 ? $pointer : '') . '<a href="' . htmlspecialchars(('index.php?id=' . $GLOBALS['TSFE']->id . '&L=0')) . '" target="_top"><img src="typo3/sysext/statictemplates/media/uploads/flag_uk.gif" width="21" height="13" hspace="5" border="0" alt="" /></a>';
+$flags[] = ($GLOBALS['TSFE']->sys_language_uid == 1 ? $pointer : '') . '<a href="' . htmlspecialchars(('index.php?id=' . $GLOBALS['TSFE']->id . '&L=1')) . '" target="_top"><img src="typo3/sysext/statictemplates/media/uploads/flag_dk' . ($langArr[1] ? '' : '_d') . '.gif" width="21" height="13" hspace="5" border="0" alt="" /></a>';
+$flags[] = ($GLOBALS['TSFE']->sys_language_uid == 2 ? $pointer : '') . '<a href="' . htmlspecialchars(('index.php?id=' . $GLOBALS['TSFE']->id . '&L=2')) . '" target="_top"><img src="typo3/sysext/statictemplates/media/uploads/flag_de' . ($langArr[2] ? '' : '_d') . '.gif" width="21" height="13" hspace="5" border="0" alt="" /></a>';
 // Make the little menu. Notice, the menu does ONLY transfer the page-id and
 // the "L" variable (which is also SPECIFIC for this website because "L" has
 // been used in the extension template used to control the language setup)
-$content = ('<table border="0" cellpadding="0" cellspacing="0"><tr><td><img src="clear.gif" width="30" height="1" alt="" /></td><td>' . implode('', $flags)) . '</td></tr></table>';
+$content = '<table border="0" cellpadding="0" cellspacing="0"><tr><td><img src="clear.gif" width="30" height="1" alt="" /></td><td>' . implode('', $flags) . '</td></tr></table>';
 ?>
\ No newline at end of file
index 7942d9d..85c1858 100644 (file)
@@ -55,8 +55,8 @@
  */
 function user_typoLinkPopUp($content, $conf)
 {
-       $aOnClick = ('openPic(\'' . $GLOBALS['TSFE']->baseUrlWrap($content['url'])) . '\',\'popupwin\',\'width=400,height=500,status=0,menubar=0\'); return false;';
-       $TAG = ((('<a href="#" onclick="' . htmlspecialchars($aOnClick)) . '"') . $content['aTagParams']) . '>';
+       $aOnClick = 'openPic(\'' . $GLOBALS['TSFE']->baseUrlWrap($content['url']) . '\',\'popupwin\',\'width=400,height=500,status=0,menubar=0\'); return false;';
+       $TAG = '<a href="#" onclick="' . htmlspecialchars($aOnClick) . '"' . $content['aTagParams'] . '>';
        return $TAG;
 }
 ?>
\ No newline at end of file
index 888ad2c..17de7b0 100644 (file)
@@ -11,7 +11,7 @@ if (!is_object($TSFE)) {
  */
 function user_printTimeExt($content, $conf)
 {
-       return ('<font color="red">Dynamic time: ' . date('H:i:s')) . '</font><br />';
+       return '<font color="red">Dynamic time: ' . date('H:i:s') . '</font><br />';
 }
 ?>