Witam,
Mam problem z tym kodem:

[PHP] pobierz, plaintext 
$p = $_GET['p'];
 
if($p == 1){
$login1 = $_GET['login'];
$aktywacja_kod = $_GET['aktywacja'];
$info = mysql_query("SELECT activ FROM login='$login1' WHERE aktywacja1");
if($info == $aktywacja_kod){
$login2 = $_GET['login'];
$info1 = mysql_query("SELECT activ FROM login='$login2' WHERE aktywacja2");
if($info1 == 0){
mysql_query("UPDATE activ SET aktywacja2='1' WHERE login='$login2'");
header("Location: index.php"); 
}
else{
echo 'Twoje konto zostalo juz aktywowane!';
}
}
else{
echo 'Niepoprawny kod';
}
}
[PHP] pobierz, plaintext 

Nie wyświetla pobiera mi kodu z bazy danych. A dokładniej to problem tkwi w $info bo jak wyświetlałem jego zawartość to była pusta.

Pozdrawiam

[PHP] pobierz, plaintext 
$info = mysql_query("SELECT activ FROM login='$login1' WHERE aktywacja1");
if($info == $aktywacja_kod){
[PHP] pobierz, plaintext 

W przyszłości, zamin zadasz pytanie przeczytaj kurs o podstawach, albo zajrzyj do manuala

[PHP] pobierz, plaintext 
$info = mysql_fetch_assoc(mysql_query("SELECT activ FROM login='$login1' WHERE aktywacja1"));
if($info['activ'] == $aktywacja_kod){
[PHP] pobierz, plaintext 

@up ty również

co do problemu - niewłaściwa składnia zapytania a dokładnie ten fragment

[SQL] pobierz, plaintext 
SELECT activ FROM login='$login1' WHERE aktywacja1
[SQL] pobierz, plaintext 

powinien wyglądać mniej więcej tak

[SQL] pobierz, plaintext 
SELECT activ FROM tabela WHERE login='".$login1."' AND aktywacja1 = '".$aktwyacja1."'
[SQL] pobierz, plaintext 

Ok, już sobie poradziłem, oto poprawny kod:

[PHP] pobierz, plaintext 
$info = mysql_fetch_assoc(mysql_query("SELECT * FROM activ WHERE login='$login1'"));
if($info['aktywacja1']==$aktywacja_kod){
[PHP] pobierz, plaintext 

//EDIT
Czyli wyszło że oboje mieliscie poniekąd racje.

Nie. Jeżeli chcesz sprawdzić czy podane argumenty znajdują się w bazie to pod żadnym pozorem nie robisz tego jak kolega wyżej opisał.

Włącz raportowanie błędów i podaj wartości, których nie ma w bazie

[PHP] pobierz, plaintext 
$p = $_GET['p'];
 
if($p == 1){
 
     $login1 = mysql_real_escape_string($_GET['login']);
     $aktywacja_kod = mysql_real_escape_string($_GET['aktywacja']);
 
     $result = mysql_query("SELECT kolumna FROM active WHERE login='".$login1."' WHERE aktywacja1 = '".$aktywacja1."'") or die(mysql_error());
 
     if(mysql_num_rows($result)) {
 
         $row = mysql_fetch_assoc($result);
         ....
     }
}
[PHP] pobierz, plaintext 

@daniel1302 ehhh... przyjrzyj się temu zapytaniu

Niekoniecznie błąd w składni. Bo WHERE sprawdza co zwraca warunek w tej klauzuli, a WHERE pole zwraca jeśli pole nie jest równe false lub puste i chyba dla 0 też warunek jest równy false. Sprawdziłem zapytanie w którym nie ma błędów, ale wszystko zależy od struktury bazy danych na tym forum prze różne dziwolągi widać, więc czasami już nie widze w tym nic dziwnego. Dokładnie jak powiedział kolega wyżej, nie robi się tego jak napisałem ja.

ten kod:
if($info['aktywacja1']==$aktywacja_kod){

Można zastąpić jak napisał użytkownik post wyżej przez dodanie w zapytaniu w klauzuli WHERE warunku: aktywacja1 = '".$aktywacja1."'

Jest to w pewnym rodzaju optymalizacja

Nie chodzi tu o wpisywanie warunków a o sposób pobierania danych

Załóżmy, że używamy "twojego" sposobu. Niech $login1 ma wartość "afeofjiwejfiow" (która nie istnieje w bazie). Taki zapis wywali błąd dla mysql_fetch_assoc() (mysql_query zwróci boolean zamiast resources)

[PHP] pobierz, plaintext 
$info = mysql_fetch_assoc(mysql_query("SELECT * FROM activ WHERE login='$login1'"));
[PHP] pobierz, plaintext 

Natomiast "mój" zapis najpierw sprawdzi czy jakiś zasób zostanie zwrócony za pomocą mysql_num_rows() a dopiero później ten zasób zostanie wykorzystany.

Ale to jest zbędne wykonywanie zapytania bo sprawdzać można za pomocą isset i wteddy jest jedno zapytanie mniej, może nie robi to różnicy przy małych projektach ale później na dole strony przy pobraniu 5 newsów widać takie głupoty jak:.