Acacha\Events\Http\Controllers\APIUsersController

Complexity

Total Complexity

5

Size/Duplication

Total Lines	68
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	index()	0	4	1
A	show()	0	4	1
A	store()	0	8	1
A	update()	0	6	1
A	destroy()	0	5	1

<?php

namespace Acacha\Events\Http\Controllers;

use Acacha\Events\Http\Requests\DestroyUser;
use Acacha\Events\Http\Requests\UpdateUser;
use Acacha\Events\Http\Requests\ListUsers;
use Acacha\Events\Http\Requests\ShowUser;
use Acacha\Events\Http\Requests\StoreUser;
use App\User;

/**
 * Class APIUsersController.
 *
 * @package App\Http\Controllers
 */
class APIUsersController extends Controller
{
    /**
     * Show list of users.
     *
     * @param ListUsers $request
     * @return \Illuminate\Database\Eloquent\Collection|static[]
     */
    public function index(ListUsers $request)

The parameter $request is not used and could be removed.

    {
        return User::all();
    }

    /**
     * Show and user
     *
     * @param ShowUser $request
     * @param User $user
     * @return User
     */
    public function show(ShowUser $request, User $user)

The parameter $request is not used and could be removed.

    {
        return $user;
    }

    /**
     * Store and user.
     *
     * @param StoreUser $request
     * @return mixed
     */
    public function store(StoreUser $request)
    {
        return User::create([
            'name' => $request->name,

The property name does not exist on object<Acacha\Events\Http\Requests\StoreUser>. Since you implemented __get, maybe consider adding a @property annotation.

            'email' => $request->email,

The property email does not exist on object<Acacha\Events\Http\Requests\StoreUser>. Since you implemented __get, maybe consider adding a @property annotation.

            'password' => bcrypt($request->password)

The property password does not exist on object<Acacha\Events\Http\Requests\StoreUser>. Since you implemented __get, maybe consider adding a @property annotation.

        ]);
    }

    /**
     * Edit and user.
     *
     * @param UpdateUser $request
     * @param $user
     * @return mixed
     */
    public function update(UpdateUser $request, User $user)
    {
        $user->update($request->only('name',''));
        $user->save();
        return $user;
    }

    /**
     * Delete user.
     *
     * @param DestroyUser $request
     * @param $user
     * @return mixed
     */
    public function destroy(DestroyUser $request, User $user)

The parameter $request is not used and could be removed.

    {
        $user->delete();
        return $user;
    }

}