Д
1. не хватает метода типа
ReadOne()
2. приходится также собирать все Reader интерфейсы постоянно в одинSize: a a a
2020 June 10
Д
или я не так тебя понял?
А
1. не хватает метода типа
ReadOne()
2. приходится также собирать все Reader интерфейсы постоянно в одинпросто к одному пакету относится )
Д
приходится также собирать все Reader интерфейсы постоянно в один. Было бы круто его иметь из коробки (
DataReaderCollection?)что за кейс?
А
что за кейс?
если посмотришь в yii-cycle, то там примерно такая же картина. Reader со всеми этими интерфейсами (пока кроме фильтра)
Кейс такой, что репозиторий возвращает DataReader со всеми плюхами (countable, offsetable, sortable..), а эти плюхи каяждая в своём интерфейсе
Кейс такой, что репозиторий возвращает DataReader со всеми плюхами (countable, offsetable, sortable..), а эти плюхи каяждая в своём интерфейсе
AM
Тут ты в точку попал :) За php под win ничего не скажу, а линуксовый версии 7.4.3 стабильно проводит все тесты, в то время как 7.4.6 выдает три ошибки.
Регрессия значит... хммм.
А
говорить, что репозиторий возвращает сакловый DataReader не корректно, нужен сторонний общий интерфейс. Т.е. надо уходить от этого https://github.com/yiisoft/yii-demo/blob/40a1914d0ed4fad132a5c68a1d687dddce7b8d95/src/Repository/UserRepository.php#L12
и от этого https://github.com/yiisoft/yii-demo/blob/40a1914d0ed4fad132a5c68a1d687dddce7b8d95/src/Blog/Post/PostRepository.php#L27
и от этого https://github.com/yiisoft/yii-demo/blob/40a1914d0ed4fad132a5c68a1d687dddce7b8d95/src/Blog/Post/PostRepository.php#L27
Д
а, я раз 100 предлагал сделать общий для всех этих countable, offsetable. @samdark не хочет)
В
Регрессия значит... хммм.
Это выглядит как баг в php: похоже,
$reflection->invokeArgs() выполняется дваждыА
AM
кстати, @samdark, я тут активно юзаю
yiisoft/data в проде, и тоже чёт в DataReader не хватает метода типа ReadOne() https://github.com/yiisoft/data/blob/master/src/Reader/DataReaderInterface.phpНу давай поправим.
AM
Это выглядит как баг в php: похоже,
$reflection->invokeArgs() выполняется дваждыAM
Нет, не похоже.
AM
В смысле дважды?
А
Ну давай поправим.
ReadFirst() или ReadOne()?что насчёт общего интерфейса для всех Data/Read?
В
В смысле дважды?
Могу кусок лога скинуть
AM
Могу кусок лога скинуть
Гм...
NO
ReadFirst() или ReadOne()?что насчёт общего интерфейса для всех Data/Read?
просто ->first()
В
Возможно, оно не выполняется дважды, а вместо этого у меня контейнер не ресетится в
setUp тестов.AM
Так у меня 7.4.3