Borra archivo con unlink

[Ghosty]

Hola amigos

Tengo un formulario para borrar un registro de la BD esta registro tiene tiene enlazado un mp3 y queiro que al borrar el registro de la BD tambien me borre el archivo y he hecho esto

Código PHP:

  if ((isset($_POST['campoID'])) && ($_POST['campoID'] != "")) { 
  $deleteSQL = sprintf("DELETE FROM mhh_temas WHERE id_tema=%s", 
                       GetSQLValueString($_POST['campoID'], "int")); 
   
  mysql_select_db($database_mhh, $mhh); 
  $Result1 = mysql_query($deleteSQL, $mhh) or die(mysql_error()); 
  unlink ($row_Rc_temas['link']);// Aki quiero que borre el archivo la ruta la trae de la BD 
   
 
  $deleteGoTo = "zona_usuario.php?id=" . $row_sesion['id_usu'] . ""; 
  if (isset($_SERVER['QUERY_STRING'])) { 
    $deleteGoTo .= (strpos($deleteGoTo, '?')) ? "&" : "?"; 
    $deleteGoTo .= $_SERVER['QUERY_STRING']; 
  } 
  header(sprintf("Location: %s", $deleteGoTo)); 
}  
   

pero no funciona...alguna idea?

[pateketrueke]

Sería bueno saber si te genera algún error, o bien, si la ruta que tienes en la BD es absoluta y real.

Código PHP:

  if (is_file($row_Rc_temas['link'])) unlink ($row_Rc_temas['link']);
else die('El archivo no existe:' . $row_Rc_temas['link']);  
   

¿Puedes mostrarnos un ejemplo?

[jotaincubus]

Una pregunta... De donde estas sacando este dato ?

$row_Rc_temas['link']

Se supone que esa es la ruta del archivo, pero no se de donde la estas sacando... de la base de datos...? Pues no veo ningún SELECT para traer ese registro, de un arreglo...? Pues no veo ningún arreglo por ningún lado.

[Ghosty]

No no me genera ningun error y la ruta esta bien por que esa misma ruta "$row_Rc_temas['link']" la utilizo para enlazarlo con un flash que reproduce el sonido. La ruta es relativa seria algo asi "mp3/nombre_delacancion.mp3"

[Ghosty]

Iniciado por jotaincubus

Una pregunta... De donde estas sacando este dato ?

$row_Rc_temas['link']

Se supone que esa es la ruta del archivo, pero no se de donde la estas sacando... de la base de datos...? Pues no veo ningún SELECT para traer ese registro, de un arreglo...? Pues no veo ningún arreglo por ningún lado.

Pero esa ruta está almacenada en la misma tabla de la que kiero borrar el registro ¿tendria que hacer un select?

[jotaincubus]

Claro amigo, vos la eliminas y después queres utilizarla ? y ademas de eso no haces un SELECT para traerla ? entonces como va a saber PHP que es lo que tiene que eliminar sino esta por ningún lado?

[Ghosty]

ok muchas gracias

[esaenz22]

mi sugerencia Ghosty es que hagas un sql almacenando en un fetch_array el campo donde se guarda el archivo mp3 y luego colocarias tu otra consulta que borra tu registro. lo pones asi Ghosty.

Código PHP:

if ((isset($_POST['campoID'])) && ($_POST['campoID'] != "")) 
{ 
$elimp3 = "SELECT * FROM mhh_temas WHERE id = '".$_POST['campoID']."'";
$rowmp3 = mysql_query($elimp3,$mhh) or die(mysql_error());
$rs_mp3 = mysql_fetch_array($rowmp3);
 
if($rs_mp3['link']!="") // si el campo $rs_mp3['link'] es diferente al vacio.
{
@unlink("$rs_mp3['link']");
}
 
// la consulta DELETE.
$deleteSQL = sprintf("DELETE FROM mhh_temas WHERE id_tema=%s",
                       GetSQLValueString($_POST['campoID'], "int"));
  
mysql_select_db($database_mhh, $mhh);
$Result1 = mysql_query($deleteSQL, $mhh) or die(mysql_error()); 
 
header(sprintf("Location: %s", $deleteGoTo)); 
 
}
 

