Fix CS

.github/workflows/ci.yml

@@ -41,3 +41,31 @@ jobs:
                 run: composer update --no-interaction --no-progress ${{ matrix.prefer }}
             -   name: PHPUnit
                 run: vendor/bin/phpunit
+    cs:
+        name: Coding standards
+        runs-on: ubuntu-latest
+        strategy:
+            matrix:
+                php: ['8.1']
+                prefer: ['']
+        steps:
+            -   uses: actions/checkout@v4
+            -   name: Setup PHP
+                uses: shivammathur/setup-php@v2
+                with:
+                    php-version: ${{ matrix.php }}
+                    tools: composer:v2, cs2pr
+                    coverage: none
+            -   name: Get composer cache directory
+                id: composer-cache
+                run: echo "dir=$(composer config cache-files-dir)" >> $GITHUB_OUTPUT
+            -   name: Cache dependencies
+                uses: actions/cache@v4
+                with:
+                    path: ${{ steps.composer-cache.outputs.dir }}
+                    key: ${{ runner.os }}-composer-${{ matrix.prefer }}-${{ hashFiles('**/composer.json') }}
+                    restore-keys: ${{ runner.os }}-composer-${{ matrix.prefer }}-
+            -   name: Install dependencies
+                run: composer update --no-interaction --no-progress ${{ matrix.prefer }}
+            -   name: PHP CS Fixer
+                run: vendor/bin/php-cs-fixer fix -vvv --dry-run --format=checkstyle | cs2pr

.php-cs-fixer.dist.php

@@ -0,0 +1,27 @@
+<?php
+
+use PhpCsFixer\Config;
+USE PhpCsFixer\Finder;
+
+$header = <<<'EOF'
+AJGL Doctrine DBAL Types
+
+Copyright (C) Antonio J. García Lagar <[email protected]>
+
+For the full copyright and license information, please view the LICENSE
+file that was distributed with this source code.
+EOF;
+
+return (new Config())
+    ->setRiskyAllowed(true)
+    ->setRules(
+        [
+            '@PER-CS' => true,
+            '@PER-CS:risky' => true,
+            '@PHP81Migration' => true,
+            '@PHP80Migration:risky' => true,
+            'header_comment' => ['header' => $header],
+        ]
+    )
+    ->setFinder(Finder::create()->in(__DIR__))
+;

composer.json

