search

AlertRED / php-tasks

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

Задание 6 #6

Open AlertRED opened 5 years ago

mx2s commented 5 years ago

https://github.com/AlertRED/phpTasks/blob/ea913c20ee635659ba424f01aee81b5d2eae0240/database/migrations/2019_03_05_210414_create_github_issues_table.php#L23 с этим осторожней - пробелов в названиях колонок быть не должно, часами потом будешь искать ошибку btw в postgres такая колонка добавится, а mysql выдаст ошибку

mx2s commented 5 years ago

image не критично - мне кажется у тебя тут опечатка - должно быть название профиля вместо 'название группы'

mx2s commented 5 years ago

при добавлении профиля с user id которого не существует - вылетает exception

mx2s commented 5 years ago

exception при редактировании существующего профиля:
Symfony \ Component \ HttpKernel \ Exception \ MethodNotAllowedHttpException No message

mx2s commented 5 years ago

https://github.com/AlertRED/phpTasks/blob/ea913c20ee635659ba424f01aee81b5d2eae0240/app/Http/Controllers/WebController/UserController.php#L32 в контроллере нельзя вызывать функции других контроллеров

mx2s commented 5 years ago

image советую в sublime поставить плагины для intellisense и проверки синтаксиса если есть, либо не писать в sublime =) vscode / vs studio / phpstorm как вариант

mx2s commented 5 years ago

image опять code style - такого быть не должно