[PHP][MySQL] Aktywacja konta przy rejestracji Opcje

[hunter1988]

Wszystko powinno działać a gdzieś jest mały błąd

[PHP] pobierz, plaintext 
	public function activation($kod)
	{
		$sql_check = "SELECT * FROM `users` WHERE `kod`='$kod' and `aktywacja`='1'";
		$sql_update = "UPDATE `users` SET `aktywacja`='1' WHERE `kod`='$kod' and `aktywacja`='0'";
 
 
		$stmt = $this->pdo -> exec($sql_check);
 
		if($stmt > 0)
		{
			echo "Aktywowałeś swoje konto już wcześniej";
		}
		else
		{
			$this->pdo -> exec($sql_update);
		}
 
 
	}
 
[PHP] pobierz, plaintext 

Połączenie z bazą jest, zapytanie $sql_update działa na 100% bo wklepywałem je bez warunków i aktywuje konto, coś jest nie tak z $sql_check.
Połączenie UPDATE działa, a już np w warunkach if nie, w tym przykładzie nie wykona się NIC, nie wyświetli się żaden błąd sql ani php, składnia więc jest "teoretycznie ok", zapytanie $sql_check na bank nie działa jak powinno (szukałem literówek i nic).

Powinno być tak skrypt sprawdza czy konto z danym kodem ma już aktywację (w bazie danych 1) i to jest warunek, funkcja exec sprawdza ilość wyników zapytania, w tym przypadku nawet gdy podaje kod i w bazie jest 0 to zawsze zwraca wynik 0 dla zapytania $sql_check, no ok ale skoro dało 0 to powinno wykonać się else czyli aktywacja konta, podczas gdy ten zabieg nie wykonuje się, mimo że $this->pdo -> exec($sql_update); działa.

Na pewno jest to jakaś głupota, ale siedzę przy tym i siedzę i nie mogę znaleźć błędu...

[phpion]

W warunkach przy select masz aktywacja=1, a przy update aktywacja=0.

[hunter1988]

W warunkach przy select masz aktywacja=1, a przy update aktywacja=0.

Kod jest przechowywany w bazie np
kod to 5005 i aktywował konto czyli 1

Pierwszy warunek sprawdza czy ktoś nie próbuje aktywować konta jeszcze raz bo np po 2 dniach przeglądał pocztę jeszcze raz i postanowił kliknąć w link wtedy exec zwróci wartość większą od 0 i update nie odbędzie się i tu jest coś nie tak z zapytaniem prawdopodobnie.
Drugie zapytanie po prostu aktywuje konto

Ten post edytował hunter1988 4.06.2013, 12:59:16