App\Http\Requests\Traits\AsksForUsers

Complexity

Total Complexity

1

Size/Duplication

Total Lines	19
Duplicated Lines	100 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	askForUsers()	12	12	1

<?php

namespace App\Http\Requests\Traits;


use App\User;

/**
 * Trait ChecksPermissions.
 */
trait AsksForUsers

This class seems to be duplicated in your project.

{
    /**
     * @param $user
     *
     * @return mixed
     */
    protected function askForUsers()
    {
        $users = User::all();

        //pluck és com si es fes un foreach buscant per nom i afegir-ho a un array.
        //el que fa es agafar el valor a partir d'una clau.
        // ho passem a array i ho mostrem.
        // + info README
        $user_names = $users->pluck('name')->toArray();
        $user_name = $this->choice('User id?', $user_names);

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

        return $users->where('name', $user_name)->first()->id;
    }
}