Переходим с phpbb2 на Drupal. Улучшение (исправление) встроенного форума.

Может труд разбить на части?

Спасибо, очень актуально! Как раз занимаюсь вопросом переноса форума из phpbb на drupal. Пока имеет место проблема некорректной перекодировки данных. Ежели решится - пойду дальше.
Автор абсолютно прав насчет того, что для начала следует натренироваться на пустом друпале. Процесс весьма нетривиален...

очень интересно

Мне удалось победить самую противную проблему - когда при переходе на тему, где много страниц - всегда попадаешь на первую из них
Может быть все же - "код в студию!"?

поддержку кириллических ссылок
И это тоже можно :) Я так понимаю - отредактированный файлик соответствий?

Ещё один "глюк" друпаловского форума - я обновил свой комментарий и он по очерёдности перескочил за сделанное после него сообщение от "On вт, 28/08/2007 - 04:40 vadbars@drupal.org says:"). Видимо, надо будет убрать обновление даты при апдэйте статистики, если уже были комментарии после первой версии комментария.

пугает только количество запросов из-за Pathauto

Удалось достаточно просто решить путём использования "кэшируемой" функции comment_num_all. Добавляем в модуль комментариев (comment.module) функцию вычисления последней страцицы:

После чего изменяем все участки, где проявляется ошибка при работе с многостраничными топиками.
• Чиним функцию, которая выбрасывала на первую страницу при редактировании комментария
<?php
function comment_form_submit($form_id, $form_values) {
  $form_values = _comment_form_submit($form_values);
  if ($cid = comment_save($form_values)) {
    return array('node/'. $form_values['nid'], _pages_count($form_values['nid']), "comment-$cid"); // замеменено NULL
  }
}
?>
• В блоке Последних комментарии тоже будет ошибка для таких топиков - исправляем.
<?php
function theme_comment_block() {
  $items = array();
  foreach (comment_get_recent() as $comment) {
    // Заменяем второй NULL ранее написанной функцией
    $items[] = l(substr($comment->comment, 0, strrpos((substr($comment->comment, 0, 160)), " ")) . ' ...', 'node/'. $comment->nid, NULL, _pages_count($comment->nid), 'comment-'. $comment->cid) .'<br />'. t('@time ago', array('@time' => format_interval(time() - $comment->timestamp)));
    // "Родная" $items[]  видоизменён - по способу http://drupal.ru/node/7262
  }
  if ($items) {
    return theme('item_list', $items);
  }
}
?>
• Кроме комментариев делаем аналогичное в forum.module с функцией отображения списка топиков
<?php
function theme_forum_topic_list($tid, $topics, $sortby, $forum_per_page) {
  global $user;
  // set "New topic"-link on sub-forum page (move from main and subcontainers)        *romby* 
    $output = '<ul>';
    if (user_access('create forum topics')) {
      $output .= '<li>'. l(t('Post new forum topic.'), "node/add/forum/$tid") .'</li>';
    }
    else if ($user->uid) {
      $output .= '<li>'. t('You are not allowed to post a new forum topic.') .'</li>';
    }
    else {
      $output .= '<li>'. t('<a href="@login">Login</a> to post a new forum topic.', array('@login' => url('user/login', drupal_get_destination()))) .'</li>';
    }
    $output .= '</ul>';
  // end changes 
  $output .= theme('pager', NULL, $forum_per_page, 0);

  global $forum_topic_list_header;
  $rows = array();
  if ($topics) {

    foreach ($topics as $topic) {
      // folder is new if topic is new or there are new comments since last visit
      
        // *romby* - add '$topic->tid != 0' for deleted topic with checked "move-flag"
      if ( ( $topic->tid != 0 ) && ( $topic->tid != $tid ) )
        // end of changes
        {
            $rows[] = array(
              array('data' => theme('forum_icon', $topic->new, $topic->num_comments, $topic->comment_mode, $topic->sticky), 'class' => 'icon'),
              array('data' => check_plain($topic->title), 'class' => 'title'),
              array('data' => l(t('This topic has been moved'), "forum/$topic->tid"), 'colspan' => '3')
            );
        }
      else {
          // *romby* - add _pages_count function
          $pages = _pages_count($topic->nid);
        $rows[] = array(
          array('data' => theme('forum_icon', $topic->new, $topic->num_comments, $topic->comment_mode, $topic->sticky), 'class' => 'icon'),
          array('data' => l($topic->title, "node/$topic->nid", NULL, $pages), 'class' => 'topic'), // <- $pages here
          array('data' => $topic->num_comments . ($topic->new_replies ? '<br />'. l(format_plural($topic->new_replies, '1 new', '@count new'), "node/$topic->nid", NULL, $pages, 'new') : ''), 'class' => 'replies'), // <- $pages here
          array('data' => _forum_format($topic), 'class' => 'created'),
          array('data' => _forum_format(isset($topic->last_reply) ? $topic->last_reply : NULL), 'class' => 'last-reply')
        );
        // end of changes
      }
    }
  }
  // change '=' to '.=' for output previos "New topic"-link    *romby*
  $output .= theme('table', $forum_topic_list_header, $rows);
  // end changes 
  $output .= theme('pager', NULL, $forum_per_page, 0);
  // set once "New topic"-link on sub-forum page        *romby* 
$output .= '<ul>';
if (user_access('create forum topics')) {
  $output .= '<li>'. l(t('Post new forum topic.'), "node/add/forum/$tid") .'</li>';
}
else if ($user->uid) {
  $output .= '<li>'. t('You are not allowed to post a new forum topic.') .'</li>';
}
else {
  $output .= '<li>'. t('<a href="@login">Login</a> to post a new forum topic.', array('@login' => url('user/login', drupal_get_destination()))) .'</li>';
}
$output .= '</ul>';
  // end changes 

  return $output;
}
?>
• Если используется "родной" (не переопределённый) трэкер ("Последние сообщения") - исправляем и его.
<?php  while ($node = db_fetch_object($result)) {
    // Determine the number of comments:
    $comments = 0;
    if ($node->comment_count) {
      $comments = $node->comment_count;

      if ($new = comment_num_new($node->nid)) {
        $comments .= '<br />';
    // *romby* remove NULL with _pages_count 
        $comments .= l(format_plural($new, '1 new', '@count new'), "node/$node->nid", NULL, _pages_count($node->nid), 'new');
    // end of changes
      }
    }
    $rows[] = array(
      check_plain(node_get_types('name', $node->type)),
    // *romby* remove NULL with _pages_count 
      l($node->title, "node/$node->nid", NULL, _pages_count($node->nid)) .' '. theme('mark', node_mark($node->nid, $node->changed)),
    // end of changes
      theme('username', $node),
      array('class' => 'replies', 'data' => $comments),
      t('!time ago', array('!time' => format_interval(time() - $node->last_updated)))
    );
  }
