Оптимизация модуля phpBB2 v. 0.21 b11 (!!! из нашего архива на сайте !!!)

[Usama]

Сообщение от pan Дык вроде и так работает, у меня же пятерка

а у меня дебагер вылез, кстати версия твоя форума ))) посмотрел, ругался на 439 строку Could not obtain post/user information.

DEBUG MODE

SELECT u.username, u.user_id, u.user_posts, u.user_from, u.user_website, u.user_email, u.user_icq, u.user_aim, u.user_yim, u.user_regdate, u.user_msnm, u.user_viewemail, u.user_rank, u.user_sig, u.user_sig_bbcode_uid, u.user_avatar, u.user_avatar_type, u.user_allowavatar, u.user_avatar_description, u.user_birthday, u.user_next_birthday_greeting, u.user_zodiac, u.user_gender, u.user_age_flag, p.*, pt.post_text, pt.post_subject, pt.bbcode_uid, u.user_reputation, u.user_reputation_plus, u.user_warnings, u.user_level FROM promo_phpbb_posts p, promo_phpbb_users u, promo_phpbb_posts_text pt WHERE p.topic_id = 65 AND pt.post_id = p.post_id AND u.user_id = p.poster_id ORDER BY p.post_time ASC LIMIT 225, 15

Line : 439
File : viewtopic.php

так что пока оставил преждний... подозреваю проблема с $_GET, так как в коммоне (от форума) как-то переименовывают вроде геты все.

У тебя Олег форум весь переведен в надлежащий вид $_GET ?

[pan]

Сообщение от Usama У тебя Олег форум весь переведен в надлежащий вид $_GET

ну видимо да, раз работает

[LARK]

pan Usama Вы сейчас о чем? Об оптимизации или о PHP 5? Если о последнем, то в отдельную тему. Ошибка появилась после установки оптимизации (данная строка вообще к оптимизации не имеет отношения) ?

Оптимизация проверялась на 5.2.4, все корректно работало.

[Usama]

Сообщение от LARK Ошибка появилась после установки оптимизации

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

[LARK]

Замени 439 строку на

PHP код:

message_die(GENERAL_ERROR, "Could not obtain post/user information.", '', __LINE__, __FILE__, $db->error()); 


и включи отображение ошибок.

Ты viewtopic.php перед эти изменял? Может дело в каких то твоих модах?

Просто данный запрос никаким боком не относится к оптимизации, а та его не затрагивает.

[Usama]

ответ прост многие столбцы по другому в базе назывались, а пару не было.

Обновил у себя все привел к соотвесвию как у Михаила в SVN.

и нашел такой баг. Что банить и предупреждения выдавали у меня пользователи и анонимные наровне с админом.
нашел вот что:
viewtopic.php строка 1341.

PHP код:

    if ($board_config['warnings_enabled'])
    {
      $personal_auth = reputation_auth($is_auth, $userdata, $postrow[$i], true);

      $user_warnings = reputation_warnings($postrow[$i], !empty($banned[$poster_id]), $personal_auth);

      if (!isset($post_warnings[$postrow[$i]['post_id']]))
      {
        if ($personal_auth['auth_warn'])
        {
          $temp_url = "reputation.php?mode=warning&" . POST_POST_URL . "=" . $postrow[$i]['post_id'] . "&sid=" . $userdata['session_id'];
          $warn_img = '<a href="' . $temp_url . '"><img src="' . $images['warn_user'] . '" alt="' . $lang['reputation_warn_user'] . '" title="' . $lang['reputation_warn_user'] . '" border="0" /></a>';
          $warn = '<a href="' . $temp_url . '">' . $lang['reputation_warn'] . '</a>';
        }

        if ($personal_auth['auth_ban'])
        {
          $temp_url = "reputation.php?mode=ban&amp;" . POST_POST_URL . "=" . $postrow[$i]['post_id'] . "&amp;sid=" . $userdata['session_id'];
          $ban_img = '<a href="' . $temp_url . '"><img src="' . $images['ban_user'] . '" alt="' . $lang['reputation_ban_user'] . '" title="' . $lang['reputation_ban_user'] . '" border="0" /></a>';
          $ban = '<a href="' . $temp_url . '">' . $lang['reputation_ban'] . '</a>';
        }
      }
    } 


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

PHP код:

if ($board_config['warnings_enabled'])
    {
      $personal_auth = reputation_auth($is_auth, $userdata, $postrow[$i], true);

      $user_warnings = reputation_warnings($postrow[$i], !empty($banned[$poster_id]), $personal_auth);

      if ( $userdata['user_id'] != $postrow[$i]['user_id'] && $postrow[$i]['user_level'] != ADMIN && $postrow[$i]['user_level'] != MOD && $postrow[$i]['user_level'] != GLOBAL_MOD && $postrow[$i]['user_level'] != MAIN_MOD && $is_auth['auth_mod'] )
      {
              if ($personal_auth['auth_warn'])
        {
          $temp_url = "reputation.php?mode=warning&" . POST_POST_URL . "=" . $postrow[$i]['post_id'] . "&sid=" . $userdata['session_id'];
          $warn_img = '<a href="' . $temp_url . '"><img src="' . $images['warn_user'] . '" alt="' . $lang['reputation_warn_user'] . '" title="' . $lang['reputation_warn_user'] . '" border="0" /></a>';
          $warn = '<a href="' . $temp_url . '">' . $lang['reputation_warn'] . '</a>';
        }

        if ( $board_config['reputation_moderators_can_ban'] == 1 )
        {
          $temp_url = "reputation.php?mode=ban&amp;" . POST_POST_URL . "=" . $postrow[$i]['post_id'] . "&amp;sid=" . $userdata['session_id'];
          $ban_img = '<a href="' . $temp_url . '"><img src="' . $images['ban_user'] . '" alt="' . $lang['reputation_ban_user'] . '" title="' . $lang['reputation_ban_user'] . '" border="0" /></a>';
          $ban = '<a href="' . $temp_url . '">' . $lang['reputation_ban'] . '</a>';
        }
      }
    } 


[LARK]

Ну вот видишь - у тебя просто другая версия модуля была. Оптимизация делалась на основе модуля, который выложен у нас в архиве.

[Michael-XIII]

Ай спасибо

P.S. Наконец-то пыха стала кому-то нужна кроме меня ;-)

[Usama]

Michael-XIII
Стоит посмотреть все файлы где идет вызов карточек.
Я сейчас его поставил как основным у себя на портале, так что тестеров загалаза )))

еще бы хак реализовать предпросмотр темы(первого поста, как на VB и на IPB реализовано).

[Usama]

Michael-XIII
Дырочка сохранилась в /forum/reputation.php?mode=ban&p=10328&sid=

Там нет проверки на принадлежность возможности бан ставить!!!
Нужно там запихать проверку на разрешение банить и ставить предупреждения!