[TASK] Replace function chr calls with base constants 22/30722/7
authorTomita Militaru <militarutomita@gmail.com>
Tue, 2 Dec 2014 11:58:36 +0000 (12:58 +0100)
committerWouter Wolters <typo3@wouterwolters.nl>
Tue, 2 Dec 2014 23:41:18 +0000 (00:41 +0100)
Instead of calling function chr multiple times, it can
be replaced with base constants:

* chr(9) - TAB
* chr(10) - LF
* chr(13) - CR

Resolves: #55043
Releases: master
Change-Id: Iad5ffd0e10d436aa8532f2e686a506c30d0dc587
Reviewed-on: http://review.typo3.org/30722
Reviewed-by: Markus Klein <klein.t3@reelworx.at>
Tested-by: Markus Klein <klein.t3@reelworx.at>
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Tested-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
33 files changed:
typo3/sysext/backend/Classes/Controller/Wizard/FormsController.php
typo3/sysext/backend/Classes/Tree/Pagetree/Commands.php
typo3/sysext/core/Classes/Cache/Frontend/PhpFrontend.php
typo3/sysext/core/Classes/Core/CliBootstrap.php
typo3/sysext/core/Classes/Resource/Service/FrontendContentAdapterService.php
typo3/sysext/dbal/Classes/Controller/ModuleController.php
typo3/sysext/dbal/Classes/Database/SqlParser.php
typo3/sysext/extbase/Classes/Command/HelpCommandController.php
typo3/sysext/extbase/Classes/Mvc/Cli/Command.php
typo3/sysext/extbase/Classes/Mvc/Web/Response.php
typo3/sysext/extbase/Classes/Reflection/DocCommentParser.php
typo3/sysext/fluid/Classes/Core/Compiler/TemplateCompiler.php
typo3/sysext/fluid/Classes/Core/ViewHelper/AbstractConditionViewHelper.php
typo3/sysext/fluid/Classes/ViewHelpers/Form/AbstractFormViewHelper.php
typo3/sysext/fluid/Classes/ViewHelpers/Form/SelectViewHelper.php
typo3/sysext/fluid/Classes/ViewHelpers/FormViewHelper.php
typo3/sysext/fluid/Classes/ViewHelpers/Format/HtmlspecialcharsViewHelper.php
typo3/sysext/form/Classes/Domain/Factory/JsonToTypoScript.php
typo3/sysext/form/Classes/View/Form/FormView.php
typo3/sysext/form/Classes/View/Mail/Html/HtmlView.php
typo3/sysext/form/Classes/View/Mail/Plain/Element/CheckboxGroupElementView.php
typo3/sysext/form/Classes/View/Mail/Plain/Element/ContainerElementView.php
typo3/sysext/form/Classes/View/Mail/Plain/Element/FieldsetElementView.php
typo3/sysext/form/Classes/View/Mail/Plain/Element/RadioGroupElementView.php
typo3/sysext/form/Classes/View/Mail/Plain/Element/SelectElementView.php
typo3/sysext/form/Classes/View/Mail/Plain/Element/TextareaElementView.php
typo3/sysext/info_pagetsconfig/Classes/Controller/InfoPageTyposcriptConfigController.php
typo3/sysext/install/Classes/Controller/Action/Tool/AllConfiguration.php
typo3/sysext/install/Classes/Updates/CompatVersionUpdate.php
typo3/sysext/linkvalidator/Classes/Report/LinkValidatorReport.php
typo3/sysext/recordlist/Classes/RecordList/DatabaseRecordList.php
typo3/sysext/rtehtmlarea/Classes/Hook/Install/DeprecatedRteProperties.php
typo3/sysext/rtehtmlarea/Classes/Hook/Install/RteAcronymButtonRenamedToAbbreviation.php

index e8527c2..568a5d1 100644 (file)
@@ -708,7 +708,7 @@ class FormsController {
                                        if ($vv['type'] == 'select' || $vv['type'] == 'radio') {
                                                $options = str_replace(',', '', $vv['options']);
                                                $options = str_replace(
-                                                       array(CR . LF, CR, LF),
+                                                       array(CRLF, CR, LF),
                                                        ', ',
                                                        $options);
                                                $thisLine[2] = $options;
index 5b55cd4..f4dead9 100644 (file)
@@ -112,7 +112,7 @@ class Commands {
                        $data['pages'][$node->getWorkspaceId()][$node->getTextSourceField()] = $updatedLabel;
                        self::processTceCmdAndDataMap(array(), $data);
                } else {
-                       throw new \RuntimeException(implode(chr(10), array('Editing title of page id \'' . $node->getWorkspaceId() . '\' failed. Editing default language is not allowed.')), 1365513336);
+                       throw new \RuntimeException(implode(LF, array('Editing title of page id \'' . $node->getWorkspaceId() . '\' failed. Editing default language is not allowed.')), 1365513336);
                }
        }
 
@@ -211,7 +211,7 @@ class Commands {
                }
                // check errors
                if (count($tce->errorLog)) {
-                       throw new \RuntimeException(implode(chr(10), $tce->errorLog), 1333754629);
+                       throw new \RuntimeException(implode(LF, $tce->errorLog), 1333754629);
                }
                return $returnValues;
        }
index e9e1fc6..dd0379e 100644 (file)
@@ -60,7 +60,7 @@ class PhpFrontend extends StringFrontend {
                                throw new \InvalidArgumentException('"' . $tag . '" is not a valid tag for a cache entry.', 1264023825);
                        }
                }
-               $sourceCode = '<?php' . chr(10) . $sourceCode . chr(10) . '#';
+               $sourceCode = '<?php' . LF . $sourceCode . LF . '#';
                $this->backend->set($entryIdentifier, $sourceCode, $tags, $lifetime);
        }
 
