Комментарии для Изучаем Web

Комментарий на [Ko3.3] Изменения в модуле Database от biakaveron

biakaveron — Fri, 11 May 2012 06:10:20 +0000

Вообще, очень много зависших тикетов, которые были один раз просмотрены, и потом их просто не замечают. Вчера решил пересмотреть свои старые тикеты, и запулил Pull Request для одного из них (для Userguide). В итоге, ночью Вуди его вмержил в 3.2 и 3.3.

Получается, надо пинать постоянно разработчиков. Один отказался, другой примет.

Комментарий на [Ko3.3] Изменения в модуле Database от zerkms

zerkms — Tue, 08 May 2012 04:33:02 +0000

Вот кстати ещё один пример раздолбайского отношения к DB & ORM в дев-тиме: http://dev.kohanaframework.org/issues/4445

tl;dr: with() ломает запрос, фикс тривиальный показал. Они упёрлись, из принципа (разумных причин не фиксить не было и нет)

Комментарий на [Ko3.3] Изменения в модуле Database от biakaveron

biakaveron — Sat, 05 May 2012 12:14:01 +0000

К примеру count_all на нормальных субд зафейлится, потому что после добавления COUNT(*) он ещё и table_name.* дорисовывает.

Хм. Проверил в 3.3 – нет такого. Генерируется SELECT COUNT(*) AS records_found FROM ...

Комментарий на [Ko3.3] Изменения в модуле ORM от biakaveron

biakaveron — Sat, 05 May 2012 12:12:07 +0000

Не уверен насчет усложнения, т.к. количество подобных методов в принципе ограничено. Раньше вообще использовался __call(), а ORM содержал список методов QBuilder’а, которые можно было заюзать. Вероятно, сейчас просто решили избавиться от одного медленного магического метода, создав много быстрых.

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

Комментарий на [Ko3.3] Изменения в модуле ORM от SVat

SVat — Fri, 04 May 2012 15:43:38 +0000

Добавлен метод using() для использования SQL-условия USING. В Query Builder он есть, а вот в ORM его добавить забыли.

Вот я для себя переписывал ORM: https://github.com/svat/ORM/blob/master/classes/kohana/orm.php там сам класс Kohana_ORM наследуется от Database_Query_Builder_Select. Соответственно при добавлении новых методов в Database_Query_Builder_Select они автоматически стают доступными и в ORM. Почему то разработчики Kohana не хотят сделать так же, а просто дублируют в ORM методы из QB, тем самым только всё усложняя.

Комментарий на [Ko3.3] Изменения в модуле Database от biakaveron

biakaveron — Fri, 04 May 2012 12:22:44 +0000

Ага, нашел этот тикет. Могу предложить разве что организовать Pull Request с уже готовыми драйверами для PDO. Например, у тебя уже есть Postgre, я для мускула быстро набросаю. На форуме вроде как есть SQLite-драйвер. А по поводу ORM - вопрос больной. Ведь по сути его особо не трогали в третьей ветке. Мало кому охота все там переделывать, насколько я понимаю.

Комментарий на [Ko3.3] Изменения в модуле Database от zerkms

zerkms — Fri, 04 May 2012 10:30:17 +0000

Видишь ещё проблема в чём – ОРМ в принципе под сферической субд в вакууме работать не будет. К примеру count_all на нормальных субд зафейлится, потому что после добавления COUNT(*) он ещё и table_name.* дорисовывает.
Потому, чтобы всё было честно, нужно ещё и орм переписать. Хотя в твиторе зомбор и писал, что текущий орм попахивает, но хз когда его хотя бы надумают переписать нормально, с тестами.

Комментарий на [Ko3.3] Изменения в модуле Database от zerkms

zerkms — Fri, 04 May 2012 10:28:34 +0000

Я уже как минимум один раз тикет создавал. Он был закрыт как wontfix

Мол уважаемые разработчики – таскайте каждый свою реализацию из проекта в проект.

Комментарий на [Ko3.3] Изменения в модуле Database от biakaveron

biakaveron — Fri, 04 May 2012 10:20:26 +0000

Дык если не создать даже тикет, как можно требовать от разработчиков нужных изменений? Можно отправить pull request и начать обсуждение на оф. форуме. Найдутся желающие потратить “уйму сил” на допиливание и внедрение этой фичи. Тем более, что скоро 3.3, надо успеть ))

Комментарий на [Ko3.3] Изменения в модуле Database от zerkms

zerkms — Fri, 04 May 2012 05:39:22 +0000

Все субд я не осилю, но решение с постгрей у меня есть. Раньше ещё были с ораклом и мускулем

ps: про тикеты – не надо, кохановская дев-тим состоит из снобов чуть менее, чем полностью. Чтобы доказать, что оно того стоит – придётся потратить уйму сил.