Hi,

I want to include a check that won't allow the message to be sent if the user's inbox has more than 25 messages.

I have tried the following query but it didn't work, can anyone see why?

PHP Code:

$messagesNum=(mysql_num_rows(mysql_query("SELECT userto, COUNT(userto FROM messages WHERE userto = '$userto'")));
if($messagesNum > 25){
$msg=$msg."<img src='error.png'> Can't send message<BR>";
$status= "NOTOK";} 


PHP Code:

$messagesNum=mysql_num_rows(mysql_query("SELECT * FROM messages WHERE userto = '$userto'")); 


that should just do the number of rows, and will be however many rows they have in the table. I dont think that COUNT() function was necessary.

A couple things

Your SQL query's parentheses are are off. As it is, you've coupled your FROM and WHERE clauses into the COUNT function. The result is probably a
userto: NULL count: 0.

I think you meant

SELECT userto, COUNT(*) FROM messages WHERE userto = '$userto'"

which again will only return a single row:
userto: John count: 43

You don't need to use both COUNT and mysql_num_rows here, since the COUNT is aggregating function. If userto is unique, only 1 row will be returned.

Using COUNT only

$result = mysql_query("SELECT userto, COUNT(*) as count FROM messages WHERE userto = '$userto'");
$row = mysql_fetch_assoc($result));
if ($row['count']>25) {
 $msg=$msg."<img src='error.png'> Can't send message<BR>";
$status= "NOTOK";
}

Using mysql_num_rows only

$messagesNum= mysql_num_rows(mysql_query("SELECT userto FROM messages WHERE userto = '$userto'"));
if ($messagesNum>25) {
 $msg=$msg."<img src='error.png'> Can't send message<BR>";
$status= "NOTOK";
}

Hmm - no need to "SELECT userto" when we already have $userto, right? Or, are we just sanitizing the data to update $userto to match the case as it is in the db?

Thanks, zxcvbnm60