AM
Ну, хотя-бы *всегда* ужасно и единообразно.
Size: a a a
2020 March 04
AM
Так мы убиваем изменения в attach(), ContainerDelegateInterface, метод delegateLookup().
AM
И вместо этого пишем чуть в readme.
DS
А метод delegateLookup() зачем?😳
AM
А на что он? В конструктор вторым необязательным аргументом $rootContainer и норм (как изначально до выпила делегатов было).
AM
Сильно проще получается.
DS
Ну, метод в таком случае останется, просто станет приватным и будет вызываться в кострукторе.
AM
и в методе будет $this->rootContainer = $container? :)
AM
всё?
AM
фейковый композит нам тоже не особо нужен
AM
он пустой
DS
Не, так не выйдет, метод надо оставлять. Многоуровневый delegate без него не получится.
AM
проще на null проверить
DS
проще на null проверить
Нет, логика с композитным контейнером в rootContainer останется
AM
так а зачем, если он пуст?
DS
так а зачем, если он пуст?
Кто пуст? rootContainer там пуст только изначально. Потом добавляются туда контейнеры на каждом слое делегирования. То есть, у нас будут работать коспозитные котнейнеры внутри композитных контейнеров. Прокси - это, кстати, один из уровней делегирования.
DS
Я эти все ситуации обкатал на тестовом коде.
DS
Вечером добавлю тесты, если согласуем текущие изменения.
DS
Сейчас делегирование работает при любом уровне вложенности композитных контейнеров.
AM
всё ещё не ясно, тесты картину прояснят, наверное