DoeSangue\Http\Controllers\Auth\AuthenticateController

Complexity

Total Complexity

8

Size/Duplication

Total Lines	109
Duplicated Lines	16.51 %

Coupling/Cohesion

Components	0
Dependencies	6

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
B	register()	0	41	2
A	logout()	0	4	1
A	userInfo()	18	18	2
A	authenticate()	0	23	3

<?php

namespace DoeSangue\Http\Controllers\Auth;

use JWTAuth;
use Tymon\JWTAuth\Exceptions\JWTException;
use DoeSangue\Http\Controllers\Controller;
use Illuminate\Http\Request;
use DoeSangue\Http\Requests\RegisterUserRequest;
use DoeSangue\Mail\UserCreated;
use Illuminate\Support\Facades\Mail;
use DoeSangue\Models\User;
use DoeSangue\Models\Donor;

class AuthenticateController extends Controller
{
      /**
       * Authenticate the user
       *
       * @param Request $request
       * @return void
      */
    public function authenticate(Request $request)
    {
        // grab credentials from the request
        $credentials = $request->only('email', 'password');

        try {
            // attempt to verify the credentials and create a token for the user
            if (!$token = JWTAuth::attempt($credentials)) {
                return response()->json([ 'error' => 'invalid_credentials' ], 401);
            }
        } catch (JWTException $e) {
            // something went wrong whilst attempting to encode the token
            return response()->json([ 'error' => 'could_not_create_token' ], 500);
        }

        // all good so return the token
        return response()->json(
            [
              'access_token' => $token,
              'token_type' => 'Bearer'
            ], 200
        );
    }

    // Register a new user
    public function register(RegisterUserRequest $request)
    {
        $user = User::create(

The method create() does not exist on DoeSangue\Models\User. Did you maybe mean created()?

            [
            'first_name' => $request->first_name,

The property first_name does not exist on object<DoeSangue\Http\Requests\RegisterUserRequest>. Since you implemented __get, maybe consider adding a @property annotation.

            'last_name' => $request->last_name,

The property last_name does not exist on object<DoeSangue\Http\Requests\RegisterUserRequest>. Since you implemented __get, maybe consider adding a @property annotation.

            'email' => $request->email,

The property email does not exist on object<DoeSangue\Http\Requests\RegisterUserRequest>. Since you implemented __get, maybe consider adding a @property annotation.

            // create the username based on first_name and last_name
            // if not provided
            /*$userName = trim(strtolower($request->first_name.'-'.$request->last_name)),

56% of this comment could be valid code. Did you maybe forget this after debugging?

            'username' => $userName,*/
            'username' => $request->username,

The property username does not exist on object<DoeSangue\Http\Requests\RegisterUserRequest>. Since you implemented __get, maybe consider adding a @property annotation.

            'phone' => $request->phone,

The property phone does not exist on object<DoeSangue\Http\Requests\RegisterUserRequest>. Since you implemented __get, maybe consider adding a @property annotation.

            'bio' => $request->bio,

The property bio does not exist on object<DoeSangue\Http\Requests\RegisterUserRequest>. Since you implemented __get, maybe consider adding a @property annotation.

            'birthdate' => $request->birthdate,

The property birthdate does not exist on object<DoeSangue\Http\Requests\RegisterUserRequest>. Since you implemented __get, maybe consider adding a @property annotation.

            'password' => bcrypt($request->password),

The property password does not exist on object<DoeSangue\Http\Requests\RegisterUserRequest>. Since you implemented __get, maybe consider adding a @property annotation.

            ]
        );

        if ($user) {
            $donor = new Donor();
            $donor->user_id = $user->id;

The property user_id does not exist on object<DoeSangue\Models\Donor>. Since you implemented __set, maybe consider adding a @property annotation.

            $donor->blood_type_id = null;

The property blood_type_id does not exist on object<DoeSangue\Models\Donor>. Since you implemented __set, maybe consider adding a @property annotation.

            $donor->save();
        }

        // Send mail to user
        Mail::to($user->email)->send(new UserCreated($user));



        $token = JWTAuth::attempt($request->only('email', 'password'));

        // all good so return the token
        return response()->json(
            [
            'access_token' => $token,
            'token_type' => 'Bearer'
            ], 201
        );
    }

    /**
     * Invalidate and log out the user
     *
     * @return void
     *
     */
    public function logout()
    {
        //
    }
    /**
     * Get the current user (logged in) information
     *
     * @return void
     */
    public function userInfo()

This method seems to be duplicated in your project.

    {
        $user = JWTAuth::parseToken()->authenticate();

        // If the token is invalid
        if (!$user) {
            return response()->json([ 'invalid_user' ], 401);
        }

        return response()->json(
            [
              'first_name' => $user->first_name,
              'last_name' => $user->last_name,
            //   'email' => $user->email,

56% of this comment could be valid code. Did you maybe forget this after debugging?

              'username' => $user->username
            ], 200
        );
    }
}