Merge pull request #1094 from Sephster/3.x

Remove ext-json

Author: Sephster

composer.json

@@ -1,84 +1,83 @@
 {
-    "name": "league/oauth2-client",
-    "description": "OAuth 2.0 Client Library",
-    "license": "MIT",
-    "keywords": [
-        "oauth",
-        "oauth2",
-        "authorization",
-        "authentication",
-        "idp",
-        "identity",
-        "sso",
-        "single sign on"
-    ],
-    "authors": [
-        {
-            "name": "Alex Bilbie",
-            "email": "hello@alexbilbie.com",
-            "homepage": "http://www.alexbilbie.com",
-            "role": "Developer"
-        },
-        {
-            "name": "Woody Gilk",
-            "homepage": "https://github.com/shadowhand",
-            "role": "Contributor"
-        }
-    ],
-    "require": {
-        "php": "~8.1.0 || ~8.2.0 || ~8.3.0 || ~8.4.0",
-        "ext-json": "*",
-        "psr/http-client": "^1.0",
-        "psr/http-factory": "^1.1",
-        "psr/http-message": "^2.0"
+  "name": "league/oauth2-client",
+  "description": "OAuth 2.0 Client Library",
+  "license": "MIT",
+  "keywords": [
+    "oauth",
+    "oauth2",
+    "authorization",
+    "authentication",
+    "idp",
+    "identity",
+    "sso",
+    "single sign on"
+  ],
+  "authors": [
+    {
+      "name": "Alex Bilbie",
+      "email": "hello@alexbilbie.com",
+      "homepage": "http://www.alexbilbie.com",
+      "role": "Developer"
     },
-    "require-dev": {
-        "ergebnis/composer-normalize": "^2.45",
-        "guzzlehttp/guzzle": "^7.4.5",
-        "mockery/mockery": "^1.6",
-        "php-parallel-lint/php-parallel-lint": "^1.4",
-        "phpstan/extension-installer": "^1.4",
-        "phpstan/phpstan": "^2.0",
-        "phpstan/phpstan-mockery": "^2.0",
-        "phpstan/phpstan-phpunit": "^2.0",
-        "phpunit/phpunit": "^10.5 || ^11.5",
-        "ramsey/coding-standard": "^2.2",
-        "squizlabs/php_codesniffer": "^3.11"
-    },
-    "autoload": {
-        "psr-4": {
-            "League\\OAuth2\\Client\\": "src/"
-        }
-    },
-    "autoload-dev": {
-        "psr-4": {
-            "League\\OAuth2\\Client\\Test\\": "test/src/"
-        }
-    },
-    "config": {
-        "allow-plugins": {
-            "dealerdirect/phpcodesniffer-composer-installer": true,
-            "ergebnis/composer-normalize": true,
-            "phpstan/extension-installer": true
-        },
-        "sort-packages": true
-    },
-    "scripts": {
-        "analyze": "phpstan analyze --memory-limit=1G",
-        "cs": "phpcs",
-        "cs-fix": "phpcbf",
-        "lint": "parallel-lint src test",
-        "test": [
-            "@lint",
-            "@cs",
-            "@analyze",
-            "phpunit --no-coverage"
-        ]
-    },
-    "scripts-descriptions": {
-        "cs": "Check for coding style errors.",
-        "cs-fix": "Check and automatically fix any fixable coding style errors.",
-        "lint": "Check source code for syntax errors.",
-        "test": "Run the full test suite."
+    {
+      "name": "Woody Gilk",
+      "homepage": "https://github.com/shadowhand",
+      "role": "Contributor"
+    }
+  ],
+  "require": {
+    "php": "~8.1.0 || ~8.2.0 || ~8.3.0 || ~8.4.0 || ~8.5.0",
+    "psr/http-client": "^1.0",
+    "psr/http-factory": "^1.1",
+    "psr/http-message": "^2.0"
+  },
+  "require-dev": {
+    "ergebnis/composer-normalize": "^2.45",
+    "guzzlehttp/guzzle": "^7.4.5",
+    "mockery/mockery": "^1.6",
+    "php-parallel-lint/php-parallel-lint": "^1.4",
+    "phpstan/extension-installer": "^1.4",
+    "phpstan/phpstan": "^2.0",
+    "phpstan/phpstan-mockery": "^2.0",
+    "phpstan/phpstan-phpunit": "^2.0",
+    "phpunit/phpunit": "^10.5 || ^11.5",
+    "ramsey/coding-standard": "^2.2",
+    "squizlabs/php_codesniffer": "^3.11"
+  },
+  "autoload": {
+    "psr-4": {
+      "League\\OAuth2\\Client\\": "src/"
     }
+  },
+  "autoload-dev": {
+    "psr-4": {
+      "League\\OAuth2\\Client\\Test\\": "test/src/"
+    }
+  },
+  "config": {
+    "allow-plugins": {
+      "dealerdirect/phpcodesniffer-composer-installer": true,
+      "ergebnis/composer-normalize": true,
+      "phpstan/extension-installer": true
+    },
+    "sort-packages": true
+  },
+  "scripts": {
+    "analyze": "phpstan analyze --memory-limit=1G",
+    "cs": "phpcs",
+    "cs-fix": "phpcbf",
+    "lint": "parallel-lint src test",
+    "test": [
+      "@lint",
+      "@cs",
+      "@analyze",
+      "phpunit --no-coverage"
+    ]
+  },
+  "scripts-descriptions": {
+    "cs": "Check for coding style errors.",
+    "cs-fix": "Check and automatically fix any fixable coding style errors.",
+    "lint": "Check source code for syntax errors.",
+    "test": "Run the full test suite."
+  }
 }

src/Provider/GenericProvider.php

@@ -55,7 +55,6 @@ class GenericProvider extends AbstractProvider
 
     /**
      * @var list<string> | null
-     * @phpstan-ignore property.unusedType
      */
     private ?array $scopes = null;
 
@@ -64,9 +63,6 @@ class GenericProvider extends AbstractProvider
     private string $responseCode;
     private string $responseResourceOwnerId = 'id';
 
-    /**
-     * @phpstan-ignore property.unusedType
-     */
     private ?string $pkceMethod = null;
 
     /**