[TASK] Make LocalConfiguration.php PSR-2 standard compliant 21/43921/3
authorNicole Cordes <typo3@cordes.co>
Thu, 8 Oct 2015 22:48:12 +0000 (00:48 +0200)
committerChristian Kuhn <lolli@schwarzbu.ch>
Thu, 8 Oct 2015 23:44:26 +0000 (01:44 +0200)
This patch changes the ArrayUtility::arrayExport function to generate
a PSR-2 compliant output.

Resolves: #70519
Releases: master
Change-Id: Id1d4e384b3de3719ce228e98ba88c8dcff7c2175
Reviewed-on: http://review.typo3.org/43921
Reviewed-by: Helmut Hummel <helmut.hummel@typo3.org>
Tested-by: Helmut Hummel <helmut.hummel@typo3.org>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
typo3/sysext/core/Classes/Configuration/ConfigurationManager.php
typo3/sysext/core/Classes/Utility/ArrayUtility.php
typo3/sysext/core/Tests/Unit/Configuration/ConfigurationManagerTest.php
typo3/sysext/core/Tests/Unit/Utility/ArrayUtilityTest.php

index 5b451ee..1353b05 100644 (file)
@@ -325,8 +325,7 @@ class ConfigurationManager
                     ArrayUtility::arrayExport(
                         ArrayUtility::renumberKeysToAvoidLeapsIfKeysAreAllNumeric($configuration)
                     ) .
-                ';' . LF .
-            '?>',
+                ';' . LF,
             true
         );
 
index 14b7335..2351097 100644 (file)
@@ -291,7 +291,7 @@ class ArrayUtility
 
     /**
      * Exports an array as string.
-     * Similar to var_export(), but representation follows the TYPO3 core CGL.
+     * Similar to var_export(), but representation follows the PSR-2 and TYPO3 core CGL.
      *
      * See unit tests for detailed examples
      *
@@ -302,7 +302,7 @@ class ArrayUtility
      */
     public static function arrayExport(array $array = array(), $level = 0)
     {
-        $lines = 'array(' . LF;
+        $lines = '[' . LF;
         $level++;
         $writeKeyIndex = false;
         $expectedKeyIndex = 0;
@@ -317,7 +317,7 @@ class ArrayUtility
         }
         foreach ($array as $key => $value) {
             // Indention
-            $lines .= str_repeat(TAB, $level);
+            $lines .= str_repeat('    ', $level);
             if ($writeKeyIndex) {
                 // Numeric / string keys
                 $lines .= is_int($key) ? $key . ' => ' : '\'' . $key . '\' => ';
@@ -326,14 +326,14 @@ class ArrayUtility
                 if (!empty($value)) {
                     $lines .= self::arrayExport($value, $level);
                 } else {
-                    $lines .= 'array(),' . LF;
+                    $lines .= '[],' . LF;
                 }
             } elseif (is_int($value) || is_float($value)) {
                 $lines .= $value . ',' . LF;
             } elseif (is_null($value)) {
-                $lines .= 'NULL' . ',' . LF;
+                $lines .= 'null' . ',' . LF;
             } elseif (is_bool($value)) {
-                $lines .= $value ? 'TRUE' : 'FALSE';
+                $lines .= $value ? 'true' : 'false';
                 $lines .= ',' . LF;
             } elseif (is_string($value)) {
                 // Quote \ to \\
@@ -345,7 +345,7 @@ class ArrayUtility
                 throw new \RuntimeException('Objects are not supported', 1342294987);
             }
         }
-        $lines .= str_repeat(TAB, ($level - 1)) . ')' . ($level - 1 == 0 ? '' : ',' . LF);
+        $lines .= str_repeat('    ', ($level - 1)) . ']' . ($level - 1 == 0 ? '' : ',' . LF);
         return $lines;
     }
 
index 17c296c..06890c6 100644 (file)
@@ -477,11 +477,10 @@ class ConfigurationManagerTest extends \TYPO3\CMS\Core\Tests\UnitTestCase
         );
         $expectedContent =
             '<?php' . LF .
-                'return array(' . LF .
-                    TAB . '\'bar\' => 23,' . LF .
-                    TAB . '\'foo\' => 42,' . LF .
-                ');' . LF .
-            '?>';
+                'return [' . LF .
+                    '    \'bar\' => 23,' . LF .
+                    '    \'foo\' => 42,' . LF .
+                '];' . LF;
 
         $this->subject->writeLocalConfiguration($pairs);
         $this->assertSame($expectedContent, file_get_contents($configurationFile));
@@ -521,8 +520,7 @@ class ConfigurationManagerTest extends \TYPO3\CMS\Core\Tests\UnitTestCase
             '<?php' . LF .
                 'return array(' . LF .
                     $uniqueContentString . ' => foo,' . LF .
