Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [SQL][PHP] Nie update'uje wyników do bazy
Forum PHP.pl > Forum > Przedszkole
Komiłaj
23.12.2010, 22:19:12
Napisałem skrypt, na update'owanie rekordów do bazy.
Kod:
[PHP] pobierz, plaintext 
  1. <?php
  2. include('includes/header.php');
  3. include('includes/menu.php'); 
  4. include('includes/body_header.php');
  5.  
  6.  
  7. if(!isset($_GET['site']))
  8. 	$_GET['site'] = 1;
  9.  
  10. $query = mysql_query("SELECT tresc, nazwa FROM strony WHERE id = '".$_GET['site']."'");
  11.  
  12.  			while ($row = mysql_fetch_array($query)) 
  13. 			{
  14. 				echo "
  15. 				<form method=\"post\" action=\"zmien.php\">
  16. 				<textarea name=\"nazwa\" cols=\"75\" rows=\"1\"/>".$row['nazwa']."</textarea><br />
  17. 				<textarea name=\"tresc\" cols=\"75\" rows=\"20\" >".$row['tresc']."</textarea><br /><br />
  18. 				<input type=\"hidden\" name=\"id\" value=\"".$_GET['site']."\" />
  19. 				<input type=\"submit\" value=\"Zmień\" />
  20. 				</form>
  21. 				";
  22. 			}
  23.  
  24.  
  25. ?>
[PHP] pobierz, plaintext



Plik zmien.php:

[PHP] pobierz, plaintext 
  1. <?php
  2. include('includes/header.php');
  3. include('includes/menu.php'); 
  4. include('includes/body_header.php');
  5.  
  6.  
  7.  
  8. 			if ($_POST)
  9. 			{
  10. 				$wstawka = ("UPDATE strony SET nazwa = ".$_POST['nazwa'].", tresc = ".$_POST['tresc']." WHERE id = ".$_POST['siteid']);
  11. 				$wstawienie = mysql_query($wstawka) or die(mysql_error());
  12. 				echo("Zmieniono zawartość strony. <a href=\"panel.php\">Wróć do panelu admina.</a><br />");
  13. 				echo $wstawka;
  14. 			}
  15. 			else
  16. 			{
  17. 			echo('Nie udało się zmienić strony.');
  18. 			}
  19.  
  20.  
  21.  
  22. ?>
  23.  
[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
23.12.2010, 22:24:56
Tu jest błąd:
[PHP] pobierz, plaintext 
  1. ".$_POST['siteid']);
[PHP] pobierz, plaintext

Brakuje ."
Komiłaj
23.12.2010, 22:29:00
To nie jest błąd. Tam nie powinno być ", bo zamknięcie łańcucha w tej zmiennej jest tu:
[PHP] pobierz, plaintext 
  1. 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
23.12.2010, 22:32:34
Każda wartość pola które jest typu varchar/text powinna być otoczona apostrofami lub cudzysłowiami
PawelC
23.12.2010, 22:35:17
Zobacz jaki masz kod w $query, a jaki w $wstawka, brak zamknięcia.
Komiłaj
23.12.2010, 22:37:40
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 
  1. <?php
  2. include('includes/header.php');
  3. include('includes/menu.php'); 
  4. include('includes/body_header.php');
  5.  
  6.  
  7. if ($_SESSION['adm'] == 1)
  8. {
  9. 			if ($_POST)
  10. 			{
  11. 				$wstawka = ("UPDATE strony SET nazwa = '".$_POST['nazwa']."', tresc = '".$_POST['tresc']."' WHERE id = '".$_POST['siteid']."'");
  12. 				$wstawienie = mysql_query($wstawka) or die(mysql_error());
  13. 				echo("Zmieniono zawartość strony. <a href=\"panel.php\">Wróć do panelu admina.</a><br />");
  14. 				echo $wstawka;
  15. 			}
  16. 			else
  17. 			{
  18. 			echo('Nie udało się zmienić strony.');
  19. 			}
  20. }
  21. else
  22. {
  23. echo("Nie jesteś fajny");
  24. }			
  25.  
  26.  
  27. ?>
[PHP] pobierz, plaintext
PawelC
23.12.2010, 22:39:26
Bo wykonanie zapytanie masz przypisane do zmiennej:
[PHP] pobierz, plaintext 
  1. $wstawienie = mysql_query($wstawka) or die(mysql_error());
[PHP] pobierz, plaintext

A powinno być, tak:
[PHP] pobierz, plaintext 
  1.  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 
  1. $sql=mysql_query("select * from user");
[PHP] pobierz, plaintext

A u Ciebie powinno być:
[PHP] pobierz, plaintext 
  1. $sql="select * from user";
[PHP] pobierz, plaintext


Edit//Poprawka
cycofiasz
23.12.2010, 22:40:32
Wyświetl sobie zmienną $wstawka i sprawdź czy wszystkie dane są poprawnie uzupelnione, wykonaj otrzymany SQL w phpMyAdmin i sprawdź rezultat
Komiłaj
23.12.2010, 22:47:31
Dzięki, ExPlOiT działa. A wcześniej w kodzie miałem już:
[PHP] pobierz, plaintext 
  1. echo $wstawka;
[PHP] pobierz, plaintext


BTW.
Cytat(ExPlOiT @ 23.12.2010, 22:39:26 ) *
A u Ciebie powinno być:
[PHP] pobierz, plaintext 
  1. $sql="select * from user");
[PHP] pobierz, plaintext


chyba raczej 
[PHP] pobierz, plaintext 
  1. $sql="select * from user";
[PHP] pobierz, plaintext
smile.gif
PawelC
23.12.2010, 22:49:41
No mały błąd, na szybko pisałem, już poprawiłem smile.gif

echo nie wykona zapytania, tylko wyświetli zawartość $wstawka
Komiłaj
23.12.2010, 22:55:08
ExPlOiT, chodziło mi jako odpowiedź cycofiaszowi na:
Cytat(cycofiasz @ 23.12.2010, 22:40:32 ) *
Wyświetl sobie zmienną $wstawka i sprawdź czy wszystkie dane są poprawnie uzupelnione, wykonaj otrzymany SQL w phpMyAdmin i sprawdź rezultat
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.