laravel UserRequest $request error

Question

laravel5.2,I create a UserRequest.php under Requests directory,but in controller,public function add(UserRequest $request) show error,but use public function add(Request $request) is normal.

UserRequest

namespace AppHttpRequests;

use AppHttpRequestsRequest;

class UserRequest extends Request
{
    /** 
     * Determine if the user is authorized to make this request. 
     * 
     * @return bool 
     */
    public function authorize()
    {
        return true;
    }

    /**
     * Get the validation rules that apply to the request.
     *
     * @return array
     */
    public function rules()
    {
        return [
            'user_sn'   => 'required|unique',
            'user_name' => 'required',
            'email'     => 'required|unique',
            'password'  => 'required',
        ];
    }
}

UserController

namespace AppHttpControllers;

use AppHttpRequestsUserRequest;
use IlluminateSupportFacadesDB;
use IlluminateSupportFacadesValidator;

class UserController extends Controller
{

    public function add(UserRequest $request)
    {
        if ($request->get('dosubmit')) {
            $validator = Validator::make($request->all(), $request
                ->rules(), $request->messages());
            if ($validator->fails()) {
                return redirect('user/add')->withErrors($validator)
                    ->withInput();
            }
        }
        $corporation_list = DB::table('corporation')->get();
        $department_list = DB::table('department')->get();

        return view('user.add', ['corporation_list' => $corporation_list, 'department_list' => $department_list]);
    }
}

Route

Route::group(['middleware'],function (){
Route::any('user/add',['as'=>'user.add','uses'=>'UserController@add']);

});

Answer 1

There are usually 2 reasons you could be having this issue.

1. You've not added the use statement for the UserRequest.

   At the top of your controller (above the class) add:

   use AppHttpRequestsUserRequest
   

   assuming that is the correct namespace.

2. You may need to run composer dump-autoload to make sure the class has been added to the autoloader.

---

Edit

Firstly, replace the add() method with the following methods:

public function create()
{
    $corporation_list = DB::table('corporation')->get();
    $department_list = DB::table('department')->get();

    return view('user.add', compact('corporation_list', 'department_list'));
}

public function store(UserRequest $request)
{
    // If you get to this point the validation will have passed

    // Process the request
}

Then change your routes from:

Route::any('user/add',['as'=>'user.add','uses'=>'UserControl‌​ler@add']) 

to:

Route::get('user/add', ['as' => 'user.add', 'uses' => 'UserControl‌​ler@create']);
Route::post('user/add', ['as' => 'user.store', 'uses' => 'UserControl‌​ler@store']);

Obviously, feel free to change the as in the Routes to whatever, they should unique though.

Lastly, I would suggest looking at Resource Controllers which is a RESTful approach.

Hope this helps!

Answer 2

The problem is that you have not identified UserController that you are using UserRequest file

use AppHttpRequestsUserRequest

It will solve the problem