Deleting data from mysql

REDDIO · 16 de Novembro de 2007

Estou a tentar apagar utilizadores, no entanto quando carrego em delete aparece a mensagem a dizer que foi apagado com sucesso, mas na verdade não apagou na base de dados.
Gostaria que dessem uma vista de olhos no codigo, porque não estou a encontrar o erro.

Apagar_utilizador.php:

<?php

$host="localhost"; // Host name
$username="root"; // Mysql username
$password=""; // Mysql password
$db_name="agencia_de_viagens"; // Database name
$tbl_name="utilizadores"; // Table name

// Connect to server and select database.
mysql_connect("$host", "$username", "$password")or die("cannot connect");
mysql_select_db("$db_name")or die("cannot select DB");

// select record from mysql
$sql="SELECT * FROM $tbl_name";
$result=mysql_query($sql);

?>
<table width="400" border="0" cellspacing="1" cellpadding="0">
<tr>
<td><table width="400%" border="0" cellpadding="3" cellspacing="1" bgcolor="#CCCCCC">
<tr>
<td bgcolor="#FFFFFF"> </td>
<td colspan="4" bgcolor="#FFFFFF"><strong>Apagar Utilizadores</strong> </td>
</tr>
<tr>
<td align="center" bgcolor="#FFFFFF"><strong>Id utilizador</strong></td>
<td align="center" bgcolor="#FFFFFF"><strong>Nome</strong></td>
<td align="center" bgcolor="#FFFFFF"><strong>Email</strong></td>
<td align="center" bgcolor="#FFFFFF"> </td>
</tr>

<?php
while($rows=mysql_fetch_array($result)){
?>
<tr>
<td bgcolor="#FFFFFF"><? echo $rows['ID_UTILIZADOR']; ?></td>
<td bgcolor="#FFFFFF"><? echo $rows['NOME']; ?></td>
<td bgcolor="#FFFFFF"><? echo $rows['EMAIL']; ?></td>

<td bgcolor="#FFFFFF"><a href="apaga.php?Id=<? echo $rows['ID_UTILIZADOR']; ?>">delete</a></td>

</tr>
<?

// close while loop
}

// close connection;
mysql_close();

?>
</table></td>
</tr>
</table>

apaga.php:

<?php

$host="localhost"; // Host name
$username="root"; // Mysql username
$password=""; // Mysql password
$db_name="agencia_de_viagens"; // Database name
$tbl_name="utilizadores"; // Table name

// Connect to server and select databse.
mysql_connect("$host", "$username", "$password")or die("cannot connect");
mysql_select_db("$db_name")or die("cannot select DB");

// get value of id that sent from address bar
$id=$_GET['ID_UTILIZADOR'];

// Delete data in mysql from row that has this id
$sql="DELETE FROM $tbl_name WHERE ID_UTILIZADOR='$ID_UTILIZADOR'";
$result=mysql_query($sql);

// if successfully deleted
if($result){
echo "Deleted Successfully";
echo "<BR>";
echo "<a href='Apagar_utilizador.php'>Back to main page</a>";
}

else {
echo "ERROR";
}

// close connection
mysql_close();
?>

xOy · 16 de Novembro de 2007

// get value of id that sent from address bar
$id=$_GET['ID_UTILIZADOR'];

// Delete data in mysql from row that has this id

$sql="DELETE FROM $tbl_name WHERE ID_UTILIZADOR='$id'";]

tenta assim...

slack_guy · 16 de Novembro de 2007

// get value of id that sent from address bar
$id=$_GET['ID_UTILIZADOR'];

// Delete data in mysql from row that has this id
$sql="DELETE FROM $tbl_name WHERE ID_UTILIZADOR='$ID_UTILIZADOR'";

O que é que acontece no caso de o user enviar

Código:

ID_UTILIZADOR = "0' OR id_utilizador <> '0"

?
Resultado:

Código:

DELETE FROM tbl_name WHERE id_utilizador = '0' OR id_utilizador <> '0'

AliFromCairo · 16 de Novembro de 2007

slack_guy disse:
O que é que acontece no caso de o user enviar

Código:

ID_UTILIZADOR = "0' OR id_utilizador <> '0"

?

Uma query sem grandes resultados no caso (mais que provável) de o criador da thread não ter desactivado a directiva magic_quotes_gpc que vem activa por defeito nas últimas releases do PHP.

slack_guy disse:
Resultado:

Código:

DELETE FROM tbl_name WHERE id_utilizador = '0' OR id_utilizador <> '0'

Código:

DELETE FROM tbl_name WHERE id_utilizador = '\"0\' OR id_utilizador <> \'0\"'

De qualquer forma, é sempre bom alertar para esse tipo de problemas, devido ao facto do magic_quotes_gpc não resolver tudo :007:

.

slack_guy · 17 de Novembro de 2007

devido ao facto do magic_quotes_gpc não resolver tudo

