solución para script

[decano]

Hola a todos:
primero que nada, esta mal que haga uba consulta con $_SESSION['login'] que contiene el nombre de usuario?.

Aparte de eso Tengo un seccion en mi pagina que les permite cambiar a los usuarios algunos datos, mi pequeño problema es que cuando algunos de esos datos nuevos que ingresan estan mal se los haga, por ejemplo su password.

Código PHP:

  if(($_POST['pais'] !='') || ($_POST['email'] !='') || ($_POST['pass'] !='')) 
 
    { 
         
       
    if((in_array('pais',$_POST['actualizar'])) and ($_POST['pais'] !='')) 
     
    { 
             
         { 
                
        $sql  = "UPDATE administradores SET pais=('$_POST[pais]') where id='$row[id]'"; 
        $r = mysql_query($sql); 
         
         } 
      
    if((in_array('email',$_POST['actualizar'])) and ($_POST['email'] !='')) 
     
     
         { 
 
            if(is_email_valid($_POST['email'])) 
 
            {   
             
            $sql2 = "UPDATE administradores SET email=('$_POST[email]') where id='$row[id]'"; 
            $r = mysql_query($sql2); 
             
            } 
 
         } 
             
    if((in_array('password',$_POST['actualizar'])) and ($_POST['pass'] !='')) 
     
        { 
 
            if(comprobar($_POST['pass']))  
 
            { 
 
                $pass = md5($_POST['pass']); 
                $sql3 = "UPDATE administradores SET password=('$pass') where id='$row[id]'"; 
                $r = mysql_query($sql3); 
     
            } 
 
        } 
         
    } 
  
        header('Location:../?your_acount='.$row['id'].'&aser=modificar'); 
 
      } 
     
         else  
     
                header('Location:../?your_acount='.$row['id'].'&aser=errores');  
   

en esta parte los redirige pero no les informa que hayan errores, solo les dice que fueron modificados o que no hizo cambios, pero quiero que si su email esta mal como lo dice la funcion les diga o que si el pawweord esta mal tambien se los diga, como puedo llevar los errores para la otra pagina.

Les cuento que es algo que quiero hacer para por separado, porque el formulario que trae la informacion les da la opcion de cambiar el dato que quieran.

Código HTML:

 <td><div align="center">Tareas de mantenimiento: modificar datos de <?=$_SESSION['login']?><br>
                            Recuerde que su password debe de tener mas de 6 caracteres.</div></td>
  </tr>
  <tr>
    <td>&nbsp;</td>
  </tr>
  <tr>
    <td><div align="center">Password -&gt;
        <input name="pass" type="password" > Actualizar <INPUT type="checkbox" name="actualizar[]" value="password">           
    </div><br>
    <div align="center">Pais &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; -&gt;
        <input name="pais" type="text" > Actualizar <INPUT type="checkbox" name="actualizar[]" value="pais">        
    </div></td>
  </tr>
  <tr>
    <td><INPUT type="hidden" value="<?=$row['id']?>" name="id"><div align="center">Actual &nbsp;&nbsp;&nbsp; <? if($row['pais'] =='')echo 'Sin Dato'; else echo $row['pais']?><br><br>Actual <?=$row['email']?></div></td>
  </tr>
  <tr>
    <td><div align="center">E-mail &nbsp;&nbsp;&nbsp; -&gt;
        <input name="email" type="text" > Actualizar <INPUT type="checkbox" name="actualizar[]" value="email">
    </div></td>
  </tr>
  <tr>
    <td>&nbsp;</td>
  </tr>
  <tr> 

gracias a todos

[Cluster]

primero que nada, esta mal que haga uba consulta con $_SESSION['login'] que contiene el nombre de usuario?.

Pues .. mm no? .. Ahora .. en lugar de un "login" con un nombre de usuario yo usaría un "ID" del usuario (más que nada para optimizar las consultas SQL).

en esta parte los redirige pero no les informa que hayan errores, solo les dice que fueron modificados o que no hizo cambios, pero quiero que si su email esta mal como lo dice la funcion les diga o que si el pawweord esta mal tambien se los diga, como puedo llevar los errores para la otra pagina.

Con la misma filosofía que aplicas en los otros casos ...

Código PHP:

  <? 
if(comprobar($_POST['pass']))   
 
            {  
 
                $pass = md5($_POST['pass']);  
                $sql3 = "UPDATE administradores SET password=('$pass') where id='$row[id]'";  
                $r = mysql_query($sql3);  
      
            } else { 
                  // Supuestamente aquí llegará el flujo del código si no pasa tu validación que aplica esa función .. en consecuencia: 
 
                 header ("Location: dondetengas queier.php ...."); 
                 exit; // Ya no me interesa ejecutar el resto del código .. voy a redireccionar a otro script para que tome el control el otro script no este ya. 
            }   

Un saludo,

[decano]

Hola y gracias, sabes es estoy haceindo sotod bajo session, or usuario $_SESSION['login'], ya que no esta en la session su id, por eso preguntaba, y con la otra parte, com exit() sale de ese script y de todo, porque me interesa que le de error y cometio en uno de los casos, porque puede que no haya cometido error en password y si en email o viceversa, no se si me explico.
saludos
graicas