what's wrong with code

whats wrong with this php script i made?
<?php
$popu      = $_POST['popularity'];
$webu      = $_POST['weburl'];
$fname      = $_POST['fname'];
$cemail      = $_POST['cemail'];
$minfo      = $_POST['minfo'];
 
 
if($popu == "" || $webu == "" || $fname == "" || $cemail == "" || $minfo == "") {
echo "<center><b>Please fill this out.</b><br><br></center>";
} else {
$line		.= "<b>----------------------------------</b>";
$addit     .= "Site: $webu<br>";
$addit     .= "Name: $fname<br>";
$addit     .= "Email: $cemail<br>";
$addit     .= "Popularity: $popu<br>";
$addit     .= "Comments: $minfo<br>$line<br>";
 
$file = "applications.html";
$fh = fopen($file, "a");
fwrite ($fh, "$addit");
fclose($fh);
echo "<center><font color='red'><b>Thanks! Application has been sent.</b></font><br><br></center>";
}
}
?>

Open in new window

fapathyAsked:
Who is Participating?
 
HiTechFailConnect With a Mentor Commented:
Remove one of the last brackets at the end. That is also a bad way of doing applications, but if you just want to fix this one then just remove bracket.
0
 
Computer101Commented:
HiTech and fapathy,

Seems these accounts both come from the same place, and it seems that it is mis appropriation of the point system.

Can you please explain what you are doing?

Thanks

Computer101
EE Admin
0
Question has a verified solution.

Are you are experiencing a similar issue? Get a personalized answer when you ask a related question.

Have a better answer? Share it in a comment.

All Courses

From novice to tech pro — start learning today.