PHP OO If seleccionar db

[OfSerker]

Hola, estoy haciendo que desde un if te diga si tienes activado (1) el valor de la db o no (0) con un echo. Forma parte del login:
Primero comprueba si tienes el código activado, después, selecciona el user y la pw de la bd y si son correctos te logueas.
La parte que no se hacer es la del código. No me sale -,-

login.php:

Código PHP:

  <?php 
session_start(); 
require_once('./config/funciones.php'); 
conectar('a', 'a', 'a', 'a'); 
  
include("mantenimiento.php"); 
 
//Variables 
$verf_config = mysql_query('SELECT * FROM users'); 
$rs = mysql_fetch_assoc($verf_config); 
$username = strip_tags($_POST['username']); 
$password = strip_tags(sha1($_POST['password'])); 
  
if($rs["codigo"] != 1){ 
echo 'Primero activa tu cuenta entrando a tu correo'; 
header("url=./register.html"); 
}else{ 
$query = @mysql_query('SELECT * FROM users WHERE user="'.mysql_real_escape_string($username).'" AND pass="'.mysql_real_escape_string($password).'"'); 
 
if($existe = @mysql_fetch_object($query)) 
{ 
    $_SESSION['logged'] = 'yes'; 
    $_SESSION['username'] = $username; 
    echo '<script>window.location="ofserker.php"</script>'; 
}else{ 
    echo 'El usuario y/o pass son incorrectos.';     
} 
?>   

Atentamente, OfSerker

[OfSerker]

¿Nadie me ayuda con el codigo? Joer...

[fiu]

Hola,

primero, no puedes hacer un header() despues de un echo o te dará error.

y luego, estás haciendo un select * de todos los usuarios, como sabes cual buscas ? te está devolviendo todos, e imagino que al no recorrer los registros luego en el fetch_assoc estarás recibiendo el ultimo.

Creo que deberias unirlo todo a la misma consulta, en realidad puedes deshacerte del primer select, y en el segundo, ya que haces un select * puedes hacer dentro la comprobación que hacias anteriormente de si lo tiene activado o no, algo así

Código PHP:

if($existe = @mysql_fetch_object($query))
{
  if($existe["codigo"] != 1) {
    echo 'Primero activa tu cuenta entrando a tu correo';
  }
  else {
    $_SESSION['logged'] = 'yes';
    $_SESSION['username'] = $username;
    echo '<script>window.location="ofserker.php"</script>';
  }
}else{
    echo 'El usuario y/o pass son incorrectos.';    
}
 

[OfSerker]

Muchas gracias, me has ayudado mucho.
Pero... ¿Como queda el código?
Ya que primero hace falta lo de comprovar el codigo y despues el login.

Atentamente, OfSerker