DS
о, не торопитесь сливать, я хочу завтра это погонять, может чего замечу, выскажу, вы скажете, что я не прав и тогда можно сливать )
Не забудь add() на addMiddleware() поменять. И да, оно теперь $this Возвращает, так что можно цепочкой добавлять.
Size: a a a
2020 February 18
2020 February 19
А
Не забудь add() на addMiddleware() поменять. И да, оно теперь $this Возвращает, так что можно цепочкой добавлять.
лучше бы
add() оставилА
туда ведь кроме мидлварей нечего добавлять, постфикс не нужен
DS
Так с первого вглядав понятно, но поменять обратно не проблема. Нужно только устроить голосование.
Д
@yiiliveext https://github.com/yiisoft/yii-web/pull/219/
это теперь можно присобачить свой мидлвар в начало?
это теперь можно присобачить свой мидлвар в начало?
А
Собсна только в начало
Д
это же хорошо
Д
теперь можно и дебагер повесить сюда, супер
А
Кто о чём... 😁
А
это же хорошо
Осталось сократить название метода до add() и будет ещё лучше
Д
Кто о чём... 😁
а ты о чем?)
Д
Осталось сократить название метода до add() и будет ещё лучше
ну да, можно оставить add
Д
@roxblnfk сейчас в шторме увидел по райтклику "include only these lines into commit". теперь еще удобнее стало :)
А
А убрать файл из индекса не завезли?
Д
вроде давно было
А
Как? В меню Альт+тильда нет этого
А
В списке горячих клавиш тоже
Д
хотя ты прав, нет такого
Д
но раньше я как-то делал
DS
ну да, можно оставить add
По названию класса непонятно что там add. Если бы там было MiddlewareCollection, то ок, тут понятно. А в MiddlewareDispatcher хз, может и обработчики событий какие-то добавляются. Ну и плюс в роутере так уже назван метод, логично придерживаться одного стиля в одинаковых задачах.