plunner\Http\Controllers\Employees\EmployeesController

Complexity

Total Complexity

3

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

0%

Importance

Changes	3
Bugs	1	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	index()	0	5	1
A	show()	0	6	1

<?php

namespace plunner\Http\Controllers\Employees;

use plunner\Http\Controllers\Controller;
use plunner\Http\Requests;
use plunner\Employee;

class EmployeesController extends Controller
{
    /**
     * @var plunner/Employee
     */
    private $user;

The property $user is not used and could be removed.

    /**
     * ExampleController constructor.
     */
    public function __construct()
    {
        config(['auth.model' => \plunner\Employee::class]);
        config(['jwt.user' => \plunner\Employee::class]);
        $this->middleware('jwt.authandrefresh:mode-en');
    }

    /**
     * Display a listing of the resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function index()
    {
        $employee = \Auth::user();
        return $employee;
    }

    //TODO this is not RESTFUL, but it can be ok

    /**
     * Display the specified resource.
     *
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function show($id)
    {
        $employee = Employee::findOrFail($id);
        $this->authorize($employee);
        return $employee;
    }

    //TODO why this? when we need this?
}