[BUGFIX] CGL violations DoubleQuoteUsageContainsVar
authorRoland Waldner <most.wanted@gmx.at>
Fri, 25 May 2012 15:54:56 +0000 (17:54 +0200)
committerJigal van Hemert <jigal@xs4all.nl>
Wed, 30 May 2012 07:13:41 +0000 (09:13 +0200)
There are violations against the CGL rule "DoubleQuoteUsageContainsVar".
These will be reduced.

Change-Id: Ibb45e3aec90dbe0299e35dcebda4a62386fbf874
Resolves: #37489
Releases: 6.0
Reviewed-on: http://review.typo3.org/11619
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert
15 files changed:
t3lib/class.t3lib_cs.php
t3lib/class.t3lib_div.php
t3lib/class.t3lib_tceforms_inline.php
t3lib/class.t3lib_tsparser.php
t3lib/file/Driver/DriverRegistry.php
t3lib/file/Driver/LocalDriver.php
t3lib/file/Storage.php
tests/Unit/t3lib/class.t3lib_autoloaderTest.php
typo3/class.browse_links.php
typo3/sysext/em/classes/index.php
typo3/sysext/em/classes/install/class.tx_em_install.php
typo3/sysext/em/classes/tools/class.tx_em_tools_unzip.php
typo3/sysext/indexed_search/mod/index.php
typo3/sysext/rtehtmlarea/mod4/class.tx_rtehtmlarea_select_image.php
typo3/sysext/rtehtmlarea/pi1/class.tx_rtehtmlarea_pi1.php

index c86b408..58e68e3 100644 (file)
@@ -1177,12 +1177,12 @@ class t3lib_cs {
 
                        switch ($cat{0}) {
                                case 'M': // mark (accent, umlaut, ...)
-                                       $mark["U+$char"] = 1;
+                                       $mark['U+' . $char] = 1;
                                        break;
 
                                case 'N': // numeric value
                                        if ($ord > 0x80 && $num != '') {
-                                               $number["U+$char"] = $num;
+                                               $number['U+' . $char] = $num;
                                        }
                        }
 
@@ -1194,7 +1194,7 @@ class t3lib_cs {
                                        $c += 32;
                                }
 
-                               $decomposition["U+$char"] = array(dechex($c));
+                               $decomposition['U+' . $char] = array(dechex($c));
                                continue;
                        }
 
@@ -1223,7 +1223,7 @@ class t3lib_cs {
                                        case '<vertical>':
                                                continue 2;
                                }
-                               $decomposition["U+$char"] = explode(' ', $match[2]);
+                               $decomposition['U+' . $char] = explode(' ', $match[2]);
                        }
                }
                fclose($fh);
@@ -1278,9 +1278,9 @@ class t3lib_cs {
                                        if ($line{0} != '#' && trim($line) != '') {
                                                list($char, $translit) = t3lib_div::trimExplode(';', $line);
                                                if (!$translit) {
-                                                       $omit["U+$char"] = 1;
+                                                       $omit['U+' . $char] = 1;
                                                }
-                                               $decomposition["U+$char"] = explode(' ', $translit);
+                                               $decomposition['U+' . $char] = explode(' ', $translit);
 
                                        }
                                }