si persiste el problema, posteas tu consulta.

[jotaincubus]

Iniciado por esaenz22

mi sugerencia Ghosty es que hagas un sql almacenando en un fetch_array el campo donde se guarda el archivo mp3 y luego colocarias tu otra consulta que borra tu registro. lo pones asi Ghosty.

Código PHP:

if ((isset($_POST['campoID'])) && ($_POST['campoID'] != "")) 
{ 
$elimp3 = "SELECT * FROM mhh_temas WHERE id = '".$_POST['campoID']."'";
$rowmp3 = mysql_query($elimp3,$mhh) or die(mysql_error());
$rs_mp3 = mysql_fetch_array($rowmp3);
 
if($rs_mp3['link']!="") // si el campo $rs_mp3['link'] es diferente al vacio.
{
@unlink("$rs_mp3['link']");
}
 
// la consulta DELETE.
$deleteSQL = sprintf("DELETE FROM mhh_temas WHERE id_tema=%s",
                       GetSQLValueString($_POST['campoID'], "int"));
  
mysql_select_db($database_mhh, $mhh);
$Result1 = mysql_query($deleteSQL, $mhh) or die(mysql_error()); 
 
header(sprintf("Location: %s", $deleteGoTo)); 
 
}
 

si persiste el problema, posteas tu consulta.

En realidad si se quiere algo mas "profesional" debería verificar si el archivo fue eliminado correctamente antes de hacer el DELETE en la base de datos.

[esaenz22]

bueno. eso lo deberia verificar el. no entiendo porque la critica.

[jotaincubus]

Iniciado por esaenz22

bueno. eso lo deberia verificar el. no entiendo porque la critica.

No es una critica es un aporte.

[Ghosty]

Iniciado por jotaincubus

Claro amigo, vos la eliminas y después queres utilizarla ? y ademas de eso no haces un SELECT para traerla ? entonces como va a saber PHP que es lo que tiene que eliminar sino esta por ningún lado?

Bueno no se si el codigo esta muy bien pero asi me funciona

Código PHP:

   
if ((isset($_POST['campoID'])) && ($_POST['campoID'] != "")) { 
     
  $deleteSQL = sprintf("DELETE FROM mhh_temas WHERE id_tema=%s", 
                       GetSQLValueString($_POST['campoID'], "int")); 
                mysql_select_db($database_mhh, $mhh); 
                $query_file = "SELECT * FROM mhh_temas"; 
                $Rc_file = mysql_query($query_file, $mhh) or die(mysql_error()); 
                $row_file = mysql_fetch_assoc($Rc_mk); 
                $totalRows_file = mysql_num_rows($Rc_mk); 
                $query_Rc_mk = "SELECT * FROM mhh_temas"; 
                $Rc_file = mysql_query($query_file, $mhh) or die(mysql_error()); 
                $row_file = mysql_fetch_assoc($Rc_file); 
                $totalRows_Rc_file = mysql_num_rows($Rc_file); 
                unlink($row_file['link']); 
 mysql_select_db($database_mhh, $mhh); 
  $Result1 = mysql_query($deleteSQL, $mhh) or die(mysql_error()); 
   
   
 
  $deleteGoTo = "zona_usuario.php?id=" . $row_sesion['id_usu'] . ""; 
  if (isset($_SERVER['QUERY_STRING'])) { 
    $deleteGoTo .= (strpos($deleteGoTo, '?')) ? "&" : "?"; 
    $deleteGoTo .= $_SERVER['QUERY_STRING']; 
  } 
  header(sprintf("Location: %s", $deleteGoTo)); 
}  
   

Como lo ves?

[jotaincubus]

Si te funciona tu eres el que decide si quieres dejarlo así o no, por lo que veo estas utilizando DW, este genera mucho código innecesario, pero como te digo si así te funciona y estas a gusto con ello pues déjalo así, es tu aplicación y estas en todo tu derecho de escoger la forma de desarrollarla.

[esaenz22]

porque tu codigo esta desordenado y porque repites la misma consulta ?. en mi opinion, lo veo desordenado. tu generas el codigo mediante dreamweaver y eso hace que tu programacion no se entienda bien.

