app\modules\auth\services\SocialAuth

Complexity

Total Complexity

5

Size/Duplication

Total Lines	59
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
B	prepareUser()	0	26	3
A	parser()	0	5	1

<?php

namespace app\modules\auth\services;

use yii\authclient\ClientInterface;
use app\models\entity\{User, UserProvider};

/**
 * SocialAuth handles successful authentication
 */
class SocialAuth
{
    /**
     * @var int
     */
    private $provider;
    /**
     * @var int
     */
    private $providerId;
    /**
     * @var ClientInterface
     */
    private $client;
    /**
     * @var array
     */
    public $parsers;

    public function __construct(ClientInterface $client)
    {
        $this->client = $client;
        $this->provider = UserProvider::getTypeByName($client->id);

Accessing id on the interface yii\authclient\ClientInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

        $this->providerId = $client->getUserAttributes()['id'];
    }

    public function prepareUser(): ?User
    {
        $user = null;
        $parser = $this->parser();

        $profileData = $parser->profileData();
        $tokenData = ['type' => $this->provider] + $parser->tokenData();

        if ($provider = UserProvider::find()->provider($this->provider, $this->providerId)->one()) {
            $user = $provider->user;

            // if exist then update access tokens
            $provider->setAttributes($tokenData);
            $provider->save();
        }

        if (!is_object($user)) {
            $user = new User();

            $user->email = $parser->email();
            $user->setProfile($profileData);
            $user->setProviders($tokenData);
        }

        return $user;
    }

    private function parser()
    {
        $parserClass = $this->parsers[$this->client->id];

Accessing id on the interface yii\authclient\ClientInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

        return new $parserClass($this->client);
    }
}