А
Это чем лучше, чем clone? ведь line изменится
$errorClass = get_class($error);
throw new $errorClass($error->getMessage());
Size: a a a
2020 March 08
А
ещё не факт, что порядок аргументов не изменён. Некоторые исключения этим грешат
А
@roxblnfk, если тебя еще не споили и ты можешь держать мержилку в руках - моргни.
Нужно вмержить этот пр https://github.com/yiisoft/proxy/pull/4
Там в трейт часть кода вынес, он переиспользуется в дебагере.
Нужно вмержить этот пр https://github.com/yiisoft/proxy/pull/4
Там в трейт часть кода вынес, он переиспользуется в дебагере.
в общем, я ещё не достаточно пьян, чтобы смёржить это вслепую :) но уже близок к тому
DS
Это чем лучше, чем clone? ведь line изменится
$errorClass = get_class($error);
throw new $errorClass($error->getMessage());
С clone трейса нормального не будет
T
For now, the best of the db and active record packages are the infinity of tests they have to prove everything.
DS
в общем, я ещё не достаточно пьян, чтобы смёржить это вслепую :) но уже близок к тому
Ничего не изменилось, оно уже было вмержено таким, я просто вынес часть кода в трейт.
DS
трейт - хорошая идея?
Почему нет?
DS
Ничего не изменилось, оно уже было вмержено таким, я просто вынес часть кода в трейт.
Да и вообще там код полностью мой и использую его пока только я, так что можешь смело мержить.
DS
Собственно оно и сейчас работает, но этот трейт дублируется в yii-debug сейчас.
А
Собственно оно и сейчас работает, но этот трейт дублируется в yii-debug сейчас.
почему не заменить на Throwable? Вроде правильно
А
а про порядок аргументов подумай
DS
почему не заменить на Throwable? Вроде правильно
То пока мелочи жизни, как тесты напишу - будем менять. А пока оно работает, то лучше не трогать, а оставить пока как есть. Это ж не окончательный вариант, у меня там еще большой кусок в отдельной ветке есть к этому пакету, так что еще будет время исправить.
А
название Proxy соответствует содержимому? может это что-то типа ErrorStateTrait? Может действительно слишком пьян, но ничего прямо относящегося к прокси не вижу :)
T
db works without active record.
DS
название Proxy соответствует содержимому? может это что-то типа ErrorStateTrait? Может действительно слишком пьян, но ничего прямо относящегося к прокси не вижу :)
Трейт используется в калассах с суфиксом Proxy, так что название логично. Ну и там еще должно добавится пару вещей в этот трейт.
А
Трейт используется в калассах с суфиксом Proxy, так что название логично. Ну и там еще должно добавится пару вещей в этот трейт.
мне кажется группировка в прокси обуславливается неймспейсом, а название должно соответствовать содержимому
T
and db no longer has magical methods, although I have seen that all who criticize them use them in one way or another.
DS
мне кажется группировка в прокси обуславливается неймспейсом, а название должно соответствовать содержимому
И? Если я туда слежующим пр добавлю специфичные именно для покси вещи, то обратно трейт будем переименовывать?
T
does anyone have a benchmark for db?
А
И? Если я туда слежующим пр добавлю специфичные именно для покси вещи, то обратно трейт будем переименовывать?
ну так добавь сразу то )