search

AlertRED / php-tasks

Работа по стажировке в компании 2UP на Laravel
0 stars 0 forks source link

Задание 2 #2

Closed AlertRED closed 5 years ago

mx2s commented 5 years ago

в миграции не добавляется foreign key, я правда пробовал на mysql у тебя все работает если создать базу заново и накатить миграции?

mx2s commented 5 years ago

пиши phpdoc'и https://ru.wikipedia.org/wiki/PHPDoc

mx2s commented 5 years ago

https://github.com/AlertRED/phpTasks/blob/fda7f60dfe723439912388c6ad73bec2f89a7270/app/Http/Controllers/v0/UserProfilesController.php#L32 тут нет переменной $id - из за этого ловлю exception, если заменить на $userId то работает

mx2s commented 5 years ago

если ты возвращаешь несколько профилей и их не нашел - abort делать не надо - просто возвращай пустой массив

AlertRED commented 5 years ago

в миграции не добавляется foreign key, я правда пробовал на mysql у тебя все работает если создать базу заново и накатить миграции?

Да. php artisan migrate:refresh работал.

AlertRED commented 5 years ago

@mxss Все исправил.