AM
Если они не нужны, значит что-то тут не так...
Size: a a a
2020 April 05
Д
@samdark а плагин поревьювишь?
DS
Если они не нужны, значит что-то тут не так...
Это самая гуманная реализация концепции дружественных классов из тех, что я пробовал. Для разработчика все прозрачно, иксепшин тот же, что и если бы он вызвал реально несуществующий метод.
DS
Ну и я предлагал вариант с передачей конфига в конструктор провайдера. Объясни, почему псрный провайдер не должен знать о псрном интерфейсе контейнера. Для меня объяснение только одно, ты хочешь оставить возможность юзать пакет yiisoft/event-dispatcher с не псрным контейнером. А такое будет в 0.001% случаев.
DS
Д
В чем сложность? Проверяешь одну ветку, потом другую относительно предыдущей
DS
В чем сложность? Проверяешь одну ветку, потом другую относительно предыдущей
Много неочевидных вещей, которые для тебя очевидны, потому что у тебя в голове есть общая картина.
AM
AM
@yiiliveext понял.
DS
Ну и я предлагал вариант с передачей конфига в конструктор провайдера. Объясни, почему псрный провайдер не должен знать о псрном интерфейсе контейнера. Для меня объяснение только одно, ты хочешь оставить возможность юзать пакет yiisoft/event-dispatcher с не псрным контейнером. А такое будет в 0.001% случаев.
Хотя конфигуратор более универсальный, только интерфейс к нему нужно сделать. Но не факт, что нужно, это пока так, мысли вслух.
DS
Вот, кстати, без сайкла скорость работы демо-приложения
Д
Много неочевидных вещей, которые для тебя очевидны, потому что у тебя в голове есть общая картина.
Каждый пр - отельный кусок законченных изменений, правда каждый следующий основывается на предыдущем.
Смотреть нужно правильность изменений.
После последнего пр можно покрыть некоторые вещи тестами. Потом можно будет дальше попилить
Смотреть нужно правильность изменений.
После последнего пр можно покрыть некоторые вещи тестами. Потом можно будет дальше попилить
DS
Каждый пр - отельный кусок законченных изменений, правда каждый следующий основывается на предыдущем.
Смотреть нужно правильность изменений.
После последнего пр можно покрыть некоторые вещи тестами. Потом можно будет дальше попилить
Смотреть нужно правильность изменений.
После последнего пр можно покрыть некоторые вещи тестами. Потом можно будет дальше попилить
Оно да, но надо вникать в плагин, чтобы понять правильность. Лучше будет, если @hiqsol отревьюит это.
AV
AV
Я на своем коде проверил yiisoft версия работает, перехожу на неё. Потом свою сделаю abandon в пользу yiisoft
Д
так, только давай тегнем для начала)
Д
https://github.com/yiisoft/yii-web/pull/236
@roxblnfk я бы не стал делать редирект в приложении. пусть лучше этим сервера занимаются
@roxblnfk я бы не стал делать редирект в приложении. пусть лучше этим сервера занимаются
Д
поднять пых, ради того, чтобы сделать редирект - очень неочень
А
https://github.com/yiisoft/yii-web/pull/236
@roxblnfk я бы не стал делать редирект в приложении. пусть лучше этим сервера занимаются
@roxblnfk я бы не стал делать редирект в приложении. пусть лучше этим сервера занимаются
не всегда это возможно настроить со стороны сервера