File Download - Broke in IE only

[Stooney]

The following code works fine when using chrome or firefox, but fails with IE.

Error: Requested site is either unavailable or cannot be found.

 

Question: Does anyone see anything here that is a known bug with IE that I don't know about?

 

public function download(){
	if($this->validateFile()){
		if(!headers_sent()){
			header('Content-type: '.$this->getType());
			header('Content-Disposition: attachment; filename="'.$this->name.'"');
			readfile($this->file);
		} else{
			echo 'Headers have already been sent and are preventing the file download from taking place.';
		}
		return true;
	}
	return false;
}

//I felt this function might be relevant to the issue.  I know, it's kind of a ghetto function, I'll improve on it some other time though.

private function getType(){
	$types=array(		'txt'	=>	'text/plain', 
					'mp3'	=>	'audio/x-mp3', 
					'jpg'	=>	'image/jpeg', 
					'jpeg'	=>	'image/jpeg', 
					'gif'	=>	'image/gif', 
					'png'	=>	'image/png', 
					'doc'	=>	'application/msword', 
					'swf'	=>	'application/x-shockwave-flash', 
					'flv'	=>	'video/flv', 
					'mpg'	=>	'video/mpeg', 
					'mpeg'	=>	'video/mpeg', 
					'avi'	=>	'video/x-msvideo', 
					'rtf'	=>	'application/rtf', 
					'pdf'	=>	'application/pdf', 
					'zip'	=>	'application/zip', 
					'exe'	=>	'application/octet-stream', 
					'xls'	=>	'application/msexcel', 
					'xlsxl'	=>	'application/msexcel');
	if(!$types[$this->getExt()]==''){
		return $types[$this->getExt()];
	}
	return 'application/octet-stream';
}

 

[.josh]

dunno if it'll fix your problem, but this line:

 

if(!$types[$this->getExt()]==''){

 

should be

 

if(isset($types[$this->getExt()])){

[JasonLewis]

Your not setting a Content-Length header either, perhaps add that is as well.

 

header("Content-Length: " . filesize($this->file));

[Stooney]

For future reference to others, the following fixed the issue for me:

 

<?php
header('Content-Description: File Transfer');
header('Content-type: '.$this->getType());
header('Content-Disposition: attachment; filename='.basename($this->file));
header('Content-Transfer-Encoding: binary');
header('Expires: 0');
header('Cache-Control: must-revalidate, post-check=0, pre-check=0');
header('Pragma: public');
header("Content-Length: " . filesize($this->file));
ob_clean();
flush();
?>