MadWeb\SocialAuth\SocialProviderManager

Complexity

Total Complexity

5

Size/Duplication

Total Lines	81
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	6

Test Coverage

Coverage

100%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	socialUserQuery()	0	4	1
A	getUserByKey()	0	4	1
A	attach()	0	11	1
A	createNewUser()	0	20	1

<?php

namespace MadWeb\SocialAuth;

use MadWeb\SocialAuth\Models\SocialProvider;
use Illuminate\Contracts\Auth\Authenticatable;
use MadWeb\SocialAuth\Events\SocialUserCreated;
use MadWeb\SocialAuth\Events\SocialUserAttached;
use Laravel\Socialite\Contracts\User as SocialUser;
use MadWeb\SocialAuth\Contracts\SocialAuthenticatable;

class SocialProviderManager
{
    /**
     * @var SocialProvider
     */
    protected $social;

    /**
     * SocialProviderManager constructor.
     * @param SocialProvider $social
     */
    public function __construct(SocialProvider $social)
    {
        $this->social = $social;
    }

    /**
     * @param string $key
     * @return \Illuminate\Database\Eloquent\Relations\BelongsToMany
     */
    public function socialUserQuery(string $key)
    {
        return $this->social->users()->wherePivot(config('social-auth.foreign_keys.socials'), $key);
    }

    /**
     * Gets user by unique social identifier.
     *
     * @param string $key
     * @return mixed
     */
    public function getUserByKey(string $key)
    {
        return $this->socialUserQuery($key)->first();
    }

    /**
     * @param SocialAuthenticatable $user
     * @param SocialUser $socialUser
     */
    public function attach(SocialAuthenticatable $user, SocialUser $socialUser)
    {
        $user->attachSocial(
            $this->social,
            $socialUser->getId(),
            $socialUser->token,

Accessing token on the interface Laravel\Socialite\Contracts\User suggest that you code against a concrete implementation. How about adding an instanceof check?

            $socialUser->expiresIn ?? null

Accessing expiresIn on the interface Laravel\Socialite\Contracts\User suggest that you code against a concrete implementation. How about adding an instanceof check?

        );

        event(new SocialUserAttached($user, $this->social, $socialUser));
    }

    /**
     * Create new system user by social user data.
     *
     * @param Authenticatable $userModel
     * @param SocialProvider $social
     * @param SocialUser $socialUser
     * @return Authenticatable
     */
    public function createNewUser(
        Authenticatable $userModel,
        SocialProvider $social,
        SocialUser $socialUser
    ): Authenticatable {
        $NewUser = $userModel->create(

The method create() does not seem to exist on object<Illuminate\Contracts\Auth\Authenticatable>.

            $userModel->mapSocialData($socialUser)

The method mapSocialData() does not seem to exist on object<Illuminate\Contracts\Auth\Authenticatable>.

        );

        $NewUser->attachSocial(
            $social,
            $socialUser->getId(),
            $socialUser->token,

Accessing token on the interface Laravel\Socialite\Contracts\User suggest that you code against a concrete implementation. How about adding an instanceof check?

            $socialUser->expiresIn ?? null

Accessing expiresIn on the interface Laravel\Socialite\Contracts\User suggest that you code against a concrete implementation. How about adding an instanceof check?

        );

        event(new SocialUserCreated($NewUser));

        return $NewUser;
    }
}