Линк друпала и punbb (продолжение)
Прислано: pihel
сб, 05/04/2008 - 16:23
Усовершенствовал модуль связи с панбб (продолжение темы http://drupal.ru/node/11593 )
1. Добавлена возможность анонса добавленного нода на форуме (Создается новая тема с первым постом)
2. Редактирование форумного поста при изменении нода
3. Удаление поста, при удалении нода
3.а. Вопрос пользователям: удалять форумное сообщение, при удалении нода?
3.б. Не обновляются данные форума last_post, last_post_id, last_poster
4. Добавлены параметры доступа к топике в результирующей переменной $node
5. Добавлены административные настойки, включающие/выключающие возможности модуля
6. Добавлена очистка бд при удалении модуля
полный ченжлог в ридми
жду замечаний и предложений
| Прикрепленный файл | Размер |
|---|---|
| punbb.zip | 7.29 кб |
- pihel's blog
- Для комментирования войдите или зарегистрируйтесь
Вааащее сырой и недоработанный модуль.
1) Постоянно выскакивает ошибка (и не по разу) что конфиг не прописан, хотя я только активировал модуль.
2) Нет прав, анонимусам выводится форма постинга
3) Анонимусы не могут на форум ничего отпостить, выводится ошибка SQL (Column 'poster_id' cannot be null query)
4) Не учитываются форматы ввода
5) Не учитываются скрытые форумы (то бишь модераторы тока должны бы видеть один раздел для постинга)
6) SET, SELECT, DELETE, WHERE, AND - по правилам, ваще-то пишутся большими :)
7) Много db_query(INSERT), результат которых не проверяется
8) В некоторых местах, в SQL-запросах {table_name}, в других - просто table_name.
Комменты на русском - тру :)
Вообще на кой черт автор сиё затеял - непонятно. Punbb 1.2 и друпал вещи весьма далекие. Ради только анонсов новостей с сайта на форум - стоило ли?
А релиз 1.3 уже не за горами, а там и хуки.
- Для комментирования войдите или зарегистрируйтесь
спасибо, будем править
- Для комментирования войдите или зарегистрируйтесь
по пункту 6, где такие правила описаны?
затеял я это восновном изза единой регистрации, а анонсы дело второе
- Для комментирования войдите или зарегистрируйтесь
1. Убрал показ этой ошибки, но после установки модуля сразу выдается ссылка на страницу настроек, где надо задать путь до форума
2. 3. в разработке
4. Можно поподробней, каким образом их учитывать? разве форматы обрабатываются не при выдаче из бд, а не при записи ?
5. сделано (страшный скл запрос в 20 строчек ))
7. добавил куда можно проверки
8. везде сделал {}, не знаю добавит ли это переносимости между БД
- Для комментирования войдите или зарегистрируйтесь
4 - имею в виду, что надо бы предупреждение юзеру выдавать. Что если он пишет на Full HTML / php, то это будет по-разному выводиться в друпале и в пунбб. Или поставить bb-code для друпала, как формат ввода по умолчанию.
6 - где описано хз, но это правила хорошего кода, как отступы. Посмотри документацию, хотя бы на http://www.mysql.ru/ - там ключевые слова везде в верхнем регистре.
8 - тоже одно из правил. Где обрабатывается - не припомню, но {node} учитывает префикс, указанный в settings.php, а просто node - не учитывает.
На орге запостил?
Помню, что на пунбб уже года два спрашивают интеграцию. На друпале вроде тоже проскакивали вопросы о интеграции, но не так активно, ибо других форумов хватает.
- Для комментирования войдите или зарегистрируйтесь
щас неделю некогда будет делать, выложу промежуточный вариант, может по нему будут какието новые дополнительные замечания
сейчас не сделано: пункты 2, 4 и нет перевода
- Для комментирования войдите или зарегистрируйтесь
слехка обновил модуль
добавилась функциональность: теперь форум может находится в любом месте, а не на этом же домене
убрал ненужные настройки, добавил нужные :)
осталась одна проблема: форум и друпал должны стоять на одной базе. Этого терпимо или нужно сделать чтобы они могли стоять на разных ?
- Для комментирования войдите или зарегистрируйтесь
Что-то я не нахожу топика на друпал.орг =\ Там народ активнее бы подключился, немало кто хочет перенести форум на друпал, а тут - интеграция есть.
Плюс пора бы на 6.х версию переносить.
Базы - некритично, но желательно.
- Для комментирования войдите или зарегистрируйтесь
О, сообщение, а на почту чтото не пришло (
ммм... хз, у меня с инглишом худо
да думаю довести доума, а потом можно в разных местах запостить, если конечно меня ктонить не опередит с аналогичным модулем
- Для комментирования войдите или зарегистрируйтесь
а есть ли тоже самое, но для шестого друпала? ;)
- Для комментирования войдите или зарегистрируйтесь
+1 за шестой друпаль )
- Для комментирования войдите или зарегистрируйтесь
+2 за шестой друпаль )
- Для комментирования войдите или зарегистрируйтесь
Да хорошо бы для шестой версии сделать.
- Для комментирования войдите или зарегистрируйтесь
Действительно есть необходимость? а то я подзабросил все дела...
и еще может подскажете: на каких-нибудь сайтах используется?
- Для комментирования войдите или зарегистрируйтесь
Очень нужен, особенно автопостинг на форуме, а то привязать к phpbb привязал, а автопостинга нет :(
И еще «Эта версия несовместима с версией 6.14 ядра Drupal.» Жаль :-(
- Для комментирования войдите или зарегистрируйтесь
Понятно, я тут опять возвращаюсь к друпал. Будет время - переделаю, думаю не так и много там переписывать.
- Для комментирования войдите или зарегистрируйтесь
я тут опять возвращаюсь к друпал.
Вот это правильно.
- Для комментирования войдите или зарегистрируйтесь
Изменил модуль.
Переработал на 6 версия друпала. Плюс за год авторизация и регистрация у пунбб сильно изменилась - произвел необходимые изменения. Соответственно со старыми форумами работать не будет.
Скачать можно тут:
http://skahin.blogspot.com/2009/11/punbb-13-drupal-6.html
- Для комментирования войдите или зарегистрируйтесь
Исправил следующие ошибки в модуле:
1. Создание пользователя форума с пустым паролем.
2. Ошибка авторизации при привязке cookie к домену (проявлялось на некоторых браузерах).
3. Установка языка форума для пользователя, основываясь на глобальных настройках.
Спасибо golden[a]speechpro.com (Golden) за тестирование.
Ссылка для скачивания прежняя (http://skahin.blogspot.com/2009/11/punbb-drupal.html)
- Для комментирования войдите или зарегистрируйтесь
Комментарии