Error descarga archivos

[alonsomaxx]

Hola amigos del foro.
Tengo un par de problemas a la hora de descargar archivos.
El primero es que si utilizo header("Content-Length: ".$size); a la hora de descargar el archivo desde localhost me da el siguiente error al abrirlo: "El archivo tiene un formato desconocido o esta dañado."

y el segundo problema es que cuando intento descargar el archivo desde un servidor al pinchar en el enlace de descarga me aparece una ventana con simbolos extraños y no me descarga el archivo.

Mi código es el siguiente:

Código PHP:

          if (!isset($_GET['archi']) || empty($_GET['archi'])) {
            exit();
        }
        
        $root = "download/";
        $file = basename($_GET['archi']);
        $path = $root . $file.".zip";      
        $type = '';
 
        if (is_file($path)) {
            $size = filesize($path);
            
            if (function_exists('mime_content_type')) {
                $type = mime_content_type($path);
                
            } else if (function_exists('finfo_file')) {
                $info = finfo_open(FILEINFO_MIME);
                $type = finfo_file($info, $path);
                finfo_close($info);
            }
            if ($type == '') {
                $type = "application/force-download";
            }
            
            // Definir headers            
            header("Content-Type: $type");
            header("Content-Disposition: attachment; filename=$file.zip");
            header("Content-Transfer-Encoding: binary");
            header("Content-Length: " . $size);  
            // Descargar archivo
            readfile($path);
        } else {
            die("El archivo no existe.");
        } 
   

Gracias por vuestro tiempo y un saludo.

[pateketrueke]

El código que muestras se ve correcto, ¿seguro que ese es el 100% del script?

[alonsomaxx]

Si que es el 100% de script.
No tengo nada modificado.

[pateketrueke]

¿Y las etiquetas de inicio y cierre de php se pintan solas o cómo?

En teoría después del readfile() deberías usar exit() para evitar salidas extra al final del mismo.

[alonsomaxx]

Esté es todo el código y con etiquetas de apertura y de cierre
Además he añadido el exit como me indicaste

Código PHP:

  <?php
    function Downloads() {
        if (!isset($_GET['archi']) || empty($_GET['archi'])) {
            exit();
        }
        
        $root = "download/";
        $file = basename($_GET['archi']);
        $path = $root . $file.".zip";      
        $type = '';
 
        if (is_file($path)) {
            $size = filesize($path);
            
            if (function_exists('mime_content_type')) {
                $type = mime_content_type($path);
                
            } else if (function_exists('finfo_file')) {
                $info = finfo_open(FILEINFO_MIME);
                $type = finfo_file($info, $path);
                finfo_close($info);
            }
            if ($type == '') {
                $type = "application/force-download";
            }
            // Definir headers
            
            header("Content-Type: $type");
            header("Content-Disposition: attachment; filename=$file.zip");
            header("Content-Transfer-Encoding: binary");
            header("Content-Length: " . $size);  //Esta linea da error en localhost
 
            // Descargar archivo
            readfile($path);
            exit(); 
 
        } else {
            die("El archivo no existe.");
        }
    }
?>   

[pateketrueke]

A ver, ¿entonces definiste Downloads() como función pero exactamente dónde y cómo la usas?

[alonsomaxx]

Esta dentro de una clase. Utilizo el Modelo vista controlador.

[pateketrueke]

¿Y no crees que el problema radique ahí entonces?

Porque el script de descarga es perfecto, no podemos adivinar el 100% de tu sistema para entender.

[alonsomaxx]

Mejor así...

Enlace:
<a href="index.php?ctl=descargas&archi=archivo">Desca rgar Archivo</a>

Controlador:

Código PHP:

  function misDescargas(){
            $sec=new secciones();            
            $sec->Downloads();
        } 
   

Clase secciones:

Código PHP:

  <?php
class secciones{
    function Downloads() {
        if (!isset($_GET['archi']) || empty($_GET['archi'])) {
            exit();
        }
        
        $root = "download/";
        $file = basename($_GET['archi']);
        $path = $root . $file.".zip";      
        $type = '';

        if (is_file($path)) {
            $size = filesize($path);
            
            if (function_exists('mime_content_type')) {
                $type = mime_content_type($path);
                
            } else if (function_exists('finfo_file')) {
                $info = finfo_open(FILEINFO_MIME);
                $type = finfo_file($info, $path);
                finfo_close($info);
            }
            if ($type == '') {
                $type = "application/force-download";
            }
            // Definir headers
            
            header("Content-Type: $type");
            header("Content-Disposition: attachment; filename=$file.zip");
            header("Content-Transfer-Encoding: binary");
            header("Content-Length: " . $size);  //Esta linea da error en localhost
            // Descargar archivo
            readfile($path);
            exit();
        } else {
            die("El archivo no existe.");
        }
    }
}
?>   

[pateketrueke]

Sigue sin entenderse nada.

A ver, en algún lado de tu sistema de MVC es que haces la carga de todo, puede que ahí metas algo que no deberías.

Inclusive un include/require con espacios al final puede arruinar todo, ¿cómo podríamos saber exactamente todos los pasos que realiza tu sistema?