Agregar fotos sin sobreescribir las que existen

kimmy · #1 (**permalink**) 10/08/2013, 11:26

Hola, tengo un código para cargar fotos pero necesito que si el usuario no ha podido cargar todas en un primer momento, pueda entrar despues y cargar las faltantes. El problema es que como lo tengo lo que hace es reemplazar las que ya existen y lo que necesito es que agregue las nuevas y conserve las que ya existen. Este es mi código:

Código HTML:

Ver original<form action="fotos_cargadas.php" name="form2" enctype="multipart/form-data" id="form2" method="post" >
        <th colspan="2">CARGAR FOTOS</th>
        <tr>
        <td>Id de Posada</td>
        <td><input name="id_posada" type="text" value="<?php echo $id_posada; ?>" readonly="readonly" />
        <input name="id_foto" type="hidden" value="<?php echo $id_foto; ?>">
        </td>
        </tr>
        <tr>
        <td>Nombre Posada</td>
        <td><input name="nombre_posada" type="text" value="<?php echo $nombre_posada; ?>" readonly="readonly" /></td>
        </tr>
        <tr>
            <td>FOTO 1</td>
                        <td><input type="file" name="foto1" id="foto1" /></td>
        </tr>    
        <tr>
            <td>FOTO 2</td>
                        <td><input type="file" name="foto2" id="foto2" /></td>
        </tr>    
        <tr>
            <td>FOTO 3</td>
                        <td><input type="file" name="foto3" id="foto3" /></td>
        </tr>    
        <tr>
            <td>FOTO 4</td>
                        <td><input type="file" name="foto4" id="foto4" /></td>
        </tr>    
        <tr>
            <td>FOTO 5</td>
                        <td><input type="file" name="foto5" id="foto5" /></td>
        </tr>    
        <tr>
            <td>FOTO 6</td>
                        <td><input type="file" name="foto6" id="foto6" /></td>
        </tr>    
        <tr>
            <td>FOTO 7</td>
                        <td><input type="file" name="foto7" id="foto7" /></td>
        </tr>    
        <tr>
            <td>FOTO 8</td>
                        <td><input type="file" name="foto8" id="foto8" /></td>
        </tr>
        <tr>
            <td>FOTO 9</td>
                        <td><input type="file" name="foto9" id="foto9" /></td>
            
        <tr>
          <td colspan="2" align="center"><input name="cargar" type="submit" id="cargar" value="Cargar" /></td>
        </tr>
        </tr>  
     </form>

y el php:

Código PHP:

Ver original$id_posada = $_POST['id_posada'];
$id_foto = $_POST['id_foto'];
 
$foto_nombre = substr($id_foto,0,-4); 
$codigo = substr($id_foto,0,-4); 
 
$ruta1 = "../../images/galeria/large/$codigo/";
$ruta2 = "../../images/galeria/thumbnails/$codigo/";
 
if (file_exists($ruta1)) {
$extension = "jpg";
 
$i = 1;
foreach($_FILES as $value){
   // Si la imagen fue subida correctamente
   if ( $value['error'] == UPLOAD_ERR_OK ){
        $image = new ResizePicture($value['tmp_name']);
 
        $image->resize(444,420);
        $w444 = $foto_nombre. '_' .$i. '.' . $extension;
        $image->save($ruta1.$w444);
 
        $image->resize(54,54);
        $w54 = $foto_nombre. '_'  .$i. '.' . $extension;
        $image->save($ruta2.$w54);
         $i++;
    }
}
} else {
mkdir($ruta1);
mkdir($ruta2);
$extension = "jpg";
 
$i = 1;
foreach($_FILES as $value){
   // Si la imagen fue subida correctamente
   if ( $value['error'] == UPLOAD_ERR_OK ){
        $image = new ResizePicture($value['tmp_name']);
        
        $image->resize(444,420);
        $w444 = $foto_nombre. '_' .$i. '.' . $extension;
        $image->save($ruta1.$w444);
        $image->resize(54,54);
        
        $w54 = $foto_nombre. '_'  .$i. '.' . $extension;
        $image->save($ruta2.$w54);
         $i++;
    }
}
}