-                ');' . LF .
-            '?>';
+                ');' . LF;
         file_put_contents(
             $factoryConfigurationAbsoluteFile,
             $validFactoryConfigurationFileContent
@@ -551,8 +549,7 @@ class ConfigurationManagerTest extends \TYPO3\CMS\Core\Tests\UnitTestCase
         $factoryConfigurationAbsoluteFile = PATH_site . $factoryConfigurationFile;
         $validFactoryConfigurationFileContent =
             '<?php' . LF .
-                'return array();' . LF .
-            '?>';
+                'return [];' . LF;
         file_put_contents(
             $factoryConfigurationAbsoluteFile,
             $validFactoryConfigurationFileContent
@@ -565,10 +562,9 @@ class ConfigurationManagerTest extends \TYPO3\CMS\Core\Tests\UnitTestCase
         $uniqueContentString = $this->getUniqueId('string_');
         $validAdditionalFactoryConfigurationFileContent =
             '<?php' . LF .
-                'return array(' . LF .
+                'return [' . LF .
                     $uniqueContentString . ' => foo,' . LF .
-                ');' . LF .
-            '?>';
+                '];' . LF;
         file_put_contents(
             $additionalFactoryConfigurationAbsoluteFile,
             $validAdditionalFactoryConfigurationFileContent
index 1edaee3..b61498e 100644 (file)
@@ -970,21 +970,21 @@ class ArrayUtilityTest extends UnitTestCase
             'qux2' => 0.000000001,
         );
         $expected =
-            'array(' . LF .
-                TAB . '\'foo\' => array(' . LF .
-                    TAB . TAB . '\'bar\' => 42,' . LF .
-                    TAB . TAB . '\'bar2\' => array(' . LF .
-                        TAB . TAB . TAB . '\'baz\' => \'val\\\'ue\',' . LF .
-                        TAB . TAB . TAB . '\'baz2\' => TRUE,' . LF .
-                        TAB . TAB . TAB . '\'baz3\' => FALSE,' . LF .
-                        TAB . TAB . TAB . '\'baz4\' => array(),' . LF .
-                    TAB . TAB . '),' . LF .
-                TAB . '),' . LF .
-                TAB . '\'baz\' => 23,' . LF .
-                TAB . '\'foobar\' => NULL,' . LF .
-                TAB . '\'qux\' => 0.1,' . LF .
-                TAB . '\'qux2\' => 1.0E-9,' . LF .
-            ')';
+            '[' . LF .
+                '    \'foo\' => [' . LF .
+                    '        \'bar\' => 42,' . LF .
+                    '        \'bar2\' => [' . LF .
+                        '            \'baz\' => \'val\\\'ue\',' . LF .
+                        '            \'baz2\' => true,' . LF .
+                        '            \'baz3\' => false,' . LF .
+                        '            \'baz4\' => [],' . LF .
+                    '        ],' . LF .
+                '    ],' . LF .
+                '    \'baz\' => 23,' . LF .
+                '    \'foobar\' => null,' . LF .
+                '    \'qux\' => 0.1,' . LF .
+                '    \'qux2\' => 1.0E-9,' . LF .
+            ']';
         $this->assertSame($expected, ArrayUtility::arrayExport($array));
     }
 
@@ -1013,11 +1013,11 @@ class ArrayUtilityTest extends UnitTestCase
             '42' => 'string key representing integer'
         );
         $expected =
-            'array(' . LF .
-                TAB . '\'foo\' => \'string key\',' . LF .
-                TAB . '23 => \'integer key\',' . LF .
-                TAB . '42 => \'string key representing integer\',' . LF .
-            ')';
+            '[' . LF .
+                '    \'foo\' => \'string key\',' . LF .
+                '    23 => \'integer key\',' . LF .
+                '    42 => \'string key representing integer\',' . LF .
+            ']';
         $this->assertSame($expected, ArrayUtility::arrayExport($array));
     }
 
@@ -1032,11 +1032,11 @@ class ArrayUtilityTest extends UnitTestCase
             2 => 'two'
         );
         $expected =
-            'array(' . LF .
-                TAB . '\'zero\',' . LF .
-                TAB . '\'one\',' . LF .
-                TAB . '\'two\',' . LF .
-            ')';
+            '[' . LF .
+                '    \'zero\',' . LF .
+                '    \'one\',' . LF .
+                '    \'two\',' . LF .
+            ']';
         $this->assertSame($expected, ArrayUtility::arrayExport($array));
     }
 
@@ -1052,12 +1052,12 @@ class ArrayUtilityTest extends UnitTestCase
             4 => 'four'
         );
         $expected =
-            'array(' . LF .
-                TAB . '0 => \'zero\',' . LF .
-                TAB . '1 => \'one\',' . LF .
-                TAB . '3 => \'three\',' . LF .
-                TAB . '4 => \'four\',' . LF .
-            ')';
+            '[' . LF .
+                '    0 => \'zero\',' . LF .
+                '    1 => \'one\',' . LF .
+                '    3 => \'three\',' . LF .
+                '    4 => \'four\',' . LF .
+            ']';
         $this->assertSame($expected, ArrayUtility::arrayExport($array));
     }