Witam, już poradziłem sobie z dodawaniem rekordów, oraz usuwaniem rekordów, ale mam drobny problem z edycją.
W tabeli `filmy` mamy takie kolumny: id, nazwa, gatunek, ocena, info.

Utworzyłem plik edytuj_film.php, który pobiera zawartości kolumny `filmy`
Obok każdego nowego filmu jest opcja EDYCJI wpisu. Wpis da się edytować, ale są problemy z zatwierdzeniem zmian.
Wyskakuje tylko komunikat że Edycja nie powiodła się.
Oto fragment kodu, odpowiadającego za edycje.

[PHP] pobierz, plaintext 
<?php
if ($_GET['akcja'] == 'edytuj') {
 
 $zapytanie = "SELECT * FROM filmy WHERE id=".$_GET['id'];
 $filmy2 = mysql_query($zapytanie);
 $rekord = mysql_fetch_assoc($filmy2);
?>
[PHP] pobierz, plaintext 

http://wklejto.pl/6822 (już nie będę śmiecił tematu

[PHP] pobierz, plaintext 
<?php
}
 
if ($_POST['zmien']) {
 
$zapytanie = "UPDATE filmy SET nazwa='".$_POST['nazwa']."', gatunek='".$_POST['gatunek']."', ocena='".$_POST['ocena']."', info='".$_POST['info']. " WHERE id=".$_POST['id'];
$filmy2 = mysql_query($zapytanie);
 
 if ($wykonaj) 
 echo "all ok";
 else 
 echo "operacja niepowiodla sie:/";
 
}
?>
[PHP] pobierz, plaintext 

Podejrzewam że błąd jest raczej w $zapytanie = "UPDATE.." więc prosiłbym o korektę.

Masz błąd z cudzysłowami w UPDATE, zamień je na

[PHP] pobierz, plaintext 
<?php
$zapytanie = "UPDATE filmy SET nazwa='".$_POST['nazwa']."', gatunek='".$_POST['gatunek']."', ocena='".$_POST['ocena']."', info='".$_POST['info']."' WHERE id=".$_POST['id'];
?>
[PHP] pobierz, plaintext 

i jeszcze;
if ($wykonaj)
zmienić też trzeba na:
if ($zapytanie)

Nie zauważyłem ;[ Ale i tak nici z tego ..

Pisze że all ok, ale gdy odświeżam stronę, rekord nadaj jest o tej samej nazwie.

Twój błąd tkwi w tym, że nigdzie nie masz zdefiniowanej zmiennej $_POST['id']

Zamiast tego użyj $_GET['id']
a w formularzu zamiast używać

action=$_SERVER['PHP_SELF']

zrób coś takiego

action=nazwapliku.php?id=$rekord['id']

nie zapomnij o walidacji danych wejsciowych

w zapytaniu SQL:

zamiast $_GET['id'] użyj (int)$_GET['id'] , bedziesz mial pewnosc ze tam bedzie cyfra a nie zastrzyk SQL
zamiast $_POST['nazwa'] używaj htmlspecialchars($_GET['nazwa']) jak wyzej, tyle ze dla lancucha tekstu

to, że danych "nie widać" bo leca przez POST zamiast GET, nie znaczy, że nie da się ich oszukać, nawet gdy jest to lista select, czy checkbox, wysłanie dowolnych danych POST do twojego skryptu jest banalnie proste, wiec zatroszcz sie ich filtrowanie, polecam 2 funkcje:

1. zapewnia wstepna filtrację danych

[PHP] pobierz, plaintext 
<?php
function prefilter(&$data) {
		if(is_array($data)) 
			array_map('prefilter', &$data); 
				else if (get_magic_quotes_gpc) 
			$data= htmlspecialchars(stripslashes($data),ENT_QUOTES); 
				else
			$data= htmlspecialchars($data,ENT_QUOTES);			 
	}
// uzycie
prefilter(&$_POST);
prefilter(&$_GET);
?>
[PHP] pobierz, plaintext 

2. wyrejestrowuje zmienne przy opcji register_globals ON

[PHP] pobierz, plaintext 
<?php
function unregister_globals() {
$arrays = array('_REQUEST', '_SESSION', '_SERVER', '_ENV', '_FILES', '_COOKIE');
	if (ini_get(register_globals)) {
		foreach ($arrays as $value) {
			foreach ($GLOBALS[$value] as $key => $var) {
				if ($var === $GLOBALS[$key]) {
					unset($GLOBALS[$key]);
				}//if
			}//foreach
		}//foreach
	} // if
 }// unrefgister_globals()
?>
[PHP] pobierz, plaintext