[SOLVED] My form isn't working

[helpmeplease2]

Why doesn't this work? Is it because the $php_self doesn't work with pages like www.example.com?p=vgrgr&gerr=S&video=name

 

if($_POST['submityes']=="yes"){
$name=$_POST['name'];
$comment=$_POST['comment'];
$query="INSERT INTO comments (Name,Comment) VALUES ('$name','$comment')";
mysql_query($query) or die('Could not query: ' . mysql_error());

}else{

echo "<form name=\"commentsform\" action=\"$PHP_SELF\" method=\"post\">
<table border=\"0\" cellspacing=\"0\" cellpadding=\"0\">
<tr><td>Name: </td><td><input type=\"text\" name=\"name\" size=\"30\"><br></td></tr>
<tr><td>Comments: </td><td><textarea name=\"comment\" rows=\"6\" cols=\"40\" wrap=\"virtual\"></textarea></td></tr></table>
<input type=\"hidden\" name=\"submityes\" value=\"yes\">
<input type=\"submit\" value=\"Add Comments\"></form>";

}

[helpmeplease2]

Anyone?

[tippy_102]

I tested it and it worked for me.....didn't test the INSERT though. What error are you getting?

 

Also, you should clean the data before adding it to the database:

 

$name= mysql_real_escape_string( $name );

[helpmeplease2]

No error, it just doesn't work.

[tippy_102]

What part doesn't work?

[fenway]

Also, why not have the HTML as HTML, and the php variables as PHP? The way you have it set up defeats the purpose...

[helpmeplease2]

It doesn't add it to the database.

[fenway]

Without an error?

[helpmeplease2]

Yup, no error.

[rpadilla]

try inserting a marker in your insert statment, to make sure its really executed like

 

mysql_query($query) or die('Could not query: ' . mysql_error());

echo 'my query was executed';

 

also try making some error codes, maybe error reporting is off.

[fenway]

Then post the query string sent to mysql.

[helpmeplease2]

It turns out it was inserting it, but not everything I wanted. I modified the code, but its still not inserting the Game and Video. The $gtitle and $vtitle are from the url.

 

if($_POST['submityes']=="yes"){
$gtitle=$_GET['game'];
$vtitle=$_GET['video'];
$result = mysql_query("SELECT * FROM videos WHERE GTitle='$gtitle' AND VTitle='$vtitle'")
or die(mysql_error());
while($row = mysql_fetch_row( $result )) {
$name=$_POST['name'];
$game=$row['GTitle'];
$video=$row['VTitle'];
$comment=$_POST['comment'];

$query="INSERT INTO comments (Name,Game,Video,Comment) VALUES ('$name','$game','$video','$comment')";
mysql_query($query) or die('Could not query: ' . mysql_error());
}
}else{

echo "<form name=\"commentsform\" action=\"$PHP_SELF\" method=\"post\">
<table border=\"0\" cellspacing=\"0\" cellpadding=\"0\">
<tr><td>Name: </td><td><input type=\"text\" name=\"name\" size=\"30\"><br></td></tr>
<tr><td>Comments: </td><td><textarea name=\"comment\" rows=\"6\" cols=\"40\" wrap=\"virtual\"></textarea></td></tr></table>
<input type=\"hidden\" name=\"submityes\" value=\"yes\">
<input type=\"submit\" value=\"Add Comments\"></form>";

}

[fenway]

Sigh.... could you just post the echo of $query?  I'm sure they're blank.

[helpmeplease2]

They are blank thats why I need your help.

[fenway]

Well, print out the $row hash.

[helpmeplease2]

Arghhh I was trying to do it the hard way! I could've just put

$name=$_POST['name'];
$game=$_GET['game'];
$video=$_GET['video'];
$comment=$_POST['comment'];
$query="INSERT INTO comments (Name,Game,Video,Comment) VALUES ('$name','$game','$video','$comment')";
mysql_query($query) or die('Could not query: ' . mysql_error());

 

That is working, thanks for the help!