borrar imagen despues de x tiempo

[WinderJerter]

hola que tal, les cuento; lo que pasa es que tengo un script para subir imagenes y tengo varios campos en una base de datos mysql, entre ellos unos que se llaman date que guarda la fecha en la que se subio la imagen en este formato '8/03/2011, 10:00:31' y el otro campo es status que pone '0' cuando se sube la imagen, y cuando se activa se cambia a '1' y hasta hay todo va bien

Ahora lo que quiero hacer es que si la imagen no es activada en 1 hora desde que se subio la imagen sea borrada. Tenia pensado crear un script que comprabara si el status de la imagen es '0' y si es asi que la borre y para eso que el script se activara cada hora con Cronjobs, pero el principal problema iba a ser que se iban a borrar imagenes que no llevaban la hora

Alguien que me pueda ayudar con un script que compruebe la hora en que se subio la imagen y si no se a activado en una hora sea borrada automticamente???

Saludos y gracias de antemano

[nachocb_66]

Hola,

lo que puedes hacer es utilizar una consulta que pregunte si el estado es cero y que ademas realice una resta entre las fecha de en que se subio y la actual, eso lo tienes que buscar, empieza por aca:

http://dev.mysql.com/doc/refman/5.0/es/date-calculations.html

espero haberte dado una idea,

saludos

[WinderJerter]

Hola que tal, muchas gracias por responder; revise el link que me pasaste pero no entendi nada , pero lo que conseguido de la siguiente maner:

Código PHP:

<?php
 
include_once("bd.php");
 
$query = "SELECT * FROM images WHERE status = '0'";
$result = mysql_query($query); 
 
while($image = mysql_fetch_assoc($result)) 
{
$time_upload = $image['date'];
$time_now = date("Y/n/j, H:i:s");
$tiempo_transcurrido = (strtotime($time_now) - strtotime($time_upload));
 
if($tiempo_transcurrido > "3600")
{
    $link = $image['location'];
    $delete_link = array("http://", ".upload.com");
    $str_link = str_replace($delete_link, "", $link);
    unlink("servers/".$str_link);
    
    $sql_del = "DELETE FROM images WHERE location = '$link'";
    $result_del = mysql_query($sql_del);
}
}
 
?>
 

Si funciona, pero quisier saber si es la forma correcta de hacerlo y si se puede mejorar les agradeceria mucho que me pudieran ayudar

Saludos y gracias de antemano

[masterpuppet]

Que tal WinderJerter,

podrias traerte los registros pasados de tiempo directamente en la query en vez de traer todos los registros y comparar la fecha, tambien podrias guardar el id del registro borrado para tirar un where in en vez de una query para cada uno, algo asi:

Código PHP:

include_once("bd.php");
 
$query  = 'SELECT * '
        . 'FROM images '
        . 'WHERE NOW() > DATE_ADD(date,INTERVAL 1 HOUR) '
        . 'AND status = 0';
 
$result = mysql_query($query);
 
$toDelete = array();
while ($image = mysql_fetch_assoc($result)) {
    $delete_link = array('http://', '.upload.com');
    $str_link    = str_replace($delete_link, '', $image['location']);
    $toDelete[]  = $image['id'];
    unlink('servers/' . $str_link);
}
 
if (count($toDelete)) {
    $sql_del    = 'DELETE FROM images WHERE id IN(' . implode(',', $toDelete) . ')';
    $result_del = mysql_query($sql_del);
}
 

te sugiero tambien que tanto las tablas como los campos no los nombres con palabras reservadas ni tipos de datos, por ejemplo el campo date, te puede dar problemas.

Saludos.

[WinderJerter]

Iniciado por masterpuppet

Que tal WinderJerter,

podrias traerte los registros pasados de tiempo directamente en la query en vez de traer todos los registros y comparar la fecha, tambien podrias guardar el id del registro borrado para tirar un where in en vez de una query para cada uno, algo asi:

Código PHP:

include_once("bd.php");
 
$query  = 'SELECT * '
        . 'FROM images '
        . 'WHERE NOW() > DATE_ADD(date,INTERVAL 1 HOUR) '
        . 'AND status = 0';
 
$result = mysql_query($query);
 
$toDelete = array();
while ($image = mysql_fetch_assoc($result)) {
    $delete_link = array('http://', '.upload.com');
    $str_link    = str_replace($delete_link, '', $image['location']);
    $toDelete[]  = $image['id'];
    unlink('servers/' . $str_link);
}
 
if (count($toDelete)) {
    $sql_del    = 'DELETE FROM images WHERE id IN(' . implode(',', $toDelete) . ')';
    $result_del = mysql_query($sql_del);
}
 

te sugiero tambien que tanto las tablas como los campos no los nombres con palabras reservadas ni tipos de datos, por ejemplo el campo date, te puede dar problemas.

Saludos.

Hola que tal masterpuppet, muchas gracias por responder, tu script funciona vien pero me he dado cuenta que solo borra la imagen del servidor pero no borra el registro de la base de datos, he trabado de solucionarlo pero no puedo.

