[PHP] Problem z edycją - przekazywanie z PHP do mysql Opcje

[--Marcin--]

Witam wszystkich,
jestem początkujący w php, mam nadziję, że nie będzie dla Was problemem to co mnie męczy już parę dni

piszę sobie (a właściwię dopiero się uczę) mały skrypcik, chcę móc dodawać pojazdy do bazy danych, usuwać, edytować, w miarę czasu i nauki będe go rozbudowywał ale to już przyszłość

dodawanie, usuwanie, wypisywanie danych z tabeli działa bez problemu, jednak z edycją jest pewien problem, podejrzewam, że błąd jest w zapytaniu, ale nie dostaję żadnego błedu - nie dokonuję się też aktualizacja

plik pojazdy.php - zarządzanie pojazdami, czyli dodawanie, edycja, usuwanie - część pliku jest w komentarzu, bo tak sobie kombinuję na różne sposoby i nie chcę pisać od nowa

[PHP] pobierz, plaintext 
<?php
	include ('naglowek.php');
	include ('form_trucks_add.php');
 
	echo '<hr width="610px">';
 
if(isset($_GET['akcja']))
{
	// kasowanie pojazdu
  if ($_GET['akcja']=='delete') 
	{
		$id_do_usuniecia = $_GET['id'];
		$zapytanie ="DELETE FROM trucks WHERE id=$id_do_usuniecia";
		$wynik2=mysql_query ($zapytanie) or die(mysql_error());
 
		if ($wynik2)
		echo "<center><p> Ilość usuniętych rekordów: ". mysql_affected_rows($polaczenie) ."</p></center>";
 
	}
	// edytowanie pojazdu
  if ($_GET['akcja']=='edit')
{
	include ('form_trucks_edit.php'); // dołączenie formularza edycji pojazdu
 
	/*
	{
 
		$id_ed = 'id';
		$nr_rej_ed = 'nr_rej';
		$marka_ed = 'marka';
		$data_prod_ed = 'data_prod';
		$nr_nadw_ed = 'nr_nadw';
 
		$zapytanie5 = "UPDATE trucks SET nr_rej='$nr_rej_ed', marka='$marka_ed', data_prod='$data_prod_ed', nr_nadw='$nr_nadw_ed' WHERE id='$id_ed'";
		$wynik5=mysql_query ($zapytanie5) or die(mysql_error());
 
			if ($wynik5) 
				include ('form_trucks_edit.php');
 
 
			else 
				echo "<p>Aktualizacja nie powiodła się!</p>";
 
	}*/
}
}
$zapytanie = 'SELECT id, nr_rej, marka, data_prod, nr_nadw FROM trucks';
$wynik = mysql_query($zapytanie, $polaczenie) or die(mysql_error());
?>
 
<center>
<br>
	<table border="0" cellpadding="3" cellspacing="3" width="610px">
		<tr>
			<th colspan="4">Pojazdy aktualnie znajdujące się w bazie:</th>
		</tr>
		<tr style="background-color: #eaeaea;">
			<th width="160px">Numer rejestracyjny</th>
			<th width="160px">Marka</th>
			<th width="110px">Data produkcji</th>
			<th width="140px">Numer nadwozia</th>
			<!--<th width="25px"></th>
			<th width="25px"></th>-->
		</tr>
 
 
<?php
 
 
//wybieranie z tabeli wszystkich pojazdów
 
while ($rekord = mysql_fetch_assoc($wynik)) {
 
	echo "<tr style=\"background-color: #eaeaea;\">";
 
	echo "<td>" . $rekord['nr_rej'] . "</td><td>" . $rekord['marka'] . "</td><td>" . $rekord['data_prod'] . "</td><td>" . $rekord['nr_nadw'] . "</td>";
?>
	<th width="20px">
	<a href="<?php echo $_SERVER['PHP_SELF'];?>?akcja=delete&id=<?php echo $rekord['id'];?>" onclick="return (confirm('Jesteś pewien, że chcesz usunąć pojazd ? \r\n \r\nOperacja jest nieodwracalna !!!'))"><img src="./img/drop.png" border="0"</a>
	</th>
	<th width="20px">
	<a href="<?php echo $_SERVER['PHP_SELF'];?>?akcja=edit&id=<?php echo $rekord['id'];?>" onclick="return (confirm('Jesteś pewien, że chcesz edytować dany pojazd ?'))"><img src="./img/edit.png" border="0"</a>
	</th>
 
<?php	
	echo "</tr>\n";
 
}
?>
</table>
 
</center>
 
 
<?php include ('stopka.php') ?>
[PHP] pobierz, plaintext 

plik form_trucks_edit.php - czyli sama edycja pojazdów

[PHP] pobierz, plaintext 
<?php
 
// formularz edycji poszczególnego rekordu
 
