[MySQL][PHP]Aktywacja Opcje

[nansss]

Witam.
Mam mały problem.
Posiadam skrypt rejestracji, lecz chcę go udoskonalić w aktywację przez email.
Mam problem, bo nie uaktualnia mi w bazie danych tabeli kod i nie dodaje tam kodu, oraz nie uaktualnia tabeli access i nie zmienia z 0 na 1.

rejestracja.php

[PHP] pobierz, plaintext 
<?php
session_start();
$connect = mysql_connect('localhost', 'root', 'password') or die('Nie mo&#x17C;na po&#x142;&#x105;czyc si&#x119; z baz&#x105; danych. Przepraszamy za k&#x142;opoty');
$db = mysql_select_db(db') or die('Nie mo&#x17C;na wybra&#x107; bazy danych. Przepraszamy za k&#x142;opoty');
$login = $_POST['login'];
$access = $_POST['0'];
$email = $_POST['email'];
$haslo = md5('haslo');
$haslo2 = md5('haslo2');
$data = date('Y-m-d-G:i:s');
$zapytanie = mysql_query("SELECT COUNT(*) FROM user WHERE login='$login' LIMIT 1");
$zajete = mysql_fetch_row($zapytanie);
$zapytanie = mysql_query("SELECT COUNT(*) FROM user WHERE email='$email' LIMIT 1");
$zajete1 = mysql_fetch_row($zapytanie);
if (empty($_POST["login"]) || empty($_POST["email"]) || empty($_POST["haslo"]) || empty($_POST["haslo2"]))
{
echo '<center>Nie wype&#x142;niono wszystkich pól.</center> <meta http-equiv="Refresh" content="3; url=index.php" />';
}
elseif($_POST["haslo"]!=$_POST["haslo2"])
{
echo '<center>Podane hasła nie s&#x105; identyczne.</center> <meta http-equiv="Refresh" content="3; url=index.php" />';
}
elseif($zajete[0] != 0)
{
echo '<center>Podana nazwa u&#x17C;ytkownika jest ju&#x17C; zaj&#x119;ta.</center> <meta http-equiv="Refresh" content="3; url=index.php" />';
}
elseif($zajete1[0] != 0)
{
echo '<center>Podany email jest zaj&#x119;ty przez innego u&#x17C;ytkownika.</center> <meta http-equiv="Refresh" content="3; url=index.php" />';
}
elseif(!$_POST["email"] || !preg_match("/^[-0-9a-zA-Z_\.]+@([-0-9a-zA-Z_\.]+\.)+([0-9a-zA-Z]){2,4}$/i", $_POST["email"]))
{
echo '<center>Podany email jest nieprawid&#x142;owy.</center> <meta http-equiv="Refresh" content="3; url=index.php" />';
}
else
{
$zapytanie = "INSERT INTO `user` (`login`, `access`, `email`, `haslo`, `data`) VALUES ('".$login."', '".$access."', '".$email."', '".$haslo."', '".$data."')";
$zap= "SELECT `email` FROM `user` ORDER BY `id` DESC";
$idzap = mysql_query($zap);
$naglowki = "Content-type: text/html; charset=iso-8859-2\r\n";
$naglowki .= "Reply-to: nansss@boo.pl <nansss@boo.pl>\r\n";
$naglowki .= "From: nansss@boo.pl <nansss@boo.pl>\r\n";
}
if ( ! mysql_query($zapytanie))
{
}
else
{
echo '<center>Twoje konto zosta&#x142;o za&#x142;o&#x17C;one.<br>
W ci&#x105;gu 10 minut otrzymasz email z następnymi instrukcjami.</center> <meta http-equiv="Refresh" content="5; url=index.php" />';
$wiersz = mysql_fetch_row($idzap);
$kod = str_shuffle("qwertyuiopasdfghjklzxcvbnm1234567890");
mail($email, 'Aktywacja konta na xxx.pl', '<b>Witaj na xxx.pl!</b><br>Konto <b>'.$login.'</b> zostało zarejestrowane na podany adres e-mail.<br>W celu potwierdzenia rejestracji konta kliknij w poniższy odnośnik:<br><a href="http://sampo.boo.pl/rejestracja.php?active='.$kod.'"> xxx.pl/rejestracja.php?active='.$kod.' </a><br><br>Jeśli nie rejestrowałeś konta o nazwie <b>'.$login.'</b> po prostu zignoruj tego maila.<br><br><br>------<br>Wiadomość wygenerowana automatycznie<br>przez serwis xxx.pl', $naglowki);
$zapy = "UPDATE `user` SET `klucz` =('".$kod."') WHERE `access` =1";
$idzapy = mysql_query($zapy);
}
if($_GET["active"])
{
mysql_query("UPDATE `user` SET `access` =1 WHERE `klucz` ='$_GET[active]' ");
if(mysql_affected_rows()==1)
{
echo '<center>Aktywacja uko&#x144;czona pomy&#x15B;lnie. Mo&#x17C;esz ju&#x17C; korzysta&#x107; z naszego serwisu.</center>';
}
else
{
echo '<center>Podano nieistniej&#x105;cy kod aktywacyjny.</center>';
}
}
mysql_close($connect);
?>
[PHP] pobierz, plaintext 

Najpradopodobniej z tym kodem coś jest nie tak, ale nie jestem pewien.

[PHP] pobierz, plaintext 
$zapy = "UPDATE `user` SET `klucz` =('".$kod."') WHERE `access` =1";
 $idzapy = mysql_query($zapy);
 }
 if($_GET["active"])
 {
 mysql_query("UPDATE `user` SET `access` =1 WHERE `klucz` ='$_GET[active]' ");
 if(mysql_affected_rows()==1)
[PHP] pobierz, plaintext 

Proszę o pomoc

[Cysiaczek]

Czy nie możesz się wysilić i wyświetlić sobie informacji jakie są zwracane przez mysql_error() ?
Czy nie możesz chociaż pokazać co zawiera $_GET['active']?
To są podstawowe rzeczy, które się robi, jeśli coś nie działa tak, jak nam się wydaje, że powinno.

Podzdrawiam

[cojack]

[PHP] pobierz, plaintext 
"UPDATE `user` SET `klucz` =('".$kod."') WHERE `access` =1";
[PHP] pobierz, plaintext 

a tu access nie powinno być 0 ?

Nie znam struktury bazy danych, więc nie wiem co Ty masz w access jako default. Strzelam.

[Spawnm]

Ten kod jest bez sensu,
$zapy = "UPDATE `user` SET `klucz` =('".$kod."') WHERE `access` =1";
po where daj id dodanego pola


Cojack -> wtedy by ustawiło wszystkim userom nowy klucz, czyli ich linki aktywacyjne by nie działały

[cojack]

Spawn, o tym samym chciałem napisać, ale przemyślałem pierw, i zobacz że on ma idzap, więc nie wiem dlaczego pomyliło mi się z id wiersza (IMG:style_emoticons/default/biggrin.gif) Ale masz rację.

Ten post edytował cojack 20.08.2009, 10:39:39

[nansss]

Problem został z tym:

[PHP] pobierz, plaintext 
if($_GET["active"])
{
mysql_query("UPDATE user SET  access = '1' WHERE klucz= 'nie wiem co tutaj dać'");
if(mysql_affected_rows()==1)
{
echo '<center>Aktywacja uko&#x144;czona pomy&#x15B;lnie. Mo&#x17C;esz ju&#x17C; korzysta&#x107; z naszego serwisu.<br><a href="zaloguj.php>Przejd&#x17A; do logowania</a></center>';
}
else
{
echo '<center>Podano nieistniej&#x105;cy kod aktywacyjny.</center>';
}
}
mysql_close($connect);
?>
[PHP] pobierz, plaintext 

Wyrzuca, że podano nieistniejący kod aktywacyjny

[Spawnm]

Zapewne dajesz tam klucz z geta active...
Zobacz w bazie czy taki istnieje, może wcześniejszy kod go zmienił i już nie pasuje.

[cojack]

Tutaj akurat miałeś dobrze:

[PHP] pobierz, plaintext 
mysql_query("UPDATE `user` SET `access` =1 WHERE `klucz` ='$_GET[active]' ");
[PHP] pobierz, plaintext 

@Spawnm

Zobacz w bazie czy taki istnieje, może wcześniejszy kod go zmienił i już nie pasuje.

nie ma już nic, nic już nie ma (IMG:style_emoticons/default/haha.gif)

@nansss dropnij baze i utwórz ponownie.

Ten post edytował cojack 20.08.2009, 11:11:39

[nansss]

Już działa. Dzięki waszej pomocy i mojego kolegi z miejscowości działa.

Dzięki wielkie (IMG:style_emoticons/default/smile.gif)

[Spawnm]

[PHP] pobierz, plaintext 
"UPDATE `user` SET `access` =1 WHERE `klucz` ='$_GET[active]' ");
[PHP] pobierz, plaintext 

