Problem z DELETE Opcje

[nevar]

Czy w tym skrypcie jest coś nie tak??
Nic nie wykonuje, pusty ekran się pojawia:(

[PHP] pobierz, plaintext 
<?
 
include ("baza.php");
$baza = mysql_connect ($mysql_host, $mysql_user, $mysql_haslo) or die ("Brak połączenia");
$db = mysql_select_db($mysql_baza);
 
$id = $_POST['id_pyt'];
$usun = "DELETE * FROM quiz where id=$id"; 
$usun = mysql_query($usun);
echo mysql_error();
echo 'Zmieniono: '.mysql_affected_rows().'<br>'; 
if ($usun) {
		echo '<p>Pytanie usunięto</p>';
	} else {
		echo mysql_error(); echo '<p>Błąd! Spróbuj ponownie</p>';
 
}
 
}
?>
[PHP] pobierz, plaintext 

[Master Miko]

Usuń * z zapytania delete, daj apostrofy pomiędzy $id i będzie śmigać, no i daj LIMIT 1 na wszelki wypadek.

Ten post edytował Master Miko 7.05.2006, 16:42:00

[nevar]

Zrobiłem tak jak mówisz

Skrypt wygląda tak:

[PHP] pobierz, plaintext 
<?
 
include ("baza.php");
$baza = mysql_connect ($mysql_host, $mysql_user, $mysql_haslo) or die ("Brak połączenia");
$db = mysql_select_db($mysql_baza);
 
$id = $_POST['id_pyt'];
$usun = "DELETE FROM quiz where id='$id' LIMIT 1"; 
$usun = mysql_query($usun);
 
echo mysql_error();
echo 'Zmieniono: '.mysql_affected_rows().'<br>'; 
if ($usun) {
		echo '<p>Pytanie usunięto</p>';
	} else {
		echo mysql_error(); echo '<p>Błąd! Spróbuj ponownie</p>';
 
}
 
}
?>
[PHP] pobierz, plaintext 