Não só não resolve como cria uma carrada de problemas maiores que aqueles que tenta resolver (é que nem todos os programadores de PHP são noobs ;-) ) Tanto quanto sei, vai ser removido no PHP6.

a directiva magic_quotes_gpc que vem activa por defeito nas últimas releases do PHP.

hum... aqui não:

Código:

Slackware 12.0

[me@mybox ] grep magic_quotes_gpc /etc/httpd/php.ini
; - magic_quotes_gpc = Off         [Performance]
magic_quotes_gpc = Off

[me@mybox ] grep magic_quotes_gpc /etc/httpd/php.ini-recommended
; - magic_quotes_gpc = Off         [Performance]
magic_quotes_gpc = Off

[me@mybox ] grep magic_quotes_gpc /etc/httpd/php.ini-dist
magic_quotes_gpc = On

REDDIO · 17 de Novembro de 2007

caros,

continuo sem conseguir resolver o problema, ja tentei as varias formas que indicaram.

mascker · 17 de Novembro de 2007

RE:

Boas,

Altera o seguinte

Código:

[FONT=Arial][SIZE=2]// Delete data in mysql from row that has this id 
$sql="DELETE FROM $tbl_name WHERE ID_UTILIZADOR='$ID_UTILIZADOR'";
$result=mysql_query($sql);
[/SIZE][/FONT]

para

Código:

[FONT=Arial][SIZE=2]// Delete data in mysql from row that has this id 
$sql="DELETE FROM $tbl_name WHERE ID_UTILIZADOR='$ID_UTILIZADOR'";
$result=mysql_query($sql) or die(mysql_error());
[/SIZE][/FONT]

Informa do erro....

Saudações

slack_guy · 17 de Novembro de 2007

continuo sem conseguir resolver o problema, ja tentei as varias formas que indicaram.

No código que colaste aqui tens 'ID_UTILIZADOR' como nome do campo. É esse o nome na BD? Assim? escrito em maiúsculas?

REDDIO · 17 de Novembro de 2007

slack_guy disse:
No código que colaste aqui tens 'ID_UTILIZADOR' como nome do campo. É esse o nome na BD? Assim? escrito em maiúsculas?

Sim e com é desta forma que tenho na base de dados!

REDDIO · 17 de Novembro de 2007

mascker disse:

Boas,

Altera o seguinte

Código:

[FONT=Arial][SIZE=2]// Delete data in mysql from row that has this id [/SIZE][/FONT]
[SIZE=2][FONT=Arial]$sql="DELETE FROM $tbl_name WHERE ID_UTILIZADOR='$ID_UTILIZADOR'";[/FONT][/SIZE]
[SIZE=2][FONT=Arial]$result=mysql_query($sql);[/FONT][/SIZE]

para

Código:

[FONT=Arial][SIZE=2]// Delete data in mysql from row that has this id [/SIZE][/FONT]
[SIZE=2][FONT=Arial]$sql="DELETE FROM $tbl_name WHERE ID_UTILIZADOR='$ID_UTILIZADOR'";[/FONT][/SIZE]
[SIZE=2][FONT=Arial]$result=mysql_query($sql) or die(mysql_error());[/FONT][/SIZE]

Informa do erro....

Saudações

Fiz desta forma e continua a dizer que foi eliminado, mas quando volto a listar continua a constar na lista...

anjo2 · 17 de Novembro de 2007

Código:

[FONT=Arial][SIZE=2]<?php[/SIZE][/FONT]
 
[FONT=Arial][SIZE=2]$host="localhost"; // Host name 
$username="root"; // Mysql username 
$password=""; // Mysql password 
$db_name="agencia_de_viagens"; // Database name 
$tbl_name="utilizadores"; // Table name [/SIZE][/FONT]
 
[FONT=Arial][SIZE=2]// Connect to server and select databse.
mysql_connect("$host", "$username", "$password")or die("cannot connect"); 
mysql_select_db("$db_name")or die("cannot select DB");[/SIZE][/FONT]
 
[FONT=Arial][SIZE=2]// get value of id that sent from address bar 
$id=$_REQUEST['Id'];[/SIZE][/FONT]

[FONT=Arial][SIZE=2]// Delete data in mysql from row that has this id 
$sql="DELETE FROM $tbl_name WHERE ID_UTILIZADOR='$id'";
$result=mysql_query($sql);[/SIZE][/FONT]
 
[FONT=Arial][SIZE=2]// if successfully deleted
if($result && $id){
echo "Deleted Successfully";
echo "<BR>";
echo "<a href='Apagar_utilizador.php'>Back to main page</a>";
}[/SIZE][/FONT]
 
[FONT=Arial][SIZE=2]else {
echo "ERROR";
}[/SIZE][/FONT]
 
[FONT=Arial][SIZE=2]// close connection 
mysql_close();
?>[/SIZE][/FONT]

