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

Удалось достаточно просто решить путём использования "кэшируемой" функции comment_num_all. Добавляем в модуль комментариев (comment.module) функцию вычисления последней страцицы:
<?php
function _pages_count($nid)
{
$pages = 0;
$count = comment_num_all( $nid );
$per_page = _comment_get_display_setting( 'comments_per_page' );

if( $count > $per_page )
{
while( $count > $per_page )
{
$pages++;
$count -= $per_page;
}
return 'page='.$pages;
}
else
{
return NULL;
}
}
?>

После чего изменяем все участки, где проявляется ошибка при работе с многостраничными топиками.
• Чиним функцию, которая выбрасывала на первую страницу при редактировании комментария

<?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) .'
'. 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 = '

';
if (user_access('create forum topics')) {
$output .= '
• '. l(t('Post new forum topic.'), "node/add/forum/$tid") .'

';
}
else if ($user->uid) {
$output .= '

• '. t('You are not allowed to post a new forum topic.') .'

';
}
else {
$output .= '

• '. t('login">Login to post a new forum topic.', array('login' => url('user/login', drupal_get_destination()))) .'

';
}
$output .= '

';
// 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 ? '
'. 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 .= '

';
if (user_access('create forum topics')) {
$output .= '
• '. l(t('Post new forum topic.'), "node/add/forum/$tid") .'

';
}
else if ($user->uid) {
$output .= '

• '. t('You are not allowed to post a new forum topic.') .'

';
}
else {
$output .= '

• '. t('login">Login to post a new forum topic.', array('login' => url('user/login', drupal_get_destination()))) .'

';
}
$output .= '

';
// 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 .= '
';
// *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 заменяем главное условие, по которому фильтруются все "небуквы"-"нецифры", под которое в том числе попадает и кириллица.
<?php
// Preserve alphanumerics, everything else becomes a separator
$pattern = '/[^a-zA-Z0-9]+/ ';
$output = preg_replace($pattern, $separator, $output);
?>
Вместо этого ставим более "мягкое" выражение:
<?php
$output = preg_replace("/[_'`!@#$%^&*()|\s\.,?\[\]{}><:;\/\"]+/", "_", $output);
?>
А чтобы при таком условии не получалось "двойных" подчёркиваний на знаках препинания (и пр.) - подрезаем их аналогично вырезанию апострофов:
<?php
// Replace or drop apostrophes based on user settings
$separator = variable_get('pathauto_separator', '-');
$quotes = variable_get('pathauto_quotes', 0);
$output = str_replace("'", ($quotes ? $separator : ''), $string);
// Добавляем ещё и ...
$vowels = array("(", ")", "[", "]", "{", "}", "*", "%", "#", "@", "`", "~", );
$output = str_replace('"', ($quotes ? $separator : ''), $output);
?>
Чтобы полностью навести красоту - запрещаем перевод больших букв в маленькие:
<?php
//$alias = drupal_strtolower($alias);
?>
Конечно, кроме всего прочего нужно подставить файлик (прилагается) "преобразований", где для кириллицы этих преобразований и нет.

Отключив ранее кнопки комментариев к отдельным постам мы сейчас не можем изменить "быстроответный" вид расположения формы ввода ответа (внизу топика), т.к. кнопка посылки комментраия на тему в этом случае будет отсутствовать. Исправляем эту проблему:
<?php
// If enabled, show new comment form.
if (user_access('post comments') && node_comment_mode($nid) == COMMENT_NODE_READ_WRITE && (variable_get('comment_form_location', COMMENT_FORM_SEPARATE_PAGE) == COMMENT_FORM_BELOW)) {
$output .= comment_form_box(array('nid' => $nid), t('Post new comment'));
}
?>
Заменяем на:
<?php
// If enabled, show new comment form.
if( user_access( 'post comments' ) && node_comment_mode( $nid ) == COMMENT_NODE_READ_WRITE )
{
$output .= '

• ';
  $output .= l( t( 'Post new comment' ), 'comment/reply/'.$nid, NULL, NULL, 'comment-form' );
  $output .= '
';
}
?>
В результате внизу топика появится кнопка
• Отправить комментарий
  

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

