Линк друпала и punbb (продолжение)

Аватар пользователя pihel

Усовершенствовал модуль связи с панбб (продолжение темы http://drupal.ru/node/11593 )
1. Добавлена возможность анонса добавленного нода на форуме (Создается новая тема с первым постом)
2. Редактирование форумного поста при изменении нода
3. Удаление поста, при удалении нода
3.а. Вопрос пользователям: удалять форумное сообщение, при удалении нода?
3.б. Не обновляются данные форума last_post, last_post_id, last_poster
4. Добавлены параметры доступа к топике в результирующей переменной $node
5. Добавлены административные настойки, включающие/выключающие возможности модуля
6. Добавлена очистка бд при удалении модуля

полный ченжлог в ридми

жду замечаний и предложений

ВложениеРазмер
Иконка пакета punbb.zip7.29 КБ
Версия Drupal:
0 Thanks

Комментарии

Аватар пользователя KCEOH
KCEOH 10 лет назад

Вааащее сырой и недоработанный модуль.

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 уже не за горами, а там и хуки.

Аватар пользователя pihel
pihel 10 лет назад

спасибо, будем править

Аватар пользователя pihel
pihel 10 лет назад

по пункту 6, где такие правила описаны?

затеял я это восновном изза единой регистрации, а анонсы дело второе

Аватар пользователя pihel
pihel 10 лет назад

1. Убрал показ этой ошибки, но после установки модуля сразу выдается ссылка на страницу настроек, где надо задать путь до форума
2. 3. в разработке
4. Можно поподробней, каким образом их учитывать? разве форматы обрабатываются не при выдаче из бд, а не при записи ?
5. сделано (страшный скл запрос в 20 строчек ))
7. добавил куда можно проверки
8. везде сделал {}, не знаю добавит ли это переносимости между БД

Аватар пользователя KCEOH
KCEOH 10 лет назад

4 - имею в виду, что надо бы предупреждение юзеру выдавать. Что если он пишет на Full HTML / php, то это будет по-разному выводиться в друпале и в пунбб. Или поставить bb-code для друпала, как формат ввода по умолчанию.
6 - где описано хз, но это правила хорошего кода, как отступы. Посмотри документацию, хотя бы на http://www.mysql.ru/ - там ключевые слова везде в верхнем регистре.
8 - тоже одно из правил. Где обрабатывается - не припомню, но {node} учитывает префикс, указанный в settings.php, а просто node - не учитывает.

На орге запостил?
Помню, что на пунбб уже года два спрашивают интеграцию. На друпале вроде тоже проскакивали вопросы о интеграции, но не так активно, ибо других форумов хватает.

Аватар пользователя pihel
pihel 10 лет назад

щас неделю некогда будет делать, выложу промежуточный вариант, может по нему будут какието новые дополнительные замечания
сейчас не сделано: пункты 2, 4 и нет перевода

Аватар пользователя pihel
pihel 2 года назад

слехка обновил модуль
добавилась функциональность: теперь форум может находится в любом месте, а не на этом же домене
убрал ненужные настройки, добавил нужные :)
осталась одна проблема: форум и друпал должны стоять на одной базе. Этого терпимо или нужно сделать чтобы они могли стоять на разных ?

Аватар пользователя KCEOH
KCEOH 9 лет назад

Что-то я не нахожу топика на друпал.орг =\ Там народ активнее бы подключился, немало кто хочет перенести форум на друпал, а тут - интеграция есть.

Плюс пора бы на 6.х версию переносить.

Базы - некритично, но желательно.

Аватар пользователя pihel
pihel 9 лет назад

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

Аватар пользователя Питон_Ъ
Питон_Ъ 9 лет назад

а есть ли тоже самое, но для шестого друпала? ;)

Аватар пользователя Yas
Yas 9 лет назад

+1 за шестой друпаль )

Аватар пользователя Paldru
Paldru 9 лет назад

+2 за шестой друпаль )

Аватар пользователя milsa
milsa 9 лет назад

Да хорошо бы для шестой версии сделать.

Аватар пользователя pihel
pihel 8 лет назад

Действительно есть необходимость? а то я подзабросил все дела...
и еще может подскажете: на каких-нибудь сайтах используется?

Аватар пользователя Balda
Balda 8 лет назад

Очень нужен, особенно автопостинг на форуме, а то привязать к phpbb привязал, а автопостинга нет :(
И еще «Эта версия несовместима с версией 6.14 ядра Drupal.» Жаль :-(

Аватар пользователя pihel
pihel 8 лет назад

Понятно, я тут опять возвращаюсь к друпал. Будет время - переделаю, думаю не так и много там переписывать.

Аватар пользователя volocuga
volocuga 8 лет назад
pihel написал:
я тут опять возвращаюсь к друпал.

Вот это правильно.

Аватар пользователя pihel
pihel 8 лет назад

Изменил модуль.
Переработал на 6 версия друпала. Плюс за год авторизация и регистрация у пунбб сильно изменилась - произвел необходимые изменения. Соответственно со старыми форумами работать не будет.

Скачать можно тут:
http://skahin.blogspot.com/2009/11/punbb-13-drupal-6.html

Аватар пользователя pihel
pihel 8 лет назад

Исправил следующие ошибки в модуле:
1. Создание пользователя форума с пустым паролем.
2. Ошибка авторизации при привязке cookie к домену (проявлялось на некоторых браузерах).
3. Установка языка форума для пользователя, основываясь на глобальных настройках.
Спасибо golden[a]speechpro.com (Golden) за тестирование.

Ссылка для скачивания прежняя (http://skahin.blogspot.com/2009/11/punbb-drupal.html)