Телеграмм чат группы yii3ru страница 501

Дмитрий

во, наконец-то)

А что наконец-то, сессия там же нормально инжектится.

22:24пожаловаться #1

хм... похоже на косяк

22:25пожаловаться #2

https://github.com/yiisoft/injector - тут

PSR-11 compatible injector. Contribute to yiisoft/injector development by creating an account on GitHub.

yiisoft/injector

22:26пожаловаться #3

Alexander Makarov

хм... похоже на косяк

Ну да, там в цикле надо проверять и ансетить если попал с типом

22:26пожаловаться #4

Ну либо в доке написать, что оно так работает и порядок важен

22:27пожаловаться #5

да, тоже отмечал это странное поведение

22:28пожаловаться #6

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

22:29пожаловаться #7

Зафиксите? Я пока дожму Yii 2 релиз.

22:37пожаловаться #8

надо сначала понять, какое поведение ожидается )

22:38пожаловаться #9

любой порядок

22:38пожаловаться #10

для типизированных зависимостей

22:39пожаловаться #11

меня не затруднит пофиксить в ближайшие 2 дня

22:41пожаловаться #12

Алексей R

меня не затруднит пофиксить в ближайшие 2 дня

Повесьте задачу на этого храбреца)

22:45пожаловаться #13

https://github.com/yiisoft/yii-rest/pull/51
смержит кто-нибудь?

Fix duplicated prefix, add config by xepozz · Pull Request #51 · yiisoft/yii-rest

Q
A

Is bugfix?
✔️

New feature?
❌

Breaks BC?
❌

Tests pass?
✔️

22:46пожаловаться #14

https://trello.com/c/HITlInXp/52-fix-injector-to-allow-specifying-named-parameters-in-any-order

22:46пожаловаться #15

Дмитрий

https://github.com/yiisoft/yii-rest/pull/51
смержит кто-нибудь?

Fix duplicated prefix, add config by xepozz · Pull Request #51 · yiisoft/yii-rest

Q
A

Is bugfix?
✔️

New feature?
❌

Breaks BC?
❌

Tests pass?
✔️

✅

22:47пожаловаться #16

https://github.com/yiisoft/yii-rest/pull/52
https://github.com/yiisoft/yii-demo/pull/71

Прошу заценить

Added ResponseConverter Middleware by xepozz · Pull Request #52 · yiisoft/yii-rest

Q
A

Is bugfix?
✔️

New feature?
✔️

Breaks BC?
✔️

Tests pass?
✔️

Example of usage you can see this yiisoft/yii-demo#71

23:25пожаловаться #17

@samdark https://github.com/yiisoft/friendly-exception/pull/5
нельзя назначить ревьюверов :(

Extend Throwable by xepozz · Pull Request #5 · yiisoft/friendly-exception

Q
A

Is bugfix?
✔️

New feature?
✔️

Breaks BC?
✔️

Tests pass?
❌

Fixed issues
#4

23:29пожаловаться #18

уже можно

23:30пожаловаться #19

если предложите норм кейс для list, то запилю. сделал просто чтобы показать, что работает