Size: a a a

Генератор непрочитанных сообщений

2020 July 23

DM

Daria Motorina in Генератор непрочитанных сообщений
Потому что одним из условий тестового было добавление курсорной навигации и фильтрации по свойствам, а этот пакет давал это искаропки
источник

DM

Daria Motorina in Генератор непрочитанных сообщений
Alexander Kladov
если бы я проверял, это был бы в моих глазах серьёзный минус, но я не знаю что задумывал автор
да
А как правильно задать валидацию доп. полей? Есть ядро сущности (Product), и Properties (разные свойства для разных продуктов), получается надо валидацию делать в зависимости от product_type или category_id
источник

DM

Daria Motorina in Генератор непрочитанных сообщений
c этим я больше всего застряла
источник

AK

Alexander Kladov in Генератор непрочитанных сообщений
можно ручками на крайний случай
источник

AK

Alexander Kladov in Генератор непрочитанных сообщений
не обязательно всё в rules выносить
источник

RV

Roma Vandolyak in Генератор непрочитанных сообщений
не смотрел много
но
как по мне такое лучше не делать
public static function boot()
   {
       parent::boot();

       $categoryId = Category::where(['name' => Category::CATEGORY_SHAMPOO])->first()->id;

       static::creating(function ($model) use ($categoryId) {
           $model->category_id = $categoryId;
       });
   }
источник

AK

Alexander Kladov in Генератор непрочитанных сообщений
ага, выглядит не очень
источник

AK

Alexander Kladov in Генератор непрочитанных сообщений
согласен
источник

RV

Roma Vandolyak in Генератор непрочитанных сообщений
ну в целом есть моментов много, но странно что не отписали точней
как то не оч так делать
источник

AK

Alexander Kladov in Генератор непрочитанных сообщений
Roma Vandolyak
не смотрел много
но
как по мне такое лучше не делать
public static function boot()
   {
       parent::boot();

       $categoryId = Category::where(['name' => Category::CATEGORY_SHAMPOO])->first()->id;

       static::creating(function ($model) use ($categoryId) {
           $model->category_id = $categoryId;
       });
   }
@Afterword хардкод таких штук это всегда плохо
источник

RV

Roma Vandolyak in Генератор непрочитанных сообщений
у нас тестовое  это блог (пост, комент) и логирование ИП
источник

AK

Alexander Kladov in Генератор непрочитанных сообщений
у тебя придет менеджер и скажет, а давай у нас будет называться не shampoo, а shampoonies
источник

RV

Roma Vandolyak in Генератор непрочитанных сообщений
и половина людей не может сделать.(
источник

AK

Alexander Kladov in Генератор непрочитанных сообщений
и нужно помнить что у тебя ищется айдишник по этому
источник

DM

Daria Motorina in Генератор непрочитанных сообщений
Я постараюсь за пару дней отрефакторить по вашим замечаниям, сможете потом глянуть, пожалуйста?)
источник

AK

Alexander Kladov in Генератор непрочитанных сообщений
конечно присылай
источник

DM

Daria Motorina in Генератор непрочитанных сообщений
спасибо)
источник

RV

Roma Vandolyak in Генератор непрочитанных сообщений
Daria Motorina
Я постараюсь за пару дней отрефакторить по вашим замечаниям, сможете потом глянуть, пожалуйста?)
могу написать точне буду, но не сегодня)
источник

RV

Roma Vandolyak in Генератор непрочитанных сообщений
public static $smells = ['без запаха', 'кокос', 'ваниль'];
зачем статика ?)
источник

DM

Daria Motorina in Генератор непрочитанных сообщений
Roma Vandolyak
public static $smells = ['без запаха', 'кокос', 'ваниль'];
зачем статика ?)
а как лучше определить список разрешенных значений в поле? Там условный enum в свойстве
источник