search

malinink / LaravelTestWeb15

Test App
GNU General Public License v2.0
0 stars 10 forks source link

Feature 68 #71

Open artesby opened 9 years ago

artesby commented 9 years ago

странно, но в модели Collector пхпкс заставил меня править аргументы (записать в столбец), а в Fruit нет.

malinink commented 9 years ago

@artesby Подправьте тогда php cs по всем другим вашим файлам в этом PR

artesby commented 8 years ago

упс что-то странное получилось...

artesby commented 8 years ago

я тут же сделаю 7 задание, не надо сливать пока

malinink commented 8 years ago

@artesby Может быть всеже откатите изменения и сделаете только фикс? Так даже интереснее будет!

artesby commented 8 years ago

кажется у меня не вышло =D

artesby commented 8 years ago

я попробовал использовать pull, всяк он мне писал, что моя ветка up-to-date, короче коммиты откатить не получилось тут добавил поле активности в младшую модель и подправил селект, правда не понял, кому и на каком этапе должна выдаваться ошибка, если в списке показано только то, что мы сами разрешаем выбрать: "проверяем каждый параметр на вхождение в этот список, иначе выдаем ошибку" или имеется в виду, что при редактировани все неактивные параметры должны быть убраны?

malinink commented 8 years ago

@artesby Не проверял как работает Ваша реализация, но по итогу должно быть так: Если у модели(старшей), по какой-то причине была выбрана ныне неактивная младшая модель, то при редактировании она должна быть в списке. И её можно также сохранить. При этом всем должна быть проверка при которой набор id младщих моделей должен состоять из всех активных младших моделей + все неактивные из выбраных у старшей модели.

Надеюсь поятно описал. Но как мне кажется сейчас у Вас немного другая логика.

artesby commented 8 years ago

у меня делается выборка всех активных + задействованных неактивных, и передается в селект а нужно брать по одной из списка младших моделей и смотреть, принадлежит ли она данному множеству, и потом передавать этот набор так? переделывать?

malinink commented 8 years ago

@artesby Да нет, просто добавьте такую же проверку в update

artesby commented 8 years ago

так мы же для update берем список из реквеста селекта, в который передаем уже отфильтрованный список как там может оказаться недопустимый id-шник? я не понимаю ._.

malinink commented 8 years ago

@artesby Ну допустим я возьму и подправлю html код в firebug.

artesby commented 8 years ago

как-то так, хотя мне кажется, что этому место в валидаторе, но у меня не получается для FormRequest сделать свое правило и по поводу cs в 94 строке контроллера не уверен, хотя phpcs заставил меня просто перенести скобку

malinink commented 8 years ago

@artesby Работать должно правильно, у меня последнийвопрос, правильно ли выбрасывание подобной ошибки в контроллере? Вожет быть воспользоваться валидецией для решения задачи?

malinink commented 8 years ago

@artesby ой, да прошу прощения, с 94 строкой - ему не понравилась длинная строка, можно выражения записать в переменные и сравнить их значения. Будет красивее. Поискал я информацию по валидации у меня сложилось две мысли

По второму варианту решение интересное, попробуйте.

artesby commented 8 years ago

я смотрел тут, но:

  1. мне кажется, что если расширить тут валидатор, то он расширится для всех реквестов, а не только для моего, т.е. нужно переопределить Validator внутри определенного FormRequest, а не глобальный
  2. по идее списки для сравнения нужно составлять в валидаторе, для этого в apprpovider придется добавить use Fruits и use Collectors, это нормально?
  3. я не понял как передать id модели в аргумент $parameters, чтобы найти объект в бд
malinink commented 8 years ago

Здравствуйте,

Прежлагаю обсудить завтра на паре все проблемы!

2016-02-12 21:17 GMT+03:00 artesby notifications@github.com:

я смотрел тут, но:

  1. мне кажется, что если расширить тут валидатор, то он расширится для всех реквестов, а не только для моего, т.е. нужно переопределить Validator внутри определенного FormRequest, а не глобальный
  2. по идее списки для сравнения нужно составлять в валидаторе, для этого в apprpovider придется добавить use Fruits и use Collectors, это нормально?
  3. я не понял как передать id модели в аргумент $parameters, чтобы найти объект в бд

— Reply to this email directly or view it on GitHub https://github.com/malinink/LaravelTestWeb15/pull/71#issuecomment-183437320 .

Константин Александрович apmath@malinink.ru

artesby commented 8 years ago

а меня, к сожалению, не будет) я приеду послезавтра только

12 февраля 2016 г., 23:04 пользователь malinink notifications@github.com написал:

Здравствуйте,

Прежлагаю обсудить завтра на паре все проблемы!

2016-02-12 21:17 GMT+03:00 artesby notifications@github.com:

я смотрел тут, но:

  1. мне кажется, что если расширить тут валидатор, то он расширится для всех реквестов, а не только для моего, т.е. нужно переопределить Validator внутри определенного FormRequest, а не глобальный
  2. по идее списки для сравнения нужно составлять в валидаторе, для этого в apprpovider придется добавить use Fruits и use Collectors, это нормально?
  3. я не понял как передать id модели в аргумент $parameters, чтобы найти объект в бд

— Reply to this email directly or view it on GitHub < https://github.com/malinink/LaravelTestWeb15/pull/71#issuecomment-183437320>

.

Константин Александрович apmath@malinink.ru

— Reply to this email directly or view it on GitHub https://github.com/malinink/LaravelTestWeb15/pull/71#issuecomment-183469243 .