[PHP][MySQL]Dziwne działanie skryptu Opcje

[adrianozo]

Witam
Napisałem mały skrypt i zastanawia mnie jego dziwne działanie:

[PHP] pobierz, plaintext 
<?php
$haslo = md5(sha1($_POST['haslo']));
$haslo1 = md5(sha1($_POST['haslo1']));
$haslo2 = md5(sha1($_POST['haslo2']));
$zapytanie = mysql_query("SELECT * FROM `user` WHERE `id` = '".$_SESSION['id']."'");
while($wiersz = mysql_fetch_array($zapytanie))
$starehaslo = $wiersz['haslo'];
echo '<form action="index.php?page=changepassword" method="post">
Podaj swoje obecne hasło: <input type="password" name="haslo" /><br /><br />';
echo 'Podaj nowe hasło: <input type="password" name="haslo1" /><br /><br />';
echo 'Potwierdź nowe hasło: <input type="password" name="haslo2" /><br /><br />';
echo '<input name="aktualizuj" type="submit" value="Aktualizuj hasło">
</form><br />';
if(isset($_POST['aktualizuj']))
{
    if(empty($haslo) || empty($haslo1)  || empty($haslo2))
    {
        echo 'Nie wypełniono wszystkich pól.<br />';
    }
    if($starehaslo != $haslo)
    {
        echo 'Podano nieprawidłowe stare hasło.<br />';
    }
    if($haslo1 != $haslo2) 
    {
        echo 'Podane hasła nie są identyczne.<br /><br />';
    }
    else
    {
        $zap = mysql_query("UPDATE `user` SET `haslo` = '".$haslo2."' WHERE `id` = '".$_SESSION['id']."'");
        if($zap === TRUE)
        {
            echo 'Hasło zostało zmienione';
        }
        else
        {
            echo 'Nie udało się zmienić hasła. Prosimy spróbować ponownie za jakiś czas.';
        }
    }
}
?>
[PHP] pobierz, plaintext 

Po przesłaniu pustego formularza otrzymuje komunikat:

Podano nieprawidłowe stare hasło.
Hasło zostało zmienione.

I hasło się zmienia w bazie na pusty ciąg. Dlaczego tak się dzieje. Co zrobiłem źle bo nie mogę zauważyć

[Agape]

powinienes stosowac if elseif
wytlumacze Ci
Sprawdzasz czy hasla nie sa puste, jesli sa wypisuje ze nie przeslano i skrypt leci dalej, kolejna instrukcja to sprawczy czy pobrane z bazy haslo (np. "kornik") pasuje do przeslanego (tu, pustego), nie pasuje wiec wypisuje komunikat i idzie dalej. Haslo1 i haslo2 sa oba przeslane przez formularz a skoro byl pusty oba sa puste i dajesz im mozliwosc modyfikacji hasla na to przeslane (czyli puste)

Musisz troche sobie to rozplanowac i nie pozwolic skryptowi isc dalej jak nie powinien. Np. jesli hasla sa puste wypisz ze sa puste i tyle, else idz dalej. Tak musisz pisac.

[adrianozo]

Tylko w tym rzecz, że sprawdzam czy są puste a gdy u mnie są puste nie otrzymuje tego komunikatu, że są puste...