-
Юбилейный DevConfX пройдет 21-22 июня в Москве. Как всегда — Вы решаете, кто попадет в программу секции Backend — голосуйте за интересные доклады
YiiFramework Опасность в архитектуре yii Active Record
- Автор темы grigori
- Дата начала
Ну в основном мелочи но все равно неприятно.
Вот недавно выкатывал кусок кода на Yii на прод. Валится с "driver not found" Вот not found и привет. Ё, какой драйвер? pdo/mysql стоит, в еррор логах пустота. Не, ну я не пугливый, ставлю прямо в перденя var_dump($exception) в его catch, вижу sqlite, офигеваю, понимаю, что сессии по дефолту в него, офигеваю еще, перебиваю на apc, все хорошо. Не, ну в sqlite по дефолту так в sqlite, ладно, но можно message оригинального exception показывать, пожалуйста? )
Вот недавно выкатывал кусок кода на Yii на прод. Валится с "driver not found" Вот not found и привет. Ё, какой драйвер? pdo/mysql стоит, в еррор логах пустота. Не, ну я не пугливый, ставлю прямо в перденя var_dump($exception) в его catch, вижу sqlite, офигеваю, понимаю, что сессии по дефолту в него, офигеваю еще, перебиваю на apc, все хорошо. Не, ну в sqlite по дефолту так в sqlite, ладно, но можно message оригинального exception показывать, пожалуйста? )
Ragazzo
TDD interested
fixxxer
да, эта проблема обсуждалась в одном из issues Кьенгом(проблема именно когда какой-либо из драйверов не найден и непонятно какую схему использовать), но там были свои скажем так тонкости) Есть еще пара моментов где нужно детальнее сообщение эксепшена сделать, Александр говорил что нужно тоже подумать над тем как в некоторых местах детальнее сообщения показывать (на хабре там был тоже какой-то пост)
да, эта проблема обсуждалась в одном из issues Кьенгом(проблема именно когда какой-либо из драйверов не найден и непонятно какую схему использовать), но там были свои скажем так тонкости) Есть еще пара моментов где нужно детальнее сообщение эксепшена сделать, Александр говорил что нужно тоже подумать над тем как в некоторых местах детальнее сообщения показывать (на хабре там был тоже какой-то пост)
Ragazzo в 2ке metadata вынесен из AR вообще. Ниче, там новый говнокод будет, еще накушаемся.
MiksIr а вот кодер мой не читал внутренности yii и не знает, что нельзя перекрывать конструктор. Он публичный - значит, можно. И в доках про запрет перекрытия конструктора ничего не сказано. А мысли он читать не умеет.
fixxxer у них 5.2+
keltanas У меня есть коммиты и багрепорты в yii, и основания злиться лично на Тяна, и еще я багфикс на это пулреквестну, а что сделал ты, чтоб ругать?
>Я не сомневаюсь что ты хороший разработчик
А я считаю, что это очередной пионэр, и буду молча вычищать раздел yii от его будущих постов.
исправление элементарное
в CActiveRecord
MiksIr а вот кодер мой не читал внутренности yii и не знает, что нельзя перекрывать конструктор. Он публичный - значит, можно. И в доках про запрет перекрытия конструктора ничего не сказано. А мысли он читать не умеет.
fixxxer у них 5.2+
keltanas У меня есть коммиты и багрепорты в yii, и основания злиться лично на Тяна, и еще я багфикс на это пулреквестну, а что сделал ты, чтоб ругать?
>Я не сомневаюсь что ты хороший разработчик
А я считаю, что это очередной пионэр, и буду молча вычищать раздел yii от его будущих постов.
исправление элементарное
в CActiveRecord
PHP:
//добавить
static protected function generateMetaData($model){return new CActiveRecordMetaData($model);
//изменить
public function getMetaData()
{
if($this->_md!==null)
return $this->_md;
else
//вместо return $this->_md=self::model(get_class($this))->_md;
return $this->_md=self::generateMetaData(get_class($this));;
}
что ответил Кьенг?
)MiksIr
miksir@home:~$
Не знаю, мне нравится. Еще бы какое-нить решение по эскейпингу имен связей в условиях... а то постгрес ругается.MiksIr
А что именно заменить то? Обычно AR хотят переписать
А вот application не очень нравится. Щас то привык, а в начале было сложно понять, как вся эта кухня работает. Особо после лимбовских цепочек фильтров. Но в общем и с этим можно жить.