Mckenziearts / laravel-oauth

src/Traits/OAuthSocialite.php

<?php

namespace Mckenziearts\LaravelOAuth\Traits;

use Carbon\Carbon;
use Illuminate\Support\Facades\DB;
use Illuminate\Support\Facades\Auth;
use Laravel\Socialite\Facades\Socialite;

trait OAuthSocialite
{
    /**
     * Redirect the user to the Provider authentication page.
     *
     * @param $provider
     * @return \Symfony\Component\HttpFoundation\RedirectResponse
     */
    public function redirectToProvider($provider)
    {
        $provider = strtolower($provider);

        if ($provider === 'facebook') {
            return Socialite::driver('facebook')->with(['auth_type' => 'rerequest'])->redirect();

The method with() 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.

        }

        return Socialite::driver($provider)->redirect();
    }

    /**
     * Obtain the user information from provider.
     *
     * @param $provider
     * @return \Illuminate\Http\Response
     */
    public function handleProviderCallback($provider)
    {
        $providerUser = Socialite::driver($provider)->user();

        // Check if user email is null
        // Get user from the database with the good provider_id or email
        if (is_null($providerUser->getEmail())) {

The condition is_null($providerUser->getEmail()) is always false.

            $this->redirectToProvider($provider);
        }

        $user = DB::table(config('laravel-oauth.users.table'))

The function config was not found. Maybe you did not declare it correctly or list all dependencies?

            ->where($provider.'_id', '=', $providerUser->getId())
            ->orWhere('email', '=', $providerUser->getEmail())
            ->first();

        if (is_null($user)) {
            // Save user to the database
            $userId = $this->registerUser($provider, $providerUser);
            Auth::loginUsingId($userId);

            return redirect()->intended($this->redirectPath());

It seems like redirectPath() must be provided by classes using this trait. How about adding it as abstract method to this trait?

The function redirect was not found. Maybe you did not declare it correctly or list all dependencies?

        }

        // Login user
        Auth::loginUsingId($user->id);

        return redirect()->intended($this->redirectPath());
    }

    /**
     * Resgiter user to the database and return ID.
     *
     * @param $provider
     * @param $user
     * @return int
     */
    public function registerUser($provider, $user)
    {
        $userId = DB::table(config('laravel-oauth.users.table'))->insertGetId([

The function config was not found. Maybe you did not declare it correctly or list all dependencies?

            'name'      => $user->getName(),
            'email'     => $user->getEmail(),
            'password'  => bcrypt('password'),

The function bcrypt was not found. Maybe you did not declare it correctly or list all dependencies?

            $provider.'_id'     => $user->getId(),
            'email_verified_at' => Carbon::now(),
            'created_at'    => Carbon::now(),
            'updated_at'    => Carbon::now(),
        ]);

        return $userId;
    }
}