(PDO) Błąd w INSERT INTO i WHERE jak to napisać ? Opcje

[Dyktus]

[PHP] pobierz, plaintext 
     try
        {
 
                $pdo -> setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
 
    $doit = $pdo -> exec('INSERT INTO `users` (`active`)   VALUES(\''.$aktywny.'\') WHERE `login`='.$login.'');
 
 
        }      catch(PDOException $e)
        {
                echo 'Wystąpił błąd biblioteki PDO: ' . $e->getMessage();
        }
 
[PHP] pobierz, plaintext 

Mam takie zapytanie do bazy danych (IMG:style_emoticons/default/smile.gif) wykonane przy użyciu PDO (oczywiście wcześniej wszystko ok łącze się z bazą itd.)

Jeśli chodzi o PDO posiłkowałem się WikiBooks by choć podstawę tego ogarnąć no i w tym kodzie próbuje ustawić użytkownika jako aktywnego. Buduje sobie po prostu taki system logowania. Wiadomo, że muszę sprawdzić którego użytkownika chcę aktywować dlatego jest to WHERE tylko czy jest Ono w dobrym miejscu ?

Oczywiście

[PHP] pobierz, plaintext 
$aktywny=1;
$login='przyklad';
[PHP] pobierz, plaintext 

Aktywny jest równy 1 ponieważ pole active w bazie danych jest typu int.

Treść błędu:

Wystąpił błąd biblioteki PDO: SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'WHERE `login`=test' at line 1test

Próbowałem chyba już na wszelkie sposoby to wpisać, nawet zwykłe zapytanie typu:

[PHP] pobierz, plaintext 
$doit = $pdo -> exec('INSERT INTO users SET `active`=1 WHERE `login`='.$login.');
[PHP] pobierz, plaintext 

Jeśli ktoś z was ma pomysł co tu jest źle to z góry dziękuję za odpowiedź.

[askone]

[PHP] pobierz, plaintext 
$doit = $pdo -> exec('INSERT INTO users SET `active`=1 WHERE `login`='.$login.');
[PHP] pobierz, plaintext 

Jeśli ktoś z was ma pomysł co tu jest źle to z góry dziękuję za odpowiedź.

Według mnie błąd jest trywialny spróbuj tak:

[PHP] pobierz, plaintext 
$doit = $pdo -> exec('INSERT INTO users SET `active`= 1 WHERE `login`=\''.$login.'\'');
[PHP] pobierz, plaintext 

Pozdrawiam

[nospor]

Oczywiście kod askone jest poprawny, ale skoro już korzystamy z PDO to czemu nie skorzystać z dobrodziejstwa jakie ono niesie i nie zacząć bindować zmiennych? Bez sensu jest używać PDO i robić wszystko nadal jak w zwykłym mysql_query()

[konole]

http://pl.wikibooks.org/wiki/PHP/Biblioteka_PDO#Podpinanie

[Dyktus]

Wielkie dzięki za pomoc (IMG:style_emoticons/default/smile.gif) nospore tak właściwie to masz racje (IMG:style_emoticons/default/smile.gif) dostosuje się dzięki wielkie