ale dalej nic nie wykonuje:(

Wcześniej wychodzę z takiego skryptu:

[PHP] pobierz, plaintext 
<?php
include ("baza.php");
$baza = mysql_connect ($mysql_host, $mysql_user, $mysql_haslo);
mysql_select_db($mysql_baza);
$wynik = mysql_query("select * from quiz");
 
 
if (mysql_num_rows ($wynik)>0)
	{	
	$i=1;
	while($pytanie = mysql_fetch_array($wynik))
	{	
		print '<font face="Arial" size="3">'.$i.'.</font>';
		print '<font face="Arial" size="3">'.$pytanie['pyt'].'</font><br>';
		print '<font face="Arial" size="2"><input type="radio" name=p'.$pytanie['id'].' value="a" />'.$pytanie['odpa'].'<br></font>';
		print '<font face="Arial" size="2"><input type="radio" name=p'.$pytanie['id'].' value="b" />'.$pytanie['odpb'].'<br></font>';
		print '<font face="Arial" size="2"><input type="radio" name=p'.$pytanie['id'].' value="c" />'.$pytanie['odpc'].'<br></font>';
		print '<font face="Arial" size="3" color="green">Poprawna odp to:'.$pytanie['odppraw'].'</font><br>';
	 print '<input type="button" value="Edytuj" onClick="window.open('edytuj.php?id_pyt='.$pytanie['id'].'', 'nazwa').focus(); return false" />';
		print '<input type="button" value="Usuń" onClick="if(window.confirm('Czy na pewno ?')){window.location='usun.php?id_pyt='.$pytanie['id'].'';}">';
		print '<br><hr>';
		$i++;
	}
}
 
?>
[PHP] pobierz, plaintext 

Klikam przycisk Usuń, jeszcze jest potwierdzenie ale dalej nic nie wykonuje

[Michasko]

Może niepotrzebnie, ale dziwi mnie to:

[PHP] pobierz, plaintext 
<?php
$usun = "DELETE FROM quiz where id='$id' LIMIT 1"; 
$usun = mysql_query($usun);
?>
[PHP] pobierz, plaintext 

zmienna usun ma teraz wartosc 'mysql_query($usun)' zamiast 'DELETE FROM quiz where id='$id' LIMIT 1' tak więc zapytanie sie raczej nie wykona, choć moge sie mylić.

[kwiateusz]

no i sie myslisz bo kod php jest od prawej do lewej przypisywany wiec najpier wykona sie zapytanie potem zostanie rpzypisane do zmienne

[nevar]

No więc co w takim razie jest nie tak z tym?

Nie wiem czy on dobrze pobiera id

Przekazywane jest tak:

[PHP] pobierz, plaintext 
<?php
<input type="button" value="Usuń" onClick="if(window.confirm('Czy na pewno ?')){window.location='usun.php?id_pyt='.$pytanie['id'].'';}">
?>
[PHP] pobierz, plaintext 

usun.php to ten skrypt z DELETE

[Master Miko]

a dałeś refresh ?

Sprawdź w skrypcie tam gdzie masz przycik "usuń"
czy ściężka to usun.php?id_pyt=no i id jakiegos tam,

w usun.php daj echo tego posta i spradz co wyjdzie.
jesli jest ok, sprawdz czy masz prawa w bazie danych co do kasowania.

[nevar]

Ścieżka jest w porządku

kiedy kliknę na przycisk usuń w linku mam np. http://wloclawek.wsinf.edu.pl/~prempals/usun.php?id_pyt=9

ale nie usuwa mi tego pytania, w ogóle nic nie robi, nic nie wyświetla

Prawa do kasowania mam, sprawdzałem przez phpMyAdmin

[Master Miko]

Daj to:

[PHP] pobierz, plaintext 
<?php
 
error_reporting(E_ALL);
 
?>
[PHP] pobierz, plaintext 

na samej górze skryptu (usuń.php) ... i powiedz co powie

[nevar]

Dałem to ale dalej nic nie wyświetla, pusty ekran

[Master Miko]

To powiedz serverowi że jest głupi i ma wszy.
Nie widzę żadnego błędu już :|


a tak serio..

wykasuj tę linię:

[PHP] pobierz, plaintext 
<?php
 
$id = $_POST['id_pyt'];
$usun = "DELETE FROM quiz where id='$id' LIMIT 1"; 
$usun = mysql_query($usun);
 
?>
[PHP] pobierz, plaintext 

i powiedz co Ci się pokaże... lub nie.

Ten post edytował Master Miko 7.05.2006, 21:35:27

[skowron-line]

moze daj w tym skrypcie echo $id i zobacz czy wogole przekazuje zmienna,
a i moze powinienies zrobic

[PHP] pobierz, plaintext 
<?php
$usun = 'DELETE FROM quiz where id="'.$id.'" LIMIT 1';
?>
[PHP] pobierz, plaintext 

ale jestem tylko czlowiekiem i moge sie mylic

Ten post edytował skowron-line 7.05.2006, 21:42:29

[nospor]

1) skoro id to liczba, to nie musi byc w ciapkach
2) skoro id jest przekazywane w url to musi byc pobierane z $_GET a nie z $_POST
3) skoro id, zapamietywana jest w parametrze o nazwie id_pyt, to na grzyba dobierac sie do niego poprzez zmienna o nazwie id ?

[PHP] pobierz, plaintext 
<?php
$usun = 'DELETE FROM quiz where id='.((int)$_GET['id_pyt']).' LIMIT 1';
?>
[PHP] pobierz, plaintext 

ten (int) to po to, by jak ktos bedzie chcial wlam zrobic, to zeby mu sie nie udalo

ps: i prosze poprawic tytul o znacznik zgodnie z zasadami forum przedszkole