?>

Для "русифицирования" ссылок в Pathauto заменяем главное условие, по которому фильтруются все "небуквы"-"нецифры", под которое в том числе попадает и кириллица.

Отключив ранее кнопки комментариев к отдельным постам мы сейчас не можем изменить "быстроответный" вид расположения формы ввода ответа (внизу топика), т.к. кнопка посылки комментраия на тему в этом случае будет отсутствовать. Исправляем эту проблему:

Спасибо! Будем посмотреть.

На drupal.org не хотите отписаться? Если проблема с некорректной ссылкой на последний комментарий действительно решена, то это порадует очень многих. Эта багофича тянется с самых ранних версий.

На drupal.org не хотите отписаться?
Таки это тоже не слишком "культурное" решение - изначально было желание исправить это на уровне ядра, чтобы не патчить все модули подряд. Просто пока торопят возмущённые пользователи форума (привыкшие к функционалу phpbb), потому решил сделать сначала хоть какое-то приличное и работающее решение. Позже попробую сделать "правильно" и уже по (любому) результату отпишусь.

Ещё один момент - при варианте с формой отправки комментария на отдельной странице (в целом для друпала и к которой мы пришли по этой пишущейся "методичке" в частности) - можно видеть лишь "заглавное" сообщение (пост нода). Такая принудительная тренировка памяти очень быстро начинает раздражать, особенно счастливых обладателей IE, где возврат на предыдущую страницу - для просмотра последних комментраев (а не только нода) весьма не быстр. Потому исправим это, отобразив кроме поста нода ещё и $comments_per_page последних комментариев на странице с формой отпавки сообщения. Для этого придётся вставить достаточно крупный кусок, хотя вообще он достаточно прост, ибо банально выдран из функции comment_render . Итак, ищем в уже не раз упомянутой функции comment_reply место сразу после вывод содержимого поста нода:

Тут найдено решение гадской проблемы Пометить все как прочитанные. Позже нужно будет "встроить" в форум, ибо патчить каждую тему сайта для вывода кнопки ни разу не кошерно.

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

