[PHP] Błąd pętli weryfikacyjnej Opcje

[Terrorizer]

Po kliknięciu przycisku zaloguj za każdym razem wyskakuje mi komunikat "podaj email i hasło"
Natomiast jak wynika z pętli powinno to wyskakiwać tylko wtedy gdy jedno z pól jest empty.
Nie potrafię dojść, z czego to wynika

Streszczony kod:

[PHP] pobierz, plaintext 
if (isset($_POST['zaloguj'])) {
 
   if (!empty($_POST['email']) || !empty($_POST['pass'])){		
 
//tutaj lecą instrukcje weryfikujące poprawność danych	
 
   } else echo 'podaj email i hasło';
}
[PHP] pobierz, plaintext 

Jeśli komuś się chce, zamieszczam poniżej kod bez wycinania:

[PHP] pobierz, plaintext 
if (isset($_POST['zaloguj'])) {
if (!empty($_POST['email']) || !empty($_POST['pass'])){
 
 
 
$email = $_POST['logemail'];
$password = $_POST['logpass'];	
 
if ((strlen($email) < 4) || (strlen($email) > 40))   {echo '<script>
                                                            document.getElementById("nick").value = "niepoprawny email";
                                                            document.getElementById("nick").className = "niepoprawnyemail"; 
															</script>';}
	if ((strlen($password) < 6) || (strlen($password) > 40))   {echo '<script>
                                                            document.getElementById("pass").value = "niepoprawne hasło";
                                                            document.getElementById("pass").className = "niepoprawnehasło"; 
															</script>';}
	if (filter_var($email, FILTER_VALIDATE_EMAIL)){
		$pass = md5(sha1($password));
		} else echo '<script>
                     document.getElementById("nick").value = "niepoprawny email";
                     document.getElementById("nick").className = "niepoprawnyemail"; 
					 </script>';
 
		$checkuser = mysql_fetch_assoc(mysql_query(" SELECT id FROM users WHERE email='$email' AND pass='$password' "));
		if($checkuser){ 
					   $_SESSION = array();
					   $_SESSION['id'] = $checkuser['id'];
								if(!empty($_SESSION['id'])){
								                            header('location: main.php');
									                        }
                                else header('location: index.php');															
}
} else echo '<script> alert("podaj email i hasło")</script>';
}
[PHP] pobierz, plaintext 

Ten post edytował Terrorizer 20.09.2014, 14:36:41

[KsaR]

zamień || (or) na && (i) inaczej walidujesz jak ktoś wpisze tylko hasło lub email.

Ps.

[PHP] pobierz, plaintext 
$pass = md5(sha1($password)); 
[PHP] pobierz, plaintext 

tak się nie robi.

Temat: podwójne hashowanie haseł.

Ten post edytował Ksar 20.09.2014, 14:46:13

[Terrorizer]

Dzięki na pewno się to przyda
Tak czy inaczej, wciąż wyskakuje mi alert, żeby podać email i hasło :/

EDIT: pozwoliłem sobie przesunąć całą linijkę 33 na miejsce 3 i teraz można się zalogować normalnie. Chociaż wciąż przy wpisaniu niepoprawnego pola wyskakuje aby podać email i hasło.
W ogóle tego nie ogarniam

Ten post edytował Terrorizer 20.09.2014, 14:55:05

[KsaR]

if (!empty($_POST['email']) (...)
$email = $_POST['logemail'];

Czemu przypisujesz różne pola?.
Tak samo z hasłem.

Prawdo podobnie to jest problem.

[Terrorizer]

Już wszystko jasne! Bardzo mi pomogłeś dzięki

email i pass to zmienne z rejestracji
logemail logpass to zmienne logowania

Zwyczajnie się zaplątałem w tym wszystkim