PHP OO Modificar contraseña

[jeyn]

Hola.

Me estoy encontrando con un problema y no doy con la solución.

Quiero que los usuarios puedan modificar su contraseña, pero no soy capaz de que me funcione el script, en teoria deberia de detenerse si no es correcta la comparacion entre los valores de las variables, pero continua con el script, y no encuentro el error

Script:

Código PHP:

  $error=""; 
if($error==""){ 
 
/*Comparo password de bbdd con oldpass */ 
        if(!comparar_cadena($password,$oldpass)){ 
        $error.="<b>Tu Antigua Contraseña No Coincide.</b><br>";         
        } 
         
/*Comparo password newpass con newpass2 y defino variable */ 
        if(!comparar_cadena($newpass,$newpass2)){ 
        $error.="<b>Tu Nueva Contraseña No Coincide.</b><br>"; 
        } 
        $newpassword = md5($newpass); 
}else{ 
echo "No Ha Sido Posible Cambiar la contraseña, Pulse <a href='javascript:history.go(-1)'><b>Aqui</b></a> Para Corregir Los Siguientes Errores<br><br>"; 
echo "Ocurrieron Los Siguientes Errores:<br><br>".$error; 
exit; 
}  
   

Función:

Código PHP:

  /*Comparar Variables*/ 
function comparar_cadena($cadena1,$cadena2){ 
if(strcmp($cadena1===$cadena2)){ 
return true; 
}else{ 
return false; 
} 
}  
   

Me podrian ayudar a encontrar el error

Gracias

Un saludo

[mogurbon]

Código PHP:

  function comparar_cadena($cadena1,$cadena2){ 
if($cadena1==$cadena2){ 
return true; 
}else{ 
return false; 
} 
}  
   

[iviamontes]

pero y pq no haces la consulta update y si la cantidad de registros afectados es 1 es que se actualizo bien, si es 0 pues no...

[jeyn]

Por que para que cambie la contraseña el usuario queria comparar que la que hay en la BBDD con la que el introduzca, y luego comparar la nueva contraseña, que va repetida por si la introduce con error alguna de las dos veces,
Ejemplo de la nueva contraeña;
Nueva: amgel
Repetida: angel
Puede tener un error al escribirla por eso quiero comparar la nueva contraseña y la vieja por seguridad para que no la pueda cambiar alguien que no sepa la antigua.

[jeyn]

Iniciado por mogurbon

Código PHP:

  function comparar_cadena($cadena1,$cadena2){ 
if($cadena1==$cadena2){ 
return true; 
}else{ 
return false; 
} 
}  
   

Tambien lo he probado y no se me detiene el script aun que no sean iguales las contraeñas.

Un saludo

[Nexus10]

¿Y donde esperas que se detenga el script?

Esta instrucción

$newpassword = md5($newpass);

se va a ejecutar siempre, independientemente de cualquier cosa.

¿Por qué debería detenerse?, ¿donde?

[mogurbon]

asi seria la funcion , ya probada y todo el pdo , pero lo que dice nexus es verdad, si no indicas que se detenga , no se va a detener jamas

Código PHP:

function comparar_cadena($cadena1,$cadena2){
    if(strcmp($cadena1,$cadena2)==0){
        return true;
    }else{
        return false;
    } 
}
 
$pass1="chana";
 
$pass2="chana";
 
if(!comparar_cadena($pass1,$pass2)){
   $mensaje="<b>Tu Antigua Contraseña No Coincide.</b><br>";        
} 
else{
    $mensaje="coincide";
}
echo $mensaje;
 

[jeyn]

Ya lo solucione era solo la negación.

Muchas gracias de todos modos.

El script se detiene en $error. al recibir false de la función.

Gracias de nuevo.

Un saludo.