А
Вернуть возможность может иметь плюс с той позиции, что совместим со слимом
Size: a a a
2020 April 03
А
Но я не знаю, насколько там это распространенная практика
AM
Вернуть возможность может иметь плюс с той позиции, что совместим со слимом
Это не важно.
AM
В общем, подумайте какой вариант выбрать. Я пока код посмотрю в PR.
DS
->addMiddleware(MyMiddleware::class, [args])?
Да-да, ща тебе туда из за одного текстового параметра инстансов остальных зависимостей напихают.
AM
Да-да, ща тебе туда из за одного текстового параметра инстансов остальных зависимостей напихают.
Выбери вариант.
AM
Там доступен контейнер, если что...
DS
Да-да, ща тебе туда из за одного текстового параметра инстансов остальных зависимостей напихают.
Не забудь такую ситуацию проработать
class MyMiddleware
{
public function __construct(MyFirstClass $firstClass, MySecondClass $secondClass, string $key)
}
AM
Да, придётся как в injector делать с именованными и вот этим всем.
AM
но мы сейчас не про реализацию
А
Напомню о вопросе по инжектор (метод мейк) - может и не будет лишним
DS
Выбери вариант.
Я выбрал, меня текущий вариант со стрелочной функцией вполне устраивает. Хотя я вообще не буду в своих проектах использовать замыкания в мидлтварях. Мне не сложно написать обертку. Чтобы даже лишний объект Closure не создавался и оно будет совместимым с декларативным объявлением роутов.
А
Я выбрал, меня текущий вариант со стрелочной функцией вполне устраивает. Хотя я вообще не буду в своих проектах использовать замыкания в мидлтварях. Мне не сложно написать обертку. Чтобы даже лишний объект Closure не создавался и оно будет совместимым с декларативным объявлением роутов.
Ну так и предлагается сделать удобную обёртку из коробки
AM
Я выбрал, меня текущий вариант со стрелочной функцией вполне устраивает. Хотя я вообще не буду в своих проектах использовать замыкания в мидлтварях. Мне не сложно написать обертку. Чтобы даже лишний объект Closure не создавался и оно будет совместимым с декларативным объявлением роутов.
Из:
- отдельный метод
- третий айтем в массив коллейбл
- дополнительный параметр на метод addMiddleware
- отдельный метод
- третий айтем в массив коллейбл
- дополнительный параметр на метод addMiddleware
DS
Из:
- отдельный метод
- третий айтем в массив коллейбл
- дополнительный параметр на метод addMiddleware
- отдельный метод
- третий айтем в массив коллейбл
- дополнительный параметр на метод addMiddleware
Из этого ничего. Все плохо.
DS
Остальные смотри, там все готово. EventConfigurator пока убрал из index.php, без стратегий мержа там только костылями. Он сейчас локально в дебагере.
AM
Из этого ничего. Все плохо.
Так мы ни к чему не придём и продолжим спорить.
AM
Остальные смотри, там все готово. EventConfigurator пока убрал из index.php, без стратегий мержа там только костылями. Он сейчас локально в дебагере.
Напиши issue про стратегии мёржа, ок?