From 3ff40471cc6ba39d5898da15342a977783a20d07 Mon Sep 17 00:00:00 2001
From: jonathangreen <jonathan@discoverygarden.ca>
Date: Thu, 1 Nov 2012 10:26:22 -0300
Subject: [PATCH] Modify datastreams to not load files into memory

We were putting potentially large datastreams into memeory in the
download and view datastream hooks. We now create a temp file then
we output that to the user.
---
 includes/datastream.inc | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/includes/datastream.inc b/includes/datastream.inc
index 69b725ac..d6596490 100644
--- a/includes/datastream.inc
+++ b/includes/datastream.inc
@@ -21,7 +21,8 @@ function islandora_view_datastream($object, $dsid, $method = 'view') {
   if (!isset($object[$dsid])) {
     return drupal_not_found();
   }
-
+  header_remove('Cache-Control');
+  header_remove('Expires');
   header('Content-type: ' . $object[$dsid]->mimetype);
   if ($object[$dsid]->controlGroup == 'M' || $object[$dsid]->controlGroup == 'X') {
     header('Content-length: ' . $object[$dsid]->size);
@@ -30,7 +31,15 @@ function islandora_view_datastream($object, $dsid, $method = 'view') {
   if ($method == 'download') {
     header("Content-Disposition: attachment; filename=\"" . $object[$dsid]->label);
   }
-  print($object[$dsid]->content);
+
+  // Disable page caching
+  drupal_page_is_cacheable(FALSE);
+
+  // Try not to load the file into PHP memory
+  $file = drupal_tempnam(file_directory_temp(), 'islandora');
+  $object[$dsid]->getContent($file);
+  readfile($file);
+  drupal_unlink($file);
   exit();
 }