Size: a a a

2020 March 01

DS

Dmitriy S in Yii Framework 3
Дмитрий
вот тут
Это для других целей, подключение коллектора логера идит в другом месте. Да и на его места можно передать другой коллектор, например ивент коллектор и он не будет работать, хотя интерфейс соответсвует, так что там однозначно надо было менять или на отдельные интерфейсы или на конкретные реализации.
источник

DS

Dmitriy S in Yii Framework 3
Дмитрий
если да, то оно может быть создано и сейчас. делаешь новый коллектор, подменяешь по ссылке аргумент $container, и подменяешь реализацию ContainerInterface на свою обёртку. готово, у тебя прокси контейнер
Не вижу смысла обсуждать, пока не выложу ПР, поскольку там уже все сделано.
источник

Д

Дмитрий in Yii Framework 3
вот смотри, сейчас в мастере лежит пакет, который работает, но с изъянами по управлению зависимостями и конфигурированию. твой PR не одну из проблем не решает, просто дробит все еще сильнее. зачем его лить в этом случае я не особо понимаю.
мысль с разбивкой интерфейсов имеет место быть, если ее правильно организовать до конца, а не как сейчас.
не думаю, что кому-либо еще понравится всё то, что сейчас ты скинул в тот ПР, соответсвенно, пр на закрытие, а старания на ветер.
вместо необдуманной и не особо планированной работы, ты бы мог запилить уже вьювер, например
источник

DS

Dmitriy S in Yii Framework 3
Результат примерно так выглядит в коллекторе.
https://i.imgur.com/Zno3IX4.png

Но там и кроме коллектора есть фичи.
источник

DS

Dmitriy S in Yii Framework 3
Дмитрий
вот смотри, сейчас в мастере лежит пакет, который работает, но с изъянами по управлению зависимостями и конфигурированию. твой PR не одну из проблем не решает, просто дробит все еще сильнее. зачем его лить в этом случае я не особо понимаю.
мысль с разбивкой интерфейсов имеет место быть, если ее правильно организовать до конца, а не как сейчас.
не думаю, что кому-либо еще понравится всё то, что сейчас ты скинул в тот ПР, соответсвенно, пр на закрытие, а старания на ветер.
вместо необдуманной и не особо планированной работы, ты бы мог запилить уже вьювер, например
В ПР дебагера не было проделано никакой существенной работы. Более того, я в процессе разработки еще немало вариантов пробовал. А с интерфейсами, я их оставлю, но верну старт/стоп. Так что насчет закрытия говорить рано.
источник

Д

Дмитрий in Yii Framework 3
Dmitriy S
В ПР дебагера не было проделано никакой существенной работы. Более того, я в процессе разработки еще немало вариантов пробовал. А с интерфейсами, я их оставлю, но верну старт/стоп. Так что насчет закрытия говорить рано.
а говоришь конечная версия)
источник

DS

Dmitriy S in Yii Framework 3
Конечная в том смысле, что я не буду туда исправлений вносить, только верну как было. Я кроме интерфейсов все остальное только для обсудить делал.
источник

Д

Дмитрий in Yii Framework 3
Dmitriy S
В ПР дебагера не было проделано никакой существенной работы. Более того, я в процессе разработки еще немало вариантов пробовал. А с интерфейсами, я их оставлю, но верну старт/стоп. Так что насчет закрытия говорить рано.
с интерфейсами я про то, чтобы стабилизировать метод collect(). чтобы уйти от плавающих аргументов к статическим и описанным в доке
источник

Д

Дмитрий in Yii Framework 3
Dmitriy S
Результат примерно так выглядит в коллекторе.
https://i.imgur.com/Zno3IX4.png

Но там и кроме коллектора есть фичи.
это ты здесь ловишь всё, что отдается в get и оборачиваешь в класс для трекинга дальнейших действий с объектом?
источник

DS

Dmitriy S in Yii Framework 3
Дмитрий
с интерфейсами я про то, чтобы стабилизировать метод collect(). чтобы уйти от плавающих аргументов к статическим и описанным в доке
В тех интерфейсах они и будут статическим, не делал пока чтобы меньше ломать тебе код, если ты там над чем-то работаешь в этом пакете.
источник

DS

Dmitriy S in Yii Framework 3
Дмитрий
это ты здесь ловишь всё, что отдается в get и оборачиваешь в класс для трекинга дальнейших действий с объектом?
Детализация (здесь максимальная стоит) и отслеживаемые интерфейсы настраиваются в конфиге. В целом да, ловится все что в гет и если оно в списке трекинга то автоматически создается пркси-класс для дальнейшего трекинга объекта.
источник

Д

Дмитрий in Yii Framework 3
зачем для этого отдельный "прокси-контейнер", если оно может быть реализовано без привязки к контейнеру? это обычный коллектор
источник

Д

Дмитрий in Yii Framework 3
это в params.php tracked_services для него создано?
источник

DS

Dmitriy S in Yii Framework 3
Дмитрий
это в params.php tracked_services для него создано?
Это для демонстрации старта дебагера без прописывания его в index.php. То есть, если стоит прокси, то просто делаешь require yiisoft/yii-debugger и он будет автоматически запущен если включен в настройках.
источник

DS

Dmitriy S in Yii Framework 3
То есть, сервис провайдер в контейнере при этом не добавляется.
источник

DS

Dmitriy S in Yii Framework 3
Дмитрий
зачем для этого отдельный "прокси-контейнер", если оно может быть реализовано без привязки к контейнеру? это обычный коллектор
Причем здесь вообще коллектор? Это прокси. Такой же как и твои LoggerProxy and EventDispatcherProxy, но с дополнительными фичами.
источник

Д

Дмитрий in Yii Framework 3
оформи эту штуки в виде прокси и давай зальем ее в yii-debugger, эта крутая фича будет
источник

DS

Dmitriy S in Yii Framework 3
Дмитрий
оформи эту штуки в виде прокси и давай зальем ее в yii-debugger, эта крутая фича будет
Как раз прокси из дебагера уедут в отдельный пакет и там вместе с этим пркси и будут. Ну я так это вижу.
источник

Д

Дмитрий in Yii Framework 3
только вот конфигурирование надо бы сделать простым, что-то вроде такого:
interface1=>3,
interface2=>5
где 3  и 5 это уровень, который будет где-нибудь описан
источник

DS

Dmitriy S in Yii Framework 3
источник