Hiding form fields that were not completed

[Manual]

Hi again ...

Below is part of a script for handling an "Inquiries" form. Thing is, if the visitor does not fill in a field, it should not show in the email that gets send. If he only completes his name and email, surname, tel no and fax should not be included in the mail.

//======== Receivers address =============
$toaddress = "[e-mail]";

//======== Subject Line =================
$subject = "Inquiry from the web site";

//========Mail Content=================
$mailcontent = "Inquiry from the web site \n"

."Name: ".$name."\n"
."Surname: ".$nachn."\n"
."E-Mail: ".$email."\n"
."Tel Nr: ".$tel."\n"
."Fax: ".$fax."\n"

$fromaddress = "From web site";
	mail($toaddress, $subject, $mailcontent, $fromaddress);

?>

[wilzy1]

One way to do it would be to include 'if' statements around each field, testing for nothing ($fieldName == '').

Ie: If the email field does not equal nothing then write it out... else, write nothing.

Same for the other fields.

[Manual]

OK .... I think ... 😕

... would the following line work?

if ($email!=="")	."E-Mail: ".$email."\n"		

[wilzy1]

Try that... there could be a cuter way to do this!

//======== Receivers address ============= 
$toaddress = "[e-mail]"; 

//======== Subject Line ================= 
$subject = "Inquiry from the web site"; 

//========Mail Content================= 
$mailcontent = "Inquiry from the web site \n" 

if($name != '')  // The ''  are  two of these  '    '   not a  "
{
."Name: ".$name."\n"
}
else{} 

if($nachn != '')
{
."Surname: ".$nachn."\n" 
}
else{}

if($email != '')
{
."E-Mail: ".$email."\n" 
}
else{}

if($tel != '')
{
."Tel Nr: ".$tel."\n" 
}
else{}

if($fax != '')
{
."Fax: ".$fax."\n";
}
else{;}


$fromaddress = "From web site"; 
    mail($toaddress, $subject, $mailcontent, $fromaddress); 

?>

[Manual]

Ok I tried the code above and got the following error message ...

Parse error: parse error, unexpected T_IF

😕 😕

[wilzy1]

Is there anymore of the error than what you just showed me?

[Manual]

Thanx a lot for the help I realy do appreciate it!🆒
Here it is, line 29 of the code would correspond to the line begining with

if($name != '') 

Parse error: parse error, unexpected T_IF in /web/comdocs/www.prodg.cc/docs/test/sendmail.php on line 29

Here is the entire code again, I created a test page, based on the above ...

//======== Receivers address =============
$toaddress = "[e-mail]";

//======== Subject Line =================
$subject = "Inquiry from the Script Test site";

//========Mail Content=================
$mailcontent = "Inquiry from the Script Test site \n"  

if($name !="") 
{ 
."Name: ".$name."\n" 
} 
else{}  

if($surname != '') 
{ 
."Surname: ".$surname."\n"  

} 
else{} 

if($email != '') 
{ 
."E-Mail: ".$email."\n"; 
} 
else{;} 


$fromaddress = "From the Script Test site"; 
    mail($toaddress, $subject, $mailcontent, $fromaddress);  

Sorry to be a pain, so far I just included the fields that were not completed. I find it a rather crude solution and would just like it to look a bit more refined. If you know what I mean.

[wilzy1]

I'll look into this a bit later.. got an RTCW game to play now.

[Manual]

Thanx, have fun ...

RTCW? 😕

[bad76]

HI,
"." is an operation, can't start a line.

Rewrite like this:

mailcontent = "Inquiry from the Script Test site \n"  ; // <-- ";" here

if($name !="")  

{  

mailcontent.="Name: ".$name."\n";  //<-- mailcontent.= instead of "." And ";"
}  

//<-- remove any else{}   here

if($surname != '')  

{  

mailcontent.=."Surname: ".$surname."\n"  ;
}  

// and so on...

If it gives not error, but not works try also with if(!empty($name)) or if(isset($name))

see you

[Manual]

Ok, now it works ... still had a couple of errors

$mailcontent= "Inquiry from the Script Test site \n" ; 

if($name !='')   

{   

$mailcontent.="Name: ".$name."\n";   

}   

if($surname != '')   

{   

$mailcontent.="Surname: ".$surname."\n"  ; 
} 

Mission accomplished, thanx a lot guys 🙂 :p 😃

[wilzy1]

Thats the one !

slaps head Hasnt done much programming lately... hence me overlooking that 🙂

Goodwork.