Size: a a a

2018 November 02

ИС

Илья Сакович in Laravel UA
постараюсь как можно больше кейсов сокращённой записи покрыть
источник

ИС

Илья Сакович in Laravel UA
Yurii Prudskyi
По поводу развития пакета: с точки зрения юзера, я бы хотел, чтоб можно было писать не только
laraflash()
 ->add()
 ->content('You have been registered successfully.')
 ->type('success')
 ->now();
а и
laraflash()->success('You have been registered successfully.')

Так как, имхо, чаще всего будет юзаться самый базовый кейс - дилеи и хопы нужны намного реже. И их можно добавить необязательными параметрами методов 'success', 'info' и т.д.
Кстати, я из доки не понял, что делает свойство important)
свойство important по факту просто выставляет true для ключа 'important' при преобразовании сообщения в массив
источник

ИС

Илья Сакович in Laravel UA
для чего это нужно: глядя на это свойство можно определить, надо ли, чтобы флэш сообщение пропадало через несколько секунд или надо, чтобы оно осталось до того момента, как юзер сам его закроет
источник

ИС

Илья Сакович in Laravel UA
в таком случае дополню описание
источник

AA

Ann Ali in Laravel UA
Привет. Кто-то использует/использовал https://github.com/andersao/l5-repository ? Если да - поделитесь впечатлениями - раздумываю использовать ли у себя на проектах. Есть некоторые шероховатости с использованием(например генерация пейджера). Сам подход очень нравится - но смущает, что пакет уж достаточно длительно время не обновлялся(9 месяцев) и похоже ментейнер им сейчас не занимается
источник

В

Вячеслав in Laravel UA
Если использовать репозитории, то нужно отчётливо понимать зачем. В основном это overhead.
источник

AA

Ann Ali in Laravel UA
Сам принцип того, что контроллер, job и тому подобное работает только с сырыми данными, а не моделью - мне кажется правильной. Плюс централизация всего в 1-м месте, и модель не разростается и хранит в себе только связи и описание свойств
источник

ОТ

Олег Таланов in Laravel UA
Ann Ali
Привет. Кто-то использует/использовал https://github.com/andersao/l5-repository ? Если да - поделитесь впечатлениями - раздумываю использовать ли у себя на проектах. Есть некоторые шероховатости с использованием(например генерация пейджера). Сам подход очень нравится - но смущает, что пакет уж достаточно длительно время не обновлялся(9 месяцев) и похоже ментейнер им сейчас не занимается
Привет! Пакет не обновляется, по всей видимости, по-тому, что это все уже есть в новых версиях Laravel без всяких там бубнов.
источник

YP

Yurii Prudskyi in Laravel UA
Ann Ali
Сам принцип того, что контроллер, job и тому подобное работает только с сырыми данными, а не моделью - мне кажется правильной. Плюс централизация всего в 1-м месте, и модель не разростается и хранит в себе только связи и описание свойств
По поводу первого пункта не совсем понял, можно пример? Что мешает в контроллере работать с моделью?
источник

AA

Ann Ali in Laravel UA
Мне кажется это приводит к более сильной связности и разрастанию контроллеров. Для меня контроллер - это принял запрос, передал данные на обработку нужному слою, получил сырые результаты, отдал запрос - и ничего больше. О внутренней структуре приложения он ничего знать не должен
источник

AH

Angry Hungry in Laravel UA
Yurii Prudskyi
По поводу первого пункта не совсем понял, можно пример? Что мешает в контроллере работать с моделью?
SOLID ?
источник

AA

Ann Ali in Laravel UA
+ этот пакет разрешает делать много удобных штук - работа с презентором, кеширование, централизированная валидация(а не только через форм-реквест) - это круто
источник

YP

Yurii Prudskyi in Laravel UA
А солид тут причем?
Вот, скажем, есть контроллер:

class EventController extends Controller
{
   public function index()
   {
       $events = Event::visible()->get();

       return view('events.index', compact('events'));
   }
}

Чем он плох и как он нарушает солид?
источник

MR

Maxim Rebitskiy in Laravel UA
А будет у тебя таких выводов 5 например, и тебе в один момент нужно будет убрать visible() например, и ты уже такой должен везде это исправлять, а юзая тот же репозиторий например ты можешь потом просто в методе изменить что нужно.
источник

MR

Maxim Rebitskiy in Laravel UA
(как пример)
источник

YP

Yurii Prudskyi in Laravel UA
>А будет у тебя таких выводов 5 например
В 5 разных методах контроллера? Маловероятно, но допустим. Как это решит репозиторий? Там же все равно будет аналог метода визибл, не? Который надо будет выпиливать из этих 5 мест
источник

AA

Ann Ali in Laravel UA
Event::visible() - со временем при таком подходе модель превращается в большую свалку из статических методов по доставанию чего-то. Модель для меня должна служить для отображения собственных свойств и связей с другими моделями
источник

YP

Yurii Prudskyi in Laravel UA
ну это вообще был скоуп, а не статический метод) Который, если он общий для многих моделей (а он похож на такой, что будет общим), можно вынести в трейт
источник

AH

Angry Hungry in Laravel UA
или в репозиторий, что будет более верно
источник

ОТ

Олег Таланов in Laravel UA
Yurii Prudskyi
ну это вообще был скоуп, а не статический метод) Который, если он общий для многих моделей (а он похож на такой, что будет общим), можно вынести в трейт
а еще можно вынести в глобальный скоуп для модели
источник