Problema al actualizar datos de la base de datos

[Nitsuga120]

Hola tengo un problema al actualizar un dato en la base de datos en este caso es una contraseña justamente es para que cambiar la contraseña de un registro

este es el codigo de php (aviso mi forma de programar es media rara y ya se lo del tema de las variables y la seguridad por ahora es para empezar )

Código PHP:

<?php
$pass = $_POST['pass1'];
$npass = $_POST['pass'];
$email = $_POST['email1'];
 
$password = sha1($npass);
if ($email = $email1)
{
$conexion = mysqli_connect("","","","") or die('Error al Conectar con la base de datos');
 
$result = mysqli_query($conexion, "SELECT `id` FROM `wisher` WHERE `email` = '".$email."' AND `password` = '".$pass."'");
if(mysqli_num_rows($result) != 0)
{
 
header("Location: indexus.php?error=error");
 
}
else if ($pass != $npass)
 
{
 
$conexion = mysqli_connect("mysql.hostinger.com.ar","","","") or die("Error al conectar");
 
$datos = "UPDATE `wisher`(`password`) SET `password`='$password'  where password='$pass'";
}
 
if(mysqli_query($conexion,$datos)){
session_start();
session_destroy();
header("Location: login.php");
 
 
}
 
else{
header("Location: indexus.php?error=error");
}
 
 
mysqli_close($conexion);
?>
 

lo que esta borrado de la conexion lo borre por tema de seguridad

[xfxstudios]

?????????????????????

Código PHP:

$email = $_POST['email1'];<<<<<<<<<<<<<<<<
 
if ($email = $email1)<<<<<<<<<<<<<<<<<<<<
 

de donde sale $email1???

Segundo: Aqui solo estas seleccionando la id y no la password que quieres actualizar, se podria analizar mejor

Tercero:
Aqui indicas que debes actualizar si la pass no coincide con el npass

Código PHP:

}
else if ($pass != $npass)
 
{
 
$conexion = mysqli_connect("mysql.hostinger.com.ar","","","") or die("Error al conectar");
 
$datos = "UPDATE `wisher`(`password`) SET `password`='$password'  where password='$pass'";
}
 

como sería eso???? es decir, que si yo envío las claves deferentes me la actualiza y si envio las claves que coincidan no??, debería de ser indiferente, lo que deberias de comparar es la clave anterior con la almacenada en la base para así verificar y permitir en consecuencia la actualización o no.

[rodocoyote15]

Código SQL:

UPDATE `wisher`(`password`) SET `password`='$password'  WHERE password='$pass'
 

Estás seguro que la consulta es correcta? Pruebala en tu SMBD y nos dices. Aquí una referencia: aquí