App\Http\Controllers\Auth\OAuthController

Complexity

Total Complexity

7

Size/Duplication

Total Lines	75
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	handleProviderCallback()	0	25	3
A	sendFailedLoginResponse()	0	4	1
A	redirectToProvider()	0	3	1
A	findOrCreateUser()	0	16	2

<?php

namespace App\Http\Controllers\Auth;

use App\User;
use Illuminate\Http\Request;
use App\Http\Controllers\Controller;
use Illuminate\Support\Facades\Auth;
use Illuminate\Support\Facades\Hash;
use Illuminate\Validation\ValidationException;
use Laravel\Socialite\Facades\Socialite;

class OAuthController extends Controller
{

    public function redirectToProvider($provider)
    {
        return Socialite::driver($provider)->redirect();
    }


    /**
     * Obtain the user information from provider.  Check if the user already exists in our
     * database by looking up their provider_id in the database.
     * If the user exists, log them in. Otherwise, create a new user then log them in. After that
     * redirect them to the authenticated users homepage.
     *
     * @return Response

The type App\Http\Controllers\Auth\Response was not found. Did you mean Response? If so, make sure to prefix the type with \.

     */
    public function handleProviderCallback($provider)
    {
        $user = null;
        $err = null;

The assignment to $err is dead and can be removed.

        if($provider == 'google') {
            $user = Socialite::driver($provider)->stateless()->user();

The method stateless() does not exist on Laravel\Socialite\Contracts\Provider. It seems like you code against a sub-type of Laravel\Socialite\Contracts\Provider such as Laravel\Socialite\Two\AbstractProvider.

        } else {
            $user = Socialite::driver($provider)->user();
        }

        $authUser = $this->findOrCreateUser($user, $provider);

        $attempt = Auth::guard('web')->attempt(['email' => $authUser->email, 'password' => $authUser->provider_id]);

        $req = new Request([

The assignment to $req is dead and can be removed.

            'email' => $authUser->email,
        ]);

        if($attempt) {
            $msg = "Selamat Datang ".$authUser->name." !";
            return redirect()->intended(route('home'))->with('info', $msg);

The expression return redirect()->intended(route('home'))->with('info', $msg) returns the type Illuminate\Http\RedirectResponse which is incompatible with the documented return type App\Http\Controllers\Auth\Response.

        }
//        $this->sendFailedLoginResponse($req);
        return redirect(route('login'));

The expression return redirect(route('login')) returns the type Illuminate\Http\RedirectResponse which is incompatible with the documented return type App\Http\Controllers\Auth\Response.

    }


    /**
     * If a user has registered before using social auth, return the user
     * else, create a new user object.
     * @param  $user Socialite user object
     * @param $provider Social auth provider

The type App\Http\Controllers\Auth\Social was not found. Maybe you did not declare it correctly or list all dependencies?

     * @return  User
     */
    private function findOrCreateUser($user, $provider)
    {
        $authUser = User::where('email', $user->getEmail())->first();
        if($authUser) {
            return $authUser;
        }

        $newUser = new User;
        $newUser->email = $user->getEmail();
        $newUser->name = $user->getName();
        $newUser->password = Hash::make($user->id);
        $newUser->provider = $provider;
        $newUser->provider_id = $user->id;
        $newUser->save();

        return $newUser;
    }


    private function sendFailedLoginResponse(Request $request)

The method sendFailedLoginResponse() is not used, and could be removed.

The parameter $request is not used and could be removed.

    {
        throw ValidationException::withMessages([
            'email' => [trans('auth.failed')],
        ]);

    }

}