Нужно повторно посмотреть https://github.com/yiisoft/yii-debug/pull/34. Впушил туда всякого... тесты проходят, их тоже можно глянуть.

Да что ж такое, опять дебагер обновлять😏

Мож вместо collected() все же лучше getCollected() ?

мне и так и так норм, но @xepozz getCollected() не понравился

странно

Мне наоборот метод collected() ни о чем не говорит, больше ожидаемо, что он вернет bool

Для меня метод должен звучать либо как команда, либо как вопрос.

Как я понял, он хочет переместить все тесты в дб, но автолоад и финал не понятно, как ему в этом помогут. Поэтому я ничего и не понял

Можно придумать другое название в одно слово :)

я переименовал в геттер

пока лучше названия нет

Если так сильно хочется, то подойдет что-то вроде export() или dump()

ну, кстати, да... название 1 в 1 как в Yii 2 :)

@xepozz ?

хм, я наугад сказал, вообще не помню подробностей реализаци. дебагера в yii2))

CollectorInterface по сути не является контрактом, потому как его значимый метод collect() неоднозначен и допускает несовместимые имплементации. Поэтому при текущей реализации нужно либо удалить этот метод из интерфейса либо целиком выбросить интерфейс, а вместо интерфеса указывать сами конкретные коллекторы. Собственно по этому интерфесу сечас и можно передать только конкретный коллектор. Например в LogProxy в конструкторе в CollectorInterface можно передать только LogCollector, а вот у же EventCollector там не подойдет, несмотря на то, что он реализует CollectorInterface.

Глянул по остальным классам, интерфейс там нужен, но без этого метода и в некоторых заменить интерфейс на конкретную реализацию.

Это создано по твоему запросу. Ты хотел сделать свои лог/эвент коллекторы. Если передавать туда не интерфейс, то своё уже не поставишь, т.к. классы финал

Да, можно)

Этот метод в интерфесе мне не нужен, можешь удалять, а в прокси заменить в конструкторе на конкретные реализации