Size: a a a

Laravel для начинающих

2020 November 24

AB

Alex B in Laravel для начинающих
Vladimir Yefremov
@mrcypukat

есть сущность - позиция в заказе - она belongsTo - от заказа;
есть таблица и модель связи UserPackage, в которой отмечаются активные пакеты - так спроектировали предшественники и менять довольно много и сложно, не хочу пока;
Я сначала привязал позицию только к заказу, но тут наворочено и позиция не активна без отметки  в UserPackage;
приходится эту позицию заказа ставить в зависимость и от UserPackage.
И вот возникает вопрос - правомерно ли такое или надо иначе?
а что этот пакет делает? почему он связан с позицией?
почему нельзя все это засунуть в обсерверы, евенты и прочие штуки?
источник

O

Ostap in Laravel для начинающих
Alex B
а что этот пакет делает? почему он связан с позицией?
почему нельзя все это засунуть в обсерверы, евенты и прочие штуки?
>а что этот пакет делает?
Очень страшные вещи

>почему он связан с позицией?
Потому что так надо. Не спрашивай "почему". Просто надо.

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

Gt

Galèriarch the Koban... in Laravel для начинающих
Alex B
а что этот пакет делает? почему он связан с позицией?
почему нельзя все это засунуть в обсерверы, евенты и прочие штуки?
Столько вопросов и так мало ответов...
источник

AB

Alex B in Laravel для начинающих
Ostap
>а что этот пакет делает?
Очень страшные вещи

>почему он связан с позицией?
Потому что так надо. Не спрашивай "почему". Просто надо.

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

VY

Vladimir Yefremov in Laravel для начинающих
Alex B
а что этот пакет делает? почему он связан с позицией?
почему нельзя все это засунуть в обсерверы, евенты и прочие штуки?
пакет - это единица товара, в него входят услуги;
дело в том, что все другие пакеты имеют название, цену - в общем товар;
а я сейчас разрабатываю пакет "конструктор", в который юзер сам может набрать пакет услуг;
можно было бы его как бы отвязать вообще - в отдельный модуль, но есть одно "но": услуги это разделы информации/статьи и они доступны по пакетам, поэтому мне надо этот конструктор с этими пакетами - с одной стороны -  как-то увязывать , а с другой - обособливать; поэтому рождаются такие кривые решения;
ну и плюс - не хочется да и нет времени переделывать все коренным образом
источник

VY

Vladimir Yefremov in Laravel для начинающих
Alex B
я имел ввиду чтобы не кормить контроллеры или сервисы
ну, сервис раскормил уже, жирный ....
источник

O

Ostap in Laravel для начинающих
Alex B
я имел ввиду чтобы не кормить контроллеры или сервисы
Да пусть кушают сколько угодно, пока solid'у следуют)
источник

VY

Vladimir Yefremov in Laravel для начинающих
Ostap
Да пусть кушают сколько угодно, пока solid'у следуют)
волей-неволей надо следовать, иначе через пару недель придется все заново переписывать, не разберу ничего :)
источник

VY

Vladimir Yefremov in Laravel для начинающих
а еще, вот такое чучело можно писать?
return ConstructorOrder::whereHas('order')->whereHas( 'userPackageActive')
связку 2-х whereHas ?
я написал - валится ошибка, вот ищу - то ли оно вообще неприемлемо, то ли просто ошибка в коде?
источник

O

Ostap in Laravel для начинающих
Если это строка полностью, то ты забыл поставить ;
источник

ПФ

Паша Финогентов... in Laravel для начинающих
Ошибку бы по-хорошему
источник

O

Ostap in Laravel для начинающих
А возврат query builder'а - это так и задумано, или get() забыл?
источник

AB

Alex B in Laravel для начинающих
Vladimir Yefremov
а еще, вот такое чучело можно писать?
return ConstructorOrder::whereHas('order')->whereHas( 'userPackageActive')
связку 2-х whereHas ?
я написал - валится ошибка, вот ищу - то ли оно вообще неприемлемо, то ли просто ошибка в коде?
order.userPackageActive вродь так можно было
источник

VY

Vladimir Yefremov in Laravel для начинающих
Ostap
А возврат query builder'а - это так и задумано, или get() забыл?
не забыл, это для краткости :) вот весь запрос
        return ConstructorOrder::whereHas('order')->has( 'userPackageActive')
           ->with('infoSection')
           ->with('infoSection.texts')
           ->with('infoSection.markets')
           ->with('infoSection.markets.locale')
           ->with('infoSection.countries')
           ->with('infoSection.countries.currentTranslate')
           ->get();
источник

VY

Vladimir Yefremov in Laravel для начинающих
Паша Финогентов
Ошибку бы по-хорошему
Array to string conversion
источник

O

Ostap in Laravel для начинающих
Vladimir Yefremov
Array to string conversion
Скинь полный контекст)
источник

ПФ

Паша Финогентов... in Laravel для начинающих
Может вообще не в методе ошибка
источник

VY

Vladimir Yefremov in Laravel для начинающих
Alex B
order.userPackageActive вродь так можно было
это так работает?
они между собой не связаны,
это 2 связи order и userPackageActive к/из ConstructorOrder
источник

O

Ostap in Laravel для начинающих
Желательно скриншотами, что бы подсветочка была
источник

VY

Vladimir Yefremov in Laravel для начинающих
Ostap
Скинь полный контекст)
как это сделать?
на пастбин?
источник