what's wrong with code

Posted on 2007-11-22
Last Modified: 2008-02-01
whats wrong with this php script i made?

$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");


echo "<center><font color='red'><b>Thanks! Application has been sent.</b></font><br><br></center>";




Open in new window

Question by:fapathy

Accepted Solution

HiTechFail earned 500 total points
ID: 20336804
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.

Expert Comment

ID: 20336865
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?


EE Admin

Featured Post

Is Your Active Directory as Secure as You Think?

More than 75% of all records are compromised because of the loss or theft of a privileged credential. Experts have been exploring Active Directory infrastructure to identify key threats and establish best practices for keeping data safe. Attend this month’s webinar to learn more.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Suggested Solutions

Title # Comments Views Activity
Permalink issue in Wordpress archive 1 32
Echo vs ?><?php  html code 4 45
php call to a non-object 3 32
echo time from sql to input type="time" 9 17
Part of the Global Positioning System A geocode ( is the major subset of a GPS coordinate (, the other parts being the altitude and t…
Password hashing is better than message digests or encryption, and you should be using it instead of message digests or encryption.  Find out why and how in this article, which supplements the original article on PHP Client Registration, Login, Logo…
Explain concepts important to validation of email addresses with regular expressions. Applies to most languages/tools that uses regular expressions. Consider email address RFCs: Look at HTML5 form input element (with type=email) regex pattern: T…
The viewer will learn how to look for a specific file type in a local or remote server directory using PHP.

910 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question

Need Help in Real-Time?

Connect with top rated Experts

21 Experts available now in Live!

Get 1:1 Help Now