Ещё один момент - при варианте с формой отправки комментария на отдельной странице (в целом для друпала и к которой мы пришли по этой пишущейся "методичке" в частности) - можно видеть лишь "заглавное" сообщение (пост нода). Такая принудительная тренировка памяти очень быстро начинает раздражать, особенно счастливых обладателей IE, где возврат на предыдущую страницу - для просмотра последних комментраев (а не только нода) весьма не быстр. Потому исправим это, отобразив кроме поста нода ещё и $comments_per_page последних комментариев на странице с формой отпавки сообщения. Для этого придётся вставить достаточно крупный кусок, хотя вообще он достаточно прост, ибо банально выдран из функции comment_render . Итак, ищем в уже не раз упомянутой функции comment_reply место сразу после вывод содержимого поста нода:
<?php
// This is the case where the comment is in response to a node. Display the node.
else if (user_access('access content')) {
$output .= node_view($node);
?>
и после него вставляем:
<?php
// ----------------------------------------------------
$output .= '
...

';
$mode = _comment_get_display_setting('mode');
$order = _comment_get_display_setting('sort');
$comments_per_page = _comment_get_display_setting('comments_per_page');

// Multiple comment view
$query_count = 'SELECT COUNT(*) FROM {comments} WHERE nid = %d';
$query = 'SELECT c.cid as cid, c.pid, c.nid, c.subject, c.comment, c.format, c.timestamp, c.name, c.mail, c.homepage, u.uid, u.name AS registered_name, u.picture, u.data, c.score, c.users, c.thread, c.status FROM {comments} c INNER JOIN {users} u ON c.uid = u.uid WHERE c.nid = %d';

$query_args = array($nid);
if (!user_access('administer comments')) {
$query .= ' AND c.status = %d';
$query_count .= ' AND status = %d';
$query_args[] = COMMENT_PUBLISHED;
}

if ($order == COMMENT_ORDER_NEWEST_FIRST) {
if ($mode == COMMENT_MODE_FLAT_COLLAPSED || $mode == COMMENT_MODE_FLAT_EXPANDED) {
$query .= ' ORDER BY c.timestamp DESC';
}
else {
$query .= ' ORDER BY c.thread DESC';
}
}
else if ($order == COMMENT_ORDER_OLDEST_FIRST) {
if ($mode == COMMENT_MODE_FLAT_COLLAPSED || $mode == COMMENT_MODE_FLAT_EXPANDED) {
$query .= ' ORDER BY c.timestamp';
}
else {

/*
** See comment above. Analysis learns that this doesn't cost
** too much. It scales much much better than having the whole
** comment structure.
*/

$query .= ' ORDER BY SUBSTRING(c.thread, 1, (LENGTH(c.thread) - 1))';
}
}

// Start a form, for use with comment control.
//$result = pager_query($query, $comments_per_page, 0, $query_count, $query_args);
// db_query_range($query, $args, $pager_page_array[$element] * $limit, $limit)
$com_num_all = comment_num_all($nid);
if( $com_num_all < $comments_per_page )
{
$com_num_all = $comments_per_page;
}
$result = db_query_range( $query, $query_args, $com_num_all - $comments_per_page, $comments_per_page );
//$output .= theme('pager', NULL, $comments_per_page, 0);

$divs = 0;
$last_depth = 0;
drupal_add_css(drupal_get_path('module', 'comment') .'/comment.css');
while ($comment = db_fetch_object($result)) {
$comment = drupal_unpack($comment);
$comment->name = $comment->uid ? $comment->registered_name : $comment->name;
$comment->depth = count(explode('.', $comment->thread)) - 1;

if ($mode == COMMENT_MODE_THREADED_COLLAPSED || $mode == COMMENT_MODE_THREADED_EXPANDED) {
if ($comment->depth > $last_depth) {
$divs++;
$output .= '

';
$last_depth--;
}
}
}

if ($mode == COMMENT_MODE_FLAT_COLLAPSED) {
$output .= theme('comment_flat_collapsed', $comment);
}
else if ($mode == COMMENT_MODE_FLAT_EXPANDED) {
$output .= theme('comment_flat_expanded', $comment);
}
else if ($mode == COMMENT_MODE_THREADED_COLLAPSED) {
$output .= theme('comment_thread_collapsed', $comment);
}
else if ($mode == COMMENT_MODE_THREADED_EXPANDED) {
$output .= theme('comment_thread_expanded', $comment);
}
}
for ($i = 0; $i < $divs; $i++) {
$output .= '

';
}
// $output .= theme('pager', NULL, $comments_per_page, 0);
$output = theme('comment_wrapper', $output);
// ===============================================

?>
Если захочется выводить лишь "постоянные" Х последних комментариев - заменяем на него $comments_per_page.

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

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

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