...
0.19	1	drupal_lookup_path	SELECT dst FROM url_alias WHERE src = 'user/2410'
0.2	1	drupal_lookup_path	SELECT dst FROM url_alias WHERE src = 'user/2783'
0.14	1	comment_num_all	SELECT comment_count FROM node_comment_statistics WHERE nid = 14077
0.17	1	drupal_lookup_path	SELECT dst FROM url_alias WHERE src = 'node/14077'
0.19	1	drupal_lookup_path	SELECT dst FROM url_alias WHERE src = 'user/378'
0.14	1	comment_num_all	SELECT comment_count FROM node_comment_statistics WHERE nid = 13962
0.17	1	drupal_lookup_path	SELECT dst FROM url_alias WHERE src = 'node/13962'
0.2	1	drupal_lookup_path	SELECT dst FROM url_alias WHERE src = 'user/1934'
0.16	1	comment_num_all	SELECT comment_count FROM node_comment_statistics WHERE nid = 14038
0.17	1	drupal_lookup_path	SELECT dst FROM url_alias WHERE src = 'node/14038'
...

Один из вариантов решения проблемы перехода на многостраничные темы форумов "на уровне ядра" - видится исправления функции отдачи адреса url. В её фрагменте:

Чтобы не забыть, продолжаю отчитываться сам себе же.
Модуль FormFilter при всей очевидной полезности заставил помучаться в тонком моменте - если с помощью него скрыть отображение выбора категорий форума, то в поле forum->tid для создаваемой темы будет занесён 0. В результате получается совсем неочевидный глюк - с одной стороны, создаваемая тема получит в node принадлежность к форуму, но в нём появится как перенесённая неизвестно куда (tid = 0). Я по глупости сначала добавил условие, чтобы форумы с tid = 0 не считались перенесёнными, в результате всё как бы работали, но - новые сообщения считались неверно и при удалении такой темы не получилось бы отметить её как перенесённую. В общем, короче - будьте осторожны и не злоупотребляйте модулем FormFilter, иначе придётся как мне :) (обновлять все созданные темы с tid=0 при включенном поле выбора подфорума - после обновления tid получили требуемое значение и всё образовалось).

По-прежнему не удаётся включить кэширование форума из-за вылезающией ошибки
Warning: gzinflate() [function.gzinflate]: data error in /home/rombyportal/public_html/includes/bootstrap.inc on line 575
Ошибка вылазит у некоторых незалогиненных пользователей, использующих IE. Все найденные подобные ошибки на drupal.org остались без чёткого решения, к сожалению. Максимально конкретный вариант решения был здесь, но и он не помог. :/
В общем, в результате пришлось хорошенько подумать над оптимизацией. После исследования загрузки страниц с помощью модуля Devel были убиты модули, которые не представляли "принципиальной" ценности, но при этом напрягали БД многочисленными запросами, отражающие явно плохо написанный код. Если кто зачем-то поставит себе модуль User Activity - гляньте подробности его работы, они могут очень не понравится. :)
Кроме удаления ещё других "непринципиальных" и нагружающих "лишними" запросами модулей, стоит задуматься над оптимизацией кэша.
При использовании модуля Pathauto на каждую ссылку ищется алиас. В результате получаем, пусть мелкие, но самые многочисленные запросы. При чём важно, что они делаются "без разбора" - в том числе на те ссылки, которые не имеют синонимов. А ведь наиболее частые ссылки у каждого из комментариев - ответить, удалить, редактировать, отправить сообщение и тп. - обычно не использую синонимы. Зачем же зря напрягать и делать запросы для таких путей? В общем, в таком случае есть смысл подредактировать функцию url, заменив вызов $path = drupal_get_path_alias( $path ); на:

Кстати, как помните, мы реализовали "русские" ссылки, однако полностью их "русскими" назвать не получалось из-за того, что автопреобразование из "процентиков" в кириллицу умеет лишь Опера и FF с дополнительным плагином. Так вот, если в вышеупомянутой функции url закомментировать строчку $path = drupal_urlencode($path);, то получим "честные" русские ссылки уже в любом броузеры. Местами такой не совсем "честный" подход даёт глюки (обычно при частом нажатии кнопки назад в IE русские буквы заменяются на коряболки), однако, во-первых, при этом чаще нормально срабатывает, во-вторых, это правильней отнести к "нестандартным" ситуациям - во всём остальном получается красота.
Хотя нет, не во всём. :) Модуль "альтернативного" кэширования Boost использующий сохранение сгенерированных страниц на диске - сохраняет русские названия некорректно, потому от него пришлось отказаться.

Очень интересные исследования и работа над Друпалом...
Мои поздравления, очень нужная тема... насчёт кэша, могу посоветовать вновь оживший fastpath_fscache... он, по моему, работает получше, чем Буст, во всяком случае проблем с зарегистрированными пользователями и анонимными - не имеет...