revisa bien el codigo que mostre. y como dijo jotaincubus, si te funciona, tu eres el que decide.

[Ghosty]

Iniciado por esaenz22

porque tu codigo esta desordenado y porque repites la misma consulta ?. en mi opinion, lo veo desordenado. tu generas el codigo mediante dreamweaver y eso hace que tu programacion no se entienda bien.

revisa bien el codigo que mostre. y como dijo jotaincubus, si te funciona, tu eres el que decide.

Ese codigo me lo genera DW mirare como hacer la consulta sin DW. Se que genera mucho codigo que no sirve pero...

Estoy mirando tu codigo a ver como va pero me dice No database selected

[esaenz22]

muestrame como lo has implementado.

[Ghosty]

Iniciado por esaenz22

muestrame como lo has implementado.

Código PHP:

    if ((isset($_POST['campoID'])) && ($_POST['campoID'] != "")) 
    { 
    $elimp3 = "SELECT * FROM mhh_temas WHERE id = '".$_POST['campoID']."'"; 
    $rowmp3 = mysql_query($elimp3,$mhh) or die(mysql_error()); 
    $rs_mp3 = mysql_fetch_array($rowmp3); 
      
    if($rs_mp3['link']!="") // si el campo $rs_mp3['link'] es diferente al vacio. 
    { 
    @unlink($rs_mp3['link']); 
    } 
      
    // la consulta DELETE. 
    $deleteSQL = sprintf("DELETE FROM mhh_temas WHERE id_tema=%s", 
                           GetSQLValueString($_POST['campoID'], "int")); 
      
    mysql_select_db($database_mhh, $mhh); 
    $Result1 = mysql_query($deleteSQL, $mhh) or die(mysql_error()); 
      
  
      
    } 
  $deleteGoTo = "zona_usuario.php?id=" . $row_sesion['id_usu'] . ""; 
  if (isset($_SERVER['QUERY_STRING'])) { 
    $deleteGoTo .= (strpos($deleteGoTo, '?')) ? "&" : "?"; 
    $deleteGoTo .= $_SERVER['QUERY_STRING']; 
  } 
  header(sprintf("Location: %s", $deleteGoTo));  
   

[esaenz22]

ok. te muestra que la base de datos no ha sido seleccionada. el dreamweaver como genera codigo php, las conexiones y las consulta se repiten siempre que llamas a un sql. y eso es un error. porque que pasaria si tienes mas de una consulta en otros archivos php.

lo recomendable es que tu deber crear el codigo para la conexion y no estar repitiendo la funcion para llamar a la bd.

volviendo al tema, seria asi el codigo (en mi sano juicio).

Código PHP:

if ((isset($_POST['campoID'])) && ($_POST['campoID'] != ""))
    {
    mysql_select_db($database_mhh, $mhh);
    $elimp3 = "SELECT * FROM mhh_temas WHERE id = '".$_POST['campoID']."'";
    $rowmp3 = mysql_query($elimp3,$mhh) or die(mysql_error());
    $rs_mp3 = mysql_fetch_array($rowmp3);
     
    if($rs_mp3['link']!="") // si el campo $rs_mp3['link'] es diferente al vacio.
    {
    @unlink($rs_mp3['link']);
    }
     
    // la consulta DELETE.
    $deleteSQL = sprintf("DELETE FROM mhh_temas WHERE id_tema=%s",
                           GetSQLValueString($_POST['campoID'], "int"));
     
    mysql_select_db($database_mhh, $mhh);
    $Result1 = mysql_query($deleteSQL, $mhh) or die(mysql_error());
 
  $deleteGoTo = "zona_usuario.php?id=" . $row_sesion['id_usu'] . "";
  if (isset($_SERVER['QUERY_STRING'])) {
    $deleteGoTo .= (strpos($deleteGoTo, '?')) ? "&" : "?";
    $deleteGoTo .= $_SERVER['QUERY_STRING'];
  }
 
  header(sprintf("Location: %s", $deleteGoTo)); 
 
}
 

[Ghosty]

Funciona perfectamente. Muchas gracias