Witam,
zrobiłem system notatek administracyjnych w moim panelu administracyjnym. Z dodawaniem notatek wszystko jest w porządku. Gorzej jest z ich usuwaniem. Mam taki kod:

[PHP] pobierz, plaintext 
$pobierzNotatka = $db->prepare("SELECT `id`, `nick`, `notatka`, `data` FROM `notatki_admin`");
 
if(!$pobierzNotatka->execute()){
	throw new Exception("Błąd zapytania z NOTATKAMI ADMINISTRATORÓW.");
}
 
foreach($pobierzNotatka as $row_not_admin)
{
	echo "</br>Nick: \t";
	echo('<span class = "red">');
	echo ''.$row_not_admin['nick'].'</br>' ;
	echo('</span>');
 
	echo "Data: \t";
	echo('<span class = "red">');
	echo ''.$row_not_admin['data'].'</br>';
	echo('</span>');
 
	echo "Notatka: \t";
	echo('<span class = "red">');
	echo ''.$row_not_admin['notatka'].'</br>';
	echo('</span>');
 
	echo '- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -<a href="notatki.php?id='.$row_not_admin['id'].'">[X]</a></br>';
	if(isset($_GET["id"]))
	{
		$id = str_replace("\'","\\\'",$_GET["id"]);
		$usunNotatka = $db->prepare("DELETE FROM `notatki_admin` WHERE `id` = :id");
		$usunNotatka->bindValue(":id", $id, PDO::PARAM_INT);
	}
}
[PHP] pobierz, plaintext 

Gdy kliknę na [X] w linku poprawnie pobiera id notatki. Niestety na tym się kończy. Notatka się nie usuwa. Co zrobić? Jakieś pomysły/rady?

[PHP] pobierz, plaintext 
	if(isset($_GET["id"]))
	{
		$usunNotatka = $db->prepare("DELETE FROM `notatki_admin` WHERE `id` = :id");
		$usunNotatka->bindValue(":id", $id, PDO::PARAM_INT);
                $usunNotatka->execute();
	}
[PHP] pobierz, plaintext 

Niestety nie działa. Po próbowałem i mam coś takiego:

[PHP] pobierz, plaintext 
$pobierzNotatka = $db->prepare("SELECT `id`, `nick`, `notatka`, `data` FROM `notatki_admin`");
 
if(!$pobierzNotatka->execute()){
	throw new Exception("Błąd zapytania z NOTATKAMI ADMINISTRATORÓW.");
}
 
foreach($pobierzNotatka as $row_not_admin)
{
	echo "</br>Nick: \t";
	echo('<span class = "red">');
	echo ''.$row_not_admin['nick'].'</br>' ;
	echo('</span>');
 
	echo "Data: \t";
	echo('<span class = "red">');
	echo ''.$row_not_admin['data'].'</br>';
	echo('</span>');
 
	echo "Notatka: \t";
	echo('<span class = "red">');
	echo ''.$row_not_admin['notatka'].'</br>';
	echo('</span>');
 
	echo '- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -<a href="notatki.php?id='.$row_not_admin['id'].'">[X]</a></br>';
	if(isset($_GET["id"]))
	{
		$usunNotatka = $db->prepare("DELETE FROM `notatki_admin` WHERE `id` = :id");
		$usunNotatka->bindValue(":id", $row_not_admin['id'], PDO::PARAM_INT);
		$usunNotatka->execute();
	}
}
[PHP] pobierz, plaintext 

lecz ten kod usuwa wszystkie notatki. I jakby tego było mało link nie reaguje na 1 kliknięcie zadziała dopiero po drugim(i wszystko usuwa). Nie wiem czemu. Co jest nie tak?

Zgubiłem jedną linijkę

Poza tym włącz sobie wyświetlanie błędów, powinno Ci od razu wyskoczyć, że nie masz $id zdefiniowanego.

[PHP] pobierz, plaintext 
if(isset($_GET["id"]))
	{
                $id = $_GET["id"]; // str_replace jest niepotrzebne jak bindujesz dane
		$usunNotatka = $db->prepare("DELETE FROM `notatki_admin` WHERE `id` = :id");
		$usunNotatka->bindValue(":id", $id, PDO::PARAM_INT); // tu też był błąd. Po co to dodałeś? "$row_not_admin['id']" ?
		$usunNotatka->execute();
	}
[PHP] pobierz, plaintext 

Ale czemu trzeba kliknąć 2 razy na [X] żeby usunęło?

foreach($pobierzNotatka as $row_not_admin)
Co to ma być Gdzie w ogóle podział się jakich fetch()

Ale jak ja mam użyć do tego fetch()? Przecież zwracam kilka wartości(więc dałem pętlę). Jak mam tego użyć?

W PDO powinno to być tak

[PHP] pobierz, plaintext 
$pobierzNotatka = $db->prepare("SELECT `id`, `nick`, `notatka`, `data` FROM `notatki_admin`");
 
if(!$pobierzNotatka->execute()){
throw new Exception("Błąd zapytania z NOTATKAMI ADMINISTRATORÓW.");
}
 
$rows = $pobierzNotatka->fetchAll(PDO::FETCH_ASSOC);
foreach($rows as ....){
 
}
[PHP] pobierz, plaintext 

No tak, ale powiedz co to zmienia? Działanie jest takie samo. A pomimo tego nadal trzeba 2 razy kliknąć, żeby usunąć.

Teoretycznie nie powinieneś mieć w ogóle tego [X] widocznego, praktycznie masz.

Daj to:

[PHP] pobierz, plaintext 
if(isset($_GET["id"]))
	{
                $id = $_GET["id"]; // str_replace jest niepotrzebne jak bindujesz dane
		$usunNotatka = $db->prepare("DELETE FROM `notatki_admin` WHERE `id` = :id");
		$usunNotatka->bindValue(":id", $id, PDO::PARAM_INT); // tu też był błąd. Po co to dodałeś? "$row_not_admin['id']" ?
		$usunNotatka->execute();
	}
[PHP] pobierz, plaintext 

na początek pliku. Może teraz nie trzeba będzie klikać 2 razy.

A to coś z tym [X] jest nie tak? Nadal gdy dam na początek trzeba 2 razy kliknąć. Pierwszy raz, żeby "wczytało link do przeglądarki", drugi raz żeby usunąć. A chodzi mi o to, żeby od razu usuwało.

[PHP] pobierz, plaintext 
	if(isset($_GET["id"]))
{
$id = str_replace("\'","\\\'",$_GET["id"]);
$usunNotatka = $db->prepare("DELETE FROM `notatki_admin` WHERE `id` = :id");
$usunNotatka->bindValue(":id", $id, PDO::PARAM_INT);
}
[PHP] pobierz, plaintext 

nie trzymaj tego w pętli...

Nie ma to znaczenia gdzie jest, czy w pętli czy nie. I tak jest tak samo.

a nie masz czasem tak że najpierw wyświetlasz/pobierasz dane a dopiero potem kasujesz?

Skoro kazałem mu dać to na początek pliku i odpisał, że nie ma to znaczenia to widocznie nie.

Tak dokładnie. Najpierw wyświetlam/pobieram dane a dopiero potem kasuję.

No a sprawdzasz w bazie czy na stronie czy się notatka usuneła?