Gameap\Http\Controllers\Admin\UsersController

Complexity

Total Complexity

9

Size/Duplication

Total Lines	111
Duplicated Lines	0 %

Test Coverage

Coverage

72.41%

Importance

Changes	2
Bugs	0	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	store()	0	6	1
A	show()	0	3	1
A	__construct()	0	4	1
A	update()	0	8	2
A	edit()	0	4	1
A	index()	0	4	1
A	destroy()	0	5	1
A	create()	0	5	1

<?php

namespace Gameap\Http\Controllers\Admin;

use Bouncer;
use Gameap\Http\Controllers\AuthController;
use Gameap\Models\User;
use Gameap\Repositories\UserRepository;
use Gameap\Http\Requests\Admin\UserCreateRequest;
use Gameap\Http\Requests\Admin\UserUpdateRequest;

class UsersController extends AuthController
{
    /**
     * The UserRepository instance.
     *
     * @var \Gameap\Repositories\UserRepository
     */
    protected $repository;

    /**
     * Create a new UserRepository instance.
     *
     * @param  \Gameap\Repositories\UserRepository $repository
     */
    public function __construct(UserRepository $repository)
    {
        parent::__construct();
        $this->repository = $repository;
    }

    /**
     * Display a listing of the users.
     *
     * @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
     */
    public function index()
    {
        return view('admin.users.list',[
            'users' => $this->repository->getAll()
        ]);
    }

    /**
     * Display new create user page
     *
     * @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
     */
    public function create()
    {
        $roles = Bouncer::role()->all();
        
        return view('admin.users.create', compact('roles'));
    }

    /**
     * Store a newly created user in storage.
     *
     * @param UserCreateRequest $request
     * @return \Illuminate\Http\RedirectResponse
     */
    public function store(UserCreateRequest $request)
    {
        $this->repository->store($request->all());

        return redirect()->route('admin.users.index')
            ->with('success', __('users.create_success_msg'));
    }

    /**
     * Display the specified resource.
     *
     * @param User $user
     * @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
     */
    public function show(User $user)
    {
        return view('admin.users.view', compact('user'));
    }

    /**
     * Show the form for editing the specified resource.
     *
     * @param User $user
     * @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
     */
    public function edit(User $user)
    {
        $roles = Bouncer::role()->all();
        return view('admin.users.edit', compact('user', 'roles'));
    }

    /**
     * Update the specified resource in storage.
     *
     * @param  \Gameap\Http\Requests\Admin\UserUpdateRequest  $request
     * @param  \Gameap\Models\User  $user
     * @return \Illuminate\Http\RedirectResponse
     */
    public function update(UserUpdateRequest $request, User $user)
    {
        if ($this->repository->update($user, $request->all())) {
            return redirect()->route('admin.users.index')
                ->with('success', __('users.update_success_msg'));
        } else {
            return redirect()->route('admin.users.index')
                ->with($user->getValidationErrors()->all());

It seems like $user->getValidationErrors()->all() can also be of type Illuminate\Database\Eloquent\Builder; however, parameter $key of Illuminate\Http\RedirectResponse::with() does only seem to accept array|string, maybe add an additional type check?

        }
    }

    /**
     * Remove the specified resource from storage.
     *
     * @param \Gameap\Models\User $user
     * @return \Illuminate\Http\RedirectResponse
     * @throws \Exception
     */
    public function destroy(User $user)
    {
        $user->delete();
        return redirect()->route('admin.users.index')
            ->with('success', __('users.delete_success_msg'));
    }
}