[SOLVED] PM System help required :)

[stelthius]

Hello guys,

 

Ive built a pm system and ive got it pretty much all covered, but one thing im stuck on is getting it to add +1 to the message count of the reciever, if anyone could take a look at my code and help me out here id be very greatull

 

 

<?
require_once("../include/session.php");
require_once("../include/checkban.php");
require_once("../skin/default/header.php");

$user = $_SESSION['username'];

//Are they logged in or not?
if ($session->logged_in){
?>
<p align="center"> </p>
<br>
<center>
<b><p><a href="inbox.php">Inbox</a> | <a href="compose.php">Compose</a> | <a href="sent.php">Sentbox</a>
<b><p> </p>
</b>
</center>
<br>
<?
//So here we get the variable submitted through the form to this page
$reciever = $_POST['username'];
$subject = $_POST['subject'];
$message = $_POST['message'];
$unread = $_POST['unread'];
$error = '0';

//If they are all blank we jsut say to compose a message
if(!$reciever AND !$subject AND !$message)
{
?>
<p><b>Please compose a message.</b></p>
<br>
<?
}

//Since this form was partially filled out we need to return an error message
else
{
if (!$reciever)
{
$error = 'You must enter a reciever to your message';
}

if (!$subject)
{
$error = 'You must enter a subject';
}

if (!$message)
{
$error = 'You must enter a message';
}

if (!unread)
{
$error = 'Can not set to unread';
}

//If the variable error is not set to zero, we have a problem and should show the error message
if($error != '0')
{
echo "<p>$error</p><br>";
}

//There are no errors so far which means the form is completely filled out 
else
{
//Are the trying to send a message to a real user or to something they just made up?
$user_check = mysql_query("SELECT username FROM users WHERE username='$reciever'");
$user_check = mysql_num_rows($user_check);

//The user is real and not made up if this is true
if($user_check > '0')
{
//There might already be a sessioned time variable, if so we need to get it for the flood check
$time = $_SESSION['time'];

//If there is a time variable already, set it to the varialbe $old_time
if($time > '0')
{
$old_time = $time;
}

//Here we get the minutes and seconds on the server time using the date function, and set that to the $time variable
//Now we find the difference between this time ($time) and the time that the page was submitted ($old_time)
$time = date('is');
$difference = $time - $old_time;

$_SESSION['time'] = $time;

//If the two times have a difference greater or equal to 15, which is 15 seconds, they can submit the message, this is for flood protection
if($difference >= '1')
{
//Get their private message count
$sql = mysql_query ("SELECT pm_count FROM users WHERE username='$reciever'");
$row = mysql_fetch_array ($sql);

$pm_count = $row['pm_count'];

//You cant have more than 50 private messages, if they try sending a message to a user with a full inbox return an error message
if(pm_count == '50')
{
$error = 'The user you are trying to send a message to has 50 private messages, sorry but we cant send your message untill that user deletes some of their messages.';
}

else
{ 
//And not we stick the message in the database with all the correct information
mysql_query("INSERT INTO messages (reciever, sender, subject, message) VALUES('$reciever', '$user', '$subject', '$message')") or die (mysql_error());
}

//Let the user know everything went ok.
echo "<p><b>You have successfully sent a private message!</b></p><br>";
}

//Since they are trying to send messages faster than every 15 seconds, give them an error message 
else
{
$error = 'You must wait 15 seconds before sending another private message';
}
}

//If they mis spelled or, made up a username, then give an error message telling them its wrong.
else
{
$error = 'That username does not exist, please try again. Remember to check your spelling.';
}
}
}

//Since we may have set the error variable to something while trying to send the messae, we need another error check
if($error != '0')
{
echo "<p>$error</p><br>";
}

else
{
//Here's the form for the input
?>
<form name="send" method="post" action="compose.php">
<table width="80%">
<tr>
<td width="150px" align="left" valign="top"><p>Username</p></td>
<td width="" align="left" valign="top"><input name="username" type="text" id="username" value="<? echo "$reciever"; ?>"></td>
</tr>

<tr>
<td width="150px" align="left" valign="top"><p>Subject</p></td>
<td width="" align="left" valign="top"><input name="subject" type="text" id="subject" value="<? echo "$subject"; ?>"></td>
</tr>

<tr>
<td width="150px" align="left" valign="top"><p>Message Body</p></td>
<td width="" align="left" valign="top"><textarea name="message" type="text" id="message" value="" cols="50" rows="10"></textarea></td>
</tr>

<tr>
<td><input type="hidden" name="unread" value="<? echo "$unread"; ?>"></td>
</tr>


<tr> 
<td></td>
<td><input type="submit" name="Submit" value="Send Message"></td>
</tr>
</table>
</form>
<!-- FOOTER TEMPLATE INCLUDE -->
<? require_once("../skin/default/footer.php"); ?>
<!-- FOOTER TEMPLATE INCLUDE -->
</center>

<?
}
} 

else
{
echo "<center>";
echo "[<a href=\"../index.php\">Home</a>]";
echo "[<a href=\"../main.php\">Login</a>]";
echo "[<a href=\"../register.php\">Register</a>]";
echo "<br>Please Login to use the PM System";
echo "</center>";
} 
?>

 

 

Thanks Rick.

 

 

P.S I know my code isnt the cleanest but it works, ill clean it when i have everything i want working

[MadTechie]

Try this (see added line)

else
{ 
//And not we stick the message in the database with all the correct information
mysql_query("INSERT INTO messages (reciever, sender, subject, message) VALUES('$reciever', '$user', '$subject', '$message')") or die (mysql_error());
mysql_query ("UPDATE `users` SET `pm_count` = `pm_count`+1 WHERE username='$reciever' LIMIT 1") or die (mysql_error()); //ADDED
}

 

PS

The reason to make it clean is not to make it pretty at the end but to make it easier to develop 

[stelthius]

thank you so much, somethign so very simple and i missed it,

 

Like your signature too lol its a good one mate.