record di codice attivo record dove, o dove?

Question

Sto utilizzando Active Record su CodeIgniter. Sono confuso su quale approccio dovrei prendere. Attualmente, il nostro sistema di accesso consente all'utente di utilizzare nome utente/email per l'accesso e la password. Ma il mio attuale record attivo sembra consentire all'utente di accedere se sceglie di usare l'email + nessuna password.

In questo momento questo è il mio query:

$this->db->select('id,level,email,username'); 
$this->db->where('email',$user); 
$this->db->or_where('username',$user); 
$this->db->where('password',$pass); 
$query = $this->db->get('users'); 

if($query->num_rows>0) 
    return TRUE; 
else 
    return FALSE; 

ingressi del campione:

• Nome utente: prova | Password: pass | Risultato: Successo
• Nome utente: prova | Password: vuoto | Risultato: Impossibile
• Nome utente: test@domain.com | Password: pass | Risultato: Successo
• Nome utente: test@domain.com | Password: vuoto | Risultato: Successo

Il quarto ingresso di prova deve essere Impossibile nel risultato, ma sembra che si registra l'utente anche se la password è vuota.

Answer 1

Il problema è probabilmente che è necessario aggiungere parentesi quando si mischiano AND e OR in una clausola WHERE. Prova questo:

$this->db->select('id,level,email,username'); 
$this->db->where("(email = '$user' OR username = '$user') 
        AND password = '$pass'"); 
$query = $this->db->get('users'); 

Answer 2

@RidIculous ha ragione. Questo è un modo corretto per farlo:

$user = $this->db->escape($user); 
$this->db->select('id,level,email,username'); 
$this->db->where("(email = $user OR username = $user)"); 
$this->db->where('password', $pass); 
$query = $this->db->get('users'); 

O un formato preferisco (PHP 5+)

$user = $this->db->escape($user); 
$query = $this->db 
    ->select('id,level,email,username') 
    ->where("(email = $user OR username = $user)") 
    ->where('password', $pass) 
    ->get('users'); 

Answer 3

$conditions = '(`username`="'.$username.'" OR `email`="'.$email.' OR `mobile`="'.$mobile.'"') AND `password`="'.$password.'"';   
$query = $this->db->get_where('table_name', $conditions); 
$result = $query->result();