I keep getting the error:

Notice: Undefined index: sending in /home/r0106985/public_html/duct-central/winawallet.php on line 8

When all my code says is:

<?php
	$title = "Win A Wallet";
	include("header.php");
	include("mainfuncs.php");
?>

<?php
	$sending = $_POST['sending'];
	if($sending == 1){
                             SOME RANDOM CODE
	}
	frame(1, 'winawallet');
?>


</body>
</html>

The problem I guess comes because sometimes $sending is passed from a form to this script and sometimes another script doesn't send that variable. How do I make it so that it won't complain if the variable doesn't exist?

Try this:

<?php
	$title = "Win A Wallet";
	include("header.php");
	include("mainfuncs.php");
?>

<?php
                if(empty($sending))
                {
                 $sending == 0;
                }
	$sending = $_POST['sending'];
	if($sending == 1){
                             SOME RANDOM CODE
	}
	frame(1, 'winawallet');
?>


</body>
</html>

Steve.

Or just simplify the whole thing ...

PHP Code:

if((!empty($_POST['sending'])) && ($_POST['sending']==1))
{
   WHATEVER YOU NEED TO DO;
}

frame(1, 'winawallet'); 


I agree with foobar but I want to point this out for the sake of Gaffer...

PHP Code:

if(empty($sending))
                {
                 $sending == 0;
                }
    $sending = $_POST['sending'];
    if($sending == 1){
                             SOME RANDOM CODE
    }
    frame(1, 'winawallet'); 


Your pretty much working backwords here as you say if $sending is empty then set it to 0 which also equivicates to boolean false. Then you set $sending to equal $_POST['sending'] which this whole thing is bad for two reasons. One you never want to reffer to $sending when $_POST['sending'] is the variable, the use of registered globals will be is being deprecated. Two, you set sending to a boolean value but if $_POST['sending'] is empty it will revert to a string as php automatically casts types. An empty string will throw the same error again.

On understanding your post, I have to disagree.

If the code I have posted senses a null for $sending, it will not run the final script. It will only run the script if it has an input for $sending.

And if you want to emphasise a problem in my coding or anyone else's, please use paragraphs and sentences, as it is easier to read.

Steve.

$sending = $_POST['sending'];

That line effectively turns $sending into a string, not null. Null in php is a special value and is not default under any circumstance (at least that I'm aware of).

Setting $sending to 0 is not the same thing as setting $_POST['sending'] to 0. In other words take out that line and the script should run fine.

$sending is taking the 'sending' from the $POST and setting it to zero. If you can understand the post format, any setting will be added to a variable when the script kicks in.

Basically:

$sending1 = $sending;

Would do the same job as:

$sending = $_POST['sending'];

And what happens when register globals is turned off? (which it is by default now)

$sending becomes a seperate variable from $_POST['sending']

Do you not understand that a php form that has a post of 'sending' will be accepted as $sending in a php form handler?

Personally I would re-write this script to my standards, but as I charge a fee, I am sure that would not be accepted.

Originally Posted by Gaffer Sports

Do you not understand that a php form that has a post of 'sending' will be accepted as $sending in a php form handler?

Do you not understand that in order for it to work like that register_globals has to be set on. By default in php it is off as its a sloppy way to code and can open yourself up to security holes.

http://www.php.net/manual/en/security.globals.php