Сергей delphinpro

Тебе когда надо достать из холодильника 5 бутылок пива, что проще - открыть холодильник, взять 5 бутылок пива и закрыть холодильник, или открыть холодильник, взять бутылку, закрыть холодильник, открыть холодильник, взять бутылку, закрыть холодильник, открыть холодильник, взять бутылку, закрыть холодильник, открыть холодильник, взять бутылку, закрыть холодильник, открыть холодильник, взять бутылку, закрыть холодильник?

Это же many to many обычный. В laravel связь belongsToMany + pivot для значения

Называется, услышал звон, да не знаю где он.

1. Сервисный слой ты превратил в контроллер.
Т.е. у тебя есть контроллер, а ты создал ещё один, т.к. где-то слышал, что нужны сервисы.
Но, сервисы не шлют ответы, это внутренний слой для отделения логики.
Ты можешь те методы вызвать через консоль, другие сервисы и т.п.
Там должен просто генерироваться return, а сам response уже в контроллере (чтоб контроллер своб функцию выполнял). Валидацию реквеста кстати тоже надо делать не в сервесе.

2. CategoryQuery - это вобще дичь дикая.
Ты опять создаёшь дубликат, потому что услышал о репозиториях, и даже интерфейс. Вот я угараю с людей, кто везде пихает интерфейсы. Интерфейсы нужны там, где они нужны. Где будет 2+ наследователей интерфейса.
Откуда у тебя тут наследники, если ты завязан на элокуент?
А ноги от индусов репозитаристов. Не нужен репозитарий, Query и интерфейсы при работе в элокуент. Он сам в себе это всё уже содержит и все повторяющиеся вещи обёрнуты уже. find, first, firstOrNew и т.д.
Так что используй элокуент и не переусложняй проект напрасно. Не надо делать сложно, делай просто.

3. CategoryAction.
Опять интерфейс...
Вот это Экшн по сути у тебя сервис. Тут ты имеено перенёс бизнес логику создания и редактирования и возвращаешь ответ данного метода. Вот удали свой CategoryAction и перенеси методы в сервис.

Не благодари

<ul class="list-inside bullet-list-item flex flex-wrap justify-between -mx-5 -my-2">
    <?php 
 if ($isAuthorized) { //тут условие проверки авторизации
        foreach ($menu as $value) {
 ?>
        <li class="px-5 py-2">
            <a class="<?= selectedMenu($value['path']) ? 'text-orange cursor-default' : 'text-gray-600 hover:text-orange' ?>"
               href="<?= $value['path'] ?>"><?= cutString($value['title']) ?></a></li>
    <?php }
}
 ?>
</ul>

Верстальщик никому ничего не должен по умолчанию просто так. Если вы готовы платить, обговорите условия, и вам все сделают. Скорее всего, в вашем случае макет попал в руки не к самому лучшему исполнителю. Не любой исполнитель может работать без макета. Но оставлять так, что при 1600 что-то не влезло в экран, это дикость.

можно

другое дело что на боевом сайте никакую разработку обычно не ведут, и гит на нем тоже держать нежелательно
если же речь о локальном сервере, то разумеется можно. собственно, гит к сайтам никакого отношения не имеет, он работает с кодом. Как говорил поручик Ржевский, "Код есть? Значит можно".

Не надо этого хотеть