if statement

[danielson2k]

I am having trouble with the following piece of code:

	$msgcount = mysql_result($results, 0, 'message_count');

	if($msgcount>5){

		header('location: home.messages.view.php?message=Message%20not%20sent.%20This%20users%20message%20box%20is%20full.');

	}

the $msgcount variable is greater than 5 (i have run die($msgcount) before the if statement to check) yet the if statement never equates to true.

I have also tried <5 and ==5 but nothing works!

Any ideas?

Thanks in advance.

[Muntjewerf]

The code you posted does not make much sense to me. Maybe post a little bit more? (could be reading wrong cause I'm drinkin beer....alot of it:p So correct me if I'm wrong!)
And, are you really certain that the url in the header will work?
Or do you have this name for one file(why would you do that anyways?):

 home.messages.view.php

[batman]

You have to make sure that there is no browser output before the header. Like you can't have any echos or prints anywhere before the header.

[danielson2k]

Theres nothing wrong with the URL, its just an easy way (for me at least 😛) to name a load of similar files.

Theres also no browser output before it.

Could it be that despite being a number, the variable is being treated as a string rather than a variable?

[Mordecai]

Since it's from MySQL, it probably doesn't know that it's just a number. Instead of being 5, it's probably actually "5" (as said in above post). Try using intval().

Additionally, I see another error. Your header() tag needs two things: a capitalized L ("Location:") and a URI ("http://...").

[danielson2k]

Thanks for the help, but im still having the problem.

If I use intval() or convert to an integer with (int) when I echo the variable nothing shows up.

This is getting a wee bit frustrating now! :p

[danielson2k]

Ok, I got this sorted now. Tis because I wasn't using exit() after the header :rolleyes: