Что не так с валидацией Laravel?

Question

[Xveeder]

Есть контроллер:

<?php

namespace App\Http\Controllers\CAuth;

use  App\Http\Classes\UserRegister;
use Illuminate\Http\Request;
use Illuminate\Foundation\Bus\DispatchesJobs;
use Illuminate\Contracts\Validation\Validator;
use Illuminate\Routing\Controller as BaseController;
use Illuminate\Foundation\Validation\ValidatesRequests;

class CustomAuthController {
    public function validate (Request $request) {
        $rules = [
            'username' => 'required|string|max:255',
            'email' => 'required|string|email|max:255',
            'password' => 'required|string|min:6',
            'invite' => 'required|integer|min:6',
        ];
            
       $this->validate($request, $rules);       
     
    }
}

Данные передаются из формы через POST-запрос, в роутах также прописан POST-запрос. Но после отработки этого контроллера браузер вылетает:

Answer 1

[Arman]

Это на продакшн? логи молчат? если дев то включайте показ ошибок.
так может что угодно быть, какие middleware не пропускают.. навскидку может в csrf полях?

Comments

[Xveeder]

Контроль ошибок включен. CSRF поля передаются. Вот что приходит в $_POST:

Array ( [email] => [e-mail] [username] => asdfghjj [password] => sdfghjk [invite] => 111111111 [_token] => uaWfaWG6vtrkJPFJF1tbFslnEVzssO3Gr1UQMJQat27 )

[iljaGolubev]

Как-то стрёмно выглядит:

class CustomAuthController {
    public function validate (Request $request) {          
       $this->validate($request, $rules);       
    }
}

[Xveeder]

iljaGolubev, что вы имеете в виду?

[iljaGolubev]

рекурсию?
$this->validate вызывает public function validate.
И вообще, переходите сразу на использование FormRequest вместо валидации в контроллере. (всё в доке ларавел по валидации).

Answer 2

[Константин Б.]

А чего ты ждал когда называл метод validate?
Обрати внимание, у тебя метод контроллера называется также как метод валидации в контроллере, который ты кстати вдобавок ко всему вызываешь из своего метода
Замени
public function validate
на другое название, должно быть либо store либо update
Ах да, и еще ты не наследуешь свой контроллер от базового, в этом тоже беда

Comments

[Xveeder]

Переписал в соответствии с вашими рекомендациями:

<?php

namespace App\Http\Controllers\CAuth;

use  App\Http\Classes\UserRegister;
use Illuminate\Http\Request;
use Illuminate\Foundation\Bus\DispatchesJobs;
use Illuminate\Contracts\Validation\Validator;
use Illuminate\Foundation\Validation\ValidatesRequests;

class CustomAuthController {
    
    public function clean (Request $request) {
        
        $rules = [
            'username' => 'required|string|max:255',
            'email' => 'required|string|email|max:255',
            'password' => 'required|string|min:6',
            'invite' => 'required|integer|min:6',
        ];
            
       $this->validate($request, $rules);  
        
        
        
    }  
}

В итоге, следующая ошибка:

[Андрей Новиков]

Xveeder, теперь роут на новый метод направь

[Константин Б.]

Xveeder, о боже.... иди собирай портфель тебе завтра в школу... что за тупые вопросы, ну серьезно без обид. подучи php, потренируй внимательность, подтяни английский. Что тут не понятного-то, приложение тебе говорит, что ты отправляешь данные например методом GET, а маршрутизации у тебя указан POST

[Xveeder]

Kostik_1993, спасибо за совет. Но проблема в том, что данные я отправляю методом post и в маршруте у меня метод post.

[Xveeder]

Андрей Новиков, в том и проблема, что направил.

[Константин Б.]

Xveeder, тогда весь код в студию, у меня дома кофе закончился, по гуще не смогу погодать

[Константин Б.]

Xveeder, и кстати ты так и не унаследовал контроллер от базового, а значит метода validate() у тебя быть не может

Answer 3

[iljaGolubev]

Должно быть

class CustomAuthController {
    public function validate (Request $request) {
        $rules = [
            'username' => 'required|string|max:255',
            'email' => 'required|string|email|max:255',
            'password' => 'required|string|min:6',
            'invite' => 'required|integer|min:6',
        ];

       // Внимание
       $request->validate($rules);            
    }
}