Que esta mal en el UPDATE

[granasturcom]

require_once "bd.php";//conecta mi base de datos
if(empty($_POST['submit']) ) //si envia formulario
{
$id=$_POST['id'];
$nombre = $_POST['nombre'];
$antigua = $_POST['antigua'];
$nueva = $_POST['nueva'];
$email = $_POST['email'];
$sql = "UPDATE login SET nombre='$nombre', direccion='$direccion',".
telefono='$telefono', email='$email' WHERE loginid=$id";
$result = mysql_query($sql);
header("Location:Usuarios.php");

[AdrianSeg]

Código PHP:

$sql = "UPDATE login SET nombre='$nombre', direccion='$direccion',telefono='$telefono', email='$email' WHERE loginid=$id";
 

Prueba asi

[maycolalvarez]

if(isset($_POST['submit']) )

[pilucho]

Aqui

,".telefono

debe ser asi como el principio

,telefono='$telefono', etc......

[granasturcom]

Creo que es asi, pero ahora tengo un problema mi contraseña esta encriptada en shal como compruebo la antigua con la que tengo en mi base de datos, que se llama en la bd password
<?php
require_once "header.php";
if(empty($_POST['submit']) ) //si envia formulario
{
$id=$_POST['id'];
$nombre = $_POST['nombre'];
$nueva = $_POST['nueva'];
$antigua=$_POST['antigua'];
$email = $_POST['email'];
$sql = "UPDATE login SET nombre='$nombre', password='$nueva',email='$email' WHERE loginid=$id";
$result = mysql_query($sql);
header("Location:Usuarios.php");
}
?>

[maycolalvarez]

simple, usa sha1 al password en el login, recuerda que es un hash y por lo tanto es no reversible

[granasturcom]

Seria asi :
<?php
require_once "header.php";
global $seed;
$consulta3=mysql_query("SELECT * from login where loginid=$id");
while ($row = mysql_fetch_array($consulta3))
{
$clav=$row["password"];
}
if(empty($_POST['submit']) ) //si envia formulario
{
$id=$_POST['id'];
$nombre = $_POST['nombre'];
$nueva = $_POST['nueva'];
$antigua=$_POST['antigua'];
$antigua=sha1($antigua.$seed))
$nueva=sha1($nueva.$seed))
$email = $_POST['email'];
if ($id and $antigua==$clav)//modificamos
{
//Editamos usuarios con clave
$sql="UPDATE usuarios SET username='$nombre',password='$nueva',email='$email ' where loginid=$id";
$result = mysql_query($sql);
header("Location:Usuarios.php");
}
if ($antigua)
{
if ($antigua!=$clav)
{
$mensaje="Clave Incorrecta, Vuelva a Introducir la clave";
}
}
if ($clave=="" and $antigua=="")
{
//Editamos usuarios sin clave dejando la antigua
$sql="UPDATE usuarios SET username='$nombre',email='$email' where loginid=$id";
$result = mysql_query($sql);
header("Location:Usuarios.php");
}


?>