СП
Я там добавил ещё вариант, когда есть и в конечном массиве тоже модификатор
Size: a a a
2020 September 03
А
кстати, а чего это о плагине мы тут разглагольствуем? есть же отдельный чатик
СП
Потому что про
yiisoft/arrays было))AM
Потому что это не плагин делает.
А
Потому что это не плагин делает.
и это частично досадно ) т.к. исключения при мерже были бы кстати (кмк)
YM
separate pull request for testing of Migration class.
What do you think?)
https://github.com/yiisoft/yii-db-migration/pull/44
What do you think?)
https://github.com/yiisoft/yii-db-migration/pull/44
СП
Сделал черновик. Мне кажется такое поведение модификатора
И в конфигах тот кто создаёт конфиг (в приложении или пакете) определяет как его конфиг объединиться с нижними конфигами, но не лезет выше.
ReverseBlockMerge более очевидно.И в конфигах тот кто создаёт конфиг (в приложении или пакете) определяет как его конфиг объединиться с нижними конфигами, но не лезет выше.
a
Насчет rbac - вот этот пример неправильный https://github.com/yiisoft/rbac#attach-permissions-to-roles
потому что нет таких методов в менеджере(getRole, getPermission), но есть в интерфейсе хранилища getPermissionByName, getRoleByName
Реализовывать этим методы в менеджере либо править документацию, но тогда еще как то нужно сделать, к примеру, метод для получения хранилища из менеджера
Сейчас можно разве что отдельно передавать хранилище и менеджер
https://github.com/yiisoft/yii-demo/pull/132/files#diff-ed1daa02756ed2c589c4ff2c51750804R25
потому что нет таких методов в менеджере(getRole, getPermission), но есть в интерфейсе хранилища getPermissionByName, getRoleByName
Реализовывать этим методы в менеджере либо править документацию, но тогда еще как то нужно сделать, к примеру, метод для получения хранилища из менеджера
Сейчас можно разве что отдельно передавать хранилище и менеджер
https://github.com/yiisoft/yii-demo/pull/132/files#diff-ed1daa02756ed2c589c4ff2c51750804R25
СП
Добавил тестов по строкам, к которым нет у меня вопросов. Добавьте на ревью.
RT
Насчет rbac - вот этот пример неправильный https://github.com/yiisoft/rbac#attach-permissions-to-roles
потому что нет таких методов в менеджере(getRole, getPermission), но есть в интерфейсе хранилища getPermissionByName, getRoleByName
Реализовывать этим методы в менеджере либо править документацию, но тогда еще как то нужно сделать, к примеру, метод для получения хранилища из менеджера
Сейчас можно разве что отдельно передавать хранилище и менеджер
https://github.com/yiisoft/yii-demo/pull/132/files#diff-ed1daa02756ed2c589c4ff2c51750804R25
потому что нет таких методов в менеджере(getRole, getPermission), но есть в интерфейсе хранилища getPermissionByName, getRoleByName
Реализовывать этим методы в менеджере либо править документацию, но тогда еще как то нужно сделать, к примеру, метод для получения хранилища из менеджера
Сейчас можно разве что отдельно передавать хранилище и менеджер
https://github.com/yiisoft/yii-demo/pull/132/files#diff-ed1daa02756ed2c589c4ff2c51750804R25
Поправить документацию
a
Поправить документацию
ок
2020 September 04
СП
https://github.com/yiisoft/arrays/issues/44
Это баг или для массивов внутри
Это баг или для массивов внутри
ArrayableInterface и не должно работать?AM
лейблы добавил
AM
DM
И вот этот на код ревью: https://github.com/yiisoft/arrays/pull/43/files
А почему без деклораций парамеры?
СП
А почему без деклораций парамеры?
В интерфейсе так
DM
Печалька, а интерфейс изменить?