Skip to content

Commit

Permalink
Fix bug bcit-ci#195
Browse files Browse the repository at this point in the history
Fixes bug bcit-ci#195 regarding non-existent user agent strings when using force_download() helper.
  • Loading branch information
AkenRoberts committed Jan 24, 2012
1 parent 96db8f9 commit 0e4d2b6
Showing 1 changed file with 10 additions and 16 deletions.
26 changes: 10 additions & 16 deletions system/helpers/download_helper.php
Original file line number Diff line number Diff line change
Expand Up @@ -88,26 +88,20 @@ function force_download($filename = '', $data = '')
{
$mime = (is_array($mimes[$extension])) ? $mimes[$extension][0] : $mimes[$extension];
}

// Generate the server headers
if (strpos($_SERVER['HTTP_USER_AGENT'], "MSIE") !== FALSE)
header('Content-Type: "'.$mime.'"');
header('Content-Disposition: attachment; filename="'.$filename.'"');
header('Expires: 0');
header("Content-Transfer-Encoding: binary");
header("Content-Length: ".strlen($data));
header('Pragma: no-cache');

// Internet Explorer-specific headers.
if (isset($_SERVER['HTTP_USER_AGENT']) && strpos($_SERVER['HTTP_USER_AGENT'], "MSIE") !== FALSE)
{
header('Content-Type: "'.$mime.'"');
header('Content-Disposition: attachment; filename="'.$filename.'"');
header('Expires: 0');
header('Cache-Control: must-revalidate, post-check=0, pre-check=0');
header("Content-Transfer-Encoding: binary");
header('Pragma: public');
header("Content-Length: ".strlen($data));
}
else
{
header('Content-Type: "'.$mime.'"');
header('Content-Disposition: attachment; filename="'.$filename.'"');
header("Content-Transfer-Encoding: binary");
header('Expires: 0');
header('Pragma: no-cache');
header("Content-Length: ".strlen($data));
}

exit($data);
Expand Down

0 comments on commit 0e4d2b6

Please sign in to comment.