@@ -23,7 +23,8 @@
         "doctrine/dbal": "^3.3 || ^4"
     },
     "require-dev": {
-        "phpunit/phpunit": "^9.5"
+        "phpunit/phpunit": "^9.5",
+        "friendsofphp/php-cs-fixer": "^3.52"
     },
     "autoload": {
         "psr-4": {

src/ArrayTypeAbstract.php

@@ -1,5 +1,16 @@
 <?php
 
+declare(strict_types=1);
+
+/*
+ * AJGL Doctrine DBAL Types
+ *
+ * Copyright (C) Antonio J. García Lagar <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
 namespace Ajgl\Doctrine\DBAL\Types;
 
 use Doctrine\DBAL\Types\Type;
@@ -35,7 +46,7 @@ public function canRequireSQLConversion(): bool
 
     public function convertToDatabaseValue($value, AbstractPlatform $platform): string
     {
-        array_walk_recursive($value, array($this, 'convertToDatabaseCallback'), $platform);
+        array_walk_recursive($value, [$this, 'convertToDatabaseCallback'], $platform);
 
         return self::parseArrayToPg($value);
     }
@@ -44,7 +55,7 @@ public function convertToPHPValue($value, AbstractPlatform $platform): ?array
     {
         if (null !== $value) {
             $value = self::parsePgToArray($value);
-            array_walk_recursive($value, array($this, 'convertToPhpCallback'), $platform);
+            array_walk_recursive($value, [$this, 'convertToPhpCallback'], $platform);
         }
 
         return $value;
@@ -59,18 +70,18 @@ public function requiresSQLCommentHint(AbstractPlatform $platform): bool
      * @see https://web.archive.org/web/20120721205048/http://www.php.net/manual/es/ref.pgsql.php#89841
      * @author [email protected]
      */
-    protected static function parsePgToArray(string $input, &$output=null, int $limit=null, int $offset=1)
+    protected static function parsePgToArray(string $input, &$output = null, int $limit = null, int $offset = 1)
     {
         if (null === $limit) {
             $limit = strlen($input) - 1;
-            $output = array();
+            $output = [];
         }
         if ('{}' != $input) {
             do {
                 if ('{' != $input[$offset]) {
                     preg_match("/(\\{?\"([^\"\\\\]|\\\\.)*\"|[^,{}]+)+([,}]+)/", $input, $match, 0, $offset);
                     $offset += strlen($match[0]);
-                    $output[] = ( '"' != $match[1][0] ? $match[1] : stripcslashes(substr($match[1], 1, -1)) );
+                    $output[] = ('"' != $match[1][0] ? $match[1] : stripcslashes(substr($match[1], 1, -1)));
                     if ('},' == $match[3]) {
                         return $offset;
                     }
@@ -107,7 +118,7 @@ public function getInnerType(): Type
      * @param mixed $v
      * @return mixed
      */
-    protected function convertToPhpCallback(&$v,  string $k, AbstractPlatform $platform)
+    protected function convertToPhpCallback(&$v, string $k, AbstractPlatform $platform)
     {
         $v = $this->getInnerType()->convertToPHPValue($v, $platform);
     }

src/BigIntArrayType.php

@@ -1,9 +1,21 @@
 <?php
+
+declare(strict_types=1);
+
+/*
+ * AJGL Doctrine DBAL Types
+ *
+ * Copyright (C) Antonio J. García Lagar <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
 namespace Ajgl\Doctrine\DBAL\Types;
 
 final class BigIntArrayType extends ArrayTypeAbstract
 {
-    const BIGINTARRAY = 'bigint[]';
+    public const BIGINTARRAY = 'bigint[]';
 
     protected string $name = self::BIGINTARRAY;
 

src/BooleanArrayType.php

@@ -1,9 +1,21 @@
 <?php
+
+declare(strict_types=1);
+
+/*
+ * AJGL Doctrine DBAL Types
+ *
+ * Copyright (C) Antonio J. García Lagar <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
 namespace Ajgl\Doctrine\DBAL\Types;
 
 final class BooleanArrayType extends ArrayTypeAbstract
 {
-    const BOOLEANARRAY = 'boolean[]';
+    public const BOOLEANARRAY = 'boolean[]';
 
     protected string $name = self::BOOLEANARRAY;
 

src/IntegerArrayType.php

@@ -1,9 +1,21 @@
 <?php
+
+declare(strict_types=1);
+
+/*
+ * AJGL Doctrine DBAL Types
+ *
+ * Copyright (C) Antonio J. García Lagar <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
 namespace Ajgl\Doctrine\DBAL\Types;
 
 final class IntegerArrayType extends ArrayTypeAbstract
 {
-    const INTEGERARRAY = 'integer[]';
+    public const INTEGERARRAY = 'integer[]';
 
     protected string $name = self::INTEGERARRAY;
 

src/SmallIntArrayType.php

@@ -1,9 +1,21 @@
 <?php
+
+declare(strict_types=1);
+
+/*
+ * AJGL Doctrine DBAL Types
+ *
+ * Copyright (C) Antonio J. García Lagar <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
 namespace Ajgl\Doctrine\DBAL\Types;
 
 final class SmallIntArrayType extends ArrayTypeAbstract
 {
-    const SMALLINTARRAY = 'smallint[]';
+    public const SMALLINTARRAY = 'smallint[]';
 
     protected string $name = self::SMALLINTARRAY;
 

src/StringArrayType.php

@@ -1,9 +1,21 @@
 <?php
+
+declare(strict_types=1);
+
+/*
+ * AJGL Doctrine DBAL Types
+ *
+ * Copyright (C) Antonio J. García Lagar <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
 namespace Ajgl\Doctrine\DBAL\Types;
 
 final class StringArrayType extends ArrayTypeAbstract
 {
-    const STRINGARRAY = 'string[]';
+    public const STRINGARRAY = 'string[]';
 
     protected string $name = self::STRINGARRAY;
 

src/TextArrayType.php

@@ -1,9 +1,21 @@
 <?php
+
+declare(strict_types=1);
+
+/*
+ * AJGL Doctrine DBAL Types
+ *
+ * Copyright (C) Antonio J. García Lagar <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
 namespace Ajgl\Doctrine\DBAL\Types;
 
 final class TextArrayType extends ArrayTypeAbstract
 {
-    const TEXTARRAY = 'text[]';
+    public const TEXTARRAY = 'text[]';
 
     protected string $name = self::TEXTARRAY;
 

src/XmlArrayType.php

@@ -1,9 +1,21 @@
 <?php
+
+declare(strict_types=1);
+
+/*
+ * AJGL Doctrine DBAL Types
+ *
+ * Copyright (C) Antonio J. García Lagar <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
 namespace Ajgl\Doctrine\DBAL\Types;
 
 final class XmlArrayType extends ArrayTypeAbstract
 {
-    const XMLARRAY = 'xml[]';
+    public const XMLARRAY = 'xml[]';
 
     protected string $name = self::XMLARRAY;
 

src/XmlType.php

@@ -1,5 +1,15 @@
 <?php
 
+declare(strict_types=1);
+
+/*
+ * AJGL Doctrine DBAL Types
+ *
+ * Copyright (C) Antonio J. García Lagar <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
 
 namespace Ajgl\Doctrine\DBAL\Types;
 
@@ -11,7 +21,7 @@
 
 final class XmlType extends TextType
 {
-    const XML = 'xml';
+    public const XML = 'xml';
 
     public function getName(): string
     {