Proszę powiedzieć mi kto tak robi?

$_SESSION['haslo'] = $_POST['haslo'];
Dobrym nawykiem jest nieprzechowywać haseł w sesji.
Druga rzecz:
Cytat
if(mysql_num_rows(mysql_query("SELECT *
FROM user WHERE login = '".$_POST['login']."'
&& haslo = '".$_POST['haslo']."' ")) > 0)
a do bazy zapisuje pan tak:
$haslo = sha1($haslo);$dodaj_user= ("INSERT INTO user SET login='$login', email='$email',
haslo='$haslo'");
Wniosek:
Porównuje Pan hasło z inputa z zakodowanym hasłem z bazy danych co w rezultacie nie przejdzie.
Niech Pan przeniesie $haslo = sha1($haslo) przed:
Cytat
if(mysql_num_rows(mysql_query("SELECT *
FROM user WHERE login = '".$_POST['login']."'
&& haslo = '".$_POST['haslo']."' ")) > 0)
,albo dodać
Cytat
if(mysql_num_rows(mysql_query("SELECT *
FROM user WHERE login = '".$_POST['login']."'
&& haslo = '".sha1($_POST['haslo'])."' ")) > 0)