index 2bc4d2d..11e9f8f 100644 (file)
@@ -34,7 +34,7 @@ class CliBootstrap {
                if (substr(php_sapi_name(), 0, 3) === 'cgi') {
                        self::initializeCgiCompatibilityLayerOrDie();
                } elseif (php_sapi_name() !== 'cli') {
-                       die('Not called from a command line interface (e.g. a shell or scheduler).' . chr(10));
+                       die('Not called from a command line interface (e.g. a shell or scheduler).' . LF);
                }
        }
 
@@ -84,10 +84,10 @@ class CliBootstrap {
                $checkEnvVars = array('HTTP_USER_AGENT', 'HTTP_HOST', 'SERVER_NAME', 'REMOTE_ADDR', 'REMOTE_PORT', 'SERVER_PROTOCOL');
                foreach ($checkEnvVars as $var) {
                        if (array_key_exists($var, $_SERVER)) {
-                               echo 'SECURITY CHECK FAILED! This script cannot be used within your browser!' . chr(10);
-                               echo 'If you are sure that we run in a shell or cronjob, please unset' . chr(10);
-                               echo 'environment variable ' . $var . ' (usually using \'unset ' . $var . '\')' . chr(10);
-                               echo 'before starting this script.' . chr(10);
+                               echo 'SECURITY CHECK FAILED! This script cannot be used within your browser!' . LF;
+                               echo 'If you are sure that we run in a shell or cronjob, please unset' . LF;
+                               echo 'environment variable ' . $var . ' (usually using \'unset ' . $var . '\')' . LF;
+                               echo 'before starting this script.' . LF;
                                die;
                        }
                }
index 8db3993..f8892d2 100644 (file)
@@ -120,7 +120,7 @@ class FrontendContentAdapterService {
                                                        $fieldContents = implode(',', $fileFieldContents[$oldFieldType]);
                                                } else {
                                                        // For all other fields, separate by newline
-                                                       $fieldContents = implode(chr(10), $fileFieldContents[$oldFieldType]);
+                                                       $fieldContents = implode(LF, $fileFieldContents[$oldFieldType]);
                                                }
                                                $row[$oldFieldName] = $fieldContents;
                                        }
