search

AlertRED / php-tasks

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

Задание 4 #4

Open AlertRED opened 5 years ago

mx2s commented 5 years ago

image 1) в контроллере не должно быть функций, которые не связаны с роутами (все это нужно выносить в Support) 2) не проще ли передавать $data в таком случае?

mx2s commented 5 years ago

image снова code style - должны быть отступы между функциями + phpdoc'и

mx2s commented 5 years ago

image нужно success возвращать всегда в самом конце

mx2s commented 5 years ago

AdminOnly & StopBanned image тебе не нужно тут проверять пароль (токен за тебя проверяет api:auth middleware) + что делать в случае если пароль отправляться не будет? У нас в параметрах везде кроме login его нет

AlertRED commented 5 years ago

image

  1. в контроллере не должно быть функций, которые не связаны с роутами (все это нужно выносить в Support)
  2. не проще ли передавать $data в таком случае?
  1. Да, я в дальнейших дз уже перенес эту функцию в сапорт
  2. Я тоже думал передавать сразу словарь, но почему-то мне не понравилось это... Изменю
AlertRED commented 5 years ago

image нужно success возвращать всегда в самом конце

не понял этого... В смысле в конце?

AlertRED commented 5 years ago

AdminOnly & StopBanned image тебе не нужно тут проверять пароль (токен за тебя проверяет api:auth middleware) + что делать в случае если пароль отправляться не будет? У нас в параметрах везде кроме login его нет

понял. По api_token проверяем роль

mx2s commented 5 years ago

не понял этого... В смысле в конце?

В прямом) Визуально когда смотришь на функцию в контроллере - всегда ожидаешь увидеть успешный return в самом низу. С точки зрения логики тут все правильно, осталось только поменять местами