Если у тебя уже заложенный такой флоу, что в моделях идёт расчет по сервисам, то лучше придерживался старого флоу. Иначе у тебя однажды что-то отвалится, а виноват будешь ты. И ты можешь сколько угодно рассказывать заказчику, что "я боролся за чистоту архитектуры". Для него это все вода. У него говнокод работал. Твой "чистый" код сломал систему напрочь. К тому же, судя по всему, опыта у тебя ещё немного, так что подобная задача тебе будет вряд ли по плечу.

Если у тебя нет задачи навести порядок в проекте, сделать рефакторинг и вытащить из заказчика кучу бабла за 0% нового функционала, то придерживайся старого стиля проекта. По крайней мере старое будет работать.

Удачи тебе, что ещё сказать :) С этими говно-легаси проектами, которые писали какие то Раджи Абдурахманы вместе со своими ручными обезьянками, можно крышей поехать.

Тогда как один DI в контроллере все сломает?)))

Пацаны, я на сиденья натянул меховые чехлы, теперь моя беха не едет.
Шо ж такое.

это ты про меня?

я не утверждал этого, лол.

🤦‍♂

А ведь багисто реально индусы писали

Не зря багисто называется.
И дело вовсе не в пакетах --,

нет, но это пакет багисто, и приходятся дополнять функционал которого нет в багисто, а с этими ценами, очень сложно, они выводятся в очень многих местах, и придется менять это в половину проекта контроллера, поэтому я делаю в модели, знаю, что не верно, но как иначе, если в очень многих местах тянется эта модель и именно этот метод из модели, и да, Nick верно сказал,у меня не много опыта, совсем немного, недавно начал работать, поэтому учусь, и я не спорил и не хочу конфликтов, просто просил совета не более. Я учусь.

спасибо, хорошее сообщение.

Непорочное зачатие

Хорошо. Расскажи что делает твой метод в сервисе.
Подробно расскажи.
Как формируется цена, все ли параметры, необходимые для расчета, присутствуют в модели?

Кстати да. Хуже уже точно не будет 😂

public function getPrice($price)
    {
        if (Auth::guard('customer')->check()) {
            // Get discount user
            $discount = $this->customer->find($this->user->id)->discount;

            if ($discount) {
                // calculate price
                return ($discount->discount / 100) * $price;
            }
        }
        return $price; // get price not with discount
    }

- если у юзера есть скидка, тогда высчитывается сумма уже с процентом, проверяю залогирован ли юзер, и если скидки нет, тогда возвращает просто сумму товара.

public function getPriceAttribute($price)
        {
            return $this->productService->getPrice($price);
        }

 - это в модели использую метод из сервиса

у них, уже в модели было так.

public function getPriceAttribute($price)
    {
        return $price;
    }

возвращают цену продукта.

как вернуться к изменениям?

пкм

реверт

не помогло