Ver Mensaje Individual
  #13 (permalink)  
Antiguo 14/04/2014, 15:16
Qazser
 
Fecha de Ingreso: abril-2014
Ubicación: Canarias!
Mensajes: 150
Antigüedad: 10 años, 8 meses
Puntos: 10
Respuesta: cancelar suscripcion por correo

Unos consejos,

-no hagas 2 veces la misma query
-NO uses @, ignorar una advertencia nunca es algo bueno si quieres ser un programador, intenta hacer un código limpio sin errores y no necesitas arrobas para ocultar fallos.
-Quita mysql error cuando no tienes un eror en la query, si algo surge no lo debe ver el usuario final.
-No uses mysql_fetch_array porque te da 2 array con lo mismo, con mysql_fetch_assoc te devuelve 1 solo array, mucho mas ligero
-Las funciones mysql son obsoletas y no son recomendadas, investiga y usa mysqli.

Con respecto a esto:
Código PHP:
$consulta=mysql_query('SELECT * FROM boletin WHERE correo="'.$_POST["suscor"].'"');
while(
$row=mysql_fetch_array($consulta)){  
$id_bo=$row['id_bo'];

Solo vas a esperar 1 fila de esta Query,porque un correo no puede estar dentro varias veces...entonces puede someterse bajo un microoptimizacion para quedar así:
Código PHP:
$consulta=mysql_query('SELECT * FROM boletin WHERE correo="'.$_POST["suscor"].'" LIMIT 1');
$row=mysql_fetch_array($consulta)
$id_bo=$row['id_bo']; 
Finalmente prueba reemplazar esto
Código PHP:
el siguiente enlacehttp://pagina.cl/suscribirse/delete.php?correo=$correo&id_bo={$row['id_bo']}",$headers); 
por esto
Código PHP:
el siguiente enlacehttp://pagina.cl/suscribirse/delete.php?correo=$correo&id_bo=$id_bo",$headers); 
__________________
Pautas php:
No usar libreria mysql, usa mysqli; Separar siempre los controladores de la vista; Procedual es feo, ama los objetos