[PHP]Formularz zapisu na newsletter Opcje

[gregi1]

Tworze formularz zapisu na newsletter i niestety napotkałem na problem:

Mam kod:

[PHP] pobierz, plaintext 
if ($akcja == 'add') {
$mail = $_POST['email'];
$token = sha1(time());
$data_zapisu=date(c);
$ip = $_SERVER['REMOTE_ADDR']; 
$sql='SELECT * FROM newsletter where e-mail ='.$mail;
$res = mysql_query($sql);
 
$num_rows = mysql_num_rows($res);
 
 
	if ($num_rows=="0") {
	$zapis='INSERT INTO newsletter(e-mail,token,data_zapisu,ip) VALUES ('.$mail.','.$token.','.$data_zapisu.','.$ip.')';
	$res = @mysql_query($zapis);
 
 
	}
 
	else {
 
	$res = mysql_query("SELECT active FROM newsletter where e-mail = \"".$mail."\" ");
	$row = mysql_fetch_array($result);
		if ($row[0] == "1") {
		echo("Adres już istnieje w bazie i jest aktywny");
		}
 
		else  {
                   echo 'Nie aktywny';
		}
	}
}
[PHP] pobierz, plaintext 

Jak wpisuje adres e-mail w formularzu to dostaje na stronie komunikat:
Warning: mysql_num_rows(): supplied argument is not a valid MySQL result resource in /home/gregi1/domains/politykroku.eu/public_html/dev/func/newsletter_show.php on line 47

Dodam ze do bazy nic sie nie zapisuje.

[radziowie]

Masz złe zapytanie do bazy. Przynajmniej jeden z argumentów jest niepoprawny. Pokaż może strukturę tabeli?

Ten post edytował radziowie 29.11.2010, 21:01:56

[markonix]

Stringi w języku sql umieszczamy w " " np. where `e-mail` = "'.$email.'";

[vistacash]

podobnie i w zapytaniu INSERT wartości tekstowe w zapytaniu musisz otoczyć ""

[PHP] pobierz, plaintext 
'INSERT INTO newsletter(e-mail,token,data_zapisu,ip) VALUES ("'.$mail.'","'.$token.'",'.$data_zapisu.',"'.$ip.'")';
[PHP] pobierz, plaintext 

[markonix]

dodaj po zapytaniu:
or die(mysql_error());

[gregi1]

dodaj po zapytaniu:
or die(mysql_error());

Nie wiem dlaczego, ale nie akceptowal pola e-mail pomogla zmiana na email w bazie i w kodzie.

[markonix]

Dozwolone znaki w bazie danych litery, cyfry i znak podkreślenia.
Przynajmniej jeśli nie dozwolone to na pewno zalecane

[markonix]

Temat świeci kolorem rozwiązanego..

[gregi1]

Temat świeci kolorem rozwiązanego..

Jeśli można prosiłbym o nie zamykanie tematu... temat dotyczy ogólnie zapisu na newsletter, a jestem dopiero na początku drogi czekają mnie jeszcze potwierdzenia zapisu, wiec myslę ze może się jeszcze przydać.

[markonix]

Zwykle jeden temat = jeden problem (przynajmniej jedna dziedzina problemu).
Nic nie płacisz na zakładanie tematów