anonymous@RULINUX.NET~# Last login: 2024-12-23 07:04:34
Регистрация Вход Новости | Разметка | Пользователи | Галерея | Форум | Статьи | Неподтвержденное | Трекер | Правила форума | F.A.Q. | Ссылки | Поиск
[#] [Добавить метку] [Редактировать]
Скрыть

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

Попробовал движок - с постгресом всё ок, а вот на мускуле и правда ошибки есть, как Id и говорил. Я одну поправил, а вот остальные где-то глубоко в base::select.

И добавь в репозиторий тему classic black, а то твой дефолтный рут-юзер её хочет видеть, а файлов нет.

P.S. я небольшой патч написал, который это дело ликвидирует: если тема, выбранная пользователем отсутствует, то берётся первая существующая.  http://paste.pocoo.org/show/358018/

SystemV(*) (2011-03-23 22:44:00)

Emacs-w3m/1.4.414 w3m/0.5.3

[Ответить на это сообщение]
[#] [Добавить метку] [Редактировать] Ответ на: [2Tuxoid] Про исходники сайта от SystemV 2011-03-23 22:44:00
avatar
Скрыть

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

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

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

Id(*)(2011-03-23 22:55:37)

Mozilla/5.0 (X11; U; Linux i686; ru; rv:1.9.1.17) Gecko/20110302 Iceweasel/3.5.17 (like Firefox/3.5.17)
[#] [Добавить метку] [Редактировать] Ответ на: Re: [2Tuxoid] Про исходники сайта от Id 2011-03-23 22:55:37
avatar
Скрыть

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 хосты.

SystemV(*)(2011-03-23 23:05:52)

Emacs-w3m/1.4.414 w3m/0.5.3
[#] [Добавить метку] [Редактировать] Ответ на: Re: [2Tuxoid] Про исходники сайта от Id 2011-03-23 22:55:37
avatar
Скрыть

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

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

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

SystemV(*)(2011-03-23 23:12:39)

Emacs-w3m/1.4.414 w3m/0.5.3
[#] [Добавить метку] [Редактировать] Ответ на: Re: [2Tuxoid] Про исходники сайта от SystemV 2011-03-23 23:05:52
avatar
Скрыть

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

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

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

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

Id(*)(2011-03-23 23:14:17)

Mozilla/5.0 (X11; U; Linux i686; ru; rv:1.9.1.17) Gecko/20110302 Iceweasel/3.5.17 (like Firefox/3.5.17)
[#] [Добавить метку] [Редактировать] Ответ на: Re: [2Tuxoid] Про исходники сайта от Id 2011-03-23 23:14:17
avatar
Скрыть

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

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

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

SystemV(*)(2011-03-23 23:18:23)

Emacs-w3m/1.4.414 w3m/0.5.3
[#] [Добавить метку] [Редактировать] Ответ на: Re: [2Tuxoid] Про исходники сайта от SystemV 2011-03-23 23:18:23
avatar
Скрыть

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

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

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

Id(*)(2011-03-23 23:48:41)

Mozilla/5.0 (X11; U; Linux i686; ru; rv:1.9.1.17) Gecko/20110302 Iceweasel/3.5.17 (like Firefox/3.5.17)
[#] [Добавить метку] [Редактировать] Ответ на: [2Tuxoid] Про исходники сайта от SystemV 2011-03-23 22:44:00
avatar
Скрыть

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

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

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

SystemV(*)(2011-03-24 01:53:18)

Emacs-w3m/1.4.414 w3m/0.5.3
[#] [Добавить метку] [Редактировать] Ответ на: [2Tuxoid] Про исходники сайта от SystemV 2011-03-23 22:44:00
avatar
Скрыть

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

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

Tux-oid(*)(2011-03-24 09:24:20)

Mozilla/5.0 (X11; Linux i686; rv:2.0.0) Gecko/20100101 Firefox/4.0
[#] [Добавить метку] [Редактировать] Ответ на: Re: [2Tuxoid] Про исходники сайта от SystemV 2011-03-23 23:12:39
avatar
Скрыть

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

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

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

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

Tux-oid(*)(2011-03-24 09:28:53)

Mozilla/5.0 (X11; Linux i686; rv:2.0.0) Gecko/20100101 Firefox/4.0
[#] [Добавить метку] [Редактировать] Ответ на: [2Tuxoid] Про исходники сайта от SystemV 2011-03-23 22:44:00
avatar
Скрыть

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

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

Tux-oid(*)(2011-03-24 09:54:07)

Mozilla/5.0 (X11; Linux i686; rv:2.0.0) Gecko/20100101 Firefox/4.0
[#] [Добавить метку] [Редактировать] Ответ на: Re: [2Tuxoid] Про исходники сайта от Tux-oid 2011-03-24 09:28:53
avatar
Скрыть

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-ом более-менее:)

SystemV(*)(2011-03-25 22:08:17)

Emacs-w3m/1.4.414 w3m/0.5.3
[#] [Добавить метку] [Редактировать] Ответ на: Re: [2Tuxoid] Про исходники сайта от SystemV 2011-03-25 22:08:17
avatar
Скрыть

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 гораздо сложнее. Я кстати его так и не осилил.

Tux-oid(*)(2011-03-25 23:03:53)

Mozilla/5.0 (X11; Linux i686; rv:2.0.0) Gecko/20100101 Firefox/4.0
[#] [Добавить метку] [Редактировать] Ответ на: Re: [2Tuxoid] Про исходники сайта от Tux-oid 2011-03-25 23:03:53
avatar
Скрыть

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, локальные ветки, команды непривычные:) Хотя, в общем-то, всё понятно.

SystemV(*)(2011-03-25 23:16:12)

Emacs-w3m/1.4.414 w3m/0.5.3
[#] [Добавить метку] [Редактировать] Ответ на: Re: [2Tuxoid] Про исходники сайта от Tux-oid 2011-03-25 23:03:53
avatar
Скрыть

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

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

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

SystemV(*)(2011-03-26 16:13:20)

Emacs-w3m/1.4.414 w3m/0.5.3
[#] [Добавить метку] [Редактировать] Ответ на: Re: [2Tuxoid] Про исходники сайта от SystemV 2011-03-26 16:13:20
avatar
Скрыть

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

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

Tux-oid(*)(2011-03-28 10:21:51)

Mozilla/5.0 (X11; Linux i686; rv:2.0.0) Gecko/20100101 Firefox/4.0
Этот тред читают 1 пользователь:
Анонимных: 1
Зарегистрированных: 0




(c) 2010-2020 LOR-NG Developers Group
Powered by TimeMachine

Valid HTML 4.01 Transitional Правильный CSS!