SocialProviderManager::getUserByKey() - Code Metrics - mad-web/laravel-social-auth - Measure and Improve Code Quality continuously with Scrutinizer

SocialProviderManager::getUserByKey() A
last analyzed 2017-10-29 22:08 UTC

↳ Parent: SocialProviderManager

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes

Metric	Value
dl	0
loc	4
ccs	2
cts	2
cp	1
rs	10
c	0
b	0
f	0
cc	1
eloc	2
nc	1
nop	1
crap	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,
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
            $socialUser->expiresIn ?? null
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
        );

        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(

            $userModel->mapSocialData($socialUser)

        );

        $NewUser->attachSocial(
            $social,
            $socialUser->getId(),
            $socialUser->token,
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
            $socialUser->expiresIn ?? null
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
        );

        event(new SocialUserCreated($NewUser));

        return $NewUser;
    }
}


1		<?php
2
3		namespace MadWeb\SocialAuth;
4
5		use MadWeb\SocialAuth\Models\SocialProvider;
6		use Illuminate\Contracts\Auth\Authenticatable;
7		use MadWeb\SocialAuth\Events\SocialUserCreated;
8		use MadWeb\SocialAuth\Events\SocialUserAttached;
9		use Laravel\Socialite\Contracts\User as SocialUser;
10		use MadWeb\SocialAuth\Contracts\SocialAuthenticatable;
11
12		class SocialProviderManager
13		{
14		/**
15		* @var SocialProvider
16		*/
17		protected $social;
18
19		/**
20		* SocialProviderManager constructor.
21		* @param SocialProvider $social
22		*/
23	51	public function __construct(SocialProvider $social)
24		{
25	51	$this->social = $social;
26	51	}
27
28		/**
29		* @param string $key
30		* @return \Illuminate\Database\Eloquent\Relations\BelongsToMany
31		*/
32	24	public function socialUserQuery(string $key)
33		{
34	24	return $this->social->users()->wherePivot(config('social-auth.foreign_keys.socials'), $key);
35		}
36
37		/**
38		* Gets user by unique social identifier.
39		*
40		* @param string $key
41		* @return mixed
42		*/
43	18	public function getUserByKey(string $key)
44		{
45	18	return $this->socialUserQuery($key)->first();
46		}
47
48		/**
49		* @param SocialAuthenticatable $user
50		* @param SocialUser $socialUser
51		*/
52	12	public function attach(SocialAuthenticatable $user, SocialUser $socialUser)
53		{
54	12	$user->attachSocial(
55	12	$this->social,
56	12	$socialUser->getId(),
57	12	$socialUser->token,
		0 ignored issues – show Bug introduced 2017-08-17 12:03 UTC by Report Bug Copy Issue Report Accessing `token` on the interface `Laravel\Socialite\Contracts\User` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
58	12	$socialUser->expiresIn ?? null
		0 ignored issues – show Bug introduced 2017-08-17 12:03 UTC by Report Bug Copy Issue Report Accessing `expiresIn` on the interface `Laravel\Socialite\Contracts\User` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
59		);
60
61	12	event(new SocialUserAttached($user, $this->social, $socialUser));
62	12	}
63
64		/**
65		* Create new system user by social user data.
66		*
67		* @param Authenticatable $userModel
68		* @param SocialProvider $social
69		* @param SocialUser $socialUser
70		* @return Authenticatable
71		*/
72	6	public function createNewUser(
73		Authenticatable $userModel,
74		SocialProvider $social,
75		SocialUser $socialUser
76		): Authenticatable {
77	6	$NewUser = $userModel->create(
		0 ignored issues – show Bug introduced 2017-08-17 12:03 UTC by Report Bug Copy Issue Report The method `create()` does not seem to exist on `object<Illuminate\Contracts\Auth\Authenticatable>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
78	6	$userModel->mapSocialData($socialUser)
		0 ignored issues – show Bug introduced 2017-08-17 12:03 UTC by Report Bug Copy Issue Report The method `mapSocialData()` does not seem to exist on `object<Illuminate\Contracts\Auth\Authenticatable>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
79		);
80
81	6	$NewUser->attachSocial(
82	6	$social,
83	6	$socialUser->getId(),
84	6	$socialUser->token,
		0 ignored issues – show Bug introduced 2017-08-17 12:03 UTC by Report Bug Copy Issue Report Accessing `token` on the interface `Laravel\Socialite\Contracts\User` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
85	6	$socialUser->expiresIn ?? null
		0 ignored issues – show Bug introduced 2017-08-17 12:03 UTC by Report Bug Copy Issue Report Accessing `expiresIn` on the interface `Laravel\Socialite\Contracts\User` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
86		);
87
88	6	event(new SocialUserCreated($NewUser));
89
90	6	return $NewUser;
91		}
92		}
93

mad-web / laravel-social-auth

SocialProviderManager::getUserByKey() A last analyzed 2017-10-29 22:08 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like

SocialProviderManager::getUserByKey() A
last analyzed 2017-10-29 22:08 UTC