BC fix

src/Provider/SeptemberFirstProvider.php

@@ -28,18 +28,38 @@ class SeptemberFirstProvider extends GenericProvider
     const API_BASE = 'https://api.1sept.ru';
 
     /**
-     * @var string Версия API
+     * @var array Разрешения (scopes) по умолчанию
      */
-    const API_VERSION = '2.0';
+    const SCOPES_DEFAULT = ['profile'];
+
+    /**
+     * @var string Разделитель перечня запрашиваемых разрешений
+     */
+    const SCOPES_SEPARATOR = ' ';
+
+    /**
+     * @var string Путь авторизации
+     */
+    const AUTHORIZE_PATH = '/oauth/authorize';
+
+    /**
+     * @var string Путь получения токена
+     */
+    const ACCESS_TOKEN_PATH = '/oauth/access_token';
+
+    /**
+     * @var string Путь получения данных пользователя
+     */
+    const USERINFO_PATH = '/2.0/userinfo';
 
     public function __construct(array $options = [], array $collaborators = [])
     {
         $defaultOptions = [
-            'urlAuthorize' => static::AUTH_BASE.'/oauth/authorize',
+            'urlAuthorize' => static::AUTH_BASE.static::AUTHORIZE_PATH,
-            'urlAccessToken' => static::API_BASE.'/oauth/access_token',
+            'urlAccessToken' => static::API_BASE.static::ACCESS_TOKEN_PATH,
-            'urlResourceOwnerDetails' => static::API_BASE.'/'.static::API_VERSION.'/userinfo',
+            'urlResourceOwnerDetails' => static::API_BASE.static::USERINFO_PATH,
-            'scopes' => ['profile'],
+            'scopes' => static::SCOPES_DEFAULT,
-            'scopeSeparator' => ' ',
+            'scopeSeparator' => static::SCOPES_SEPARATOR,
         ];
 
         parent::__construct(array_merge($defaultOptions, $options), $collaborators);
@@ -50,8 +70,8 @@ class SeptemberFirstProvider extends GenericProvider
      */
     protected function checkResponse(ResponseInterface $response, $data): void
     {
-        if (!empty($data['error'])) {
+        if (! empty($data['error'])) {
-            throw new IdentityProviderException($data['error'].': '.$data['message'], null, $response);
+            throw new IdentityProviderException($data['error'].': '.$data['message'], 0, $response);
         }
     }
 

src/Provider/SeptemberFirstUser.php

@@ -152,20 +152,20 @@ class SeptemberFirstUser implements ResourceOwnerInterface
      */
     public function getBirthday(): ?\DateTime
     {
-        return !empty($this->data['birthday']) ? new \DateTime($this->data['birthday']) : null;
+        return ! empty($this->data['birthday']) ? new \DateTime($this->data['birthday']) : null;
     }
 
     /**
      * URL аватарки (150x150)
      *
-     * @param bool $addVersion Использовать версию аватарки для улучшенного кэширования
+     * @param bool $rejectDefaultAvatar
      * @return string|null
-     * 
+     *
      * @example https://avatar.1sept.ru/12121212-3456-7243-2134-432432144221.jpeg?v=12345
      */
-    public function getAvatarUrl(bool $addVersion = true): ?string
+    public function getAvatarUrl(bool $rejectDefaultAvatar = false): ?string
     {
-        return $this->getField('avatar') . ($addVersion ? $this->getAvatarVersionQuery() : '');
+        return ($rejectDefaultAvatar && $this->isDefaultAvatar()) ? null : $this->getField('avatar');
     }
 
     /**
@@ -224,9 +224,9 @@ class SeptemberFirstUser implements ResourceOwnerInterface
     /**
      * Является ли аватарка заглушкой
      *
-     * @return boolean
+     * @return bool|null
      */
-    public function isDefaultAvatar(): bool
+    public function isDefaultAvatar(): ?bool
     {
         return (bool) $this->getField('avatar_default');
     }
@@ -279,7 +279,7 @@ class SeptemberFirstUser implements ResourceOwnerInterface
      * СНИЛС
      *
      * @return string|null
-     * @example 123-123-123 56
+     * @example 123-456-789 01
      */
     public function getSnils(): ?string
     {
@@ -330,7 +330,7 @@ class SeptemberFirstUser implements ResourceOwnerInterface
     {
         return $this->getField('address.country_id');
     }
-    
+
     /**
      * ID региона страны адреса
      *
@@ -341,7 +341,7 @@ class SeptemberFirstUser implements ResourceOwnerInterface
     {
         return $this->getField('address.region_id');
     }
-    
+
     /**
      * Почтовый индекс
      *