EDIT: Este código deve funcionar.

p3dro · 17 de Novembro de 2007

Posso estar a ver mal

mas acho nunca passas a variável "ID_UTILIZADOR"
por ex.:

PHP:

<a href="apaga.php?Id=<? echo $rows['ID_UTILIZADOR']; ?>">delete</a>

Mas depois vais buscar como

PHP:

$id=$_GET['ID_UTILIZADOR'];

Não deveria antes ser ...

PHP:

// get value of id that sent from address bar
$id=$_GET["Id"];

// Delete data in mysql from row that has this id
$sql="DELETE FROM $tbl_name WHERE ID_UTILIZADOR='$id' ";

K0mA · 17 de Novembro de 2007

Ah granda p3dro !

p3dro · 17 de Novembro de 2007

K0mA disse:
Ah granda p3dro !

uups, afinal o anjo2 já tinha respondido :x2:

K0mA · 17 de Novembro de 2007

p3dro disse:
uups, afinal o anjo2 já tinha respondido

Ah granda anjo2 !

anjo2 · 17 de Novembro de 2007

p3dro disse:
uups, afinal o anjo2 já tinha respondido

Não tinha não, apenas tinha dito a ele como identificar o erro, a partir de echo de cada variável, mas como já tinhas visto o erro, editei e apenas pus a solução

REDDIO · 17 de Novembro de 2007

anjo2 disse:

Código:

[FONT=Arial][SIZE=2]<?php[/SIZE][/FONT]
 
[FONT=Arial][SIZE=2]$host="localhost"; // Host name [/SIZE][/FONT]
[SIZE=2][FONT=Arial]$username="root"; // Mysql username [/FONT][/SIZE]
[SIZE=2][FONT=Arial]$password=""; // Mysql password [/FONT][/SIZE]
[SIZE=2][FONT=Arial]$db_name="agencia_de_viagens"; // Database name [/FONT][/SIZE]
[SIZE=2][FONT=Arial]$tbl_name="utilizadores"; // Table name [/FONT][/SIZE]
 
[FONT=Arial][SIZE=2]// Connect to server and select databse.[/SIZE][/FONT]
[SIZE=2][FONT=Arial]mysql_connect("$host", "$username", "$password")or die("cannot connect"); [/FONT][/SIZE]
[SIZE=2][FONT=Arial]mysql_select_db("$db_name")or die("cannot select DB");[/FONT][/SIZE]
 
[FONT=Arial][SIZE=2]// get value of id that sent from address bar [/SIZE][/FONT]
[SIZE=2][FONT=Arial]$id=$_REQUEST['Id'];[/FONT][/SIZE]
 
[FONT=Arial][SIZE=2]// Delete data in mysql from row that has this id [/SIZE][/FONT]
[SIZE=2][FONT=Arial]$sql="DELETE FROM $tbl_name WHERE ID_UTILIZADOR='$id'";[/FONT][/SIZE]
[SIZE=2][FONT=Arial]$result=mysql_query($sql);[/FONT][/SIZE]
 
[FONT=Arial][SIZE=2]// if successfully deleted[/SIZE][/FONT]
[SIZE=2][FONT=Arial]if($result && $id){[/FONT][/SIZE]
[SIZE=2][FONT=Arial]echo "Deleted Successfully";[/FONT][/SIZE]
[SIZE=2][FONT=Arial]echo "<BR>";[/FONT][/SIZE]
[SIZE=2][FONT=Arial]echo "<a href='Apagar_utilizador.php'>Back to main page</a>";[/FONT][/SIZE]
[SIZE=2][FONT=Arial]}[/FONT][/SIZE]
 
[FONT=Arial][SIZE=2]else {[/SIZE][/FONT]
[SIZE=2][FONT=Arial]echo "ERROR";[/FONT][/SIZE]
[SIZE=2][FONT=Arial]}[/FONT][/SIZE]
 
[FONT=Arial][SIZE=2]// close connection [/SIZE][/FONT]
[SIZE=2][FONT=Arial]mysql_close();[/FONT][/SIZE]
[SIZE=2][FONT=Arial]?>[/FONT][/SIZE]

EDIT: Este código deve funcionar.

Hey obrigado, com este codigo ja funciona!!

slack_guy · 17 de Novembro de 2007

mas o problema de segurança mantém-se...

REDDIO · 17 de Novembro de 2007

Obrigado a todos pela ajuda!!

anjo2 · 17 de Novembro de 2007

slack_guy disse:
mas o problema de segurança mantém-se...

Sim, mas a solução não é difícil, é só verificar se é um inteiro por exemplo.

Deleting data from mysql

Power Member

Power Member

Power Member

Power Member

Power Member

Power Member

Power Member

Power Member

Power Member

Power Member

Power Member

Power Member

Power Member

Power Member

Power Member

Power Member

Power Member

Power Member

Power Member

Power Member