plunner\Http\Controllers\Companies\Employees\EmployeesController

Complexity

Total Complexity

6

Size/Duplication

Total Lines	94
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	index()	0	9	1
A	store()	0	8	1
A	show()	0	7	1
A	update()	0	9	1
A	destroy()	0	8	1

<?php

namespace plunner\Http\Controllers\Companies\Employees;

use Illuminate\Http\Request;
use plunner\Company;
use plunner\Employee;
use plunner\Http\Controllers\Controller;
use plunner\Http\Requests\Companies\EmployeeRequest;


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

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

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


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

    /**
     * Store a newly created resource in storage.
     *
     * @param  EmployeeRequest  $request
     * @return \Illuminate\Http\Response
     */
    public function store(EmployeeRequest $request)
    {
        //
        $company = \Auth::user();
        $input = $request->all();
        $employee = $company->employees()->create($input);
        return $employee;
    }

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

The method findOrFail does not exist on object<plunner\Employee>? Since you implemented __callStatic, maybe consider adding a @method annotation.

        $this->authorize($employee);
        return $employee;
    }

    /**
     * Update the specified resource in storage.
     *
     * @param  EmployeeRequest  $request
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function update(EmployeeRequest $request, $id)
    {
        //
        $employee = Employee::findOrFail($id);

The method findOrFail does not exist on object<plunner\Employee>? Since you implemented __callStatic, maybe consider adding a @method annotation.

        $this->authorize($employee);
        $input = $request->all();
        $employee->update($input);
        return $employee;
    }

    /**
     * Remove the specified resource from storage.
     *
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function destroy($id)
    {
        //
        $employee = Employee::findOrFail($id);

The method findOrFail does not exist on object<plunner\Employee>? Since you implemented __callStatic, maybe consider adding a @method annotation.

        $this->authorize($employee);
        $employee->delete();
        return $employee;
    }
}