UPD to php 8.4

.gitignore

@@ -2,3 +2,5 @@ build
 composer.lock
 docs
 vendor
+.php-cs-fixer.cache
+.phpcs-cache

composer.json

@@ -1,5 +1,6 @@
 {
     "name": "1sept/oauth2-vkontakte",
+    "version": "1.2",
     "description": "VK provider for league/oauth2-client",
     "keywords": [
         "league",
@@ -13,7 +14,7 @@
         }
     ],
     "require": {
-        "php" : ">=8.1",
+        "php" : ">=8.3",
         "league/oauth2-client": "^2.0"
     },
     "require-dev": {

phpcs.xml.dist

@@ -0,0 +1,20 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+<ruleset xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+        xsi:noNamespaceSchemaLocation="vendor/squizlabs/php_codesniffer/phpcs.xsd">
+
+    <arg name="basepath" value="."/>
+    <arg name="cache" value=".phpcs-cache"/>
+    <arg name="colors"/>
+    <arg name="extensions" value="php"/>
+
+    <rule ref="PSR12">
+        <exclude name="Generic.Files.LineLength"/>
+        <exclude name="PSR1.Methods.CamelCapsMethodName.NotCamelCaps"/>
+    </rule>
+
+    <file>src/</file>
+
+</ruleset>
+
+<!-- composer require "squizlabs/php_codesniffer=*" dev -->

phpstan.dist.neon

@@ -0,0 +1,14 @@
+parameters:
+    level: 5
+    paths:
+        - bin/
+        - config/
+        - public/
+        - src/
+
+    errorFormat: table
+    editorUrl: 'vscode://file/%%file%%:%%line%%'
+    editorUrlTitle: '%%relFile%%:%%line%%'
+
+    ignoreErrors:
+        - '#Construct empty\(\) is not allowed. Use more strict comparison.#'

src/VkProvider.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Sept\OAuth2\Client\Provider;
 
 use League\OAuth2\Client\Provider\AbstractProvider;
@@ -9,10 +11,10 @@ use Psr\Http\Message\ResponseInterface;
 
 class VkProvider extends AbstractProvider
 {
-    protected $baseOAuthUri = 'https://oauth.vk.com';
+    protected string $baseOAuthUri = 'https://oauth.vk.com';
-    protected $baseUri      = 'https://api.vk.com/method';
+    protected string $baseUri = 'https://api.vk.com/method';
-    protected $version      = '5.236';
+    protected float $version = 5.199;
-    protected $language     = null;
+    protected ?string $language = null;
 
     /**
      * @see https://vk.com/dev/permissions
@@ -38,8 +40,8 @@ class VkProvider extends AbstractProvider
         // 'status',
         // 'video',
     ];
+
     /**
-     * @type array
      * @see  https://new.vk.com/dev/fields
      */
     public array $userFields = [
@@ -60,6 +62,8 @@ class VkProvider extends AbstractProvider
         'photo_max_orig',
         'screen_name',
         'sex',
+        'deactivated',
+        'is_closed',
         // 'about',
         // 'activities',
         // 'blacklisted',
@@ -111,12 +115,9 @@ class VkProvider extends AbstractProvider
         // 'wall_comments',
     ];
 
-    /**
-     * @param string $language
-     */
     public function setLanguage(string $language): self
     {
-        $this->language = (string)$language;
+        $this->language = $language;
 
         return $this;
     }
@@ -137,17 +138,19 @@ class VkProvider extends AbstractProvider
             'fields' => $this->userFields,
             'access_token' => $token->getToken(),
             'v' => $this->version,
-            'lang'         => $this->language
+            'lang' => $this->language,
         ];
         $query = $this->buildQueryString($params);
         $url = "$this->baseUri/users.get?$query";
 
         return $url;
     }
-    protected function getDefaultScopes()
+
+    protected function getDefaultScopes(): array
     {
         return $this->scopes;
     }