Como puedo hacer esto????

Gracias

Nemutagk · #2 (**permalink**) 10/08/2013, 12:59

Podría ser algo así....

Código PHP:

Ver original$image->resize(444,420);
        $w444 = $ruta1.$foto_nombre. '_' .$i. '.' . $extension;
        if (file_exists($w444))
                $w444 = rand(1000,9999).'_'.$w444;
        $image->save($w444);
 
        $image->resize(54,54);
        $w54 = $ruta2.$foto_nombre. '_' .$i. '.' . $extension;
        if (file_exists($w54))
                $w444 = rand(1000,9999).'_'.$w54;
        $image->save($w54);

kimmy · #3 (**permalink**) 10/08/2013, 14:48

Hola Nemutagk gracias por contestar, pero no funciona... Sigue reemplazando las fotos y no agrega nuevas ???????

HalconVigia · #4 (**permalink**) 10/08/2013, 21:52

buenas!

se me ocurre que en lugar de tomar en cuenta el nombre unicamente tambien tomes en cuenta la hora, minutos y segundos para crear el nombre, porque es imposible que todo suba al mismo y eso hace que tu variable sea mas complicada de repetirse y asi evitar sobrescribirlo

saludos

kimmy · #5 (**permalink**) 10/08/2013, 21:58

Si estaba bien lo que me indicaste pero ya se porque no me funciona. Es porque tengo que incrementar en 1 y no se como. Yo las guardo asi P0001_1, P0001_2, P0001_3 que viene de:

$codigo = substr($id_foto,0,-4);
$extension = "jpg";
$w444 = $codigo. '_' .$i. '.' . $extension;

que va en:

Código PHP:

Ver originalif (file_exists($ruta1)) {
$extension = "jpg";
$i = 1;
foreach($_FILES as $value){
   // Si la imagen fue subida correctamente
   if ( $value['error'] == UPLOAD_ERR_OK ){
        $image = new ResizePicture($value['tmp_name']);
        $image->resize(444,420);
        
        $w444 = $codigo. '_' .$i. '.' . $extension;
        $image->save($ruta1.$w444);
 
        $image->resize(54,54);
        $w54 = $codigo. '_'  .$i. '.' . $extension;
        $image->save($ruta2.$w54);
         $i++;
    }
}
}

Entonces la siguiente debería llamarse P0001_4 (por ejemplo) y no se como lograr esto

???

Nemutagk · #6 (**permalink**) 10/08/2013, 22:10

Cielos. hasta me da pena decirlo por no verlo desde un principio....

Todo tu código esta mal, no puedes hacer un foreach de $_FILES porque $_FILES contiene varios items que se asocian a un mismo archivo (tmp_name, name, type, size y error) por lo tanto en varios loop se hace referente al mismo archivo, AÍ

kimmy · #7 (**permalink**) 10/08/2013, 22:15

Pero me funciona bien para guardarlas, solo no funciona para agregar nuevas. Que me sugieres que puedo hacer porque esto me esta volviendo loca. He tratado de buscar un tutorial para hacer un gestor galeria de fotos pero no he conseguido nada que sirva.

Nemutagk · #8 (**permalink**) 10/08/2013, 22:32

Cielos. hasta me da pena decirlo por no verlo desde un principio....

Todo tu código esta mal, no puedes hacer un foreach de $_FILES porque $_FILES contiene varios items que se asocian a un mismo archivo (tmp_name, name, type, size y error) por lo tanto en varios loop se hace referente al mismo archivo, así que todo esta mal, hagamos las cosas bien =)

Primero, cambia todos los name de los input:file a "fotos[]", checa con especial cuidado los corchetes en el name, son muy importantes, todos deben tener el mismo name!

Ejemplo:

Código HTML:

Ver original<td><input type="file" name="foto[]" id="foto1" /></td>
<td><input type="file" name="foto[]" id="foto2" /></td>
<td><input type="file" name="foto[]" id="foto3" /></td>
<td><input type="file" name="foto[]" id="foto4" /></td>

Ahora, en el código PHP haremos esto!

Código PHP:

