O problema todo é que você está fazendo o download junto do conteúdo do index.php:
<?php
require 'config/Config.php';
require 'config/tratarUrl.php';
?>
<html lang="pt-br">
<link rel="stylesheet" href="<?=DIR_CSS?>Style.css">
<link rel="icon" href="<?= DIR_IMAGES?>icon.png"
<meta charset="UTF-8">
<title>Site</title>
<head>
<meta charset="UTF-8">
</head>
<body>
<?php
include $pag;
?>
</body>
</html>
Ou seja quando baixa o arquivo, isso está vindo junto:
<html lang="pt-br">
<link rel="stylesheet" href="Style.css">
<link rel="icon" href="icon.png">
<meta charset="UTF-8">
<title>Site</title>
<head>
<meta charset="UTF-8">
</head>
<body>
CONTEUDO DO ARQUIVO AQUI
</body>
</html>
Eu recomendo separar o download.php ou simplesmente criar um if.
Usando if:
<?php
require 'config/Config.php';
require 'config/tratarUrl.php';
if (preg_match('#pages/download\.php$#', $pag) > 0) {
include $pag;
exit;
}
?>
<html lang="pt-br">
<link rel="stylesheet" href="<?=DIR_CSS?>Style.css">
<link rel="icon" href="<?= DIR_IMAGES?>icon.png"
<meta charset="UTF-8">
<title>Site</title>
<head>
<meta charset="UTF-8">
</head>
<body>
<?php
include $pag;
?>
</body>
</html>
Ou então você pode reescrever o .htaccess assim:
RewriteEngine On
RewriteBase /
RewriteCond %{SCRIPT_FILENAME} !-f
RewriteCond %{SCRIPT_FILENAME} !-d
#se a url vier com prefix download
RewriteRule ^download/ /pages/download.php [QSA,L]
#outras urls
RewriteRule (.*) /index.php [QSA,L]
Então o seu pages/download.php
deve ficar algo mais ou menos assim:
<?php
require 'config/Config.php';
//... Conteudo do seu download.php ...
$file_path = DIR_ARQUIVOS.$key.'.zip';
$file_name = $key.'.zip';
$new_name = $sql->getKeyName($key);
set_time_limit(0);
// Verifica se o arquivo não existe
if (!file_exists($file_path)) {
// Exiba uma mensagem de erro caso ele não exista
exit;
}
// Configuramos os headers que serão enviados para o browser
header('Content-Type: "application/zip"');
header('Content-Disposition: attachment; filename="'.basename($file_path).'"');
header("Content-Transfer-Encoding: binary");
header('Expires: 0');
header('Pragma: no-cache');
// Envia o arquivo para o cliente
readfile($file_name);
Problemas com filesize
O filesize
não era o problema do teu script, mas sim o conteúdo misturado como citei, uma coisa interessante de enviar o Content-Length
é que a barra de progresso dos navegadores costumar dar uma estimativa em porcentagem do download mais precisa, então seria interessante definir isto. Todavia é bom notar que filesize
usa int
e dependendo da arquitetura do processador você terá limites, o que pode ocasionar problemas como o que citei em filesize para arquivos maiores que 2GB em plataformas x86
Uma solução alternativa seria usando o CURL
(que retorna em string
e não em int
), assim:
function filesizecurl($arquivo)
{
if (is_file($arquivo) === false) {
return false;
}
$arquivo = realpath(preg_replace('#^file:#', '', $arquivo));
$ch = curl_init('file://' . ltrim($arquivo, '/'));
curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1); //Faz o retorno ser salvo na variável
curl_setopt($ch, CURLOPT_HEADER, 1); //Faz retornar os headers
curl_setopt($ch, CURLOPT_NOBODY, 1); //Evita retornar o corpo
$headers = curl_exec($ch);
curl_close($ch);
$ch = null;
//Com preg_match extraímos o tamanho retornado de Content-Length
if (preg_match('#(^c|\sc)ontent\-length:(\s|)(\d+)#i', $headers, $matches) > 0) {
return $matches[3];
}
return false;
}
// Configuramos os headers que serão enviados para o browser
header('Content-Length: ' . filesizecurl($file_name));
header('Content-Type: "application/zip"');
header('Content-Disposition: attachment; filename="'.basename($file_path).'"');
header("Content-Transfer-Encoding: binary");
header('Expires: 0');
header('Pragma: no-cache');
// Envia o arquivo para o cliente
readfile($file_name);