Один из вариантов решения проблемы перехода на многостраничные темы форумов "на уровне ядра" - видится исправления функции отдачи адреса url. В её фрагменте:
<?php
// The special path '' links to the default front page.
if (!empty($path) && $path != '') {
$path = drupal_get_path_alias($path);
$path = drupal_urlencode($path);
if (!$clean_url) {
if (isset($query)) {
return $base . $script .'?q='. $path .'&'. $query . $fragment;
}
else {
// тут для "грязных"...
return $base . $script .'?q='. $path . $fragment;
}
}
else {
if (isset($query)) {
return $base . $path .'?'. $query . $fragment;
}
else {
// тут для "чистых" ссылок - добавляем проверку и вычисление страницы...
return $base . $path . $fragment;
}
}
}
?>
... в случае $query == NULL (иначе - это запрос от пэйджера или др. функции, что уже "посчитала" страницу) делаем проверку полученного алиаса, чтобы он начинался с "/forum/" (или другое - позже можно добавить в настраиваемое значение) - значит это форумная ссылка (иначе будем зря дёргать запросами БД по каждой ссылке). И ежели так - получаем nid и определяем кол-во комментариев с последующим установлением нужного значения в $query. Вот только вопрос - не слишком ли "коряво" выдирать номер нода из строки $path?

Чтобы не забыть, продолжаю отчитываться сам себе же.
Модуль 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 ); на:
<?php
if (!empty($path) && $path != '')
{
if( ! ( substr( $path, 0, 7 ) == 'comment'
|| substr( $path, 0, 8 ) == 'node/add'
|| substr( $path, 0, 5 ) == 'admin'
|| substr( $path, 0, 6 ) == 'logout'
|| substr( $path, 0, 6 ) == 'search'
|| substr( $path, 0, 10 ) == 'userpoints'
|| substr( $path, 0, 15 ) == 'user2userpoints'
|| substr( $path, 0, 10 ) == 'privatemsg' ) )
{
if( substr( $path, 0, 6 ) == 'NoPath' ) // для "ручного" отключения вычисления синонима
{
$path = substr( $path, 6, strlen( $path ) );
}
else
{
$path = drupal_get_path_alias( $path );
}
}
?>
Сделано не слишком оптимально, однако результат на лицо - количество обращений drupal_lookup_path уменьшилось в три-четрыре раза.
Кроме того, всякие выборки, что были дописаны в ходе улучшения юзабельности (например, банальное количество сообщений пользователя в его comment.tpl.php) - правильней сделать "кэшируемыми", например, так:
function user_comments_count( $uid )
<?php
{
static $cache;
if (!isset($cache[$uid]))
{
$cache[$uid] = db_result(db_query('SELECT COUNT(cid) FROM {comments} WHERE uid = %d', $uid)); // AND status = %d , COMMENT_PUBLISHED
}
return $cache[$uid];
}
?>
После всех таких действий, даже без установки альтернативных алгоритмов кэширования количество запросов к БД уменьшилось местами почти на половину, что на глаз ускорило работу (примерно в полтора раза).

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

Кстати, упомянутая проблема с тем, что перейдя на строго плоский развёрнутый вид форума, получается весьма гадская неприятность, когда при правке какого-то поста он "улетает" в самый конец, меняя видимую хронологию (последовательность) сообщений - из-за того, что ему ставится более свежая дата, а отображение комментариев в comment_render как раз сортируется по timestamp - решается убийством обновления даты при редатировании поста в comment_save:
<?php
db_query("UPDATE {comments} SET status = %d, subject = '%s', comment = '%s', format = %d, uid = %d, name = '%s', mail = '%s', homepage = '%s' WHERE cid = %d", $edit['status'], $edit['subject'], $edit['comment'], $edit['format'], $edit['uid'], $edit['name'], $edit['mail'], $edit['homepage'], $edit['cid']);
?>
Как вариант, чтобы таки был виден факт редактирования поста после его размещения можно добавить перед вышеописанной записью нечто типа:
<?php
$edit['comment'] .= '

';
?>
Такая версия не слишком удобна из-за того, что она добавляется в том числе и в самый последний пост (который обычно очень часто редактируется автором при размещении), потому нужно бы добавить условие. Однако как просто решить проблему определения "последнего поста" пока не знаю - как узнаю отпишусь.
п.с. кстати, на мой взгляд, это (обновление даты редактируемого поста) - есть именно баг, ибо сложно увидеть, где по "философии" всей структуры друпала оправдана данная операция, изменяющая цепочку сообщений.

shamaner wrote:

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

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