+
     protected function checkResponse(ResponseInterface $response, $data): void
     {
         // Metadata info
@@ -157,12 +160,13 @@ class VkProvider extends AbstractProvider
         // Response info
         $responseCode = $response->getStatusCode();
         $responseMessage = $response->getReasonPhrase();
+
         // Data info
         $error = !empty($data['error']) ? $data['error'] : null;
         $errorCode = !empty($error['error_code']) ? $error['error_code'] : $responseCode;
         $errorDescription = !empty($data['error_description']) ? $data['error_description'] : null;
         $errorMessage = !empty($error['error_msg']) ? $error['error_msg'] : $errorDescription;
-        $message          = $errorMessage ?: $responseMessage;
+        $message = (bool) $errorMessage ? $errorMessage : $responseMessage;
 
         // Request/meta validation
         if (399 < $responseCode) {
@@ -170,89 +174,93 @@ class VkProvider extends AbstractProvider
         }
 
         // Content validation
-        if ('application/json' != $contentType) {
+        if ('application/json' !== $contentType) {
             throw new IdentityProviderException($message, $responseCode, $data);
         }
         if ($error) {
             throw new IdentityProviderException($errorMessage, $errorCode, $data);
         }
     }
+
     protected function createResourceOwner(array $response, AccessToken $token): VkUser
     {
         $response = reset($response['response']);
         $additional = $token->getValues();
+
         if (!empty($additional['email'])) {
             $response['email'] = $additional['email'];
         }
-        if (!empty($response['uid']) && 4 === floor($this->version)) {
+
+        if (!empty($response['uid']) && 4 === (int) floor($this->version)) {
             $response['id'] = $response['uid'];
         }
+
         if (!empty($additional['user_id'])) {
             $response['id'] = $additional['user_id'];
         }
 
-        return new VkUser($response, $response['id']);
+        return new VkUser($response);
     }
 
     /**
+     * Возвращает массив объектов пользователей.
+     *
      * @see https://vk.com/dev/users.get
-     *
-     * @param integer[]        $ids
-     * @param AccessToken|null $token Current user if empty
-     * @param array            $params
-     *
-     * @return User[]
      */
-    public function usersGet(array $ids = [], AccessToken $token = null, array $params = []): array
+    public function usersGet(array $ids = [], ?AccessToken $token = null, array $params = []): array
     {
-        if (empty($ids) && !$token) {
+        if (empty($ids) && null === $token) {
             throw new \InvalidArgumentException('Some of parameters usersIds OR access_token are required');
         }
 
         $default = [
             'user_ids' => implode(',', $ids),
             'fields' => $this->userFields,
-            'access_token' => $token ? $token->getToken() : null,
+            'access_token' => null !== $token ? $token->getToken() : null,
             'v' => $this->version,
-            'lang'         => $this->language
+            'lang' => $this->language,
         ];
+
         $params = array_merge($default, $params);
         $query = $this->buildQueryString($params);
         $url = "$this->baseUri/users.get?$query";
 
         $response = $this->getResponse($this->createRequest(static::METHOD_GET, $url, $token, []))['response'];
         $users = !empty($response['items']) ? $response['items'] : $response;
-        $array2user = function ($userData) {
+
+        $array2user = function (mixed $userData) {
             return new VkUser($userData);
         };
 
         return array_map($array2user, $users);
     }
+
     /**
+     * Возвращает список идентификаторов (id) друзей пользователя,
+     * если параметр fields не использовался. При использовании
+     * параметра fields возвращает список объектов пользователей,
+     * но не более 5000.
+     *
      * @see https://vk.com/dev/friends.get
-     *
-     * @param integer          $userId
-     * @param AccessToken|null $token
-     * @param array            $params
-     *
-     * @return User[]
      */
-    public function friendsGet($userId, AccessToken $token = null, array $params = []): array
+    public function friendsGet(int $userId, ?AccessToken $token = null, array $params = []): array
     {
         $default = [
             'user_id' => $userId,
             'fields' => $this->userFields,
-            'access_token' => $token ? $token->getToken() : null,
+            'access_token' => null !== $token ? $token->getToken() : null,
             'v' => $this->version,
-            'lang'         => $this->language
+            'lang' => $this->language,
         ];
+
         $params = array_merge($default, $params);
         $query = $this->buildQueryString($params);
         $url = "$this->baseUri/friends.get?$query";
 
         $response = $this->getResponse($this->createRequest(static::METHOD_GET, $url, $token, []))['response'];
         $friends = !empty($response['items']) ? $response['items'] : $response;
-        $array2friend = function ($friendData) {
+
+        $array2friend = function (mixed $friendData) {
             if (is_numeric($friendData)) {
                 $friendData = ['id' => $friendData];
             }

src/VkUser.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Sept\OAuth2\Client\Provider;
 
 use League\OAuth2\Client\Provider\ResourceOwnerInterface;
@@ -10,7 +12,7 @@ use League\OAuth2\Client\Provider\ResourceOwnerInterface;
 class VkUser implements ResourceOwnerInterface
 {
     /**
-     * @type array
+     * @var array
      */
     protected $response;
 
@@ -29,21 +31,21 @@ class VkUser implements ResourceOwnerInterface
 
     public function getId(): int
     {
-        return (int)($this->getField('uid') ?: $this->getField('id'));
+        return (int) ((bool) $this->getField('uid') ? $this->getField('uid') : $this->getField('id'));
     }
 
     /**
-     * Helper for getting user data
+     * Helper for getting user data.
      */
     protected function getField(string $key): mixed
     {
-        return !empty($this->response[$key]) ? $this->response[$key] : null;
+        return \array_key_exists($key, $this->response) ? $this->response[$key] : null;
     }
 
     /**
      * @return string|null DD.MM.YYYY
      */
-    public function getBirthday():string
+    public function getBirthday(): ?string
     {
         return $this->getField('bdate');
     }
@@ -51,7 +53,7 @@ class VkUser implements ResourceOwnerInterface
     /**
      * @return array [id =>, title => string]
      */
-    public function getCity(): array
+    public function getCity(): ?array
     {
         return $this->getField('city');
     }
@@ -59,28 +61,33 @@ class VkUser implements ResourceOwnerInterface
     /**
      * @return array [id =>, title => string]
      */
-    public function getCountry(): array
+    public function getCountry(): ?array
     {
         return $this->getField('country');
     }
 
     /**
-     * Short address to user page
+     * Short address to user page.
      */
-    public function getDomain(): string
+    public function getDomain(): ?string
     {
         return $this->getField('domain');
     }
 
-    public function getFirstName(): string
+    /**
+     * Return firstname.
+     */
+    public function getFirstName(): ?string
     {
         return $this->getField('first_name');
     }
 
     /**
+     * Friend status.
+     *
      * @return int 0|1|2|3 => nobody|resquest_sent|incoming_request|friends
      */
-    public function getFriendStatus(): int
+    public function getFriendStatus(): ?int
     {
         return $this->getField('friend_Status');
     }
@@ -93,58 +100,77 @@ class VkUser implements ResourceOwnerInterface
         return (bool) $this->getField('has_photo');
     }
 
-    public function getHomeTown(): string
+    public function getHomeTown(): ?string
     {
         return $this->getField('home_town');
     }
 
     /**
-     * Detect if current user is freind to this
+     * Detect if current user is freind to this.
      */
     public function isFriend(): bool
     {
         return (bool) $this->getField('is_friend');
     }
 
-    public function getLastName(): string
+    public function getLastName(): ?string
     {
         return $this->getField('last_name');
     }
 
-    public function getMaidenName(): string
+    public function getMaidenName(): ?string
     {
         return $this->getField('maiden_name');
     }
 
-    public function getNickname(): string
+    public function getNickname(): ?string
     {
         return $this->getField('nickname');
     }
+
     /**
      * It's square!
      */
-    public function getPhotoMax():string
+    public function getPhotoMax(): ?string
     {
         return $this->getField('photo_max');
     }
 
-    public function getPhotoMaxOrig(): string
+    public function getPhotoMaxOrig(): ?string
     {
         return $this->getField('photo_max_orig');
     }
 
-    public function getScreenName(): string
+    public function getScreenName(): ?string
     {
         return $this->getField('screen_name');
     }
 
-    public function getSex(): int
+    public function getSex(): ?int
     {
         return $this->getField('sex');
     }
 
-    public function getEmail(): string
+    public function getEmail(): ?string
     {
         return $this->getField('email');
     }
+
+    /**
+     * Поле возвращается, если страница пользователя удалена или заблокирована, содержит значение deleted или banned. В этом случае опциональные поля не возвращаются.
+     *
+     * @return string null|deleted|banned
+     */
+    public function getDeactivated(): ?string
+    {
+        return $this->getField('deactivated');
+    }
+
+    /**
+     * Скрыт ли профиль пользователя настройками приватности.
+     */
+    public function isClosed(): bool
+    {
+        return $this->getField('is_closed');
+    }
 }