[SQL][PHP] Nie update'uje wyników do bazy Opcje

[Komiłaj]

Napisałem skrypt, na update'owanie rekordów do bazy.
Kod:

[PHP] pobierz, plaintext 
<?php
include('includes/header.php');
include('includes/menu.php'); 
include('includes/body_header.php');
 
 
if(!isset($_GET['site']))
	$_GET['site'] = 1;
 
$query = mysql_query("SELECT tresc, nazwa FROM strony WHERE id = '".$_GET['site']."'");
 
 			while ($row = mysql_fetch_array($query)) 
			{
				echo "
				<form method=\"post\" action=\"zmien.php\">
				<textarea name=\"nazwa\" cols=\"75\" rows=\"1\"/>".$row['nazwa']."</textarea><br />
				<textarea name=\"tresc\" cols=\"75\" rows=\"20\" >".$row['tresc']."</textarea><br /><br />
				<input type=\"hidden\" name=\"id\" value=\"".$_GET['site']."\" />
				<input type=\"submit\" value=\"Zmień\" />
				</form>
				";
			}
 
 
?>
[PHP] pobierz, plaintext 

Plik zmien.php:

[PHP] pobierz, plaintext 
<?php
include('includes/header.php');
include('includes/menu.php'); 
include('includes/body_header.php');
 
 
 
			if ($_POST)
			{
				$wstawka = ("UPDATE strony SET nazwa = ".$_POST['nazwa'].", tresc = ".$_POST['tresc']." WHERE id = ".$_POST['siteid']);
				$wstawienie = mysql_query($wstawka) or die(mysql_error());
				echo("Zmieniono zawartość strony. <a href=\"panel.php\">Wróć do panelu admina.</a><br />");
				echo $wstawka;
			}
			else
			{
			echo('Nie udało się zmienić strony.');
			}
 
 
 
?>
 
[PHP] pobierz, plaintext 

Wywala mi

Kod

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 'szkoda, że nie działa...

Co jest nie tak?

[PawelC]

Tu jest błąd:

[PHP] pobierz, plaintext 
".$_POST['siteid']);
[PHP] pobierz, plaintext 

Brakuje ."

Ten post edytował ExPlOiT 23.12.2010, 22:25:50

[Komiłaj]

To nie jest błąd. Tam nie powinno być ", bo zamknięcie łańcucha w tej zmiennej jest tu:

[PHP] pobierz, plaintext 
WHERE id = "
[PHP] pobierz, plaintext 

Potem dołączam do tego $_POST['site']

Zresztą jak dodam " to wywala błąd parsowania, czyli mam rację.
Nadal czekam na rozwiązanie...

[cycofiasz]

Każda wartość pola które jest typu varchar/text powinna być otoczona apostrofami lub cudzysłowiami

[PawelC]

Zobacz jaki masz kod w $query, a jaki w $wstawka, brak zamknięcia.

Ten post edytował ExPlOiT 23.12.2010, 22:36:40

[Komiłaj]

Dzięki. cycofiaszowe pomogło. Teraz nie ma tego syntaxu, ale jak update'uję, to nic się nie zmienia w bazie.

zmien.php po edycji:

[PHP] pobierz, plaintext 
<?php
include('includes/header.php');
include('includes/menu.php'); 
include('includes/body_header.php');
 
 
if ($_SESSION['adm'] == 1)
{
			if ($_POST)
			{
				$wstawka = ("UPDATE strony SET nazwa = '".$_POST['nazwa']."', tresc = '".$_POST['tresc']."' WHERE id = '".$_POST['siteid']."'");
				$wstawienie = mysql_query($wstawka) or die(mysql_error());
				echo("Zmieniono zawartość strony. <a href=\"panel.php\">Wróć do panelu admina.</a><br />");
				echo $wstawka;
			}
			else
			{
			echo('Nie udało się zmienić strony.');
			}
}
else
{
echo("Nie jesteś fajny");
}			
 
 
?>
[PHP] pobierz, plaintext 

Ten post edytował Komiłaj 23.12.2010, 22:40:02

[PawelC]

Bo wykonanie zapytanie masz przypisane do zmiennej:

[PHP] pobierz, plaintext 
$wstawienie = mysql_query($wstawka) or die(mysql_error());
[PHP] pobierz, plaintext 

A powinno być, tak:

[PHP] pobierz, plaintext 
 mysql_query($wstawka) or die(mysql_error());
[PHP] pobierz, plaintext 

Bez przypisanie do zmiennej. I wywal te nawiasy z zapytanie, bo one powinny być, ale w wypadku gdy masz:

[PHP] pobierz, plaintext 
$sql=mysql_query("select * from user");
[PHP] pobierz, plaintext 

A u Ciebie powinno być:

[PHP] pobierz, plaintext 
$sql="select * from user";
[PHP] pobierz, plaintext 

Edit//Poprawka

Ten post edytował ExPlOiT 23.12.2010, 22:50:36

[cycofiasz]

Wyświetl sobie zmienną $wstawka i sprawdź czy wszystkie dane są poprawnie uzupelnione, wykonaj otrzymany SQL w phpMyAdmin i sprawdź rezultat

[Komiłaj]

Dzięki, ExPlOiT działa. A wcześniej w kodzie miałem już:

[PHP] pobierz, plaintext 
echo $wstawka;
[PHP] pobierz, plaintext 

BTW.

A u Ciebie powinno być:

[PHP] pobierz, plaintext 
$sql="select * from user");
[PHP] pobierz, plaintext 

chyba raczej

[PHP] pobierz, plaintext 
$sql="select * from user";
[PHP] pobierz, plaintext 

Ten post edytował Komiłaj 23.12.2010, 22:48:05

[PawelC]

No mały błąd, na szybko pisałem, już poprawiłem

echo nie wykona zapytania, tylko wyświetli zawartość $wstawka

Ten post edytował ExPlOiT 23.12.2010, 22:50:19

[Komiłaj]

ExPlOiT, chodziło mi jako odpowiedź cycofiaszowi na:

Wyświetl sobie zmienną $wstawka i sprawdź czy wszystkie dane są poprawnie uzupelnione, wykonaj otrzymany SQL w phpMyAdmin i sprawdź rezultat

Ten post edytował Komiłaj 23.12.2010, 22:55:34