[php/sql] nie udało się: Unknown column '$id' in 'where clause', gdzie popełniam błąd Opcje

[wojtekwro]

[PHP] pobierz, plaintext 
<?php
session_start();
if ($_SESSION['LOGGED'] != TRUE) {
 echo 'nie masz uprawnień';
 exit;
}
if ($_SESSION['pelno'] != 1) { echo 'brak uprawnień';}
else {
$id=$_POST['id'];
$akcep=$_POST['akcep'];
	//laczenie z bd
	$connection = @mysql_connect('localhost', 'root', '')
	or die('brak polaczenia z BD');
$db = @mysql_select_db("ankiety")
or die('Nie mogę połączyć się z bazą danych');
$sql = 'UPDATE `szkolenia` SET `akcep` = '1' WHERE `szkolenia`.`id` = $id';
$sql = mysql_query($sql)
or die("nie udało się: " . mysql_error());
 
}
?>
[PHP] pobierz, plaintext 

[SQL] pobierz, plaintext 
szkolenia
Pole 	Typ 	NULL 	Domyślnie
id 	int(11) 	Nie  	  	 
temat 	varchar(50) 	Nie  	  	 
uczestnicy 	varchar(30) 	Nie  	  	 
DATA 	varchar(10) 	Nie  	yyyy-mm-dd  	 
organizator 	varchar(20) 	Nie  	  	 
doki 	varchar(30) 	Nie  	  	 
cena 	varchar(10) 	Nie  	  	 
akcep 	varchar(2) 	Nie  	  	 
oc1 	varchar(2) 	Nie  	  	 
oc2 	varchar(2) 	Nie
[SQL] pobierz, plaintext 

sypie mie takim błedem jak w topicu jeżeli usunę z zapytani sql= warunek where id=$id to zapisze mi wartość pola akcep dla całej kolumny =1 ja chcę aby robił to tylko dla id z poprzedniego skryptu. Brak pomysłu jak to obejśc poprawić proszę o pomoc.

Ten post edytował wojtekwro 23.05.2007, 10:38:55

[nospor]

No chyba wyraźnie masz napisane: Nieznana kolumna '$id' . TO juz mozna sie probowac domyslic, ze zamiast przekazac wartosc zmiennej $id to przekazales tekst '$id'

powinno byc:

[PHP] pobierz, plaintext 
<?php
$sql = 'UPDATE `szkolenia` SET `akcep` = '1' WHERE `szkolenia`.`id` ='. $id;
?>
[PHP] pobierz, plaintext 

Poczytaj w manualu jaka jest roznica miedzy apostrofem a cudzyslowiem

[wojtekwro]

jak poprawie to tak jak mówisz wali dalszym błędem

[PHP] pobierz, plaintext 
<?php
nie udało się: 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 ''1' WHERE `szkolenia`.`id` ='. $id' at line 1
?>
[PHP] pobierz, plaintext 

poniżej zamieszcze fragment skryptu, który przekazuje $id dalej

[PHP] pobierz, plaintext 
<?php
mysql_connect ("127.0.0.1","root","");
mysql_select_db (ankiety);
$res = mysql_query("SELECT  id, akcep, oc1, oc2 FROM szkolenia WHERE id=$id LIMIT 1");
$row= mysql_fetch_array($res);
$wiersz['id']=$id;
$wiersz['akcep']=$akcep;
$wiersz['oc1']=$il2;
$wiersz['oc2']=$il3;
echo 'akceptacja: <b>';	
echo $akcep;
$ile = $row["akcep"];
$il2 = $row["oc1"];
$il3 = $row["oc2"];
$id = $row["id"];
 
 
if($ile !='1')
	{echo '</b>oczekuje do akceptacji<br>';if ($_SESSION['pelno'] != 1) {} 
	else 
	{echo '<form action="akceptuj.php" method="post">
			<input type="hidden" name="id" value="$id">
			<input type="checkbox" name="akcep" value="1" />akceptacja  szkolenia.<br>
			<input type="submit" value="dodaj" />
				</form>
		';}
?>
[PHP] pobierz, plaintext 

[nospor]

po pierwsze: co ty mi za kod pokazujesz? Przeciez on ma sie nijak do komunikatu bledu. Blad masz w tym samym kodzie co pokazales na poczatku

po drugie: kombinacji Ctrl+C, Ctrl+v to też trzeba umiec uzywac. Zobacz co ja ci kazalem wstawic, a spojrz co ty wstawiles w swoim kodzie (oczywiscie wnioskuje, ze źle skopiowales, po komunikacie bledu, a nie po kodzie, gdyz kodu po poprawce nie pokazales)

[wojtekwro]

sorki źle wkleiłem ale jak wlepiłem tak jak kazałeś sypie tym samym

[PHP] pobierz, plaintext 
<?php
nie udało się: Unknown column '$id' in 'where clause'
?>
[PHP] pobierz, plaintext 

[nospor]

sorki źle wkleiłem ale jak wlepiłem tak jak kazałeś sypie tym samym

Nie ma prawa.
Pokaz kod po moich zmianach.
Dla pewnosci zrob tez: echo $id; i powiedz mi co to wyswietla

[Darti]

użyj innych cudzysłowów, nie pojedynczych (') a podwójnych ("):

[PHP] pobierz, plaintext 
<?php
$sql = "UPDATE `szkolenia` SET `akcep` = '1' WHERE `szkolenia`.`id` = $id";
?>
[PHP] pobierz, plaintext 

albo daj $id poza stringa tak jak napisał Ci @nospor

[jasiek]

powinno pomoc :

$akcept="1";
$sql = "UPDATE szkolenia SET akcep = $akcept WHERE id=$id";

[wojtekwro]

[PHP] pobierz, plaintext 
<?php
session_start();
if ($_SESSION['LOGGED'] != TRUE) {
 echo 'nie masz uprawnień';
 exit;
}
if ($_SESSION['pelno'] != 1) { echo 'brak uprawnień';}
else {
$id=$_POST['id'];
$akcep=$_POST['akcep'];
 
	//laczenie z bd
	$connection = @mysql_connect('localhost', 'root', '')
	or die('brak polaczenia z BD');
$db = @mysql_select_db("ankiety")
or die('Nie mogę połączyć się z bazą danych');
$sql = 'UPDATE `szkolenia` SET `akcep` = '1' WHERE `szkolenia`.`id` ='. $id;
$sql = mysql_query($sql)
or die("nie udało się: " . mysql_error());
print $sql;
}
?>
[PHP] pobierz, plaintext 

echo $id; wyświetliło mi $id czyli nie pobiera zmiennej z poprzedniego skryptu tam szukac błędu

[nospor]

w tamtym skrypcie masz ten sam blad, czyli nie:

[PHP] pobierz, plaintext 
<?php
echo '<form action="akceptuj.php" method="post">
			<input type="hidden" name="id" value="$id">....
?>
[PHP] pobierz, plaintext 

a:

[PHP] pobierz, plaintext 
<?php
echo '<form action="akceptuj.php" method="post">
			<input type="hidden" name="id" value="'.$id.'">....
?>
[PHP] pobierz, plaintext 

Mowilem bys poczytal o roznicy miedzy ' a "
http://pl.php.net/manual/pl/language.types.string.php

[wojtekwro]

ruszyło dzięki wszystkim za pomoc a manuala przeczytam dzisiaj w domu ;]

pozdrawiam