Skip to main content
Olvidé dos breaks
Origen Enlace
OscarGarcia
  • 28.5k
  • 3
  • 29
  • 64
    function miniatura($archivo, $local, $ancho, $alto){    
        $arrNombre = explode(".", $archivo);
        $extension = $arrNombre[1];
        switch($extension) {
            case "jpg":
            case "jpeg":
                $nuevo = imagecreatefromjpeg($archivo);
                break;
            case "png":
                $nuevo = imagecreatefrompng($archivo);
                break;
            case "gif":
                $nuevo = imagecreatefromgif($archivo);
        }
        // Lo haremos de un tamaño 100x100
        $thumb = imagecreatetruecolor($ancho, $alto);
        $ancho_original = imagesx($nuevo);
        $alto_original = imagesy($nuevo);
        imagecopyresampled($thumb, $nuevo, 0, 0, 0, 0, $ancho, $alto, $ancho_original, $alto_original);
        $thumb_name = "miniaturas/$local.$extension";
        switch($extension) {
            case "jpg":
            case "jpeg":
                // 70 es la calidad de compresión
                imagejpeg($thumb, $thumb_name, 70);
                break;
            case "png":
                imagepng($thumb, $thumb_name);
                break;
            case "gif":
                imagegif($thumb, $thumb_name);
        }
    }
    function miniatura($archivo, $local, $ancho, $alto){    
        $arrNombre = explode(".", $archivo);
        $extension = $arrNombre[1];
        switch($extension) {
            case "jpg":
            case "jpeg":
                $nuevo = imagecreatefromjpeg($archivo);
                break;
            case "png":
                $nuevo = imagecreatefrompng($archivo);
            case "gif":
                $nuevo = imagecreatefromgif($archivo);
        }
        // Lo haremos de un tamaño 100x100
        $thumb = imagecreatetruecolor($ancho, $alto);
        $ancho_original = imagesx($nuevo);
        $alto_original = imagesy($nuevo);
        imagecopyresampled($thumb, $nuevo, 0, 0, 0, 0, $ancho, $alto, $ancho_original, $alto_original);
        $thumb_name = "miniaturas/$local.$extension";
        switch($extension) {
            case "jpg":
            case "jpeg":
                // 70 es la calidad de compresión
                imagejpeg($thumb, $thumb_name, 70);
                break;
            case "png":
                imagepng($thumb, $thumb_name);
            case "gif":
                imagegif($thumb, $thumb_name);
        }
    }
    function miniatura($archivo, $local, $ancho, $alto){    
        $arrNombre = explode(".", $archivo);
        $extension = $arrNombre[1];
        switch($extension) {
            case "jpg":
            case "jpeg":
                $nuevo = imagecreatefromjpeg($archivo);
                break;
            case "png":
                $nuevo = imagecreatefrompng($archivo);
                break;
            case "gif":
                $nuevo = imagecreatefromgif($archivo);
        }
        // Lo haremos de un tamaño 100x100
        $thumb = imagecreatetruecolor($ancho, $alto);
        $ancho_original = imagesx($nuevo);
        $alto_original = imagesy($nuevo);
        imagecopyresampled($thumb, $nuevo, 0, 0, 0, 0, $ancho, $alto, $ancho_original, $alto_original);
        $thumb_name = "miniaturas/$local.$extension";
        switch($extension) {
            case "jpg":
            case "jpeg":
                // 70 es la calidad de compresión
                imagejpeg($thumb, $thumb_name, 70);
                break;
            case "png":
                imagepng($thumb, $thumb_name);
                break;
            case "gif":
                imagegif($thumb, $thumb_name);
        }
    }
Modificación de la función de miniatura
Origen Enlace
OscarGarcia
  • 28.5k
  • 3
  • 29
  • 64

Estás guardando la miniatura siempre con el mismo nombre de archivo, por lo que podríaspodría llegarse a dar el caso de ver la miniatura generada por otro usuario diferente. 

Deberías guardar las miniaturas en un directorio (para poder realizar una limpieza posterior de miniaturas antiguas que no se usan) y con el nombre de archivo basado en la clave primaria (para evitar duplicidad de miniaturas por compartir nombre de archivo).

<!-- Ojo que aquí estás forzando la extensión -->
<img src="miniaturas/<?= urlencodehtmlspecialchars($id) ?>.jpg" />
/* Pasamos como parámetro el nombre (sin extensión) que debería tener */
miniatura($nombre_imagen, 'miniaturas/' .$i, 650, 400);

