Re: [2Tuxoid] Про исходники сайта

У меня с потгресом постоянное андифенед pg_escape_string, это вообще чем лечится ?

Какие ты настройки включал в http сервере ?

Re: [2Tuxoid] Про исходники сайта

>У меня с потгресом постоянное андифенед pg_escape_string, это вообще чем лечится ?
Там всё просто - Туксоид, видимо, копировал мускульный код из постгресного, в результате появилась такая штука: на строке 39 в classes/base/mysql.php используется энтот pg_escape_string (а надо бы mysql_escape_string или как его там). А у тебя постгрес не установлен, и функция поэтому не определена. Так что либо ставь постгрес, либо правь исходник:)

>Какие ты настройки включал в http сервере ?
Ничего не делал особенного. Я взял богомерзкий апач 2.2.17 и пхп 5.3. Вот конфиг виртуального хоста для апача:

 http://paste.pocoo.org/show/358482/

У меня port-based хосты.

Re: [2Tuxoid] Про исходники сайта

Да, там сразу ерроров выше крыши с мускулем, как минимум в classes/threads.class.php на 107 строке надо убрать экранированные кавычки в запросе у LIMIT и OFFSET.

Может сегодня остальное посмотрю, только вдруг Туксоид всё у себя уже пофиксил, а я просто время трачу:)

Re: [2Tuxoid] Про исходники сайта

Ну с мускулом я так и сделал, а сейчас накатил на слаку пакет постгреса, вот с ним то и выдает андифинед.

Сдается мне PHP чего то не хватает :)

И да, в слаке апач с дефолтным конфигом не пашет (даже в сключенными коментами), пашет только с конфигом для пыховых разработчиков ...

Re: [2Tuxoid] Про исходники сайта

>Ну с мускулом я так и сделал, а сейчас накатил на слаку пакет постгреса, вот с ним то и выдает андифинед.
Хм, у тебя даже с постгресом есть такой еррор? А ты вписал в php.ini конфиг extension=pgsql.so ?

>И да, в слаке апач с дефолтным конфигом не пашет (даже в сключенными коментами), пашет только с конфигом для пыховых разработчиков ...
Странный там дефолтный конфиг видимо.

Re: [2Tuxoid] Про исходники сайта

Ага только не проверил факт нахождения модуля.

Все хорошо в слаке, но как надо что то не богоугодное, компилять и пересобирать крайне не удобно, особенно когда инет хреновый, и каждый мегабайт с обрывами качаешь по полчаса ...

Re: [2Tuxoid] Про исходники сайта

Ох, я нашел причину кучи ошибок с мускулем.

Во многих функциях класса base вызывается mysql_real_escape_string, а уже потом вызывается mysql_connect. А mysql_real_escape_string требует коннекта, поэтому первый её вызов возвращает False. Затем результат подставляется в запрос в виде пустой строки, ну и соответственно появляется ошибка.

Re: [2Tuxoid] Про исходники сайта

pg_escape_string я еще вчера нашел. Заменил ее на mysql_real_escape_string

Re: [2Tuxoid] Про исходники сайта

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

З.Ы. Если что-то найдете еще, то отписывайтесь в этой теме.

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

Re: [2Tuxoid] Про исходники сайта

Добавил в гиториус тему classic-black.

Re: [2Tuxoid] Про исходники сайта

Я ещё потыкал движок с мускулем.

1. mysql_real_escape_string таки не срабатывает у меня, если соединения с БД нет (так и должно быть). А его нет в момент первого вызова base::select, base::update и прочих. Получается, что функция возвращает false и первый запрос к БД неправильный. Я вылечил это переносом mysql_connect в начало каждой функции.

2. В classes/threads.class.php ещё остались ненужные кавычки около LIMIT (которые \'::1::\'), практически в каждой функции.

3. В classes/sections.class.php вызывается base::select, в котором $limit_end = 'NULL', в запросе получается LIMIT 'NULL', и mysql ругается и не работает. Я исправил, поправив в base::select проверку $limit_end в виде if(!empty($limit_end) && $limit_end != 'NULL').

4. В auth.class.php, в месте где устанавливается время последнего посещения, в base::update не передаётся id пользователя:). И вообще, есть места с пропущенными аргументами у функций, но это не сильно мешает.

В общем, я немного это пофиксил, и оно даже заработало. Могу выложить как-нибудь в виде репозитория (на том же гиториусе), чтобы не расписывать кучей слов. Вот только разберусь с этим инопланетным git-ом более-менее:)

Re: [2Tuxoid] Про исходники сайта

>В classes/sections.class.php вызывается base::select, в котором $limit_end = 'NULL', в запросе получается LIMIT 'NULL', и mysql ругается и не работает.
O_o Странно. Я специально проверял это на мускуле и постгрессе. Все работало.

>В auth.class.php, в месте где устанавливается время последнего посещения, в base::update не передаётся id пользователя:). И вообще, есть места с пропущенными аргументами у функций, но это не сильно мешает.
auth.class.php я не переписывал. Я приткнул его из старой версии. Его нужно переписать полностью.

>Вот только разберусь с этим инопланетным git-ом более-менее:)
O_o Он же простой как 5 копеек. Subversion гораздо сложнее. Я кстати его так и не осилил.

Re: [2Tuxoid] Про исходники сайта

>O_o Странно. Я специально проверял это на мускуле и постгрессе. Все работало.
Там делается например такой запрос: SELECT ALL count(id) AS cnt FROM threads WHERE section = '4' AND subsection = '1' GROUP BY subsection LIMIT NULL

А получается: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'NULL' at line 1

MySQL 5.5.10.

>O_o Он же простой как 5 копеек. Subversion гораздо сложнее. Я кстати его так и не осилил.
Я просто обычно с mercurial работаю, а тут всё не так. Какие-то rebase, локальные ветки, команды непривычные:) Хотя, в общем-то, всё понятно.

Re: [2Tuxoid] Про исходники сайта

Вот как-то так

 https://gitorious.org/~systemv/lor-ng-engine/systemv-lor-ng-engine/commits/devel

Re: [2Tuxoid] Про исходники сайта

Проверил и принял коммит с соединением паралельно адаптировав его для postgresql. А так-же добавил доп. проверку на магические ковычки. Коммит с ковычками после limit тоже принял. Проверку сессии в core.php отклонил потому-что старт сессии в core.php это временная заглушка и со временем она уберется.