filtruj dane .

[nansss]

Nastąpił według mnie ostatni problem.

Tworzę konto wyskakuje komunikat Twoje konto zostało założone.
W ciągu 10 minut otrzymasz email z następnymi instrukcjami.

Przychodzi email, klikam w link aktywacyjny i pokazuje: Nie wypełniono wszystkich pól. Aktywacja ukończona pomyślnie. Możesz już korzystać z naszego serwisu.
Przejdź do logowania

Zamiast Aktywacja ukończona pomyślnie. Możesz już korzystać z naszego serwisu.
Przejdź do logowania. Ma nie wyskakiwać Nie wypełniono wszystkich pól.

Nie wiem jak to naprawić. Najprawdopodobniej jest to błąd w warunkach if. Takty wszystko działa.

rejestracja.php

[PHP] pobierz, plaintext 
<?php
 session_start();
 $connect = mysql_connect('localhost', 'root', 'password') or die('Nie mo&#x17C;na po&#x142;&#x105;czyc si&#x119; z baz&#x105; danych. Przepraszamy za k&#x142;opoty');
 $db = mysql_select_db('db') or die('Nie mo&#x17C;na wybra&#x107; bazy danych. Przepraszamy za k&#x142;opoty');
 $login = $_POST['login'];
 $access = $_POST['0'];
 $email = $_POST['email'];
 $haslo = md5(md5('haslo'));
 $haslo2 = md5(md5('haslo2'));
 $data = date('Y-m-d-G:i:s');
 $zapytanie = mysql_query("SELECT COUNT(*) FROM user WHERE login='$login' LIMIT 1");
 $zajete = mysql_fetch_row($zapytanie);
 $zapytanie = mysql_query("SELECT COUNT(*) FROM user WHERE email='$email' LIMIT 1");
 $zajete1 = mysql_fetch_row($zapytanie);
 if (empty($_POST["login"]) || empty($_POST["email"]) || empty($_POST["haslo"]) || empty($_POST["haslo2"]))
 {
 echo '<center>Nie wype&#x142;niono wszystkich pól.</center>';
 }
 elseif($_POST["haslo"]!=$_POST["haslo2"])
 {
 echo '<center>Podane hasła nie s&#x105; identyczne.</center> <meta http-equiv="Refresh" content="3; url=index.php" />';
 }
 elseif($zajete[0] != 0)
 {
 echo '<center>Podana nazwa u&#x17C;ytkownika jest ju&#x17C; zaj&#x119;ta.</center> <meta http-equiv="Refresh" content="3; url=index.php" />';
 }
 elseif($zajete1[0] != 0)
 {
 echo '<center>Podany email jest zaj&#x119;ty przez innego u&#x17C;ytkownika.</center> <meta http-equiv="Refresh" content="3; url=index.php" />';
 }
 elseif(!$_POST["email"] || !preg_match("/^[-0-9a-zA-Z_\.]+@([-0-9a-zA-Z_\.]+\.)+([0-9a-zA-Z]){2,4}$/i", $_POST["email"]))
 {
 echo '<center>Podany email jest nieprawid&#x142;owy.</center> <meta http-equiv="Refresh" content="3; url=index.php" />';
 }
 else
 {
 $zapytanie = "INSERT INTO `user` (`login`, `access`, `email`, `haslo`, `data`) VALUES ('".$login."', '".$access."', '".$email."', '".$haslo."', '".$data."')";
 $zap= "SELECT `email` FROM `user` ORDER BY `id` DESC";
 $idzap = mysql_query($zap);
 $naglowki = "Content-type: text/html; charset=iso-8859-2\r\n";
 $naglowki .= "Reply-to: xxx@boo.pl <xxx@boo.pl>\r\n";
 $naglowki .= "From: xxx@boo.pl <xxx@boo.pl>\r\n";
 }
 if ( ! mysql_query($zapytanie))
 {
 }
 else
 {
 echo '<center>Twoje konto zosta&#x142;o za&#x142;o&#x17C;one.<br>
 W ci&#x105;gu 10 minut otrzymasz email z następnymi instrukcjami.</center> <meta http-equiv="Refresh" content="5; url=index.php" />';
 $wiersz = mysql_fetch_row($idzap);
 $kod = str_shuffle("qwertyuiopasdfghjklzxcvbnm1234567890");
 mail($email, 'Aktywacja konta na xxx.pl', '<b>Witaj na xxx.pl!</b><br>Konto <b>'.$login.'</b> zostało zarejestrowane na podany adres e-mail.<br>W celu potwierdzenia rejestracji konta kliknij w poniższy odnośnik:<br><a href="http://xxx.pl/rejestracja.php?active='.$kod.'"> <a href="http://xxx.pl/rejestracja.php?active='.$kod.'" target="_blank">http://xxx.pl/rejestracja.php?active='.$kod.'</a> </a><br><br>Jeśli nie rejestrowałeś konta o nazwie <b>'.$login.'</b> po prostu zignoruj tego maila.<br><br><br>------<br>Wiadomość wygenerowana automatycznie<br>przez serwis sampo.boo.pl', $naglowki);
 $zapy = "UPDATE user SET klucz = ('".$kod."') WHERE access = 0";
 $idzapy = mysql_query($zapy);
 }
 if($_GET["active"])
 {
 mysql_query("UPDATE user SET  access = '1' WHERE klucz = '$_GET[active]'");
 if(mysql_affected_rows()==1)
 {
 echo '<center>Aktywacja uko&#x144;czona pomy&#x15B;lnie. Mo&#x17C;esz ju&#x17C; korzysta&#x107; z naszego serwisu.<br><a href="zaloguj.php>Przejd&#x17A; do logowania</a></center>';
 }
 else
 {
 echo '<center>Podano nieistniej&#x105;cy kod aktywacyjny.</center>';
 }
 }
 mysql_close($connect);
 ?>
[PHP] pobierz, plaintext 

[Spawnm]

Skoro wiesz gdzie błąd to go popraw.
Jelenia szukasz? Zamykam bo znowu jaja sobie robisz.