DoeSangue\Http\Controllers\Auth\AuthenticateController

Complexity

Total Complexity

6

Size/Duplication

Total Lines	100
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	7

Test Coverage

Coverage

25%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	authenticate()	0	28	3
A	register()	0	32	1
A	logout()	0	16	2

<?php

namespace DoeSangue\Http\Controllers\Auth;

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

class AuthenticateController extends Controller
{
      /**
       * Authenticate the user
       *
       * @param  Request $request
       * @return \Illuminate\Http\JsonResponse
       */
    public function authenticate(Request $request)
    {

        try {

            // grab credentials from the request
            // attempt to verify the credentials and create a token for the user
            if (!$token = JWTAuth::attempt(
                $request->only('email', 'password'), [
                'exp' => Carbon::now()->addWeek()->timestamp,
                ]
            )
            ) {
                return response()->json([ 'error' => 'invalid_credentials' ], 401);

The method json does only exist in Illuminate\Contracts\Routing\ResponseFactory, but not in Illuminate\Http\Response.

            }
        } 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
     *
     * @param  RegisterUserRequest $request
     * @return \Illuminate\Http\JsonResponse
     */
    public function register(RegisterUserRequest $request)
    {
        $user = User::create(
            [
            '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.

            '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.

            'country_code' => $request->country_code,

The property country_code 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.

            'blood_type_id' => $request->blood_type_id,

The property blood_type_id 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.

            ]
        );

        // 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(

The method json does only exist in Illuminate\Contracts\Routing\ResponseFactory, but not in Illuminate\Http\Response.

            [
            'access_token' => $token,
            'token_type' => 'Bearer'
            ], 201
        );
    }

    /**
     * Invalidate and log out the user
     *
     * @param Request $request
     * @return \Illuminate\Http\JsonResponse
     */
    public function logout(Request $request)
    {
      $this->validate($request, [ 'token' => 'required' ]);

      try {
        JWTAuth::invalidate($request->input('token'));
        return response()->json(

The method json does only exist in Illuminate\Contracts\Routing\ResponseFactory, but not in Illuminate\Http\Response.

          [
            'success' => true
          ]
        );
      } catch (JWTException $e) {
        // Something went wrong whilst attemping to encode the token
        return response()->json([ 'success' => false, 'error' => 'Failed to logout, please try again.', 500 ]);
      }
    }

}