if (isset($_GET['akcja']) == 'edit')
{
 
 
	$zapytanie = "SELECT * FROM trucks WHERE id=".$_GET['id'];
	$produkt = mysql_query($zapytanie);
	$rekord = mysql_fetch_assoc($produkt);
 
 
 
// edytowanie pojazdu w bazie
 
	if (isset($_POST['zmien']))
	{
 
	// odbieramy dane z formularza
 
 
		$nr_rej = $_POST['nr_rej'];
		$marka = $_POST['marka'];
		$data_prod = $_POST['data_prod'];
		$nr_nadw = $_POST['nr_nadw'];
		//sprawdzenie poprawności wpisanych danych, później zabezpieczyć !!!
		if (!empty($nr_rej) && !empty($marka) && is_numeric($data_prod) && !empty($nr_nadw))
		{
 
		// uaktualniamy tabelę
 
 
		$zapytanie = "UPDATE trucks SET nr_rej='$nr_rej', marka='$marka', data_prod='$data_prod', nr_nadw='$nr_nadw' WHERE id='$id'";
 
		$wynik = mysql_query($zapytanie) or die ('Nie dodano rekordu, błąd :'. mysql_error());
		echo "<p /> Poprawnie zedytowano w bazie ". mysql_affected_rows($polaczenie) ." pojazd";
 
		}
			else 
			{
			echo "Błednie podane dane";
			}
	}
}
?>	
 
 
	<center><form action="pojazdy.php" method="POST">
 
		ID: <input readonly name="id" value="<?php echo $rekord['id'] ?>" size="2"/>
 
		Nr rej: <input name="nr_rej" value="<?php echo $rekord['nr_rej'] ?>" size="10" maxlength="9"/>
 
		Marka: <input name="marka" value="<?php echo $rekord['marka'] ?>" size="15" />
 
		Data prod: <input name="data_prod" value="<?php echo $rekord['data_prod'] ?>" size="5" />
 
		Nr nadw: <input name="nr_nadw" value="<?php echo $rekord['nr_nadw'] ?>" size="20" />
 
		<input type="submit" name="zmien" value="Zapisz" />
 
	</form></center>
[PHP] pobierz, plaintext 

będę wdzięczny za choćby naprowadzenie na błąd jaki zrobiłem

[Barcelona]

Wyechuj te zmienne z tego zapytania UPDATE trucks SET nr_rej='$nr_rej', marka='$marka', data_prod='$data_prod', nr_nadw='$nr_nadw' WHERE id='$id i zobacz czy faktycznie pobiera Ci dane.

[--Marcin--]

zrobiłem tak, wklejam tylko fragment kodu

[PHP] pobierz, plaintext 
		$zapytanie = "UPDATE trucks SET nr_rej='$nr_rej', marka='$marka', data_prod='$data_prod', nr_nadw='$nr_nadw' WHERE id='$id'";
 
 
			echo $nr_rej;
			echo $marka;
			echo $data_prod;
			echo $nr_nadw;
			echo $id;
 
 
 
		$wynik = mysql_query($zapytanie) or die ('Nie dodano rekordu, błąd :'. mysql_error());
[PHP] pobierz, plaintext 

nie wyświetliły się zmienne

[nospor]

if (isset($_GET['akcja']) == 'edit')
Przecież wysyłając forma, nie masz ustawionego parametru akcja w get.

[--Marcin--]

to jak to powinno wyglądać, bo nie mogę zrozumieć

coś kombinuję, ale nie wychodzi

[nospor]

No skoro warunkujesz to istnieniem akcja w get, no to dodaj to do forma.....

<form action="pojazdy.php?akcja=edit" method="POST">
Mysl troszke.

Poza tym, isset zwraca TRUE lub FALSE i nie możesz tego porównywać do tekstu edit...
nie:if (isset($_GET['akcja']) == 'edit')
a: if (isset($_GET['akcja']) && $_GET['akcja'] == 'edit')

[Barcelona]

Tudzież dla zwiększenia bezpieczeństwa użyj

[HTML] pobierz, plaintext 
<form action="pojazdy.php" method="POST">
<input type="hidden" name="akcja" value="edit" />
....
</form>
[HTML] pobierz, plaintext 

[--Marcin--]

wielkie dzięki za pomoc, teraz działa, choć pokazały się dwa błędy, ale nimi powinienem dać sobie radę

[nospor]

@Barcelona przecież to nie zwiększa żadnego bezpieczeństwa.

[Barcelona]

@Nospor, nie wiem, ja tam zawsze wolę nie pokazywać userom co się dzieje w skrypcie i daje wszystkie warunki postem. User nie będzie widział pojazdy.php?akcja=edit, tylko samo pojazdy.php
Jak ktoś jest dociekliwy to sobie spojrzy w source.
Ale i tak masz rację że to nie zwiększa bezpieczeństwa.