index 52589b3..6eff5c6 100644 (file)
@@ -237,7 +237,7 @@ updateQryForm(\'' . $input['QUERY'] . '\');
         */
        protected function createFieldsValuesArray($in) {
                $ret = array();
-               $in = explode(chr(10), $in);
+               $in = explode(LF, $in);
                foreach ($in as $v) {
                        $fv = explode('=', $v);
                        $ret[$fv[0]] = $fv[1];
index 020a629..42004b3 100644 (file)
@@ -297,7 +297,7 @@ class SqlParser extends \TYPO3\CMS\Core\Database\SqlParser {
                                $tableOptions = array('postgres' => 'WITHOUT OIDS');
                                // Fetch table/index generation query:
                                $tableName = $this->databaseConnection->quoteName($components['TABLE'], NULL, TRUE);
-                               $query = array_merge($this->databaseConnection->handlerInstance[$this->databaseConnection->lastHandlerKey]->DataDictionary->CreateTableSQL($tableName, implode(',' . chr(10), $fieldsKeys), $tableOptions), $indexKeys);
+                               $query = array_merge($this->databaseConnection->handlerInstance[$this->databaseConnection->lastHandlerKey]->DataDictionary->CreateTableSQL($tableName, implode(',' . LF, $fieldsKeys), $tableOptions), $indexKeys);
                                break;
                }
                return $query;
index 6f17b92..cdcf549 100644 (file)
@@ -153,7 +153,7 @@ class HelpCommandController extends \TYPO3\CMS\Extbase\Mvc\Controller\CommandCon
                if ($command->getDescription() !== '') {
                        $this->outputLine();
                        $this->outputLine('DESCRIPTION:');
-                       $descriptionLines = explode(chr(10), $command->getDescription());
+                       $descriptionLines = explode(LF, $command->getDescription());
                        foreach ($descriptionLines as $descriptionLine) {
                                $this->outputLine('%-2s%s', array(' ', $descriptionLine));
                        }
index cc5ca8c..77f9f61 100644 (file)
@@ -124,7 +124,7 @@ class Command {
         * @return string A short description
         */
        public function getShortDescription() {
-               $lines = explode(chr(10), $this->getCommandMethodReflection()->getDescription());
+               $lines = explode(LF, $this->getCommandMethodReflection()->getDescription());
                return count($lines) > 0 ? trim($lines[0]) : '<no description available>';
        }
 
@@ -136,7 +136,7 @@ class Command {
         * @return string A longer description of this command
         */
        public function getDescription() {
-               $lines = explode(chr(10), $this->getCommandMethodReflection()->getDescription());
+               $lines = explode(LF, $this->getCommandMethodReflection()->getDescription());
                array_shift($lines);
                $descriptionLines = array();
                foreach ($lines as $line) {
@@ -145,7 +145,7 @@ class Command {
                                $descriptionLines[] = $trimmedLine;
                        }
                }
-               return implode(chr(10), $descriptionLines);
+               return implode(LF, $descriptionLines);
        }
 
        /**
index f5b6c22..10e4e2e 100644 (file)
@@ -282,7 +282,7 @@ class Response extends \TYPO3\CMS\Extbase\Mvc\Response {
         */
        public function shutdown() {
                if (count($this->getAdditionalHeaderData()) > 0) {
-                       $this->getTypoScriptFrontendController()->additionalHeaderData[] = implode(chr(10), $this->getAdditionalHeaderData());
+                       $this->getTypoScriptFrontendController()->additionalHeaderData[] = implode(LF, $this->getAdditionalHeaderData());
                }
                $this->sendHeaders();
                return parent::shutdown();
index 2af4743..2e457e6 100644 (file)
@@ -39,12 +39,12 @@ class DocCommentParser {
        public function parseDocComment($docComment) {
                $this->description = '';
                $this->tags = array();
-               $lines = explode(chr(10), $docComment);
+               $lines = explode(LF, $docComment);
                foreach ($lines as $line) {
                        if (strlen($line) > 0 && strpos($line, '@') !== FALSE) {
                                $this->parseTag(substr($line, strpos($line, '@')));
                        } elseif (count($this->tags) === 0) {
-                               $this->description .= preg_replace('/\\s*\\/?[\\\\*]*(.*)$/', '$1', $line) . chr(10);
+                               $this->description .= preg_replace('/\\s*\\/?[\\\\*]*(.*)$/', '$1', $line) . LF;
                        }
                }
                $this->description = trim($this->description);
index b0b3a9d..40ea836 100644 (file)
@@ -205,29 +205,29 @@ EOD;
         * @see convert()
         */
        protected function convertViewHelperNode(\TYPO3\CMS\Fluid\Core\Parser\SyntaxTree\ViewHelperNode $node) {
-               $initializationPhpCode = '// Rendering ViewHelper ' . $node->getViewHelperClassName() . chr(10);
+               $initializationPhpCode = '// Rendering ViewHelper ' . $node->getViewHelperClassName() . LF;
 
                // Build up $arguments array
                $argumentsVariableName = $this->variableName('arguments');
-               $initializationPhpCode .= sprintf('%s = array();', $argumentsVariableName) . chr(10);
+               $initializationPhpCode .= sprintf('%s = array();', $argumentsVariableName) . LF;
 
                $alreadyBuiltArguments = array();
                foreach ($node->getArguments() as $argumentName => $argumentValue) {
                        $converted = $this->convert($argumentValue);
                        $initializationPhpCode .= $converted['initialization'];
-                       $initializationPhpCode .= sprintf('%s[\'%s\'] = %s;', $argumentsVariableName, $argumentName, $converted['execution']) . chr(10);
+                       $initializationPhpCode .= sprintf('%s[\'%s\'] = %s;', $argumentsVariableName, $argumentName, $converted['execution']) . LF;
                        $alreadyBuiltArguments[$argumentName] = TRUE;
                }
 
                foreach ($node->getUninitializedViewHelper()->prepareArguments() as $argumentName => $argumentDefinition) {
                        if (!isset($alreadyBuiltArguments[$argumentName])) {
-                               $initializationPhpCode .= sprintf('%s[\'%s\'] = %s;', $argumentsVariableName, $argumentName, var_export($argumentDefinition->getDefaultValue(), TRUE)) . chr(10);
+                               $initializationPhpCode .= sprintf('%s[\'%s\'] = %s;', $argumentsVariableName, $argumentName, var_export($argumentDefinition->getDefaultValue(), TRUE)) . LF;
                        }
                }
 
                // Build up closure which renders the child nodes
                $renderChildrenClosureVariableName = $this->variableName('renderChildrenClosure');
-               $initializationPhpCode .= sprintf('%s = %s;', $renderChildrenClosureVariableName, $this->wrapChildNodesInClosure($node)) . chr(10);
+               $initializationPhpCode .= sprintf('%s = %s;', $renderChildrenClosureVariableName, $this->wrapChildNodesInClosure($node)) . LF;
 
                if ($node->getUninitializedViewHelper() instanceof \TYPO3\CMS\Fluid\Core\ViewHelper\Facets\CompilableInterface) {
                        // ViewHelper is compilable
@@ -245,13 +245,13 @@ EOD;
                // ViewHelper is not compilable, so we need to instanciate it directly and render it.
                $viewHelperVariableName = $this->variableName('viewHelper');
 
-               $initializationPhpCode .= sprintf('%s = $self->getViewHelper(\'%s\', $renderingContext, \'%s\');', $viewHelperVariableName, $viewHelperVariableName, $node->getViewHelperClassName()) . chr(10);
-               $initializationPhpCode .= sprintf('%s->setArguments(%s);', $viewHelperVariableName, $argumentsVariableName) . chr(10);
-               $initializationPhpCode .= sprintf('%s->setRenderingContext($renderingContext);', $viewHelperVariableName) . chr(10);
+               $initializationPhpCode .= sprintf('%s = $self->getViewHelper(\'%s\', $renderingContext, \'%s\');', $viewHelperVariableName, $viewHelperVariableName, $node->getViewHelperClassName()) . LF;
+               $initializationPhpCode .= sprintf('%s->setArguments(%s);', $viewHelperVariableName, $argumentsVariableName) . LF;
+               $initializationPhpCode .= sprintf('%s->setRenderingContext($renderingContext);', $viewHelperVariableName) . LF;
 
-               $initializationPhpCode .= sprintf('%s->setRenderChildrenClosure(%s);', $viewHelperVariableName, $renderChildrenClosureVariableName) . chr(10);
+               $initializationPhpCode .= sprintf('%s->setRenderChildrenClosure(%s);', $viewHelperVariableName, $renderChildrenClosureVariableName) . LF;
 
-               $initializationPhpCode .= '// End of ViewHelper ' . $node->getViewHelperClassName() . chr(10);
+               $initializationPhpCode .= '// End of ViewHelper ' . $node->getViewHelperClassName() . LF;
 
                return array(
                        'initialization' => $initializationPhpCode,
@@ -277,22 +277,22 @@ EOD;
         * @see convert()
         */
        protected function convertArrayNode(\TYPO3\CMS\Fluid\Core\Parser\SyntaxTree\ArrayNode $node) {
-               $initializationPhpCode = '// Rendering Array' . chr(10);
+               $initializationPhpCode = '// Rendering Array' . LF;
                $arrayVariableName = $this->variableName('array');
 
-               $initializationPhpCode .= sprintf('%s = array();', $arrayVariableName) . chr(10);
+               $initializationPhpCode .= sprintf('%s = array();', $arrayVariableName) . LF;
 
                foreach ($node->getInternalArray() as $key => $value) {
                        if ($value instanceof \TYPO3\CMS\Fluid\Core\Parser\SyntaxTree\AbstractNode) {
                                $converted = $this->convert($value);
                                $initializationPhpCode .= $converted['initialization'];
-                               $initializationPhpCode .= sprintf('%s[\'%s\'] = %s;', $arrayVariableName, $key, $converted['execution']) . chr(10);
+                               $initializationPhpCode .= sprintf('%s[\'%s\'] = %s;', $arrayVariableName, $key, $converted['execution']) . LF;
                        } elseif (is_numeric($value)) {
                                // this case might happen for simple values
-                               $initializationPhpCode .= sprintf('%s[\'%s\'] = %s;', $arrayVariableName, $key, $value) . chr(10);
+                               $initializationPhpCode .= sprintf('%s[\'%s\'] = %s;', $arrayVariableName, $key, $value) . LF;
                        } else {
                                // this case might happen for simple values
-                               $initializationPhpCode .= sprintf('%s[\'%s\'] = \'%s\';', $arrayVariableName, $key, $this->escapeTextForUseInSingleQuotes($value)) . chr(10);
+                               $initializationPhpCode .= sprintf('%s[\'%s\'] = \'%s\';', $arrayVariableName, $key, $this->escapeTextForUseInSingleQuotes($value)) . LF;
                        }
                }
                return array(
@@ -319,13 +319,13 @@ EOD;
                                return $converted;
                        default:
                                $outputVariableName = $this->variableName('output');
-                               $initializationPhpCode = sprintf('%s = \'\';', $outputVariableName) . chr(10);
+                               $initializationPhpCode = sprintf('%s = \'\';', $outputVariableName) . LF;
 
                                foreach ($node->getChildNodes() as $childNode) {
                                        $converted = $this->convert($childNode);
 
-                                       $initializationPhpCode .= $converted['initialization'] . chr(10);
-                                       $initializationPhpCode .= sprintf('%s .= %s;', $outputVariableName, $converted['execution']) . chr(10);
+                                       $initializationPhpCode .= $converted['initialization'] . LF;
+                                       $initializationPhpCode .= sprintf('%s .= %s;', $outputVariableName, $converted['execution']) . LF;
                                }
 
                                return array(
@@ -341,7 +341,7 @@ EOD;
         * @see convert()
         */
        protected function convertBooleanNode(\TYPO3\CMS\Fluid\Core\Parser\SyntaxTree\BooleanNode $node) {
-               $initializationPhpCode = '// Rendering Boolean node' . chr(10);
+               $initializationPhpCode = '// Rendering Boolean node' . LF;
                if ($node->getComparator() !== NULL) {
                        $convertedLeftSide = $this->convert($node->getLeftSide());
                        $convertedRightSide = $this->convert($node->getRightSide());
@@ -374,10 +374,10 @@ EOD;
         */
        public function wrapChildNodesInClosure(\TYPO3\CMS\Fluid\Core\Parser\SyntaxTree\AbstractNode $node) {
                $closure = '';
-               $closure .= 'function() use ($renderingContext, $self) {' . chr(10);
+               $closure .= 'function() use ($renderingContext, $self) {' . LF;
                $convertedSubNodes = $this->convertListOfSubNodes($node);
                $closure .= $convertedSubNodes['initialization'];
-               $closure .= sprintf('return %s;', $convertedSubNodes['execution']) . chr(10);
+               $closure .= sprintf('return %s;', $convertedSubNodes['execution']) . LF;
                $closure .= '}';
                return $closure;
        }
index e4d0b39..00058ea 100644 (file)
@@ -141,13 +141,13 @@ abstract class AbstractConditionViewHelper extends \TYPO3\CMS\Fluid\Core\ViewHel
                                && $childNode->getViewHelperClassName() === \TYPO3\CMS\Fluid\ViewHelpers\ThenViewHelper::class) {
 
                                $childNodesAsClosure = $templateCompiler->wrapChildNodesInClosure($childNode);
-                               $initializationPhpCode .= sprintf('%s[\'__thenClosure\'] = %s;', $argumentsVariableName, $childNodesAsClosure) . chr(10);
+                               $initializationPhpCode .= sprintf('%s[\'__thenClosure\'] = %s;', $argumentsVariableName, $childNodesAsClosure) . LF;
                        }
                        if ($childNode instanceof \TYPO3\CMS\Fluid\Core\Parser\SyntaxTree\ViewHelperNode
                                && $childNode->getViewHelperClassName() === \TYPO3\CMS\Fluid\ViewHelpers\ElseViewHelper::class) {
 
                                $childNodesAsClosure = $templateCompiler->wrapChildNodesInClosure($childNode);
-                               $initializationPhpCode .= sprintf('%s[\'__elseClosure\'] = %s;', $argumentsVariableName, $childNodesAsClosure) . chr(10);
+                               $initializationPhpCode .= sprintf('%s[\'__elseClosure\'] = %s;', $argumentsVariableName, $childNodesAsClosure) . LF;
                        }
                }
                return \TYPO3\CMS\Fluid\Core\Compiler\TemplateCompiler::SHOULD_GENERATE_VIEWHELPER_INVOCATION;
index 20c56b4..f808b59 100644 (file)
@@ -71,12 +71,12 @@ abstract class AbstractFormViewHelper extends \TYPO3\CMS\Fluid\Core\ViewHelper\A
                // Using that one breaks re-submission of data in forms in case of an error.
                $identifier = $object->getUid();
                if ($identifier === NULL) {
-                       return chr(10) . '<!-- Object of type ' . get_class($object) . ' is without identity -->' . chr(10);
+                       return LF . '<!-- Object of type ' . get_class($object) . ' is without identity -->' . LF;
                }
                $name = $this->prefixFieldName($name) . '[__identity]';
                $this->registerFieldNameForFormTokenGeneration($name);
 
-               return chr(10) . '<input type="hidden" name="' . $name . '" value="' . $identifier . '" />' . chr(10);
+               return LF . '<input type="hidden" name="' . $name . '" value="' . $identifier . '" />' . LF;
        }
 
        /**
index dacf904..6ff7c2a 100644 (file)
@@ -134,11 +134,11 @@ class SelectViewHelper extends \TYPO3\CMS\Fluid\ViewHelpers\Form\AbstractFormFie
                if ($this->hasArgument('prependOptionLabel')) {
                        $value = $this->hasArgument('prependOptionValue') ? $this->arguments['prependOptionValue'] : '';
                        $label = $this->arguments['prependOptionLabel'];
-                       $output .= $this->renderOptionTag($value, $label, FALSE) . chr(10);
+                       $output .= $this->renderOptionTag($value, $label, FALSE) . LF;
                }
                foreach ($options as $value => $label) {
                        $isSelected = $this->isSelected($value);
-                       $output .= $this->renderOptionTag($value, $label, $isSelected) . chr(10);
+                       $output .= $this->renderOptionTag($value, $label, $isSelected) . LF;
                }
                return $output;
        }
index f5b0123..3ce3337 100644 (file)
@@ -133,9 +133,9 @@ class FormViewHelper extends \TYPO3\CMS\Fluid\ViewHelpers\Form\AbstractFormViewH
                $formContent = $this->renderChildren();
 
                if ($this->arguments['hiddenFieldClassName'] !== NULL) {
-                       $content = chr(10) . '<div class="' . htmlspecialchars($this->arguments['hiddenFieldClassName']) . '">';
+                       $content = LF . '<div class="' . htmlspecialchars($this->arguments['hiddenFieldClassName']) . '">';
                } else {
-                       $content = chr(10) . '<div>';
+                       $content = LF . '<div>';
                }
 
                $content .= $this->renderHiddenIdentityField($this->arguments['object'], $this->getFormObjectName());
@@ -145,7 +145,7 @@ class FormViewHelper extends \TYPO3\CMS\Fluid\ViewHelpers\Form\AbstractFormViewH
                // Render the trusted list of all properties after everything else has been rendered
                $content .= $this->renderTrustedPropertiesField();
 
-               $content .= chr(10) . '</div>' . chr(10);
+               $content .= LF . '</div>' . LF;
                $content .= $formContent;
                $this->tag->setContent($content);
                $this->removeFieldNamePrefixFromViewHelperVariableContainer();
@@ -183,7 +183,7 @@ class FormViewHelper extends \TYPO3\CMS\Fluid\ViewHelpers\Form\AbstractFormViewH
                        $additionalIdentityProperties = $this->viewHelperVariableContainer->get(\TYPO3\CMS\Fluid\ViewHelpers\FormViewHelper::class, 'additionalIdentityProperties');
                        $output = '';
                        foreach ($additionalIdentityProperties as $identity) {
-                               $output .= chr(10) . $identity;
+                               $output .= LF . $identity;
                        }
                        return $output;
                }
@@ -203,14 +203,14 @@ class FormViewHelper extends \TYPO3\CMS\Fluid\ViewHelpers\Form\AbstractFormViewH
                $vendorName = $request->getControllerVendorName();
                $controllerName = $request->getControllerName();
                $actionName = $request->getControllerActionName();
-               $result = chr(10);
-               $result .= '<input type="hidden" name="' . $this->prefixFieldName('__referrer[@extension]') . '" value="' . $extensionName . '" />' . chr(10);
+               $result = LF;
+               $result .= '<input type="hidden" name="' . $this->prefixFieldName('__referrer[@extension]') . '" value="' . $extensionName . '" />' . LF;
                if ($vendorName !== NULL) {
-                       $result .= '<input type="hidden" name="' . $this->prefixFieldName('__referrer[@vendor]') . '" value="' . $vendorName . '" />' . chr(10);
+                       $result .= '<input type="hidden" name="' . $this->prefixFieldName('__referrer[@vendor]') . '" value="' . $vendorName . '" />' . LF;
                }
-               $result .= '<input type="hidden" name="' . $this->prefixFieldName('__referrer[@controller]') . '" value="' . $controllerName . '" />' . chr(10);
-               $result .= '<input type="hidden" name="' . $this->prefixFieldName('__referrer[@action]') . '" value="' . $actionName . '" />' . chr(10);
-               $result .= '<input type="hidden" name="' . $this->prefixFieldName('__referrer[arguments]') . '" value="' . htmlspecialchars($this->hashService->appendHmac(base64_encode(serialize($request->getArguments())))) . '" />' . chr(10);
+               $result .= '<input type="hidden" name="' . $this->prefixFieldName('__referrer[@controller]') . '" value="' . $controllerName . '" />' . LF;
+               $result .= '<input type="hidden" name="' . $this->prefixFieldName('__referrer[@action]') . '" value="' . $actionName . '" />' . LF;
+               $result .= '<input type="hidden" name="' . $this->prefixFieldName('__referrer[arguments]') . '" value="' . htmlspecialchars($this->hashService->appendHmac(base64_encode(serialize($request->getArguments())))) . '" />' . LF;
 
                return $result;
        }
index 0130242..2b28f53 100644 (file)
@@ -78,7 +78,7 @@ class HtmlspecialcharsViewHelper extends \TYPO3\CMS\Fluid\ViewHelpers\Format\Abs
         */
        public function compile($argumentsVariableName, $renderChildrenClosureVariableName, &$initializationPhpCode, \TYPO3\CMS\Fluid\Core\Parser\SyntaxTree\AbstractNode $syntaxTreeNode, \TYPO3\CMS\Fluid\Core\Compiler\TemplateCompiler $templateCompiler) {
                $valueVariableName = $templateCompiler->variableName('value');
-               $initializationPhpCode .= sprintf('%s = (%s[\'value\'] !== NULL ? %s[\'value\'] : %s());', $valueVariableName, $argumentsVariableName, $argumentsVariableName, $renderChildrenClosureVariableName) . chr(10);
+               $initializationPhpCode .= sprintf('%s = (%s[\'value\'] !== NULL ? %s[\'value\'] : %s());', $valueVariableName, $argumentsVariableName, $argumentsVariableName, $renderChildrenClosureVariableName) . LF;
 
                return sprintf('(!is_string(%s) ? %s : htmlspecialchars(%s, (%s[\'keepQuotes\'] ? ENT_NOQUOTES : ENT_COMPAT), (%s[\'encoding\'] !== NULL ? %s[\'encoding\'] : \\TYPO3\\CMS\\Fluid\\Core\\Compiler\\AbstractCompiledTemplate::resolveDefaultEncoding()), %s[\'doubleEncode\']))',
                                $valueVariableName, $valueVariableName, $valueVariableName, $argumentsVariableName, $argumentsVariableName, $argumentsVariableName, $argumentsVariableName);
index 92d3e2b..037451f 100644 (file)
@@ -381,24 +381,24 @@ class JsonToTypoScript {
 
                        case 'typo3-form-wizard-elements-basic-textline':
                                if ($value === 'back') {
-                                       $parent[$elementCounter . '.']['layout'] = '<input />' . chr(10) . '<label />';
+                                       $parent[$elementCounter . '.']['layout'] = '<input />' . LF . '<label />';
                                }
                                break;
                        case 'typo3-form-wizard-elements-basic-checkbox':
 
                        case 'typo3-form-wizard-elements-basic-radio':
                                if ($value === 'front') {
-                                       $parent[$elementCounter . '.']['layout'] = '<label />' . chr(10) . '<input />';
+                                       $parent[$elementCounter . '.']['layout'] = '<label />' . LF . '<input />';
                                }
                                break;
                        case 'typo3-form-wizard-elements-basic-select':
                                if ($value === 'back') {
-                                       $parent[$elementCounter . '.']['layout'] = '<select>' . chr(10) . '<elements />' . chr(10) . '</select>' . chr(10) . '<label />';
+                                       $parent[$elementCounter . '.']['layout'] = '<select>' . LF . '<elements />' . LF . '</select>' . LF . '<label />';
                                }
                                break;
                        case 'typo3-form-wizard-elements-basic-textarea':
                                if ($value === 'back') {
-                                       $parent[$elementCounter . '.']['layout'] = '<textarea />' . chr(10) . '<label />';
+                                       $parent[$elementCounter . '.']['layout'] = '<textarea />' . LF . '<label />';
                                }
                                break;
                        default:
@@ -542,18 +542,18 @@ class JsonToTypoScript {
        protected function typoscriptArrayToString(array $typoscriptArray, $addKey = '', $tabCount = -1) {
                $typoscript = '';
                if ($addKey != '') {
-                       $typoscript .= str_repeat(chr(9), $tabCount) . str_replace('.', '', $addKey) . ' {' . chr(10);
+                       $typoscript .= str_repeat(TAB, $tabCount) . str_replace('.', '', $addKey) . ' {' . LF;
                }
                $tabCount++;
                foreach ($typoscriptArray as $key => $value) {
                        if (!is_array($value)) {
-                               if (strstr($value, chr(10))) {
-                                       $typoscript .= str_repeat(chr(9), $tabCount) . $key . ' (' . chr(10);
-                                       $value = str_replace(chr(10), chr(10) . str_repeat(chr(9), ($tabCount + 1)), $value);
-                                       $typoscript .= str_repeat(chr(9), ($tabCount + 1)) . $value . chr(10);
-                                       $typoscript .= str_repeat(chr(9), $tabCount) . ')' . chr(10);
+                               if (strstr($value, LF)) {
+                                       $typoscript .= str_repeat(TAB, $tabCount) . $key . ' (' . LF;
+                                       $value = str_replace(LF, LF . str_repeat(TAB, ($tabCount + 1)), $value);
+                                       $typoscript .= str_repeat(TAB, ($tabCount + 1)) . $value . LF;
+                                       $typoscript .= str_repeat(TAB, $tabCount) . ')' . LF;
                                } else {
-                                       $typoscript .= str_repeat(chr(9), $tabCount) . $key . ' = ' . $value . chr(10);
+                                       $typoscript .= str_repeat(TAB, $tabCount) . $key . ' = ' . $value . LF;
                                }
                        } else {
                                $typoscript .= $this->typoscriptArrayToString($value, $key, $tabCount);
@@ -561,7 +561,7 @@ class JsonToTypoScript {
                }
                if ($addKey != '') {
                        $tabCount--;
-                       $typoscript .= str_repeat(chr(9), $tabCount) . '}' . chr(10);
+                       $typoscript .= str_repeat(TAB, $tabCount) . '}' . LF;
                }
                return $typoscript;
        }
index 26314c6..8e37fea 100644 (file)
@@ -55,7 +55,7 @@ class FormView extends \TYPO3\CMS\Form\View\Form\Element\ContainerElementView {
         */
        public function get() {
                $node = $this->render('element', FALSE);
-               $content = chr(10) . html_entity_decode($node->saveXML($node->firstChild), ENT_QUOTES, 'UTF-8') . chr(10);
+               $content = LF . html_entity_decode($node->saveXML($node->firstChild), ENT_QUOTES, 'UTF-8') . LF;
                return $content;
        }
 
index 8ad8b28..1f9550d 100644 (file)
@@ -94,7 +94,7 @@ class HtmlView extends \TYPO3\CMS\Form\View\Mail\Html\Element\ContainerElementVi
         */
        public function get() {
                $node = $this->render('element', FALSE);
-               $content = chr(10) . html_entity_decode($node->saveXML($node->firstChild), ENT_QUOTES, 'UTF-8') . chr(10);
+               $content = LF . html_entity_decode($node->saveXML($node->firstChild), ENT_QUOTES, 'UTF-8') . LF;
                return $content;
        }
 
index c6d51d5..9f50854 100644 (file)
@@ -26,7 +26,7 @@ class CheckboxGroupElementView extends \TYPO3\CMS\Form\View\Mail\Plain\Element\C
        public function render() {
                $content = '';
                if ($this->model->additionalIsSet('legend')) {
-                       $content = $this->model->getAdditionalValue('legend') . ': ' . chr(10);
+                       $content = $this->model->getAdditionalValue('legend') . ': ' . LF;
                }
                $content .= $this->renderChildren($this->model->getElements(), $this->spaces + 4);
                return str_repeat(chr(32), $this->spaces) . $content;
index 2daf097..9d002df 100644 (file)
@@ -49,7 +49,7 @@ class ContainerElementView extends \TYPO3\CMS\Form\View\Mail\Plain\Element\Abstr
                        $childElement = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance($className, $modelChild, $spaces);
                        $elementContent = $childElement->render();
                        if ($elementContent != '') {
-                               $content = $childElement->render() . chr(10);
+                               $content = $childElement->render() . LF;
                        }
                }
                return $content;
index a6f6c82..6d4e84e 100644 (file)
@@ -27,7 +27,7 @@ class FieldsetElementView extends \TYPO3\CMS\Form\View\Mail\Plain\Element\Contai
        public function render() {
                $content = '';
                if ($this->model->additionalIsSet('legend')) {
-                       $content = $this->model->getAdditionalValue('legend') . chr(10);
+                       $content = $this->model->getAdditionalValue('legend') . LF;
                }
                $content .= $this->renderChildren($this->model->getElements(), $this->spaces + 4);
                return str_repeat(chr(32), $this->spaces) . $content;
index 4f31404..2ffce61 100644 (file)
@@ -27,7 +27,7 @@ class RadioGroupElementView extends \TYPO3\CMS\Form\View\Mail\Plain\Element\Cont
        public function render() {
                $content = '';
                if ($this->model->additionalIsSet('legend')) {
-                       $content = $this->model->getAdditionalValue('legend') . ': ' . chr(10);
+                       $content = $this->model->getAdditionalValue('legend') . ': ' . LF;
                }
                $content .= $this->renderChildren($this->model->getElements(), $this->spaces + 4);
                return str_repeat(chr(32), $this->spaces) . $content;
index 01469bd..5f662b0 100644 (file)
@@ -25,7 +25,7 @@ class SelectElementView extends \TYPO3\CMS\Form\View\Mail\Plain\Element\Containe
         * @return string
         */
        public function render() {
-               $content = $this->getLabel() . ': ' . chr(10) . $this->getValues();
+               $content = $this->getLabel() . ': ' . LF . $this->getValues();
                return str_repeat(chr(32), $this->spaces) . $content;
        }
 
index e35d819..5bb4159 100644 (file)
@@ -25,7 +25,7 @@ class TextareaElementView extends \TYPO3\CMS\Form\View\Mail\Plain\Element\Abstra
         * @return string
         */
        public function render() {
-               $content = $this->getLabel() . ': ' . chr(10) . str_repeat(chr(32), ($this->spaces + 4)) . $this->getData();
+               $content = $this->getLabel() . ': ' . LF . str_repeat(chr(32), ($this->spaces + 4)) . $this->getData();
                return str_repeat(chr(32), $this->spaces) . $content;
        }
 
@@ -46,7 +46,7 @@ class TextareaElementView extends \TYPO3\CMS\Form\View\Mail\Plain\Element\Abstra
         * @return string
         */
        protected function getData() {
-               $value = str_replace(chr(10), chr(10) . str_repeat(chr(32), ($this->spaces + 4)), $this->model->getData());
+               $value = str_replace(LF, LF . str_repeat(chr(32), ($this->spaces + 4)), $this->model->getData());
                return $value;
        }
 
index d281f6c..366a3af 100644 (file)
@@ -91,7 +91,7 @@ class InfoPageTyposcriptConfigController extends \TYPO3\CMS\Backend\Module\Abstr
                                                $onclickUrl = BackendUtility::editOnClick($params, $GLOBALS['BACK_PATH'], '');
                                                $editIcon = '<a href="#" onclick="' . htmlspecialchars($onclickUrl) . '" title="' . $GLOBALS['LANG']->getLL('editTSconfig', TRUE) . '">' . \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIcon('actions-document-open') . '</a>';
                                        }
-                                       $TScontent = nl2br(htmlspecialchars(trim($v) . chr(10)));
+                                       $TScontent = nl2br(htmlspecialchars(trim($v) . LF));
                                        $tsparser = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance(\TYPO3\CMS\Core\TypoScript\Parser\TypoScriptParser::class);
                                        $tsparser->lineNumberOffset = 0;
                                        $TScontent = $tsparser->doSyntaxHighlight(trim($v) . LF);
index 4663485..4fbdb1d 100644 (file)
@@ -69,7 +69,7 @@ class AllConfiguration extends Action\AbstractAction {
                                if (isset($GLOBALS['TYPO3_CONF_VARS_extensionAdded'][$sectionName][$key])) {
                                        // Don't allow editing stuff which is added by extensions
                                        // Make sure we fix potentially duplicated entries from older setups
-                                       $potentialValue = str_replace(array('\'.chr(10).\'', '\' . LF . \''), array(LF, LF), $value);
+                                       $potentialValue = str_replace(array('\' . LF . \'', '\' . LF . \''), array(LF, LF), $value);
                                        while (preg_match('/' . preg_quote($GLOBALS['TYPO3_CONF_VARS_extensionAdded'][$sectionName][$key], '/') . '$/', '', $potentialValue)) {
                                                $potentialValue = preg_replace('/' . preg_quote($GLOBALS['TYPO3_CONF_VARS_extensionAdded'][$sectionName][$key], '/') . '$/', '', $potentialValue);
                                        }
@@ -86,7 +86,7 @@ class AllConfiguration extends Action\AbstractAction {
                                        $itemData['description'] = $description;
                                        if ($isTextarea) {
                                                $itemData['type'] = 'textarea';
-                                               $itemData['value'] = str_replace(array('\'.chr(10).\'', '\' . LF . \''), array(LF, LF), $value);
+                                               $itemData['value'] = str_replace(array('\' . LF . \'', '\' . LF . \''), array(LF, LF), $value);
                                        } elseif (preg_match('/^(<.*?>)?boolean/i', $description)) {
                                                $itemData['type'] = 'checkbox';
                                                $itemData['value'] = $value ? '1' : '0';
index 18d7c06..cfc67aa 100644 (file)
@@ -216,8 +216,8 @@ class CompatVersionUpdate extends AbstractUpdate {
                $upgradeWizardBoxes = '';
                foreach ($GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['ext/install']['compat_version'] as $internalName => $details) {
                        if ($details['version'] > $oldVersion && $details['version'] <= $newVersion) {
-                               $description = str_replace(chr(10), '<br />', $details['description']);
-                               $description_acknowledge = isset($details['description_acknowledge']) ? str_replace(chr(10), '<br />', $details['description_acknowledge']) : '';
+                               $description = str_replace(LF, '<br />', $details['description']);
+                               $description_acknowledge = isset($details['description_acknowledge']) ? str_replace(LF, '<br />', $details['description_acknowledge']) : '';
                                $upgradeWizardBoxes .= '
                                        <div style="border: 1px solid; padding: 0 10px 10px 10px; margin: 10px; width: 500px;">
                                                <h3>' . (isset($details['title']) ? $details['title'] : $internalName) . '</h3>
index acb55b6..7328ae5 100644 (file)
@@ -435,7 +435,7 @@ class LinkValidatorReport extends \TYPO3\CMS\Backend\Module\AbstractFunctionModu
                                        while (($row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res)) !== FALSE) {
                                                $items[] = $this->renderTableRow($row['table_name'], $row, $brokenLinksItemTemplate);
                                        }
-                                       $brokenLinkItems = implode(chr(10), $items);
+                                       $brokenLinkItems = implode(LF, $items);
                                } else {
                                        $brokenLinksMarker = $this->getNoBrokenLinkMessage($brokenLinksMarker);
                                }
index d0630f5..7360b32 100644 (file)
@@ -1786,7 +1786,7 @@ class DatabaseRecordList extends AbstractDatabaseRecordList {
                // versions lower then 9. See for more information: http://support.microsoft.com/kb/323308
                header("Cache-Control: ''");
                // Printing the content of the CSV lines:
-               echo implode(chr(13) . chr(10), $this->csvLines);
+               echo implode(CRLF, $this->csvLines);
                // Exits:
                die;
        }
index 6105419..4e161a5 100644 (file)
@@ -209,7 +209,7 @@ class DeprecatedRteProperties extends AbstractUpdate {
                }
                $db = $this->getDatabaseConnection();
                $res = $db->exec_SELECTquery($fields, $table, $where);
-               $dbQueries[] = str_replace(chr(10), ' ', $db->debug_lastBuiltQuery);
+               $dbQueries[] = str_replace(LF, ' ', $db->debug_lastBuiltQuery);
                if ($db->sql_error()) {
                        $customMessages = 'SQL-ERROR: ' . htmlspecialchars($db->sql_error());
                }
@@ -256,7 +256,7 @@ class DeprecatedRteProperties extends AbstractUpdate {
                                'TSconfig' => $page['TSconfig']
                        );
                        $db->exec_UPDATEquery($table, $where, $field_values);
-                       $dbQueries[] = str_replace(chr(10), ' ', $db->debug_lastBuiltQuery);
+                       $dbQueries[] = str_replace(LF, ' ', $db->debug_lastBuiltQuery);
                        if ($db->sql_error()) {
                                $customMessages .= 'SQL-ERROR: ' . htmlspecialchars($db->sql_error()) . LF . LF;
                        }
index c27f879..5e092a1 100644 (file)
@@ -111,7 +111,7 @@ class RteAcronymButtonRenamedToAbbreviation extends AbstractUpdate {
                $where = 'TSConfig LIKE BINARY "%acronym%"';
                $db = $this->getDatabaseConnection();
                $res = $db->exec_SELECTquery($fields, $table, $where);
-               $dbQueries[] = str_replace(chr(10), ' ', $db->debug_lastBuiltQuery);
+               $dbQueries[] = str_replace(LF, ' ', $db->debug_lastBuiltQuery);
                if ($db->sql_error()) {
                        $customMessages = 'SQL-ERROR: ' . htmlspecialchars($db->sql_error());
                }
@@ -156,7 +156,7 @@ class RteAcronymButtonRenamedToAbbreviation extends AbstractUpdate {
                                'TSconfig' => $page['TSconfig']
                        );
                        $db->exec_UPDATEquery($table, $where, $field_values);
-                       $dbQueries[] = str_replace(chr(10), ' ', $db->debug_lastBuiltQuery);
+                       $dbQueries[] = str_replace(LF, ' ', $db->debug_lastBuiltQuery);
                        if ($db->sql_error()) {
                                $customMessages .= 'SQL-ERROR: ' . htmlspecialchars($db->sql_error()) . LF . LF;
                        }