Ver Mensaje Individual
  #20 (permalink)  
Antiguo 03/09/2010, 08:47
Avatar de mayid
mayid
Colaborador
 
Fecha de Ingreso: marzo-2009
Ubicación: BsAs
Mensajes: 4.014
Antigüedad: 15 años, 1 mes
Puntos: 101
Respuesta: mostar todas las fotos de un directorio con opcion de borrarlas

Te felicito por resolverlo. Ayer cruzamos mensajes al mismo tiempo, por eso se presta a confusión la seguidilla, y de ahí tu pregunta. Yo respondía a "me estoy volviendo loco" mientras vos escribías "lo he logrado ". Pero en fin, como siempre a altas horas quedan cosas que luego hay que revisar, dejé pasar el momento.

Viendo el código hoy, me surgen dudas. Lo primero a resolver es esto:

Cita:
if(unlink($dir));
Realmente no estás haciendo uso de ese condicional. Si quisieras aprovecharlo, harías esto:
Cita:
if(unlink($dir))
echo "archivo borrado"
else
echo "error de permisos al borrar archivo?";
Hay otras cosas que no me convencen. Aquí estás usando una variable PHP dentro de comillas simples. Y creo que eso no es bueno (tengo entendido que no se interpretan!)

<a href='proceso-eliminar-foto.php?files=carpeta-1/$file'>

Yo usaría comillas dobles o haría esto:
Cita:
<a href='proceso-eliminar-foto.php?files=carpeta-1/'.$file>
Por último, qué te devuelve esta variable? Solo lo que hay detrás de "?" o la url entera? Según es script, lo que necesitás es solo una subcadena, no todo:

$deleteGoTo .= $_SERVER['QUERY_STRING'];


Ahora, cosas aparte, que no hacen al funcionamiento del script. Por razones de elegancia yo cambiaría las llaves estas por un "continue" (que daría el mismo resultado)
if (($file == '.')||($file == '..'))
{
}
Cita:
if (($file == '.')||($file == '..')) continue;
Y a esto lo pondría comillas para cada atributo. Y mejor aún, usaría CSS para declarar el alto y ancho de las imagenes (a todas a la vez):
<td><img src=carpeta-1/$file width=200 height=200 /></td>