чуть получше,перед return nl добавь, так нужно
+ подумай над тем, что бы вынести это отдельным методом в модель
не встречал nl
nl return view(
Sonder
SarcasmIO
nl это new line
SarcasmIO
return'ы выделяют явно таким образом
Anton
https://paste.laravel.io/b285d8f2-1e67-4c18-9d07-dd5e78b6d408
для чего тебе отдельный контроллер для Add?
SarcasmIO
https://paste.laravel.io/b285d8f2-1e67-4c18-9d07-dd5e78b6d408
замени валидацию на формреквест
Anton
сделай просто PartnerController
Volodymyr
Шедуль артисана только посредник между твоим кодом и кроном
Спасибо! А втаком случае в крон табе 0ли прописывать ?
SarcasmIO
а инстанс формреквеста, уже провалидированный
SarcasmIO
https://laravel.com/docs/5.6/validation#creating-form-requests
Daniil
Спасибо! А втаком случае в крон табе 0ли прописывать ?
Когда хочешь например 0 часов или 0 минут
Daniil
5 звездочек - каждую минуту
Volodymyr
зачем тогда это шедуль
Daniil
Смотри, ты указываешь один раз пять звездочек в кроне а дальше кодом решаешь когда и как запускать таски
Daniil
Чтоб не делать тысячу крон задач
Volodymyr
Смотри, ты указываешь один раз пять звездочек в кроне а дальше кодом решаешь когда и как запускать таски
В таком случае крон будет каждую минуту смотреьть в задачу а лара уже на своих счетчиках решает внутри когда и как что запускать
Daniil
Причем у лары есть еще механизм препятствующий пересечению долгих тасков
Daniil
Удобно короче
Anton
@huz0634 для чего тебе целый контроллер для метода Add? Еще один контроллер на Update?
Volodymyr
Удобно короче
This Cron will call the Laravel command scheduler every minute. Then, Laravel evaluates your scheduled jobs and runs the jobs that are due. It couldn't be easier!
Опять не дочитал))
Sonder
@huz0634 для чего тебе целый контроллер для метода Add? Еще один контроллер на Update?
я по урокам смотрел, через консоль создало
Sonder
показали бы пример идеально оптимизированного контроллера)
Anton
https://paste.laravel.io/7f919531-32ee-4d8d-86d8-545da2a60346 вот попробуй что то подобное. Валидацию в отдельный класс.
Anton
метод post уже в роутинге, надо ли проверять
Sonder
но там нету переменн.. $input
Sonder
SarcasmIO
https://paste.laravel.io/7f919531-32ee-4d8d-86d8-545da2a60346 вот попробуй что то подобное. Валидацию в отдельный класс.
я бы не смешивал
breadcrumbs и данные все таки
Anton
ну возможно
Anton
как то так
Anton
чота уже подзабываю ларавел. на гребанном yii2 щас. такое гавнище
Romuald
а че это
PartnersAddRequest
Romuald
ептить
Anton
показали бы пример идеально оптимизированного контроллера)
https://paste.laravel.io/1f006c54-ebfd-432b-9a94-cd2a79cd5514
Anton
ну как то так
Anton
может что то упустил, не проверял. на глаз
Bakhriddin
https://paste.laravel.io/1f006c54-ebfd-432b-9a94-cd2a79cd5514
private $this->partnerModel по-моему это не правильно )
SarcasmIO
все таки можно доебаться,если захотеть)
аля unused local variables,фасад,return value небулево)
но в целом норм
Bakhriddin
private $partnerModel
SarcasmIO
я так понимаю у тебя весь проект:
команда/запрос?
Anton
да
Anton
https://paste.laravel.io/c1afd1b1-8239-482a-a50f-fe74e8b4a7f7
Sonder
у вас очень классный код выходит)
Anton
туплю уже чота
SarcasmIO
как поживается с CQS/CQRS ?
Anton
да по сути ни чем.
Sonder
https://paste.laravel.io/c1afd1b1-8239-482a-a50f-fe74e8b4a7f7
Route::match(['get','post'], '/add', ['uses'=>'PartnersController','as'=>'partnersAdd']);
тут вроде ошибаюсь я
Anton
это у тебя CQRS типа?
Anton
Route::post('/partners', 'PartnersController@add');
Route::get('/partners/{id}', 'PartnersController@edit');
чота как то так
Anton
все засыпаю.
Anton
помоему фасады можно мокать уже давно.
Anton
мне больше DependencyInjection по душе. в ларке контейнер хорош
Anton
я не юзал фасады )
Anton
для чего тебе префиск partners?
SarcasmIO
Anton
выкинь его
Anton
ему рано еще
Anton
до recsource
Anton
он еще в роутинге не разбирается
Anton
php artisan route:list
Anton
Anton
/partners/partners ?
Romuald
да отправьте доки читать
Romuald
или тут фаст курс
Anton
пойду спать. дальше сам
Sonder
и вот я читаю, делаю
А потом оказывается, что так плохо))
Romuald
плохо будет всегда, надо смириться
Anton
ты не читаешь доку, ты читаешь только примеры кода, пропуская описание
Sonder
и то правда