Пакет видел когда-то для этих дел, но там тоже крайне некрасивый подход

пожалуй, буду первым

#crosscodereview

https://github.com/coderello/laraflash

У меня тоже только так работает. Ждём новой версии карбон. Вроде обещают лучшую локализацию.

По поводу моего вчерашнего вопроса по локализации карбона. В следующей версии ларавеля(5.8) уже будет возможность использовать Carbon 2 https://laravel-news.com/carbon-2-laravel

Всем привет. Ребята подскажите плиз как лучше всего организовать мультиязычность? Может библиотекой какой-то? В интернете есть информация, но лучше спросить еще)

Спасибо, Илья!

Первый желающий на ревью кода - @hivokas
Прикрепил его работу, смотрим, комментируем, предлагаем 🙂

А какую из? Или все?

Смотри в прикреплённом сообщении

https://github.com/coderello/laraflash/blob/353459c12a844611b29b4c6b2bdbd580edb4f39b/tests/FlashMessageBagTest.php#L98

Имхо, тут  в сетап нужно добавить сообщение с ненулевым делеем, чтоб проверить, что оно не рэди. А то этот тест будет зеленым даже при условии, что метод рэди просто возвращает все сообщения)

Ну и я б рассмотрел ларавелевские коллекции, как альтернативу массивам. Тогда

public function ready():
   {
       return array_filter($this->messages, function (FlashMessage $message) {
           return $message->toArray()['delay'] === 0;
       });
   }

Можно заменить на
public function ready():
   {
       return $this-messages->whereStrict('delay', 0);
   }

И короче, и нагляднее

А что делает метод whereStrict ?

А понял

с первым согласен, дополню тест

на счёт коллекций
я задумывался над их использованием в этой либе при её написании, но тогда причиной их не использовать стало такое поведение

допустим свойство messages - коллекция

// FlashMessagesBag

public function all(): Collection
{
    return $this->messages;
}

// Let's play with it

>>> laraflash()->add()->content('hello')
=> Coderello\Laraflash\FlashMessage {#3348}
>>> $messages = laraflash()->all();
=> Illuminate\Support\Collection {#3355
     all: [
       Coderello\Laraflash\FlashMessage {#3348},
     ],
   }
>>> $messages->push('some garbage')
=> Illuminate\Support\Collection {#3355
     all: [
       Coderello\Laraflash\FlashMessage {#3348},
       "some garbage",
     ],
   }
>>> laraflash()->all();
=> Illuminate\Support\Collection {#3355
     all: [
       Coderello\Laraflash\FlashMessage {#3348},
       "some garbage",
     ],
   }
>>> 

хотя сейчас понимаю, что это решается таким образом:

// FlashMessagesBag

public function all(): Collection
{
    return $this->messages->make();
}

в целом согласен, спасибо
думаю, переведу на коллекции

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

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

да, тоже хотел написать это, сделать методы success, info, error, warning

На счёт сокращённой записи согласен
На днях как раз подобное issue на себя создал
https://github.com/coderello/laraflash/issues/12