Tymon\JWTAuth\Support\auth\RegistersUsers

Complexity

Total Complexity

4

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes	5
Bugs	2	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	customClaims()	0	4	2
A	postRegister()	0	16	2

<?php
/**
 * Created by PhpStorm.
 * User: Claudio Cardinale <cardi@thecsea.it>
 * Date: 18/11/15
 * Time: 16.23
 * This program is free software; you can redistribute it and/or
 * modify it under the terms of the GNU General Public License
 * as published by the Free Software Foundation; either version 2
 * of the License, or (at your option) any later version.
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
 * You should have received a copy of the GNU General Public License
 * along with this program; if not, write to the Free Software
 * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
 */

namespace Tymon\JWTAuth\Support\auth;

use Illuminate\Http\JsonResponse;
use Illuminate\Http\Request;
use Tymon\JWTAuth\Facades\JWTAuth;

/**
 * Class RegistersUsers
 * @package Tymon\JWTAuth\Support\auth
 * @author Claudio Cardinale <cardi@thecsea.it>
 * @copyright 2015 Claudio Cardinale
 * @version 1.0.0
 */
trait RegistersUsers
{
    /**
     * Handle a registration request for the application.
     *
     * @param  \Illuminate\Http\Request  $request
     * @return \Illuminate\Http\Response
     */
    public function postRegister(Request $request)
    {
        $validator = $this->validator($request->all());

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

        if ($validator->fails()) {
            $this->throwValidationException(

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

                $request, $validator
            );
        }

        $token = JWTAuth::fromUser($this->create($request->all()), $this->customClaims());

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

        $response = new JsonResponse(['token' => $token], 200);
        $response->header('Authorization', 'Bearer ' . $token);
        return $response;
    }

    /**
     * Get the custom claims
     *
     * @return string
     */
    public function customClaims()
    {
        return property_exists($this, 'custom') ? $this->custom : [];

The property custom does not exist. Did you maybe forget to declare it?

    }

}