NO
Ага. Так в чëм суть кейса? Почему он не работает в мастере?
Size: a a a
2020 June 29
DS
Если ты про тот тест, который @yiiliveext добавил, то он не проходит в мастере
Как не проходит? У меня локально проходит, в пр проходит, где не ок?
В
Я спрашиваю, что именно тестирует этот тест) и что сделать чтобы он зафейлил
В композитный контейнер добавить 2 других контейнера, в одном из которых объявить, что реализацией класса1 является класс2. В общем, прямо как в PR)
В
И запустить это на мастере. Тест сфейлит.
DS
В
Ага. Так в чëм суть кейса? Почему он не работает в мастере?
В
Ага. Так в чëм суть кейса? Почему он не работает в мастере?
Потому что в контейнере идет проверка на
class_exists, и он действительно есть. Но идея в том, чтобы подменить его другой реализацией.DS
Это статический анализатор, причем здесь тест?
NO
Так может просто исправить "композитный контейнер"?
В
Так может просто исправить "композитный контейнер"?
Как?
NO
Сейчас посмотрю.
DS
В любом случае, согласно текущего пср задача не имеет адекватного решения, можно только минимизировать риски несовместимости.
В
О @samdark, призываю тебя для разрешения разногласий
NO
А почему в мастере уже параметры в get?
DS
А почему в мастере уже параметры в get?
Забыли убрать, когда я убрал в контейнере
NO
В любом случае, согласно текущего пср задача не имеет адекватного решения, можно только минимизировать риски несовместимости.
Согласно "текущего PSR" как я понял, никакого "композитного контейнера" не существует, так что это не к PSR относится. Проблема в реализации композитного контейнера, а не в PSR.
В
Согласно "текущего PSR" как я понял, никакого "композитного контейнера" не существует, так что это не к PSR относится. Проблема в реализации композитного контейнера, а не в PSR.
Композит реализует интерфейс, в этом плане все ок
В
Идея-то в том, что в проекте может быть несколько контейнеров. И удобно получать к ним доступ через единую точку входа, такой же контейнер, который сам ничего резолвить не умеет, но знает, кто это умеет.
NO
Один из вариантов решения, сделать композит вложенным. Тогда определение будет запрашиваться не в цикле, а в рекурсии, и вернёт по чесноку самое первое найденное определение, либо отдаст фолбек с созданием объекта.
NO
Есть ещё много вариантов, один из которых - оставить текущее поведение ) Так как оно достаточно предсказуемо. Но с рекурсией мне больше нравится )