Ver original<?php
//aquí 
$id_posada = $_POST['id_posada'];
$id_foto = $_POST['id_foto'];
 
$foto_nombre = substr($id_foto,0,-4); 
$codigo = substr($id_foto,0,-4);
$extension = "jpg";
 
$ruta1 = "../../images/galeria/large/$codigo/";
$ruta2 = "../../images/galeria/thumbnails/$codigo/";
 
foreach($_FILES['foto']['error'] as $key => $value) {
    if ($value == UPLOAD_ERR_OK) {
        $image = new ResizePicture($_FILES['foto']['tmp_name'][$key]);
 
        $image->resize(444,420);
        $w444 = $foto_nombre. '_' .$key. '.' . $extension;
        $image->save($ruta1.$w444);
 
        $image->resize(54,54);
        $w54 = $foto_nombre. '_'  .$key. '.' . $extension;
        $image->save($ruta2.$w54);
    }
}

Ahora, tu código tiene un problema, y es que siempre se asume que es la primera vez que se suben los archivos, por lo tanto el ID de cada archivo será 1, así que primero hay que verificar si ya existe la carpeta, si existe, ver cuantos archivos hay dentro de la carpeta y así asignar el ID indicado!

Código PHP:

Ver original<?php
$id_posada = $_POST['id_posada'];
$id_foto = $_POST['id_foto'];
 
$foto_nombre = substr($id_foto,0,-4); 
$codigo = substr($id_foto,0,-4);
$extension = "jpg";
 
$ruta1 = "../../images/galeria/large/$codigo/";
$ruta2 = "../../images/galeria/thumbnails/$codigo/";
 
//si la carpeta existe contamos los archivos que hay dentro!
if (is_dir($ruta1)) {
    // - 1 porque scandir agrega 2 items al array, el . y el .., así obtenemos el ID sigueinte!
    $idFile = count(scandir($ruta1)) - 1;
    mkdir($ruta1);
    mkdir($ruta2); 
}else {
    $idFile = 1;
}
 
foreach($_FILES['foto']['error'] as $key => $value) {
    if ($value == UPLOAD_ERR_OK) {
        $image = new ResizePicture($_FILES['foto']['tmp_name'][$key]);
 
        $image->resize(444,420);
        $w444 = $foto_nombre. '_' .$idFile. '.' . $extension;
        $image->save($ruta1.$w444);
 
        $image->resize(54,54);
        $w54 = $foto_nombre. '_'  .$idFile. '.' . $extension;
        $image->save($ruta2.$w54);
        $idFile++;
    }
}

Prueba el código y comenta como va!

kimmy · #9 (**permalink**) 10/08/2013, 22:44

Hola, muchas gracias. Lo probé y me da este error:

Warning: imagejpeg() [function.imagejpeg]: Unable to open '../../images/galeria/large/P0001/P0001_1.' for writing: No such file or directory in C:\AppServ\www\phpMyAdmin\vdv\func\ResizePicture.p hp on line 50

Warning: imagejpeg() [function.imagejpeg]: Unable to open '../../images/galeria/thumbnails/P0001/P0001_1.' for writing: No such file or directory in C:\AppServ\www\phpMyAdmin\vdv\func\ResizePicture.p hp on line 50

Nemutagk · #10 (**permalink**) 10/08/2013, 22:47

Upppsss, fíjate que eh puesto los mkdir() en la parte del if que indica que ya existe, cuando deben estar en el else!!! hay que cambiar de lugar >.<

Código PHP:

Ver originalif (is_dir($ruta1)) {
    // - 1 porque scandir agrega 2 items al array, el . y el .., así obtenemos el ID sigueinte!
    $idFile = count(scandir($ruta1)) - 1;
}else {
    $idFile = 1;
    mkdir($ruta1);
    mkdir($ruta2); 
}

kimmy · #11 (**permalink**) 10/08/2013, 22:55

Waoooo fabuloso, ahora si funciona. De verdad te lo agradezco mucho porque esta página web me ha dado muchos dolores de cabeza. Tengo varios años con php y sigo con un motón de lagunas que no me dejan avanzar en muchas cosas. Mil gracias.