Merge pull request #8 from answear/support-symfony6

Support Symfony6

.github/workflows/phpunit.yml

@@ -12,6 +12,8 @@ jobs:
             matrix:
                 php-version:
                     - "7.4"
+                    - "8.0"
+                    - "8.1"
                 deps:
                     - "normal"
                 include:

.github/workflows/static-analysis.yml

@@ -12,7 +12,8 @@ jobs:
             matrix:
                 php-version:
                     - "7.4"
-
+                    - "8.0"
+                    - "8.1"
         steps:
             -   name: "Checkout"
                 uses: "actions/checkout@v2"

.gitignore

@@ -1,6 +1,5 @@
 /vendor/
-\.php_cs\.cache
-.phpunit.result.cache
+*.cache
 composer.lock
 /.idea
 /.env*

.php_cs → .php-cs-fixer.php

@@ -1,14 +1,19 @@
 <?php
 
-$finder = PhpCsFixer\Finder::create()
+declare(strict_types=1);
+
+$config = new PhpCsFixer\Config();
+
+$finder = $config->getFinder()
     ->exclude(
         [
             'vendor',
         ]
     )
     ->in(__DIR__);
 
-return PhpCsFixer\Config::create()
+return $config
+    ->registerCustomFixers([])
     ->setRules(
         [
             '@Symfony' => true,
@@ -21,6 +26,15 @@
             'phpdoc_var_without_name' => false,
             'phpdoc_to_comment' => false,
             'single_line_throw' => false,
+            'modernize_types_casting' => true,
+            'function_declaration' => false,
+            'ordered_imports' => [
+                'imports_order' => [
+                    'class',
+                    'function',
+                    'const',
+                ],
+                'sort_algorithm' => 'alpha',
+            ],
         ]
-    )
-    ->setFinder($finder);
+    );

composer.json

