Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

* add GroupByDialectCriteriaTest #54

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

crazedr0m
Copy link
Contributor

В некоторых случаях groupBy из критерии в OSQL попадает как DBField а в некоторых как DBValue на MyDialect это заметно из-за разного квотинга
сломалось это где-то после 06318bc, точнее не искал

@crazedr0m
Copy link
Contributor Author

с помощью git bisect найдено: 6575005 - работает. 5d93988 - сломано

@dovg
Copy link
Member

dovg commented Nov 21, 2011

Печаль. Тем коммитом я пытался исправить DialectString для Sqlite. В свободное время попробую поправить.

@AlexeyDsov
Copy link
Member

А в чем там проблема у sqlite была? Что он не умеет?

@dovg
Copy link
Member

dovg commented Nov 21, 2011

Текущая имплементация зависела от порядкового номера поля, а не его имени. Отсюда некоторые проблемы с алиасами.

ps. Проблему с sqlite породил не я, судя по коду, она была всегда, но не было теста, способного ее воспроизвести.

@crazedr0m
Copy link
Contributor Author

Можно немного доработать, добавив остальные диалекты и положить вместе с остальными тестами. некоторая сложность в том что Mysql для escape нужно подключение к базе.

@AlexeyDsov
Copy link
Member

Да, в mysql есть такая засада. По sqlite - все хочу надосуге его переписать на использование через PDO, но руки не доходят. Причина - через pdo драйвер sqlite версии 3, а сейчас версии 2. Третья версия круче хотя бы тем, что вроде б поддерживает вложенные транзакции через savepoint'ы.

@crazedr0m
Copy link
Contributor Author

Ну и как тут чего?

@AlexeyDsov
Copy link
Member

я до этой штуки все никак не соберусь добраться

@crazedr0m
Copy link
Contributor Author

пинг.

@AlexeyDsov
Copy link
Member

Прочитал еще раз комментарии. Забавно, но теперь есть SqLite через PDO, класс SQLitePDO я его когда-то запилил :) И теперь, кстати, ему тоже требует соединения с базой для эскейпинга. Если не ошибаюсь то нынче эскейпинг тупо требует соединения с базой т.к. в зависимости от версии базы эскейпинг может отличаться и функции необходимо знать как правильно эскейпить.

@AlexeyDsov
Copy link
Member

Ещё раз подебажил, посмотрел и пришел к следующему фиксу - #225.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants