AM
влил
Size: a a a
2020 March 17
DS
ну вроде все норм с мастера работает
AM
отлично
AM
спасибо 👍
Д
VarDumper::dump($object) - остётся как есть
VarDumper::create($object)->asString();
VarDumper::create($object)->asJson();
VarDumper::create($object)->asSomething();
VarDumper::create($object)->export();
VarDumper::create($object)->asString();
VarDumper::create($object)->asJson();
VarDumper::create($object)->asSomething();
VarDumper::create($object)->export();
Хорошее предложение
RM
А
Чет там все проверки проводились
DS
Хорошее предложение
Что ты там с фабрикой для айди хотел? Ее там делать через замыкание две минуты. Но сначала обоснуй почему плох текущий метод.
Д
ну как. есть публичный метод, который устанавливает debug id (setDebugId()) но этот метод работает только для первого вызова, т.е. метод(публичный) особо бесполезен на внешке.
давать интерфейсу Storage какой-то метод setDebugId() вообще не круто, тут ломается любая абстракция
менять ФАЙЛ в рантайме - плохо. менять стореджы в райнайме - позволительно
давать интерфейсу Storage какой-то метод setDebugId() вообще не круто, тут ломается любая абстракция
менять ФАЙЛ в рантайме - плохо. менять стореджы в райнайме - позволительно
DS
А вот это вот тянуть все коммиты в новый пакет из пакета yii2 кто придумал?
Как минимум надо было экспортировать одним коммитом, а еще лучше пофайлово/поклассово по мере необходимости. Потому как сейчас местами выглядит трешово.
Как минимум надо было экспортировать одним коммитом, а еще лучше пофайлово/поклассово по мере необходимости. Потому как сейчас местами выглядит трешово.
AM
Надо порезать. Но я пока не нашёл норм способ.
DS
ну как. есть публичный метод, который устанавливает debug id (setDebugId()) но этот метод работает только для первого вызова, т.е. метод(публичный) особо бесполезен на внешке.
давать интерфейсу Storage какой-то метод setDebugId() вообще не круто, тут ломается любая абстракция
менять ФАЙЛ в рантайме - плохо. менять стореджы в райнайме - позволительно
давать интерфейсу Storage какой-то метод setDebugId() вообще не круто, тут ломается любая абстракция
менять ФАЙЛ в рантайме - плохо. менять стореджы в райнайме - позволительно
Он и не нужен во внешке. DebugId нужен любому хранилищу, по нему идентифицируется дебаг. Так что его нужно либо в конструкторе передавать, либо делать метод.
DS
Он и не нужен во внешке. DebugId нужен любому хранилищу, по нему идентифицируется дебаг. Так что его нужно либо в конструкторе передавать, либо делать метод.
Тут скорее с точки зрения рр надо смотреть, потому как дебаг запросо-зависимый.
DS
Надо порезать. Но я пока не нашёл норм способ.
Д
Он и не нужен во внешке. DebugId нужен любому хранилищу, по нему идентифицируется дебаг. Так что его нужно либо в конструкторе передавать, либо делать метод.
не нужен ему этот debug id. хранилище может сохранять в файл, название которого никак не пересекается с debug id, и это будет нормой
DS
не нужен ему этот debug id. хранилище может сохранять в файл, название которого никак не пересекается с debug id, и это будет нормой
Нет, любые дебаг данные должны быть идентифицированы по айди дебага
Д
вот ты и завязался на конфликте имен
DS
вот ты и завязался на конфликте имен
В смысле? Ты в заголовке айди дебага передаешь, и по нему ссылка на дебаг формируется.
DS
Сторедж должен отдавать по айди дебага либо сами старые данные (восстановленный массив коллекторов), либо объект, который с этими данными работает и имеет стандартный интерфейс (адаптер/провайдер). Я пошел по второму пути.
DS
Еще одна вариация на тему работы с рр
https://github.com/yiisoft/di/pull/120
https://github.com/yiisoft/di/pull/120