help

[Anon]

hi, this is my code, there's no error on exectuing but it doesn's add anithing to database... anyone know why?

<?
if ($accion == "agregar") {
$db = mysql_connect("localhost","mexska","juarez");
mysql_select_db("s",$db);

if ($acoplado == "si") { 
$bandas = "Varios"; 
}

if ($autor != "" || $titulo != "" || $banda != "") {
  $sql = "INSERT INTO catalogo (autor,titulo,banda,es,es2,genero,comentario) VALUES ('$autor','$titulo','$banda','$es','$es2','$genero','$comentario')";
  $result = mysql_query($sql);
  echo "Gracias por Insertar su CD";
	echo "$autor,$titulo,$banda";
}
else {
echo "debes llenar los campos \n <br>";
echo "<a href=aver.php>regresar</a>";
}

}

else{

?>

<form method="post" action="aver.php?accion=agregar">
Usted es: <input type=text name=autor> <br>
EL Titulo del CD: <input type=text name=titulo> <br>
La Banda :<input type=text name=banda> <br> es acoplado ? <input type=checkbox name=acoplado value=si> <br>
es.. <input type=text name=es> <br>
es mas--<input type=text name=es2> <br>
genero: <input type=text name=genero> <br>
comentario: <textarea name=comentario> </textarea>
<input type="Submit" name="submit" value="Enviar!">
</form>

<?
}
?>

[nevermore2181]

That's because your form elements are incorrect. The values for each element must be placed in quotes. So your <input type=text name=titulo> must be <input type="text" name="titulo"> instead.

[[deleted]]

My guess is that you had a SQL error. Add a print(mysql_error()); after your sql statement and see what it says.

If you can't figure out what's wrong in your query, print it: print($sql);

It's always good to check for errors anyway 🙂