A que se debe que no borra el registro???

Saludos!!!

[masterpuppet]

Asumi que tenias un campo id en image, si no es el caso se puede adaptar para que borre por el campo location(como lo tenias tu), algo asi:

Código PHP:

include_once("bd.php");
 
$query  = 'SELECT * '
        . 'FROM images '
        . 'WHERE NOW() > DATE_ADD(date,INTERVAL 1 HOUR) '
        . 'AND status = 0';
 
$result = mysql_query($query);
 
$toDelete = array();
while ($image = mysql_fetch_assoc($result)) {
    $delete_link = array('http://', '.upload.com');
    $str_link    = str_replace($delete_link, '', $image['location']);
    $toDelete[]  = $image['location'];
    unlink('servers/' . $str_link);
}
 
if (count($toDelete)) {
    $sql_del    = "DELETE FROM images WHERE location IN('" . implode("','", $toDelete) . "')";
    $result_del = mysql_query($sql_del);
}
 

Saludos.

[WinderJerter]

Gracias, mira lo he solucionado de esta manera; que te parese???

Código PHP:

<?php
 
include_once("bd.php");
 
$query  = 'SELECT * '
        . 'FROM images '
        . 'WHERE NOW() > DATE_ADD(date,INTERVAL 1 HOUR) '
        . 'AND status = 0';
 
$result = mysql_query($query);
 
while ($image = mysql_fetch_assoc($result)) {
    $delete_link = array('http://', '.upload.com');
    $str_link    = str_replace($delete_link, '', $image['location']);
    unlink('servers/' . $str_link);
 
    $sql_del = $sql_del = "DELETE FROM images WHERE location = '".$image['location']."'";
    $result_del = mysql_query($sql_del);
}
 
 
?>
 

Saludos y gracias

[masterpuppet]

mmmmmm, bueno volve para atras y dejalo para que borre por id, y vamos a debugear,

Código PHP:

...
var_dump($toDelete);
if (count($toDelete)) {
    $sql_del    = 'DELETE FROM images WHERE id IN(' . implode(',', $toDelete) . ')';
    echo $sql_del . PHP_EOL;
    $result_del = mysql_query($sql_del) or die(mysql_error());
}
 

en caso de que no de error ejecuta la query en phpmyadmin o el programa que utilices.

[WinderJerter]

Iniciado por masterpuppet

mmmmmm, bueno volve para atras y dejalo para que borre por id, y vamos a debugear,

Código PHP:

...
var_dump($toDelete);
if (count($toDelete)) {
    $sql_del    = 'DELETE FROM images WHERE id IN(' . implode(',', $toDelete) . ')';
    echo $sql_del . PHP_EOL;
    $result_del = mysql_query($sql_del) or die(mysql_error());
}
 

en caso de que no de error ejecuta la query en phpmyadmin o el programa que utilices.

porque cuando le hago un echo a $toDelete de imprime Array

[masterpuppet]

WinderJerter, de la forma que propusiste en el post anterior tira una query por imagen, si lo haces con where in es una única query para todas las imagenes, cual elegir depende de ti, yo creo que es mejor aprender a hacer las cosas bien.
Con respecto al echo, no puede ser un echo porque justamente es un array, hacele como te puse en el ejemplo un var_dump, deberia mostrarte un array con los id's.

Saludos.

[WinderJerter]

Iniciado por masterpuppet

WinderJerter, de la forma que propusiste en el post anterior tira una query por imagen, si lo haces con where in es una única query para todas las imagenes, cual elegir depende de ti, yo creo que es mejor aprender a hacer las cosas bien.
Con respecto al echo, no puede ser un echo porque justamente es un array, hacele como te puse en el ejemplo un var_dump, deberia mostrarte un array con los id's.

Saludos.

Muchas gracias masterpuppet, ya funciona de maravilla tu script

Saludos y muchas gracias

[WinderJerter]

hola que tal, perdon por revivir el tema, lo que queria preguntar es que si solo quiero eliminar una imagen por consulta puede usar el mismo codigo que es este:

Código PHP:

include_once("bd.php");
 
$query  = 'SELECT * '
        . 'FROM images '
        . 'WHERE NOW() > DATE_ADD(date,INTERVAL 1 HOUR) '
        . 'AND status = 0';
 
$result = mysql_query($query);
 
$toDelete = array();
while ($image = mysql_fetch_assoc($result)) {
    $delete_link = array('http://', '.upload.com');
    $str_link    = str_replace($delete_link, '', $image['location']);
    $toDelete[]  = $image['location'];
    unlink('servers/' . $str_link);
}
 
if (count($toDelete)) {
    $sql_del    = "DELETE FROM images WHERE location IN('" . implode("','", $toDelete) . "')";
    $result_del = mysql_query($sql_del);
}
 

O se puede simplificar para que se realize la peticion en menos tiempo???

Saludos