Switch to official coding standard (#5038)

* Switch to official coding standard

* Update .no-header.php-cs-fixer.dist.php

Co-authored-by: Abdul Malik Ikhsan <samsonasik@gmail.com>

.github/workflows/test-phpunit.yml

@@ -123,7 +123,7 @@ jobs:
       - name: Install dependencies
         run: |
           composer update --ansi --no-interaction
-          composer remove --ansi --dev --unused -W rector/rector phpstan/phpstan friendsofphp/php-cs-fixer nexusphp/cs-config
+          composer remove --ansi --dev --unused -W rector/rector phpstan/phpstan friendsofphp/php-cs-fixer nexusphp/cs-config codeigniter/coding-standard
           php -r 'file_put_contents("vendor/laminas/laminas-zendframework-bridge/src/autoload.php", "");'
         env:
           COMPOSER_AUTH: ${{ secrets.COMPOSER_AUTH }}

.no-header.php-cs-fixer.dist.php

@@ -11,29 +11,29 @@
  * the LICENSE file that was distributed with this source code.
  */
 
+use CodeIgniter\CodingStandard\CodeIgniter4;
 use Nexus\CsConfig\Factory;
+use Nexus\CsConfig\Fixer\Comment\SpaceAfterCommentStartFixer;
+use Nexus\CsConfig\FixerGenerator;
 use PhpCsFixer\Finder;
-use Utils\PhpCsFixer\CodeIgniter4;
-use Utils\PhpCsFixer\Fixer\Comment\SpaceAfterCommentStartFixer;
 
 $finder = Finder::create()
     ->files()
     ->in([
         __DIR__ . '/admin',
         __DIR__ . '/app',
         __DIR__ . '/public',
-    ]);
+    ])
+    ->notName('#Logger\.php$#');
 
 $overrides = [];
 
 $options = [
     'cacheFile'    => 'build/.no-header.php-cs-fixer.cache',
     'finder'       => $finder,
-    'customFixers' => [
-        new SpaceAfterCommentStartFixer(),
-    ],
-    'customRules' => [
-        'CodeIgniter4/space_after_comment_start' => true,
+    'customFixers' => FixerGenerator::create('vendor/nexusphp/cs-config/src/Fixer', 'Nexus\\CsConfig\\Fixer'),
+    'customRules'  => [
+        SpaceAfterCommentStartFixer::name() => true,
     ],
 ];
 

.php-cs-fixer.dist.php

@@ -11,10 +11,11 @@
  * the LICENSE file that was distributed with this source code.
  */
 
+use CodeIgniter\CodingStandard\CodeIgniter4;
 use Nexus\CsConfig\Factory;
+use Nexus\CsConfig\Fixer\Comment\SpaceAfterCommentStartFixer;
+use Nexus\CsConfig\FixerGenerator;
 use PhpCsFixer\Finder;
-use Utils\PhpCsFixer\CodeIgniter4;
-use Utils\PhpCsFixer\Fixer\Comment\SpaceAfterCommentStartFixer;
 
 $finder = Finder::create()
     ->files()
@@ -37,11 +38,9 @@
 $options = [
     'cacheFile'    => 'build/.php-cs-fixer.cache',
     'finder'       => $finder,
-    'customFixers' => [
-        new SpaceAfterCommentStartFixer(),
-    ],
-    'customRules' => [
-        'CodeIgniter4/space_after_comment_start' => true,
+    'customFixers' => FixerGenerator::create('vendor/nexusphp/cs-config/src/Fixer', 'Nexus\\CsConfig\\Fixer'),
+    'customRules'  => [
+        SpaceAfterCommentStartFixer::name() => true,
     ],
 ];
 

admin/framework/composer.json

@@ -15,10 +15,11 @@
         "psr/log": "^1.1"
     },
     "require-dev": {
+        "codeigniter/coding-standard": "^1.1",
         "fakerphp/faker": "^1.9",
-        "friendsofphp/php-cs-fixer": "^3.0",
+        "friendsofphp/php-cs-fixer": "^3.1",
         "mikey179/vfsstream": "^1.6",
-        "nexusphp/cs-config": "^3.1",
+        "nexusphp/cs-config": "^3.3",
         "phpunit/phpunit": "^9.1",
         "predis/predis": "^1.1"
     },

app/Config/Services.php

@@ -19,13 +19,14 @@
  */
 class Services extends BaseService
 {
-    // public static function example($getShared = true)
-    // {
-    //     if ($getShared)
-    //     {
-    //         return static::getSharedInstance('example');
-    //     }
-    //
-    //     return new \CodeIgniter\Example();
-    // }
+    /*
+     * public static function example($getShared = true)
+     * {
+     *     if ($getShared) {
+     *         return static::getSharedInstance('example');
+     *     }
+     *
+     *     return new \CodeIgniter\Example();
+     * }
+     */
 }

composer.json

@@ -15,10 +15,11 @@
         "psr/log": "^1.1"
     },
     "require-dev": {
+        "codeigniter/coding-standard": "^1.1",
         "fakerphp/faker": "^1.9",
-        "friendsofphp/php-cs-fixer": "^3.0",
+        "friendsofphp/php-cs-fixer": "^3.1",
         "mikey179/vfsstream": "^1.6",
-        "nexusphp/cs-config": "^3.1",
+        "nexusphp/cs-config": "^3.3",
         "nexusphp/tachycardia": "^1.0",
         "phpstan/phpstan": "^0.12.91",
         "phpunit/phpunit": "^9.1",

system/Database/BaseConnection.php

@@ -1004,7 +1004,7 @@ public function protectIdentifiers($item, bool $prefixSingle = false, ?bool $pro
             // we have nothing more to do other than escape the item
             //
             // NOTE: The ! empty() condition prevents this method
-            //       from breaking when QB isn't enabled.
+            // from breaking when QB isn't enabled.
             if (! empty($this->aliasedTables) && in_array($parts[0], $this->aliasedTables, true)) {
                 if ($protectIdentifiers === true) {
                     foreach ($parts as $key => $val) {

system/Email/Email.php

@@ -1459,8 +1459,8 @@ protected function prepQEncoding($str)
 
         if ($this->charset === 'UTF-8') {
             // Note: We used to have mb_encode_mimeheader() as the first choice
-            //       here, but it turned out to be buggy and unreliable. DO NOT
-            //       re-add it! -- Narf
+            // here, but it turned out to be buggy and unreliable. DO NOT
+            // re-add it! -- Narf
             if (extension_loaded('iconv')) {
                 $output = @iconv_mime_encode('', $str, [
                     'scheme'           => 'Q',

system/Session/Session.php

@@ -565,7 +565,7 @@ public function __set(string $key, $value)
     public function __get(string $key)
     {
         // Note: Keep this order the same, just in case somebody wants to
-        //       use 'session_id' as a session data key, for whatever reason
+        // use 'session_id' as a session data key, for whatever reason
         if (isset($_SESSION[$key])) {
             return $_SESSION[$key];
         }

system/Test/Mock/MockIncomingRequest.php

@@ -15,11 +15,6 @@
 
 class MockIncomingRequest extends IncomingRequest
 {
-    //    public function populateHeaders()
-    //    {
-    //        // Don't do anything... force the tester to manually set the headers they want.
-    //    }
-
     protected function detectURI($protocol, $baseURL)
     {
         // Do nothing...

system/Typography/Typography.php

@@ -155,12 +155,12 @@ public function autoTypography(string $str, bool $reduceLinebreaks = false): str
                 continue;
             }
 
-            //  Force a newline to make sure end tags get processed by _format_newlines()
+            // Force a newline to make sure end tags get processed by _format_newlines()
             if ($i === $c) {
                 $chunks[$i] .= "\n";
             }
 
-            //  Convert Newlines into <p> and <br /> tags
+            // Convert Newlines into <p> and <br /> tags
             $str .= $this->formatNewLines($chunks[$i]);
         }
 

tests/system/CLI/CLITest.php

@@ -75,14 +75,15 @@ public function testWait()
         // The problem: if the block below is enabled, the phpunit tests
         // go catatonic when it is executed, presumably because of
         // the CLI::input() waiting for a key press
-        //      // test the press any key to continue...
-        //      stream_filter_register('CLITestKeyboardFilter', 'CodeIgniter\CLI\CLITestKeyboardFilter');
-        //      $spoofer = stream_filter_append(STDIN, 'CLITestKeyboardFilter');
-        //      $time = time();
-        //      CLITestKeyboardFilter::$spoofed = ' ';
-        //      CLI::wait(0);
-        //      stream_filter_remove($spoofer);
-        //      $this->assertEquals(0, time() - $time);
+        //
+        // // test the press any key to continue...
+        // stream_filter_register('CLITestKeyboardFilter', 'CodeIgniter\CLI\CLITestKeyboardFilter');
+        // $spoofer = stream_filter_append(STDIN, 'CLITestKeyboardFilter');
+        // $time = time();
+        // CLITestKeyboardFilter::$spoofed = ' ';
+        // CLI::wait(0);
+        // stream_filter_remove($spoofer);
+        // $this->assertEquals(0, time() - $time);
     }
 
     public function testIsWindows()

tests/system/Cache/Handlers/FileHandlerTest.php

@@ -221,8 +221,8 @@ public function testDecrement()
         $this->fileHandler->save(self::$key1, 10);
         $this->fileHandler->save(self::$key2, 'value');
 
-        //  Line following commented out to force the cache to add a zero entry for key3
-        //      $this->fileHandler->save(self::$key3, 0);
+        // Line following commented out to force the cache to add a zero entry for key3
+        // $this->fileHandler->save(self::$key3, 0);
 
         $this->assertSame(9, $this->fileHandler->decrement(self::$key1, 1));
         $this->assertFalse($this->fileHandler->decrement(self::$key2, 1));

tests/system/Config/ServicesTest.php

@@ -147,17 +147,6 @@ public function testNewImage()
         $this->assertInstanceOf(ImageHandlerInterface::class, $actual);
     }
 
-    //  public function testNewMigrationRunner()
-    //  {
-    //      //FIXME - docs aren't clear about setting this up to just make sure that the service
-    //      // returns a MigrationRunner
-    //      $config = new \Config\Migrations();
-    //      $db = new \CodeIgniter\Database\MockConnection([]);
-    //      $this->expectException('InvalidArgumentException');
-    //      $actual = Services::migrations($config, $db);
-    //      $this->assertInstanceOf(\CodeIgniter\Database\MigrationRunner::class, $actual);
-    //  }
-    //
     public function testNewNegotiatorWithNullConfig()
     {
         $actual = Services::negotiator(null);

tests/system/I18n/TimeTest.php

@@ -643,19 +643,6 @@ public function testToFormattedDateString()
         $this->assertSame('May 10, 2017', $time->toFormattedDateString());
     }
 
-    /**
-     * Unfortunately, ubuntu 14.04 (on TravisCI) fails this test and
-     * shows a numeric version of the month instead of the textual version.
-     * Confirmed on CentOS 7 as well.
-     * Example: format 'MMM' for November returns 'M02' instead of 'Nov'
-     * Not sure what the fix is just yet....
-     */
-    //    public function testToFormattedDateString()
-    //    {
-    //        $time = Time::parse('February 10, 2017', 'America/Chicago');
-    //        $this->assertEquals('Feb 10, 2017', $time->toFormattedDateString());
-    //    }
-
     public function testToTimeString()
     {
         $time = Time::parse('January 10, 2017 13:20:33', 'America/Chicago');

tests/system/Test/TestCaseTest.php

@@ -22,15 +22,6 @@
  */
 final class TestCaseTest extends CIUnitTestCase
 {
-    //  protected function tearDown(): void
-    //  {
-    //      $buffer = ob_clean();
-    //      if (ob_get_level() > 0)
-    //      {
-    //          ob_end_clean();
-    //      }
-    //  }
-    //
     public function testGetPrivatePropertyWithObject()
     {
         $obj    = new __TestForReflectionHelper();
@@ -86,7 +77,6 @@ public function testPHPUnitHeadersEmitted()
         $response->send();
         ob_end_clean();
 
-        // Did PHPunit do its thing?
         $this->assertHeaderEmitted('Content-type: text/html;');
         $this->assertHeaderNotEmitted('Set-Cookie: foo=bar;');
     }