Cuando pulsas el boton Descarga, se lanza el formulario que lo engloba, y te lleva al mismo archivo que estas ejecutando. Se esta recargando toda la pagina, y aunque en ese momento Descargar
esta definido, NO SE ENVIAN las cabeceras de archivo que tienes en la funcion descargar(), puesto que ya se han enviado las de html que forman tu pagina web.
Te sugiero 2 formas:
Que la accion de descargar esté en otro archivo:
genera un archivo (por ej: downloadQr.php
) con el siguiente contenido:
<?php
include "includes/functions.php";
$PNG_TEMP_DIR = dirname(__FILE__).DIRECTORY_SEPARATOR.'php_action'.DIRECTORY_SEPARATOR.'temp'.DIRECTORY_SEPARATOR;
//ofcourse we need rights to create temp dir
if (!file_exists($PNG_TEMP_DIR))
mkdir($PNG_TEMP_DIR);
$filename = $PNG_TEMP_DIR.'test.png';
if(isset($_GET['Descargar'])) {
descargar($filename);
}
?>
y en tu formulario que contiene la llamada del boton Descargar, cambia el action hacia este nuevo archivo. Algo como:
<form method="get" action="downloadQr.php">
<button class="btn btn-default button3" data-toggle="submit"
name="Descargar" data-target = "Descarga"><i class="glyphicon glyphicon-
download-alt"></i> Descargar Qr en su Computadora </button>
</form>
Con esto, cuando pulses el boton Descargar, se llama a la accion de descargar del archivo en downloadQr.php. Se enviaran las cabeceras que defines en la funcion, y te descargará el archivo.
Gestiona la llamada de descarga al principio de tu archivo: Cambia la gestion de crear el archivo, hacia el principio de tu codigo. Pon lo primero del todo esto:
<?php
include "includes/functions.php";
$PNG_TEMP_DIR = dirname(__FILE__).DIRECTORY_SEPARATOR.'php_action'.DIRECTORY_SEPARATOR.'temp'.DIRECTORY_SEPARATOR;
//ofcourse we need rights to create temp dir
if (!file_exists($PNG_TEMP_DIR))
mkdir($PNG_TEMP_DIR);
$filename = $PNG_TEMP_DIR.'test.png';
if(isset($_GET['Descargar'])) {
descargar($filename);
} else {
?>
date cuenta que he puesto un else
, esto es para que primero se vea si esta definido descargar y SOLO haga la descarga (enviando las cabeceras de archivo).
Este else deberás cerrarlo al final de tu archivo, para que, si no se ha pedido descargar el QR, que te muestre el resto de tu accion.
MI RECOMENDACION:
Todo este problema te surge porque mezclas html y proceso de php, y no esta muy claro (y se ve dificil) organizar estas cosas. Te recomendaria que empieces a utilizar MVC, revisar o utilizar algun framework (tipo symfony, codeigniter, laravel) para ver como se organizan los procesos en controladores y vistas.