Отправлять на мыло или нет - тебе решать, конечно)

Код

if ( $this->ipsclass->vars['reportpost_method'] == 'email' OR $data['members_disable_pm']

Это условие нужно убрать: в большинстве случаев оно не будет выполняться. Оно только для оптравки на мыло нужно.
И если не собираешься отправлять на мыло, то можно вообще выкинуть этот кусок:

Код

$this->ipsclass->DB->simple_construct( array( 'select' => 'id, members_disable_pm', 'from' => 'members', 'where' => "id={$post['author_id']}" ) ); // идет запрос к базе, e-mail убираю с целью уменбшить число запросов $this->ipsclass->DB->simple_exec(); $data = $this->ipsclass->DB->fetch_row();

И вместо $data['id'] использовать $post['author_id'].

Когда проверишь, расскажи

Неа, что-то не работает. Ошибку сказать не могу, потому как pHP 5 не выводит ошибки
Хотя может я что-то не туда вставил. Вот модифицированная функция удаления поста:

Код

function delete_post()         {                 // Get this post id.                 $this->ipsclass->input['p'] = intval($this->ipsclass->input['p']);                 $this->ipsclass->DB->simple_construct( array( 'select' => 'pid, author_id, post_date, new_topic', 'from' => 'posts', 'where' => "topic_id={$this->topic['tid']} and pid={$this->ipsclass->input['p']}" ) );                 $this->ipsclass->DB->simple_exec();                 if ( ! $post = $this->ipsclass->DB->fetch_row() )                 {                         $this->moderate_error();                 }                 $passed = 0;                 if ($this->ipsclass->member['g_is_supmod'] == 1)                 {                         $passed = 1;                 }                 else if ($this->moderator['delete_post'] == 1)                 {                         $passed = 1;                 }                 else if ( ($this->ipsclass->member['g_delete_own_posts'] == 1) and ( $this->ipsclass->member['id'] == $post['author_id'] ) )                 {                         $passed = 1;                 }                 else                 {                         $passed = 0;                 }                 if ($passed != 1) $this->moderate_error();                 //-----------------------------------------                 // Check to make sure that this isn't the first post in the topic..                 //-----------------------------------------                 if ($post['new_topic'] == 1)                 {                         $this->moderate_error('no_delete_post');                 }                 if ( $this->trash_forum and $this->trash_forum != $this->forum['id'] )                 {                         //-----------------------------------------                         // Set up and pass to split topic handler                         //-----------------------------------------                         $this->ipsclass->input['checked'] = 1;                         $this->ipsclass->input['fid']     = $this->trash_forum;                         $this->ipsclass->input['title']   = $this->ipsclass->lang['mod_from']." ".$this->topic['title'];                         $this->ipsclass->input['desc']    = $this->ipsclass->lang['mod_from_id']." ".$this->topic['tid'];                         $this->ipsclass->input[ 'post_'.$this->ipsclass->input['p'] ] = 1;                         $this->trash_inuse = 1;                         $this->multi_split_topic();                         $this->trash_inuse = 0;                 }                 else                 {                         $this->modfunc->post_delete( $this->ipsclass->input['p'] );                         $this->modfunc->forum_recount( $this->forum['id'] );                 }                 $this->ipsclass->print->redirect_screen( $this->ipsclass->lang['post_deleted'], "act=ST&f=".$this->forum['id']."&t=".$this->topic['tid']."&st=".$this->ipsclass->input['st'] );         }         // mod                 //Если юзер не сам удаляет свой пост, то...            if ($this->ipsclass->member['id'] != $post['author_id'] )                 {                    //    $this->ipsclass->DB->simple_construct( array( 'select' => 'id, members_disable_pm', 'from' => 'members', 'where' => "id={$post['author_id']}" ) ); // идет запрос к базе, e-mail убираю с целью уменбшить число запросов                    //    $this->ipsclass->DB->simple_exec();                    //   $data = $this->ipsclass->DB->fetch_row();       //    if ( $this->ipsclass->vars['reportpost_method'] == $data['members_disable_pm'] ) // вот тут я прировнял метод к ПМ              //       {                         $this->lib->to_by_id    = $post['author_id'];                         $this->lib->from_member = $this->ipsclass->member;                         $this->lib->msg_title   = 'Внимание! Ваше сообщение удалено';                         $this->lib->msg_post    = "Ваше сообщение удалено модератором, так как не соответствуе правилам форума. Ознакомиться с правилами форума вы можете <a href='{$ibforums->base_url}act=boardrules'>здесь</a> <br /> А полностью прочитать тему <a href='{$ibforums->base_url}showtopic={$ibforums->input['t']}'>здесь</a>";     ///тут я немного подправил";                         $this->lib->force_pm    = 1; // думаю это проверка, включена ли ситема ПМ у пользователя                         $this->lib->send_pm();                     }                 }            }                 $this->ipsclass->print->redirect_screen( $this->ipsclass->lang['post_deleted'], "act=ST&f=".$this->forum['id']."&t=".$this->topic['tid']."&st=".$this->ipsclass->input['st'] );         }         // end

Это сообщение отредактировал(а) 2man - 9.11.2005, 17:14

Если не работает именно этот фрагмент, то ты забыл закомментировать одну закрывающуюся скобку:

Код

//    if ( $this->ipsclass->vars['reportpost_method'] == $data['members_disable_pm'] ) // вот тут я прировнял метод к ПМ              //       {                         $this->lib->to_by_id    = $post['author_id'];                         $this->lib->from_member = $this->ipsclass->member;                         $this->lib->msg_title   = 'Внимание! Ваше сообщение удалено';                         $this->lib->msg_post    = "Ваше сообщение удалено модератором, так как не соответствуе правилам форума. Ознакомиться с правилами форума вы можете <a href='{$ibforums->base_url}act=boardrules'>здесь</a> <br /> А полностью прочитать тему <a href='{$ibforums->base_url}showtopic={$ibforums->input['t']}'>здесь</a>";     ///тут я немного подправил";                         $this->lib->force_pm    = 1; // думаю это проверка, включена ли ситема ПМ у пользователя                         $this->lib->send_pm();                     }  //!!! Вот эту.

А РНР5 не выводит ошибки вообще или там так по умолчанию установлено (я просто по-ламерски юзаю Деневер, жизни не знаю)? Для отладки, имхо, это не дело...

Bastet, а ведь правда забыл Да я просто хочу тестировать на PHP 5, как на сервере, а то сделаю под PHP 4, а потом все равно надо будет править. Попробую поставить на Денвере, посмотреть, что получится. Какую ошибку выдаст, если таковая есть

В общем проверил я на локалхосте. Кое-что пришлось подправить, а мод поставить до оригинального редиректа, иначе пост удалялся, происходил редиркт, а до мода дело не успевало доходить
Вот так я решил свои проблемы:

Код

//      $this->ipsclass->print->redirect_screen( $this->ipsclass->lang['post_deleted'], "act=ST&f=".$this->forum['id']."&t=".$this->topic['tid']."&st=".$this->ipsclass->input['st'] );

;) Не могу понять почему мне выдает ошибку: Call to undefined function: send_pm()
Перепроверил, по указанному пути в файле func_msg есть таковая

Наверное, редирект стоит включить, но после отправки сообщения.

Хм... странно, странно... А библиотека с этой функцией не должна специально подключаться?
Если ничего не придумаем, можно будет вставить кусочек кода прямо из самой функции send_pm().

Не, редирект у нас есть:

Код

$this->ipsclass->print->redirect_screen( $this->ipsclass->lang['post_deleted'], "act=ST&f=".$this->forum['id']."&t=".$this->topic['tid']."&st=".$this->ipsclass->input['st'] );

Кажется мы забыли вот этот кусок кода из PM

Код

if ( $this->lib->error )                                 {                                         print $this->error;                                         exit();                                 }

Хотя это не важно. Наверное дело в том, что мы забыли указать модуль отправки PM

Код

require_once( ROOT_PATH.'sources/lib/func_msg.php' );                 $this->lib = new func_msg();                 $this->lib->ipsclass =& $this->ipsclass;                 $this->lib->init();

Да. Точно. Я так и знал Осталось только подправить само сообщение, а именно:
1. Ссылку на правила форума
2. Ссылку на тему, откуда удален пост
Итак, первое:

Код

<a href='{$ibforums->base_url}index.php?act=boardrules'>здесь</a>

Со вторым пришлось немного повозиться, но я выиграл компьютер

Код

$this->topic['tid']

Вот она моя переменная топика
Спасибо огромное за помощь, сам бы вряд ли справился.
Да, еще хотел сказать: мы неплохо сработались. Может еще что-нибудь попробуем сделать, у меня голова полна идей

2man, ну, можно попробовать

Bastet, вот и первый глюк. Что-то мы тут с тобой накосячили, что ПМ в одном скине не работает вообще. И это при том, что файлы скинов я не трогал вообще. Странности какие-то творятся, я разобрался с чем была проблема, в файле скина, в котором не работал ПМ, стоял кривой редирект на ПМ, НО я поставил тот, который стоит в работающем скине и что-то не вышло. Может есть какие-нибудь сообращения? Ссылка в профиле

2man, а давай ты мне дашь ссылку на сам скрипт форума (мне просто искать лениво). Так очень тяжело судить, что именно там может не работать...

Давай рассуждать логически. Если работает в одном скине, а в другом нет, то надо смотреть файлы скина. Дальше, когда отправляем ПМ ссылка выглядит вот так http://forum.vingrad.ru/index.php?act=Msg&CODE=...&MID=.... значит смотрю в папке скина ключевое сочетание Msg&CODE. Выдает два файла: skin_topic.php и skin_msg.php. В первом ничего особого про ПМ нет, просто кнопка, а вот во втором как я обнаружил была проблема с редиректом странице при щелчке на кнопку. Во втором файле нашлась ссылка:

Код

[ <a href="{$this->ipsclass->base_url}CODE=04&amp;act=Msg&amp;MSID={$post['mt_id']}&amp;MID={$author['id']}&amp;fwd=1">{$this->ipsclass->lang['vm_forward_pm']}</a> | <a href="{$this->ipsclass->base_url}CODE=04&amp;act=Msg&amp;MID={$author['id']}&amp;MSID={$post['mt_id']}">{$this->ipsclass->lang['pm_reply_link']}</a> ]

ЧТо не так я не знаю. Я беру даже заменяю эти файлы теми, которые работают - не помогает
Добавлено @ 00:42
При том что ссылка рендерится правильная
http://www.mega-forum.net.ru/index.php?act=Msg&CODE=4&MID=52
Значит проблема с рендером самой страницы получается

Радикальное решение - выкинуть глюченный скин.
Не могу помочь даже логически - не вижу скинов, не могу тестировать.

Кстати, еще одна проблемка на будущее: когда удаляется топик, удаляются и все посты к нему? Значит, будут рассылаться и ПМ авторам всех постов, кому-то даже по многу штук. Нужно ввести какой-нить флаг в функции удаления топика, чтобы отличать.

Ага, точно будут. Но тема удаляется в том случае в том случае, если грубое нарушение правил, там никто нескольких постов не допустит, хотя можно сделать на всякий случай, чтоб при удалении топика уведомление шло только первому участнику, создавшего тему
Добавлено @ 01:51
Но мне кажется, что delete_topic и delete_post это разные вещи, а мод установлен только для delete_post.