PD: En tu función no usas para nada el parámetro $local, terminas forzando el nombre de la miniatura. Deberías cambiarlo:

    function miniatura($archivo, $local, $ancho, $alto){    
        $arrNombre = explode(".", '$archivo);
        $extension = $arrNombre[1];
        switch($extension) {
            case "jpg":
            case "jpeg":
                $nuevo = imagecreatefromjpeg($archivo);
                break;
            case "png":
                $nuevo = imagecreatefrompng($archivo);
            case "gif":
                $nuevo = imagecreatefromgif($archivo);
        }
        // Lo haremos de un tamaño 100x100
        $thumb = imagecreatetruecolor($ancho, $alto);
        $ancho_original = imagesx($nuevo);
        $alto_original = imagesy($nuevo);
        imagecopyresampled($thumb, $nuevo, 0, 0, 0, 0, $ancho, $alto, $ancho_original, $alto_original);
        $thumb_name = "miniaturas/$local.jpg'$extension";
        switch($extension) {
            case "jpg":
            case "jpeg":
                // 70 es la calidad de compresión
                imagejpeg($thumb, 650$thumb_name, 40070);
                break;
            case "png":
                imagepng($thumb, $thumb_name);
            case "gif":
                imagegif($thumb, $thumb_name);
        }
    }

Estás guardando la miniatura siempre con el mismo nombre de archivo, por lo que podrías llegarse a dar el caso de ver la miniatura generada por otro usuario diferente. Deberías guardar las miniaturas en un directorio (para poder realizar una limpieza posterior de miniaturas antiguas que no se usan) y con el nombre de archivo basado en la clave primaria (para evitar duplicidad de miniaturas por compartir nombre de archivo).

<img src="miniaturas/<?= urlencode($id) ?>.jpg" />
miniatura($nombre_imagen, 'miniaturas/' .$i . '.jpg', 650, 400);

Estás guardando la miniatura siempre con el mismo nombre de archivo, por lo que podría llegarse a dar el caso de ver la miniatura generada por otro usuario diferente. 

Deberías guardar las miniaturas en un directorio (para poder realizar una limpieza posterior de miniaturas antiguas que no se usan) y con el nombre de archivo basado en la clave primaria (para evitar duplicidad de miniaturas por compartir nombre de archivo).

<!-- Ojo que aquí estás forzando la extensión -->
<img src="miniaturas/<?= htmlspecialchars($id) ?>.jpg" />
/* Pasamos como parámetro el nombre (sin extensión) que debería tener */
miniatura($nombre_imagen, $i, 650, 400);

PD: En tu función no usas para nada el parámetro $local, terminas forzando el nombre de la miniatura. Deberías cambiarlo:

    function miniatura($archivo, $local, $ancho, $alto){    
        $arrNombre = explode(".", $archivo);
        $extension = $arrNombre[1];
        switch($extension) {
            case "jpg":
            case "jpeg":
                $nuevo = imagecreatefromjpeg($archivo);
                break;
            case "png":
                $nuevo = imagecreatefrompng($archivo);
            case "gif":
                $nuevo = imagecreatefromgif($archivo);
        }
        // Lo haremos de un tamaño 100x100
        $thumb = imagecreatetruecolor($ancho, $alto);
        $ancho_original = imagesx($nuevo);
        $alto_original = imagesy($nuevo);
        imagecopyresampled($thumb, $nuevo, 0, 0, 0, 0, $ancho, $alto, $ancho_original, $alto_original);
        $thumb_name = "miniaturas/$local.$extension";
        switch($extension) {
            case "jpg":
            case "jpeg":
                // 70 es la calidad de compresión
                imagejpeg($thumb, $thumb_name, 70);
                break;
            case "png":
                imagepng($thumb, $thumb_name);
            case "gif":
                imagegif($thumb, $thumb_name);
        }
    }
Origen Enlace
OscarGarcia
  • 28.5k
  • 3
  • 29
  • 64

Estás guardando la miniatura siempre con el mismo nombre de archivo, por lo que podrías llegarse a dar el caso de ver la miniatura generada por otro usuario diferente. Deberías guardar las miniaturas en un directorio (para poder realizar una limpieza posterior de miniaturas antiguas que no se usan) y con el nombre de archivo basado en la clave primaria (para evitar duplicidad de miniaturas por compartir nombre de archivo).

Para empezar, cambiamos la imagen a mostrar de la siguiente manera:

<img src="miniaturas/<?= urlencode($id) ?>.jpg" />

Y posteriormente deberemos guardar la imagen de la siguiente manera:

miniatura($nombre_imagen, 'miniaturas/' .$i . '.jpg', 650, 400);