if (member['posts'] <= '1' && .....

[gta]

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

Как сделать так :

 

if (мембер пост <= 10 и в сообщение содержиться слово Порно) {

 

$this->ipsclass->$Db->query("UPDATE members SET mgroup = '5' WHERE name='$member['name']');

 

}

[gta]

ни кто совсем не знает?

[ddvhouse]

Это при каждом обновлении форума должно проверяться или как?

[Garret]

Так и сделай, например в topics.php, там уже есть кол-во постов пользователя и сам пост, регвыром выбирай из поста нужное тебе поле.

[gta]

вы кодом лучше говорите сразу :blush:

 

 

а куда добавит я сам знаю

[Destruction]

if($this->ipsclass->member['posts'] && preg_match("/порно|реклама|и другие слова/i",$POST)){

$this->ipsclass->$Db->query("UPDATE members SET mgroup = '5' WHERE name='$member['name']');

}

Изменено 30 Июня 2006 пользователем Destruction

[gta]

if($this->ipsclass->member['posts'] && preg_match("/порно|реклама|и другие слова/i",$POST){

$this->ipsclass->$Db->query("UPDATE members SET mgroup = '5' WHERE name='$member['name']');

}

 

 

Parse error: parse error, unexpected '{' in /home/bh2foru/public_html/sources/classes/post/class_post.php on line 797

[Destruction]

Следи за парностью скобок.

[OverHerz]

так наверно правильней будет

if($this->ipsclass->member['posts'] && preg_match("/порно|реклама|и другие слова/i",$POST))
{
$this->ipsclass->$Db->query("UPDATE members SET mgroup = '5' WHERE name='$member['name']'");
}

[Arhar]

нене, если после иф выполняется только одно действие, {} не нужны

[ddvhouse]

нене, если после иф выполняется только одно действие, {} не нужны

 

но они и мешат не будут, если стоят

[gta]

так наверно правильней будет

if($this->ipsclass->member['posts'] && preg_match("/порно|реклама|и другие слова/i",$POST))
{
$this->ipsclass->$Db->query("UPDATE members SET mgroup = '5' WHERE name='$member['name']'");
}

Parse error: parse error, unexpected T_ENCAPSED_AND_WHITESPACE, expecting T_STRING or T_VARIABLE or T_NUM_STRING in /home/bh2foru/public_html/sources/classes/post/class_post.php on line

[ddvhouse]

так наверно правильней будет

if($this->ipsclass->member['posts'] && preg_match("/порно|реклама|и другие слова/i",$POST))
{
$this->ipsclass->$Db->query("UPDATE members SET mgroup = '5' WHERE name='$member['name']'");
}

Parse error: parse error, unexpected T_ENCAPSED_AND_WHITESPACE, expecting T_STRING or T_VARIABLE or T_NUM_STRING in /home/bh2foru/public_html/sources/classes/post/class_post.php on line

 

Ну то что я так на взляд увидел это:

WHERE name='$member['name']'");

 

там куча ковычек которые, то открывают переменную то закрываю не там где надо, так пропиши:

WHERE name='$member[name]'");

[Arhar]

а лучше присвоить прям перед этим $membername= $member['name']; и писать будет проще

[ddvhouse]

а лучше присвоить прям перед этим $membername= $member['name']; и писать будет проще

 

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

[gta]

Всё хорошо конечнно, дык только в баннед не переносит.

[ddvhouse]

Всё хорошо конечнно, дык только в баннед не переносит.

 

Знаешь, что ещё меня смущает?

$Db->

вместо

$DB->

[gta]

Всё хорошо конечнно, дык только в баннед не переносит.

 

Знаешь, что ещё меня смущает?

$Db->

вместо

$DB->

 

 

сёрно пользователь. и вообще вроде написание переменой ( с маленькой, с большой) вроде как безразлично.

[ddvhouse]

сёрно пользователь. и вообще вроде написание переменой ( с маленькой, с большой) вроде как безразлично.

Совсем не безразлично. Для php это разные переменные.

Может не туда вставляешь?

[gta]

соурцес/классес/пост/класс_пост.пхп.

 

ставлю я туда. мой старый вариант работал. тока он не то делал )

[gta]

ну,и?

[gta]

нарооод, ё маё.

[Tim]

там куча ковычек которые, то открывают переменную то закрываю не там где надо, так пропиши:

WHERE name='$member[name]'");

так низя делать, ключ массива в кавычках должен быть

а куда дели проверку на кол-во постов?

и вообще по-моему надо добавлять "$this->ipsclass->"

 

$spamer=$this->ipsclass->member['name'];
if(($this->ipsclass->member['posts']<10) && (preg_match("/порно|реклама|и другие слова/i",$POST)))
{
$this->ipsclass->$DB->query("UPDATE members SET mgroup = '5' WHERE name='$spamer'");
}

Изменено 12 Июля 2006 пользователем Tim

[Miklsi]

Здравствуйте ! Столкнулся с такой же проблемкой на форуме. Т.е. один человечек всё время спамит на форуме о порно или какой-либо рекламе - надоело, жуть ! Хотелось бы узнать, а последний код, который прописал Ув.*Tim* работает? С Уважением, Алексей!

[catbegemot.net]

Уже было: http://www.ibresource.ru/forums/index.php?showtopic=13088

 

И даже работает

[ddvhouse]

там куча ковычек которые, то открывают переменную то закрываю не там где надо, так пропиши:

WHERE name='$member[name]'");

так низя делать, ключ массива в кавычках должен быть

 

Неправда.

Только так и можно.

Запись '$member[name]' уже подразумевает подключение массива,

а name='$member['name']' - это name присваивается $member[, а потом начинается бред - что такое name']' компилятор вообще не поймет.