@@ -1294,11 +1294,11 @@ class t3lib_cs {
 
                        while ($code_value = array_shift($to)) {
                                        // Do recursive decomposition
-                               if (isset($decomposition["U+$code_value"])) {
-                                       foreach (array_reverse($decomposition["U+$code_value"]) as $cv) {
+                               if (isset($decomposition['U+' . $code_value])) {
+                                       foreach (array_reverse($decomposition['U+' . $code_value]) as $cv) {
                                                array_unshift($to, $cv);
                                        }
-                               } elseif (!isset($mark["U+$code_value"])) { // remove mark
+                               } elseif (!isset($mark['U+' . $code_value])) { // remove mark
                                        array_push($code_decomp, $code_value);
                                }
                        }
index 1306add..13ac802 100644 (file)
@@ -4289,7 +4289,7 @@ final class t3lib_div {
 
                        // Check prefix is valid:
                if ($checkPrefix && !self::hasValidClassPrefix($funcRef, array($checkPrefix))) {
-                       $errorMsg = "Function/class '$funcRef' was not prepended with '$checkPrefix'";
+                       $errorMsg = 'Function/class \'' . $funcRef . '\' was not prepended with \'' . $checkPrefix . '\'';
                        if ($errorMode == 2) {
                                throw new InvalidArgumentException($errorMsg, 1294585864);
                        } elseif (!$errorMode) {
index f04f8f6..35ae087 100644 (file)
@@ -470,7 +470,7 @@ class t3lib_TCEforms_inline {
                $objectId = $this->inlineNames['object'] . self::Structure_Separator . $foreign_table . self::Structure_Separator . $rec['uid'];
                $expandSingle = $config['appearance']['expandSingle'] ? 1 : 0;
                        // We need the returnUrl of the main script when loading the fields via AJAX-call (to correct wizard code, so include it as 3rd parameter)
-               $onClick = "return inline.expandCollapseRecord('" . htmlspecialchars($objectId) . "', $expandSingle, '" . rawurlencode(t3lib_div::getIndpEnv('REQUEST_URI')) . "')";
+               $onClick = "return inline.expandCollapseRecord('" . htmlspecialchars($objectId) . "', " . $expandSingle . ", '" . rawurlencode(t3lib_div::getIndpEnv('REQUEST_URI')) . "')";
 
                        // Pre-Processing:
                $isOnSymmetricSide = t3lib_loadDBGroup::isOnSymmetricSide($parentUid, $config, $rec);
@@ -951,7 +951,7 @@ class t3lib_TCEforms_inline {
                                $icon = 'actions-document-new';
                                $className = 'typo3-newRecordLink';
                                $attributes['class'] = 'inlineNewButton ' . $this->inlineData['config'][$nameObject]['md5'];
-                               $attributes['onclick'] = "return inline.createNewRecord('$objectPrefix')";
+                               $attributes['onclick'] = "return inline.createNewRecord('" . $objectPrefix . "')";
                                if (isset($conf['inline']['inlineNewButtonStyle']) && $conf['inline']['inlineNewButtonStyle']) {
                                        $attributes['style'] = $conf['inline']['inlineNewButtonStyle'];
                                }
@@ -963,14 +963,14 @@ class t3lib_TCEforms_inline {
                                $title = $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_misc.xml:localizeAllRecords', 1);
                                $icon = 'actions-document-localize';
                                $className = 'typo3-localizationLink';
-                               $attributes['onclick'] = "return inline.synchronizeLocalizeRecords('$objectPrefix', 'localize')";
+                               $attributes['onclick'] = "return inline.synchronizeLocalizeRecords('" . $objectPrefix . "', 'localize')";
                        break;
                        case 'synchronize':
                                $title = $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_misc.xml:synchronizeWithOriginalLanguage', 1);
                                $icon = 'actions-document-synchronize';
                                $className = 'typo3-synchronizationLink';
                                $attributes['class'] = 'inlineNewButton ' . $this->inlineData['config'][$nameObject]['md5'];
-                               $attributes['onclick'] = "return inline.synchronizeLocalizeRecords('$objectPrefix', 'synchronize')";
+                               $attributes['onclick'] = "return inline.synchronizeLocalizeRecords('" . $objectPrefix . "', 'synchronize')";
                        break;
                }
                        // Create the link:
@@ -1196,8 +1196,8 @@ class t3lib_TCEforms_inline {
                        $jsonArray = array(
                                'data' => $item,
                                'scriptCall' => array(
-                                       "inline.domAddNewRecord('bottom','" . $this->inlineNames['object'] . "_records','$objectPrefix',json.data);",
-                                       "inline.memorizeAddRecord('$objectPrefix','" . $record['uid'] . "',null,'$foreignUid');"
+                                       "inline.domAddNewRecord('bottom','" . $this->inlineNames['object'] . "_records','" . $objectPrefix . "',json.data);",
+                                       "inline.memorizeAddRecord('" . $objectPrefix . "','" . $record['uid'] . "',null,'" . $foreignUid . "');"
                                )
                        );
 
@@ -1206,15 +1206,15 @@ class t3lib_TCEforms_inline {
                        $jsonArray = array(
                                'data' => $item,
                                'scriptCall' => array(
-                                       "inline.domAddNewRecord('after','" . $domObjectId . '_div' . "','$objectPrefix',json.data);",
-                                       "inline.memorizeAddRecord('$objectPrefix','" . $record['uid'] . "','" . $current['uid'] . "','$foreignUid');"
+                                       "inline.domAddNewRecord('after','" . $domObjectId . '_div' . "','" . $objectPrefix. "',json.data);",
+                                       "inline.memorizeAddRecord('" . $objectPrefix . "','" . $record['uid'] . "','" . $current['uid'] . "','" . $foreignUid . "');"
                                )
                        );
                }
                $this->getCommonScriptCalls($jsonArray, $config);
                        // Collapse all other records if requested:
                if (!$collapseAll && $expandSingle) {
-                       $jsonArray['scriptCall'][] = "inline.collapseAllRecords('$objectId', '$objectPrefix', '" . $record['uid'] . "');";
+                       $jsonArray['scriptCall'][] = "inline.collapseAllRecords('" . $objectId . "', '" . $objectPrefix . "', '" . $record['uid'] . "');";
                }
                        // Tell the browser to scroll to the newly created record
                $jsonArray['scriptCall'][] = "Element.scrollTo('" . $objectId . "_div');";
@@ -1357,7 +1357,7 @@ class t3lib_TCEforms_inline {
                        $row = $this->getRecord($this->inlineFirstPid, $current['table'], $item);
                        $selectedValue = ($foreignSelector ? "'" . $row[$foreignSelector] . "'" : 'null');
                        $data .= $this->renderForeignRecord($parent['uid'], $row, $parent['config']);
-                       $jsonArrayScriptCall[] = "inline.memorizeAddRecord('$nameObjectForeignTable', '" . $item . "', null, $selectedValue);";
+                       $jsonArrayScriptCall[] = "inline.memorizeAddRecord('" . $nameObjectForeignTable . "', '" . $item . "', null, " . $selectedValue . ");";
                                // Remove possible virtual records in the form which showed that a child records could be localized:
                        if (isset($row[$transOrigPointerField]) && $row[$transOrigPointerField]) {
                                $jsonArrayScriptCall[] = "inline.fadeAndRemove('" . $nameObjectForeignTable . self::Structure_Separator . $row[$transOrigPointerField] . '_div' . "');";
@@ -1367,7 +1367,7 @@ class t3lib_TCEforms_inline {
                        $jsonArray['data'] = $data;
                        array_unshift(
                                $jsonArrayScriptCall,
-                               "inline.domAddNewRecord('bottom', '" . $nameObject . "_records', '$nameObjectForeignTable', json.data);"
+                               "inline.domAddNewRecord('bottom', '" . $nameObject . "_records', '" . $nameObjectForeignTable . "', json.data);"
                        );
                }
 
index 69326f9..fac573e 100644 (file)
@@ -748,7 +748,7 @@ class t3lib_TSparser {
                                        }
 
                                                // Insert reference to the file in the rest content
-                                       $restContent[] = "<INCLUDE_TYPOSCRIPT: source=\"FILE:$fileName\">";
+                                       $restContent[] = '<INCLUDE_TYPOSCRIPT: source="FILE: ' . $fileName . '">';
 
                                                // Reset variables (preparing for the next commented include statement)
                                        $fileContent = array();
index 6e1009d..9cd8d89 100644 (file)
@@ -78,7 +78,7 @@ class t3lib_file_Driver_DriverRegistry implements t3lib_Singleton {
        public function registerDriverClass($className, $shortName = NULL, $label = NULL, $flexFormDataStructurePathAndFilename = NULL) {
                        // check if the class is available for TYPO3 before registering the driver
                if (!class_exists($className)) {
-                       throw new InvalidArgumentException("Class $className does not exist.", 1314979197);
+                       throw new InvalidArgumentException('Class ' . $className . ' does not exist.', 1314979197);
                }
 
                if ($shortName === '') {
@@ -86,7 +86,7 @@ class t3lib_file_Driver_DriverRegistry implements t3lib_Singleton {
                }
 
                if (array_key_exists($shortName, $this->drivers)) {
-                       throw new InvalidArgumentException("Driver $shortName is already registered.", 1314979451);
+                       throw new InvalidArgumentException('Driver ' . $shortName . ' is already registered.', 1314979451);
                }
 
                $this->drivers[$shortName] = $className;
@@ -97,7 +97,7 @@ class t3lib_file_Driver_DriverRegistry implements t3lib_Singleton {
                        'flexFormDS' => $flexFormDataStructurePathAndFilename
                );
 
-               t3lib_div::sysLog("Registered driver $shortName ($className) " . t3lib_utility_Debug::debugTrail(), 't3lib_file_Driver_DriverRegistry');
+               t3lib_div::sysLog('Registered driver ' . $shortName . ' (' . $className . ') ' . t3lib_utility_Debug::debugTrail(), 't3lib_file_Driver_DriverRegistry');
 
                return TRUE;
        }
index 872237f..02b5159 100644 (file)
@@ -246,7 +246,7 @@ class t3lib_file_Driver_LocalDriver extends t3lib_file_Driver_AbstractDriver {
                        // don't use $this->fileExists() because we need the absolute path to the file anyways, so we can directly
                        // use PHP's filesystem method.
                if (!file_exists($absoluteFilePath)) {
-                       throw new InvalidArgumentException("File $fileIdentifier does not exist.", 1314516809);
+                       throw new InvalidArgumentException('File ' . $fileIdentifier . ' does not exist.', 1314516809);
                }
 
                return $this->extractFileInformation($absoluteFilePath, $dirPath);
@@ -305,7 +305,7 @@ class t3lib_file_Driver_LocalDriver extends t3lib_file_Driver_AbstractDriver {
                        // Strip trailing dots and return
                $cleanFileName = preg_replace('/\.*$/', '', $cleanFileName);
                if (!$cleanFileName) {
-                       throw new t3lib_file_exception_InvalidFileNameException("File name $cleanFileName is invalid.", 1320288991);
+                       throw new t3lib_file_exception_InvalidFileNameException('File name ' . $cleanFileName . ' is invalid.', 1320288991);
                }
 
                return $cleanFileName;
@@ -328,7 +328,7 @@ class t3lib_file_Driver_LocalDriver extends t3lib_file_Driver_AbstractDriver {
                $realPath = rtrim($this->absoluteBasePath . trim($path, '/'), '/') . '/';
 
                if (!is_dir($realPath)) {
-                       throw new InvalidArgumentException("Cannot list items in directory $path - does not exist or is no directory", 1314349666);
+                       throw new InvalidArgumentException('Cannot list items in directory ' . $path. ' - does not exist or is no directory', 1314349666);
                }
 
                if ($start > 0) {
@@ -601,7 +601,7 @@ class t3lib_file_Driver_LocalDriver extends t3lib_file_Driver_AbstractDriver {
                                break;
                        default:
                                throw new RuntimeException(
-                                       "Hash algorithm $hashAlgorithm is not implemented.",
+                                       'Hash algorithm ' . $hashAlgorithm . ' is not implemented.',
                                        1329644451
                                );
                }
@@ -639,7 +639,7 @@ class t3lib_file_Driver_LocalDriver extends t3lib_file_Driver_AbstractDriver {
                }
 
                if ($moveResult !== TRUE) {
-                       throw new RuntimeException("Moving file $localFilePath to $targetPath failed.", 1314803096);
+                       throw new RuntimeException('Moving file ' . $localFilePath . ' to ' . $targetPath . ' failed.', 1314803096);
                }
 
                clearstatcache();
@@ -746,7 +746,7 @@ class t3lib_file_Driver_LocalDriver extends t3lib_file_Driver_AbstractDriver {
 
                $result = rename($localFilePath, $filePath);
                if ($result === FALSE) {
-                       throw new RuntimeException("Replacing file $filePath with $localFilePath failed.", 1315314711);
+                       throw new RuntimeException('Replacing file ' . $filePath . ' with ' . $localFilePath . ' failed.', 1315314711);
                }
 
                $fileInfo = $this->getFileInfoByIdentifier($file->getIdentifier());
@@ -770,7 +770,7 @@ class t3lib_file_Driver_LocalDriver extends t3lib_file_Driver_AbstractDriver {
                $result = copy($localFilePath, $absoluteFilePath);
 
                if ($result === FALSE || !file_exists($absoluteFilePath)) {
-                       throw new RuntimeException("Adding file $localFilePath at $fileIdentifier failed.");
+                       throw new RuntimeException('Adding file ' . $localFilePath . ' at ' . $fileIdentifier . ' failed.');
                }
 
                return $fileIdentifier;
@@ -790,7 +790,7 @@ class t3lib_file_Driver_LocalDriver extends t3lib_file_Driver_AbstractDriver {
                $result = unlink($targetPath);
 
                if ($result === FALSE || file_exists($targetPath)) {
-                       throw new RuntimeException("Deleting file $identifier failed.", 1320381534);
+                       throw new RuntimeException('Deleting file ' . $identifier . ' failed.', 1320381534);
                }
 
                return TRUE;
@@ -832,7 +832,7 @@ class t3lib_file_Driver_LocalDriver extends t3lib_file_Driver_AbstractDriver {
 
                $result = rename($sourcePath, $this->absoluteBasePath . $targetIdentifier);
                if ($result === FALSE) {
-                       throw new RuntimeException("Moving file $sourcePath to $targetIdentifier failed.", 1315314712);
+                       throw new RuntimeException('Moving file ' . $sourcePath . ' to ' . $targetIdentifier . ' failed.', 1315314712);
                }
 
                return $targetIdentifier;
@@ -902,7 +902,7 @@ class t3lib_file_Driver_LocalDriver extends t3lib_file_Driver_AbstractDriver {
 
                $result = rename($sourcePath, $targetPath);
                if ($result === FALSE) {
-                       throw new RuntimeException("Moving folder $sourcePath to $targetPath failed.", 1320711817);
+                       throw new RuntimeException('Moving folder ' . $sourcePath . ' to ' . $targetPath . ' failed.', 1320711817);
                }
 
                        // Create a mapping from old to new identifiers
@@ -975,7 +975,7 @@ class t3lib_file_Driver_LocalDriver extends t3lib_file_Driver_AbstractDriver {
 
                $result = rename($sourcePath, $targetPath);
                if ($result === FALSE) {
-                       throw new RuntimeException("Renaming file $sourcePath to $targetPath failed.", 1320375115);
+                       throw new RuntimeException('Renaming file ' . $sourcePath . ' to ' . $targetPath . ' failed.', 1320375115);
                }
 
                return $newIdentifier;
@@ -990,7 +990,7 @@ class t3lib_file_Driver_LocalDriver extends t3lib_file_Driver_AbstractDriver {
        protected function checkFilePath($filePath) {
                        // filePath must be valid
                if (!$this->isPathValid($filePath)) {
-                       throw new t3lib_file_exception_InvalidPathException("File $filePath is not valid (\"..\" and \"//\" is not allowed in path).", 1320286857);
+                       throw new t3lib_file_exception_InvalidPathException('File ' . $filePath . ' is not valid (".." and "//" is not allowed in path).', 1320286857);
                }
        }
 
@@ -1195,7 +1195,7 @@ class t3lib_file_Driver_LocalDriver extends t3lib_file_Driver_AbstractDriver {
                $result = touch($this->absoluteBasePath . $filePath);
                clearstatcache();
                if ($result !== TRUE) {
-                       throw new RuntimeException("Creating file $filePath failed.", 1320569854);
+                       throw new RuntimeException('Creating file ' . $filePath . ' failed.', 1320569854);
                }
                $fileInfo = $this->getFileInfoByIdentifier($filePath);
 
index 6cbfcfd..12c0835 100644 (file)
@@ -493,7 +493,7 @@ class t3lib_file_Storage {
                if ($this->driver->folderExists($folderIdentifier) === FALSE) {
                                // if there is an error, this is important and should be handled
                                // as otherwise the user would see the whole storage without any restrictions for the filemounts
-                       throw new t3lib_file_exception_FolderDoesNotExistException("Folder for file mount $folderIdentifier does not exist.", 1334427099);
+                       throw new t3lib_file_exception_FolderDoesNotExistException('Folder for file mount ' . $folderIdentifier . ' does not exist.', 1334427099);
                }
 
                $folderObject = $this->driver->getFolder($folderIdentifier);
@@ -1140,7 +1140,7 @@ class t3lib_file_Storage {
                        // Max upload size (kb) for files.
                $maxUploadFileSize = t3lib_div::getMaxUploadFileSize() * 1024;
                if ($uploadedFileSize >= $maxUploadFileSize) {
-                       throw new t3lib_file_exception_UploadSizeException("The uploaded file exceeds the size-limit of $maxUploadFileSize bytes", 1322110041);
+                       throw new t3lib_file_exception_UploadSizeException('The uploaded file exceeds the size-limit of ' . $maxUploadFileSize . ' bytes', 1322110041);
                }
 
                        // Check if targetFolder is writable
@@ -1150,7 +1150,7 @@ class t3lib_file_Storage {
 
                        // Check for a valid file extension
                if (!$this->checkFileExtensionPermission($targetFileName)) {
-                       throw new t3lib_file_exception_IllegalFileExtensionException("Extension of file name is not allowed in \"$targetFileName\"!", 1322120271);
+                       throw new t3lib_file_exception_IllegalFileExtensionException('Extension of file name is not allowed in "' . $targetFileName . '"!', 1322120271);
                }
        }
 
@@ -1585,7 +1585,7 @@ class t3lib_file_Storage {
                // TODO access checks
                $returnObject = NULL;
                if ($this->driver->folderExistsInFolder($newName, $folderObject)) {
-                       throw new InvalidArgumentException("The folder $newName already exists in folder " . $folderObject->getIdentifier(), 1325418870);
+                       throw new InvalidArgumentException('The folder ' . $newName . ' already exists in folder ' . $folderObject->getIdentifier(), 1325418870);
                }
 
                $this->emitPreFolderRenameSignal($folderObject, $newName);
@@ -1745,7 +1745,7 @@ class t3lib_file_Storage {
         */
        public function getFolder($identifier) {
                if (!$this->driver->folderExists($identifier)) {
-                       throw new t3lib_file_exception_FolderDoesNotExistException("Folder $identifier does not exist.", 1320575630);
+                       throw new t3lib_file_exception_FolderDoesNotExistException('Folder ' . $identifier . ' does not exist.', 1320575630);
                }
 
                $folderObject = $this->driver->getFolder($identifier);
index 1fa3a5d..5f0ec0b 100644 (file)
@@ -93,8 +93,8 @@ class t3lib_autoloaderTest extends Tx_Phpunit_TestCase {
         */
        protected function createFakeExtension() {
                $extKey = strtolower(uniqid('testing'));
-               $absExtPath = PATH_site . "typo3temp/$extKey/";
-               $relPath = "typo3temp/$extKey/";
+               $absExtPath = PATH_site . 'typo3temp/' . $extKey . '/';
+               $relPath = 'typo3temp/' . $extKey . '/';
                t3lib_div::mkdir($absExtPath);
 
                $GLOBALS['TYPO3_LOADED_EXT'][$extKey] = array(
@@ -136,14 +136,14 @@ class t3lib_autoloaderTest extends Tx_Phpunit_TestCase {
         */
        public function autoloadFindsClassFileDefinedInExtAutoloadFile() {
                $extKey = $this->createFakeExtension();
-               $extPath = PATH_site . "typo3temp/$extKey/";
+               $extPath = PATH_site . 'typo3temp/' . $extKey . '/';
                $autoloaderFile = $extPath . "ext_autoload.php";
 
                $class = strtolower("tx_${extKey}_" . uniqid(''));
                $file = $extPath . uniqid('') . '.php';
 
                file_put_contents($file, "<?php\n\nthrow new RuntimeException('', 1310203812);\n\n?>");
-               file_put_contents($autoloaderFile, "<?php\n\nreturn array('$class' => '$file');\n\n?>");
+               file_put_contents($autoloaderFile, "<?php\n\nreturn array('" . $class. "' => '" . $file . "');\n\n?>");
 
                        // Inject a dummy for the core_phpcode cache to force the autoloader
                        // to re calculate the registry
@@ -165,14 +165,14 @@ class t3lib_autoloaderTest extends Tx_Phpunit_TestCase {
         */
        public function unregisterAutoloaderWritesLowerCasedClassFileToCache() {
                $extKey = $this->createFakeExtension();
-               $extPath = PATH_site . "typo3temp/$extKey/";
+               $extPath = PATH_site . 'typo3temp/' . $extKey . '/';
                $autoloaderFile = $extPath . "ext_autoload.php";
 
                        // A case sensitive key (FooBar) in ext_autoload file
                $class = "tx_${extKey}_" . uniqid('FooBar');
                $file = $extPath . uniqid('') . '.php';
 
-               file_put_contents($autoloaderFile, "<?php\n\nreturn array('$class' => '$file');\n\n?>");
+               file_put_contents($autoloaderFile, "<?php\n\nreturn array('" . $class . "' => '" . $file . "');\n\n?>");
 
                        // Inject a dummy for the core_phpcode cache to force the autoloader
                        // to re calculate the registry
@@ -195,7 +195,7 @@ class t3lib_autoloaderTest extends Tx_Phpunit_TestCase {
         */
        public function autoloadFindsCamelCasedClassFileIfExtAutoloadEntryIsLowerCased() {
                $extKey = $this->createFakeExtension();
-               $extPath = PATH_site . "typo3temp/$extKey/";
+               $extPath = PATH_site . 'typo3temp/' . $extKey . '/';
 
                        // A case sensitive key (FooBar) in ext_autoload file
                $class = "tx_${extKey}_" . uniqid('FooBar');
@@ -224,7 +224,7 @@ class t3lib_autoloaderTest extends Tx_Phpunit_TestCase {
         */
        public function autoloadFindsClassFileThatRespectsExtbaseNamingSchemeWithoutExtAutoloadFile() {
                $extKey = $this->createFakeExtension();
-               $extPath = PATH_site . "typo3temp/$extKey/";
+               $extPath = PATH_site . 'typo3temp/' . $extKey . '/';
 
                        // Create a class named Tx_Extension_Foo123_Bar456
                        // to find file extension/Classes/Foo123/Bar456.php
@@ -252,7 +252,7 @@ class t3lib_autoloaderTest extends Tx_Phpunit_TestCase {
         */
        public function unregisterAutoloaderWritesClassFileThatRespectsExtbaseNamingSchemeToCacheFile() {
                $extKey = $this->createFakeExtension();
-               $extPath = PATH_site . "typo3temp/$extKey/";
+               $extPath = PATH_site . 'typo3temp/' . $extKey . '/';
 
                $pathSegment = 'Foo' . uniqid();
                $fileName = 'Bar' . uniqid();
@@ -278,7 +278,7 @@ class t3lib_autoloaderTest extends Tx_Phpunit_TestCase {
         */
        public function unregisterAutoloaderWritesClassFileLocationOfClassRespectingExtbaseNamingSchemeToCacheFile() {
                $extKey = $this->createFakeExtension();
-               $extPath = PATH_site . "typo3temp/$extKey/";
+               $extPath = PATH_site . 'typo3temp/' . $extKey . '/';
 
                $pathSegment = 'Foo' . uniqid();
                $fileName = 'Bar' . uniqid();
@@ -305,13 +305,13 @@ class t3lib_autoloaderTest extends Tx_Phpunit_TestCase {
        public function getClassPathByRegistryLookupFindsClassPrefixedWithUxRegisteredInExtAutoloadFile() {
                        // Create a dummy extension with a path to a 'ux_' prefixed php file
                $extKey = $this->createFakeExtension();
-               $extPath = PATH_site . "typo3temp/$extKey/";
+               $extPath = PATH_site . 'typo3temp/' . $extKey . '/';
                $autoloaderFile = $extPath . "ext_autoload.php";
 
                $class = strtolower("ux_tx_${extKey}_" . uniqid(''));
                $file = $extPath . uniqid('') . '.php';
 
-               file_put_contents($autoloaderFile, "<?php\n\nreturn array('$class' => '$file');\n\n?>");
+               file_put_contents($autoloaderFile, "<?php\n\nreturn array('" . $class . "' => '" . $file . "');\n\n?>");
 
                        // Inject a dummy for the core_phpcode cache to force the autoloader
                        // to re calculate the registry
@@ -332,17 +332,17 @@ class t3lib_autoloaderTest extends Tx_Phpunit_TestCase {
        public function getClassPathByRegistryLookupFindsDeprecatedXclassFilePathRegisteredInTypo3ConfVars() {
                        // Create a fake extension
                $extKey = $this->createFakeExtension();
-               $extPath = PATH_site . "typo3temp/$extKey/";
+               $extPath = PATH_site . 'typo3temp/' . $extKey . '/';
                $autoloaderFile = $extPath . "ext_autoload.php";
 
                        // Feed ext_autoload with a base file
                $class = strtolower("tx_${extKey}_" . uniqid(''));
                $fileName = uniqid('') . '.php';
                $file = $extPath . $fileName;
-               file_put_contents($autoloaderFile, "<?php\n\nreturn array('$class' => '$file');\n\n?>");
+               file_put_contents($autoloaderFile, "<?php\n\nreturn array('" . $class . "' => '" . $file . "');\n\n?>");
 
                        // Register a xclass for the base file
-               $GLOBALS['TYPO3_CONF_VARS'][TYPO3_MODE]['XCLASS']["typo3temp/$extKey/$fileName"] = "typo3temp/$extKey/xclassFile";
+               $GLOBALS['TYPO3_CONF_VARS'][TYPO3_MODE]['XCLASS']['typo3temp/' . $extKey . '/' . $fileName] = 'typo3temp/' . $extKey . '/xclassFile';
 
                        // Inject a dummy for the core_phpcode cache to force the autoloader
                        // to re calculate the registry
@@ -354,7 +354,7 @@ class t3lib_autoloaderTest extends Tx_Phpunit_TestCase {
                t3lib_autoloader::registerAutoloader();
 
                        // See if the xclass lookup is successful
-               $this->assertSame("typo3temp/$extKey/xclassFile", t3lib_autoloader::getClassPathByRegistryLookup("ux_$class"));
+               $this->assertSame('typo3temp/' . $extKey . '/xclassFile', t3lib_autoloader::getClassPathByRegistryLookup('ux_' . $class));
        }
 
        /**
@@ -363,19 +363,19 @@ class t3lib_autoloaderTest extends Tx_Phpunit_TestCase {
        public function unregisterAutoloaderWritesDeprecatedTypo3ConfVarsRegisteredXclassClassFoundByGetClassPathByRegistryLookupToCache() {
                        // Create a fake extension
                $extKey = $this->createFakeExtension();
-               $extPath = PATH_site . "typo3temp/$extKey/";
+               $extPath = PATH_site . 'typo3temp/' . $extKey . '/';
                $autoloaderFile = $extPath . "ext_autoload.php";
 
                        // Feed ext_autoload with a base file and the class file
                $class = strtolower("tx_${extKey}_" . uniqid(''));
                $fileName = uniqid('') . '.php';
                $file = $extPath . $fileName;
-               $xClassFile = "typo3temp/$extKey/xclassFile";
-               file_put_contents($autoloaderFile, "<?php\n\nreturn array('$class' => '$file');\n\n?>");
+               $xClassFile = 'typo3temp/' . $extKey . '/xclassFile';
+               file_put_contents($autoloaderFile, "<?php\n\nreturn array('" . $class . "' => '" . $file . "');\n\n?>");
                file_put_contents(PATH_site . $xClassFile, "<?php\n\ndie();\n\n?>");
 
                        // Register a xclass for the base file
-               $GLOBALS['TYPO3_CONF_VARS'][TYPO3_MODE]['XCLASS']["typo3temp/$extKey/$fileName"] = $xClassFile;
+               $GLOBALS['TYPO3_CONF_VARS'][TYPO3_MODE]['XCLASS']['typo3temp/' . $extKey . '/' . $fileName] = $xClassFile;
 
                        // Inject a dummy for the core_phpcode cache to force the autoloader
                        // to re calculate the registry
@@ -389,7 +389,7 @@ class t3lib_autoloaderTest extends Tx_Phpunit_TestCase {
                t3lib_autoloader::unregisterAutoloader();
                t3lib_autoloader::registerAutoloader();
 
-               t3lib_autoloader::getClassPathByRegistryLookup("ux_$class");
+               t3lib_autoloader::getClassPathByRegistryLookup('ux_' . $class);
                t3lib_autoloader::unregisterAutoloader();
        }
 }
index ae2a605..b32cef0 100644 (file)
@@ -2261,7 +2261,7 @@ class browse_links {
                                        'fileExt'  => $fileExtension,
                                        'fileIcon' => $icon,
                                );
-                               $ATag = "<a href=\"#\" onclick=\"return BrowseLinks.File.insertElement('file_$filesIndex');\">";
+                               $ATag = "<a href=\"#\" onclick=\"return BrowseLinks.File.insertElement('file_" . $filesIndex . "');\">";
                                $ATag_alt = substr($ATag, 0, -4).",1);\">";
                                $bulkCheckBox = '<input type="checkbox" class="typo3-bulk-item" name="file_'.$filesIndex.'" value="0" /> ';
 
index f1c9983..4d08c9c 100644 (file)
@@ -2208,7 +2208,7 @@ class SC_mod_tools_em_index extends t3lib_SCbase {
                } else {
                        $sure = $GLOBALS['LANG']->getLL('extUpdateEMCONF_sure');
                        $updateEMConf = $GLOBALS['LANG']->getLL('extUpdateEMCONF_file');
-                       $onClick = "if (confirm('$sure')) {window.location.href='" . t3lib_div::linkThisScript(array(
+                       $onClick = "if (confirm('" . $sure . "')) {window.location.href='" . t3lib_div::linkThisScript(array(
                                'CMD[showExt]' => $extKey,
                                'CMD[doUpdateEMCONF]' => 1
                        )) . "';}";
index 9053044..5df555b 100644 (file)
@@ -738,7 +738,7 @@ class tx_em_Install {
                        } else {
                                $areYouSure = $GLOBALS['LANG']->getLL('extDelete_sure');
                                $deleteFromServer = $GLOBALS['LANG']->getLL('extDelete_from_server');
-                               $onClick = "if (confirm('$areYouSure')) {window.location.href='" . t3lib_div::linkThisScript(array(
+                               $onClick = "if (confirm('" . $areYouSure . "')) {window.location.href='" . t3lib_div::linkThisScript(array(
                                        'CMD[showExt]' => $extKey,
                                        'CMD[doDelete]' => 1,
                                        'CMD[absPath]' => rawurlencode($absPath)
index de37b53..e6f0064 100644 (file)
@@ -1076,7 +1076,7 @@ class tx_em_Tools_Unzip {
                // Create the directory
                if (!@mkdir($p_dir, 0777)) {
                        $this->_errorLog(ARCHIVE_ZIP_ERR_DIR_CREATE_FAIL,
-                               "Unable to create directory '$p_dir'");
+                               'Unable to create directory "' . $p_dir . '"');
                        return tx_em_Tools_Unzip::errorCode();
                }
 
index 1e70cf4..36f26ac 100755 (executable)
@@ -495,7 +495,7 @@ class SC_mod_tools_isearch_index {
                $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('count(*),item_type', 'index_phash', '', 'item_type', 'item_type');
                while($row = $GLOBALS['TYPO3_DB']->sql_fetch_row($res)) {
                        $iT = $row[1];
-                       $recList[] = array($this->tableHead($revTypes[$iT]." ($iT)"), $this->countUniqueTypes($iT)."/".$row[0]);
+                       $recList[] = array($this->tableHead($revTypes[$iT] . ' (' . $iT . ')'), $this->countUniqueTypes($iT)."/".$row[0]);
                }
 
                return $recList;
index 6014447..479bc37 100644 (file)
@@ -1167,7 +1167,7 @@ class tx_rtehtmlarea_select_image extends browse_links {
                                                'fileExt'  => $fileExtension,
                                                'fileIcon' => $icon,
                                        );
-                                       $ATag = "<a href=\"#\" onclick=\"return BrowseLinks.File.insertElement('file_$filesIndex');\">";
+                                       $ATag = "<a href=\"#\" onclick=\"return BrowseLinks.File.insertElement('file_" . $filesIndex . "');\">";
                                        $ATag_alt = substr($ATag, 0, -4).",1);\">";
                                        $ATag_e='</a>';
                                }
index dc11338..66ff401 100644 (file)
@@ -369,7 +369,7 @@ var selectedDictionary = "' . $this->dictionary . '";
                                        $AspellAnswer = shell_exec($AspellCommand);
                                        $AspellResultLines = array();
                                        $AspellResultLines = t3lib_div::trimExplode(LF, $AspellAnswer, 1);
-                                       if(substr($AspellResultLines[0], 0, 6) == 'Error:') echo("{$AspellAnswer}");
+                                       if(substr($AspellResultLines[0], 0, 6) == 'Error:') echo('{' . $AspellAnswer . '}');
                                        t3lib_div::unlink_tempfile($tmpFileName);
                                        if(substr($AspellResultLines['1'], 0, 1) != '*') {
                                                if(!in_array($word, $this->misspelled)) {