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

Add RemoveArrayKey by xepozz · Pull Request #25 · yiisoft/arrays

https://github.com/yiisoft/arrays/pull/25
@samdark поможешь с описанием назначения класса?

GitHub

Q
A

Is bugfix?
❌

New feature?
✔️

Breaks BC?
❌

Tests pass?
✔️

14:47пожаловаться #1

Дмитрий

а может еще сделаем интерфейсы на эти "маркеры"?

И как их использовать?

15:04пожаловаться #2

я хотел чтобы этот класс был не просто маркером

15:04пожаловаться #3

А этот RemoveArrayKey юзается в самом Array helper?

15:04пожаловаться #4

а чтобы и логика обработки была в нём

15:04пожаловаться #5

ну вот да

15:04пожаловаться #6

то есть И маркер И обработка

15:05пожаловаться #7

Alexander Makarov

И как их использовать?

закинул пищу для размышления

15:05пожаловаться #8

Andrii Vasyliev

я хотел чтобы этот класс был не просто маркером

я бы тоже хотел, но как это будет выглядеть дальше в коде?

15:06пожаловаться #9

Alexander Makarov

А этот RemoveArrayKey юзается в самом Array helper?

там нет, но можно будет добавить вместе с тестами

15:06пожаловаться #10

я могу сделать, посмотрите

15:06пожаловаться #11

ну, так же как и UnsetArrayValue.php

15:06пожаловаться #12

1 в 1

15:06пожаловаться #13

https://github.com/yiisoft/arrays/pull/25#discussion_r410689410

GitHub

Add RemoveArrayKey by xepozz · Pull Request #25 · yiisoft/arrays

Q
A

Is bugfix?
❌

New feature?
✔️

Breaks BC?
❌

Tests pass?
✔️

15:07пожаловаться #14

Andrii Vasyliev

я могу сделать, посмотрите

давай

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

Постораюсь сегодня

15:13пожаловаться #16

Alexander Makarov

ну, так же как и UnsetArrayValue.php

да, ты прав. не проверил как-то)

15:41пожаловаться #17

кстати, я багу починил, которая вызывается при первом создании конфигов :)

16:41пожаловаться #18

Дмитрий

кстати, я багу починил, которая вызывается при первом создании конфигов :)

Это которая composer update первый раз с путями?

17:14пожаловаться #19

ага