@@ -1,50 +1,53 @@
 {
-  "name": "answear/econt-pickup-point-bundle",
-  "description": "API Client for EcontBundle",
-  "type": "symfony-bundle",
-  "license": "MIT",
-  "require": {
-    "php": ">=7.4",
-    "ext-json": "*",
-    "guzzlehttp/guzzle": "^6.0",
-    "marc-mabe/php-enum": "^3.0|^4.3",
-    "symfony/http-kernel": "^4.4|^5.0",
-    "symfony/serializer": "^4.4|^5.0",
-    "webmozart/assert": "^1.3"
-  },
-  "require-dev": {
-    "friendsofphp/php-cs-fixer": "^2.16",
-    "phpro/grumphp": "^1.3.0",
-    "phpstan/phpstan": "^0.12.32",
-    "phpstan/phpstan-webmozart-assert": "^0.12.2",
-    "phpunit/phpunit": "^8.4",
-    "roave/security-advisories": "dev-master",
-    "symfony/phpunit-bridge": "^5.0"
-  },
-  "autoload": {
-    "psr-4": {
-      "Answear\\EcontBundle\\": "src/"
+    "name": "answear/econt-pickup-point-bundle",
+    "description": "API Client for EcontBundle",
+    "type": "symfony-bundle",
+    "license": "MIT",
+    "require": {
+        "php": ">=7.4|^8.0",
+        "ext-json": "*",
+        "guzzlehttp/guzzle": "^6.0|^7.0",
+        "marc-mabe/php-enum": "^3.0|^4.3",
+        "symfony/http-kernel": "^5.4|^6.0",
+        "symfony/serializer": "^5.4|^6.0",
+        "webmozart/assert": "^1.3"
+    },
+    "require-dev": {
+        "friendsofphp/php-cs-fixer": "^3.4",
+        "phpro/grumphp": "^1.5.0",
+        "phpstan/phpstan": "^1.4",
+        "phpstan/phpstan-webmozart-assert": "^1.0",
+        "phpunit/phpunit": "^9.5",
+        "roave/security-advisories": "dev-master",
+        "symfony/phpunit-bridge": "6.1.*"
+    },
+    "autoload": {
+        "psr-4": {
+            "Answear\\EcontBundle\\": "src/"
+        }
+    },
+    "autoload-dev": {
+        "psr-4": {
+            "Answear\\EcontBundle\\Tests\\": "tests/"
+        }
+    },
+    "scripts": {
+        "post-install-cmd": [
+            "GrumPHP\\Composer\\DevelopmentIntegrator::integrate"
+        ],
+        "post-update-cmd": [
+            "GrumPHP\\Composer\\DevelopmentIntegrator::integrate"
+        ]
+    },
+    "extra": {
+        "grumphp": {
+            "config-default-path": "grumphp.yaml"
+        }
+    },
+    "config": {
+        "sort-packages": true,
+        "allow-plugins": {
+            "phpro/grumphp": true
+        }
     }
-  },
-  "autoload-dev": {
-    "psr-4": {
-      "Answear\\EcontBundle\\Tests\\": "tests/"
-    }
-  },
-  "scripts": {
-    "post-install-cmd": [
-      "GrumPHP\\Composer\\DevelopmentIntegrator::integrate"
-    ],
-    "post-update-cmd": [
-      "GrumPHP\\Composer\\DevelopmentIntegrator::integrate"
-    ]
-  },
-  "extra": {
-    "grumphp": {
-      "config-default-path": "grumphp.yaml"
-    }
-  },
-  "config": {
-    "sort-packages": true
-  }
 }

grumphp.yaml

@@ -18,7 +18,7 @@ grumphp:
         phpcsfixer:
             allow_risky: true
             cache_file: ~
-            config: './.php_cs'
+            config: './.php-cs-fixer.php'
             using_cache: true
             verbose: true
         phpstan:

phpstan-baseline.neon

@@ -0,0 +1,16 @@
+parameters:
+	ignoreErrors:
+		-
+			message: "#^Property Answear\\\\EcontBundle\\\\Request\\\\GetCitiesRequest\\:\\:\\$countryCode is never read, only written\\.$#"
+			count: 1
+			path: src/Request/GetCitiesRequest.php
+
+		-
+			message: "#^Property Answear\\\\EcontBundle\\\\Request\\\\GetOfficesRequest\\:\\:\\$cityId is never read, only written\\.$#"
+			count: 1
+			path: src/Request/GetOfficesRequest.php
+
+		-
+			message: "#^Property Answear\\\\EcontBundle\\\\Request\\\\GetOfficesRequest\\:\\:\\$countryCode is never read, only written\\.$#"
+			count: 1
+			path: src/Request/GetOfficesRequest.php

phpstan.neon

@@ -1,5 +1,6 @@
 includes:
     - vendor/phpstan/phpstan-webmozart-assert/extension.neon
+    - phpstan-baseline.neon
 
 parameters:
     level: 5

src/Client/Serializer.php

@@ -19,7 +19,7 @@ public function serialize(Request $request): string
     {
         return $this->getSerializer()->serialize(
             $request,
-            static::FORMAT,
+            self::FORMAT,
             [Normalizer\AbstractObjectNormalizer::SKIP_NULL_VALUES => true]
         );
     }

src/ConfigProvider.php

@@ -8,6 +8,7 @@ class ConfigProvider
 {
     private const URL = 'http://ee.econt.com/';
     private const SERVICE_URI = '/services/Nomenclatures/';
+
     private string $user;
     private string $password;
 
@@ -27,12 +28,12 @@ public function getRequestHeaders(): array
 
     public function getUrl(): string
     {
-        return static::URL;
+        return self::URL;
     }
 
     public function getServiceURI(): string
     {
-        return static::SERVICE_URI;
+        return self::SERVICE_URI;
     }
 
     public function getUser(): string

src/Request/GetCitiesRequest.php

@@ -18,11 +18,11 @@ public function __construct(?string $countryCode = null)
 
     public function getEndpoint(): string
     {
-        return static::ENDPOINT;
+        return self::ENDPOINT;
     }
 
     public function getMethod(): string
     {
-        return static::HTTP_METHOD;
+        return self::HTTP_METHOD;
     }
 }

src/Request/GetOfficesRequest.php

@@ -20,11 +20,11 @@ public function __construct(?string $countryCode = null, ?int $cityId = null)
 
     public function getEndpoint(): string
     {
-        return static::ENDPOINT;
+        return self::ENDPOINT;
     }
 
     public function getMethod(): string
     {
-        return static::HTTP_METHOD;
+        return self::HTTP_METHOD;
     }
 }

src/Response/ErrorResponse.php

@@ -31,7 +31,7 @@ public static function isErrorResponse(array $response): bool
 
     public static function fromArray(array $response): self
     {
-        if (!static::isErrorResponse($response)) {
+        if (!self::isErrorResponse($response)) {
             throw new RuntimeException('Cannot create ErrorResponse');
         }