Кстати, упомянутая проблема с тем, что перейдя на строго плоский развёрнутый вид форума, получается весьма гадская неприятность, когда при правке какого-то поста он "улетает" в самый конец, меняя видимую хронологию (последовательность) сообщений - из-за того, что ему ставится более свежая дата, а отображение комментариев в comment_render как раз сортируется по timestamp - решается убийством обновления даты при редатировании поста в comment_save:

Если проблема с некорректной ссылкой на последний комментарий действительно решена, то это порадует очень многих. Эта багофича тянется с самых ранних версий.
Вадим, на ДО уже давно видел патч для этого, правда ссылку не дам - не добавил в закладки(((

Первая часть сообщения - хорошая, вторая - не очень. :) "Что ж, будем искать с перламутровыми пуговицами" (с) Бриллиантовая рука.

Может и не в тему чуть, но я увидел, что об uie не очень хорошего мнения, лично я считаю его единственным форумом, для которого не надо дополнительные модули для реализации моста между друпой и движком форума. Он взят за основу и будет развиваться, проблемы в нем конечно есть и функционала маловато, но мне например нравится вобла и функции воблы будут потиху переносится в этот форум. Все остальные форумы это костыли увы, сделаны не как дополнение к движку а как отдельный нормально функционирующий сайт-форум.

Есть такой баг (правда кажется создатель модуля считает его фичей): у всех анонимных постеров при переносе пропадают имена (все становятся Anonymous).
Решается так:
находим:

      //construct the node
      $comment = array(
        'pid' => $pid,
        'nid' => $nid,
        'uid' => $uid,
        'subject' => $post->post_subject,
        'comment' => $post->post_text,
        'hostname' => $hostname,
        'timestamp' => $post->post_time
      );

      //construct the node
      $comment = array(
      	'name' => $post->post_username,
        'pid' => $pid,
        'nid' => $nid,
        'uid' => $uid,
        'subject' => $post->post_subject,
        'comment' => $post->post_text,
        'hostname' => $hostname,
        'timestamp' => $post->post_time
      );

ещё исправление для phpbb2drupal - как сделать чтоб пустые темы не всплывали после импорта вверх, а оставались на положенных местах:

находим:

заменяем на:

находим:

вставляем после этого:

apple_rom, возникла проблема с применением кода, который отвечает за нормальные ссылки на последние комментарии.

Функцию в comment.module добавил, кроме того изменил код в блоке последних комментариев, который ты привел.

Посмотрел - в блоке у всех комментариев значится "..." вместо сабжа, а также ссылки в блоке на комментарии глючат - везде указывается page=1, даже у комментариев, которые на первой странице, хотя у них должно стоять page=0.

Как решить проблему?

Мнение уместно в случае создания (нового) сайта/форума, а не переноса уже имеющегося phpbb2-форума. Чего не поддерживает и, по заверению автора, не будет поддерживать, uie.

brooho
Большое спасибо за добавления - вмести с Вашими (и изложенным выше мною) поправками модуль phpbb2drupal станет, наконец, относительно полноценным.

К сожалению, возниклим проблемы при исправлении просмотра последней страницы вместо первой... Друпал 5 версии. Модулей comment.module и forum.module в папке с модулями аж две штуки - в самой папке и в подпапках comment и module. Какие из них нужно править и в какую часть текста размещать участки кода?

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

Возможно, кому-то понадобится

Длинный пост на друпал.орге о многостраничных комментариях. http://drupal.org/node/6162

Нда, видно что это проблема уже долго тянется...

К сожалению, из-за огромного количества патчей, решений, я так и не смогла выбрать себе наиболее нормальное. Если кто-нибудь разбирающийся посмотрит, напишите, пж-ста, какой там наиболее нормальный!

интересно, как в 6 все эти хаки и хуки

Да, очень даже полезная статья. Под шестерку вроде тоже есть модуль. Вообще очень интересный опыт!

Замечательная статья, но почему ни слова об advanced forum?
Он сразу решает половину перечисленных и неперечисленных проблем :) Во время написания статьи его наверное еще не было?

- есть в AF

- решено в AF

плюс более симпатичный вид главной страницы форумов и возможность менять темы оформления.

А вот проблемы с многостраничными комментариями до сих пор актуальны, автору спасибо, всё работает на 5.8.

Может быть кто подскажет как аналогично доработать comment_notify.module ? В каком месте там надо заменить NULL на функцию вычисления последней страницы, чтобы уведомление приходило с правильной ссылкой? Моих собственных способностей хватает только на copy-paste :(

адвансед говорят сильно нагружает систему
проверить не могу.
а вообще, щас актуально решения под 6-ю версию