Witam!

Napisałam sobie skrypt weryfikacji użytkownika poprzez email. No i tutaj pojawił się problem. Mianowicie...

Oto funkcja wraz z plikiem sprawdzającym:

[PHP] pobierz, plaintext 
<?php
include ("init.php");
 
function user_activation ()
{
   $password = $_GET['hash'];
   $timestamp = base64_decode($_GET['timestamp']);
	$query = mysql_query("UPDATE users SET status=1 WHERE (password = '$password') AND (timestamp = '$timestamp')") or die (mysql_error());
}	
 
if (!user_activation ()){
echo 'nieaktywne';
} else {
echo ' konto aktywowane';
}
 
?>
[PHP] pobierz, plaintext 

Problem polega nie na działaniu funkcji, ale na moim zapewne nie rozumieniu jeszcze pewnych elementów php.

Jak zapewne widzicie, w momencie wejścia użytkownika na podany link, zostają pobrane i sprawdzone z bazą dane. To można pominąć, bo wszystko działa jak należy. Wysyłanie także działa.

Chciałabym uzyskać coś innego, otóż... W momencie, kiedy użytkownik wchodzi na link, aktywuje konto, no i dostaje ładną wiadomość powitalną. Ale czy jest możliwość jakiegoś czasowego ograniczenia poprawności linku?

Wybaczcie moje, może ciutkę zagmatwane, wyjaśnienia, ale mój parser nie potrafi interpretować jeszcze wszystkich wiadomości, a poza tym, z php mam doczynienia raptem dwa dni.

PS Czy można gdzieś znaleść manual w postaci pliku PDF do wydrukowania?

PS 2 Błagam...zmieńcie captcha, bo za cholerę nie mogę jej przeczytać.

Pozdrawiam
Anka aka ColdSand

No i zapomniałam o jeszcze jednym. Kiedy wysyła dane do bazy wyświetla mi echo, że konto jest nieaktywne. Ale kiedy nie wysyła tych danych, bo już w bazie są, wysyła to samo. Tego nie rozumiem.

zajrzyj do manuala i zobacz jak dzialaja funkcje.
Jak nic nie zwracasz to tak jakbys zwrocila false, czyli:
if (!user_activation ()){
jest zawsze spelnione.

Funkcja musi zwracac wynik dzialania.

@nospor: Problem w tym, że chyba za dużo manuala mam na raz;)

Zmieniłam fragment sprawdzający na:

[PHP] pobierz, plaintext 
<?php
if (user_activation ()){
echo 'Your account has been activated.';
} else {
echo 'Your account has been already activated.';
}
?>
[PHP] pobierz, plaintext 

I za każdym razem zwraca mi drugą opcję.

Nie wiem czy nie za późno ale może nie
Aby zadziałało:


funkcja user_activation() musi zwracać wartość: 'return'. Samo wysłanie zapytania nic nie da. Jeśli jest ok to return true inaczej return false. Poza tym raczej nie powinno się wysyłać hasła użytkownika w adresie :/ Raczej dodatkowe pole w tabeli z wygenerowanym hashem. Może być osobna tabela.



Jeśli chcesz ograniczyć czasowo aktywacje to trzeba raczej w tabeli w nowym polu np: czas_rejestraji typu DATE wstawić przy rejestracji NOW() a pozniej funkcja DATE_DIFF() porównać.

Tak bym kombinował