Form CHECKBOX Error

[ded]

I know this is probably a simple fix.

 

I have 4 checkboxs on the form:

<input name="chalking" type="checkbox" value="Y" />
                    Chalking<br />
                    <input name="setup" type="checkbox" value="Y" />
                    Monday Setup Crew<br />
                    <input name="teardown" type="checkbox" value="Y" />
                    Sunday Tear Down Crew<br />
                    <input name="ticketbooth" type="checkbox" value="Y" />
                    Ticket/Booth

 

If I check all 4 boxes, the post.php works fine.

 

If I check anything less, if receive an error.

Notice: Undefined index: chalking in /home1/website/public_html/web/help/postindiv.php on line 10

 

Notice: Undefined index: ticketbooth in /home1/website/public_html/web/help/postindiv.php on line 13

 

 

$chalking = $_POST['chalking'];
$setup = $_POST['setup'];
$teardown = $_POST['teardown'];
$ticketbooth = $_POST['ticketbooth'];

Any ideas?

[Yesideez]

You've got strict mode enabled - nothing to worry about.

 

You can suppress the notice by using this:

$chalking = @$_POST['chalking'];

 

Or redefine error reporting using this:

error_reporting(E_ALL ^ E_NOTICE);

That will allow error messages to get through but disable those notices. That notice you're getting just means you're checking something before it has been defined. In this case $_POST['chalking'] (and the other one) won't always be defined.

[AdRock]

Beaten again

 

Or you could use

 

if(isset($_POST['chalking'])) $chalking = $_POST['chalking'];

 

etc

[Yesideez]

AdRock's is probably the best solution.

[Omzy]

Does the '@' symbol just supress the message, or does it perform the same function as the isset()?

[mrMarcus]

Does the '@' symbol just supress the message, or does it perform the same function as the isset()?

suppresses the error message.

[laffin]

I prefer to use AdRocks suggestion.

 

[Yesideez]

@ should be used with care. I rarely use it anywhere other than on $_POST or $_GET at the start of a form (that generate annoying notices) otherwise you can be hiding error messages that need to be seen.

 

Each to their own way of coding

[Omzy]

Well basically on one of my contact forms I have:

 

@mail($to,$subject,$message,$headers);

 

I have this because I usually test on my localhost machine, which does not have an smtp function.

 

Without the @ symbol I was getting the error message on screen, so I put the @ symbol in as I thought it was used to check if there is a mail server present.

 

But it's clear now, it's still trying to send the message but just not displaying the error anymore... how can I tell it not to try and send an email if there is no mail server defiined?

[Yesideez]

mail() returns boolean - true on success, false on error so you can do this:

if (@mail($a,$b,$c,$d)) {
  //it worked!
} else {
  //failed
}

[laffin]

or as he asked check the ini

<?php
if(!empty(ini_get('smtp')) mail($a,$b,$c,$d);

 

 

this means just see if smtp php.ini variable is not empty

but again this can also be 'sendmail_path' for linux systems

[Omzy]

thanks guys i'll hav a play around with it