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

ребята из hiqdev сделают

20:46пожаловаться #1

сможешь составить такой репорт?

20:48пожаловаться #2

так сначала надо решить, что нам ожидать от мержа )

20:49пожаловаться #3

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

20:49пожаловаться #4

ну не всегда

20:50пожаловаться #5

хотя в большинстве случаев да

20:50пожаловаться #6

да и оптимизированнее это как-то с текущим контейнером

20:51пожаловаться #7

это да

20:51пожаловаться #8

Add cache tags by yiiliveext · Pull Request #120 · yiisoft/di

Сделал resetable container, давайте обсудим да я уже на все это тесты напишу
https://github.com/yiisoft/di/pull/120/commits/dfcfd78c4809173601e93cc5fd9b98bfbeb5cae7

GitHub

Q
A

Is bugfix?
❌

New feature?
✔️

Breaks BC?
❌

Tests pass?
✔️

Merge yiisoft/yii-web#230 first.
When the __cacheTag is changed, the cache will be reset and a new instance of a singleton ...

20:53пожаловаться #9

Хм, не все подхавтило, ща

20:56пожаловаться #10

идеальный код

20:56пожаловаться #11

я бы ещё strict_types добавил

20:56пожаловаться #12

но это две лишних строчки

20:57пожаловаться #13

https://github.com/yiisoft/di/pull/120/commits/8773407e80f6a5ee9a7d77a43517e9503ae8620d

Dmitriy S

Хм, не все подхавтило, ща

GitHub

Add cache tags by yiiliveext · Pull Request #120 · yiisoft/di

20:57пожаловаться #14

Алексей R

идеальный код

Ты сомневался в моем мастерстве?😏

20:59пожаловаться #15

Dmitriy S

Ты сомневался в моем мастерстве?😏

просто льщу :D

20:59пожаловаться #16

Нам нужно поведение в контейнер, чтобы сервис можно было сделать не синглтоном?

21:09пожаловаться #17

@yiiliveext , в методе invalidateCache мого $this->instances[$id] можно заменить на переменную

21:09пожаловаться #18

Dmitriy S

Нам нужно поведение в контейнер, чтобы сервис можно было сделать не синглтоном?

чтобы фабрикой не наследоваться от Factory?

21:10пожаловаться #19

Алексей R

чтобы фабрикой не наследоваться от Factory?

чтобы были доступны глобальные зависимости для несинглтонов. В фабрике с этим проблема. Но мы это поменяем, надо только @samdark в отпуск отправить😉