Enable the APP attachment file download to work correctly rather than just returning a lot of php notices. Fixes WordPress#10372 for trunk props mysz.

westi · westi · commit 31387fd45c5d · 2009-08-13T20:31:12.000Z
git-svn-id: https://develop.svn.wordpress.org/trunk@11810 602fd350-edb4-49c9-b593-d223f7449a82
diff --git a/wp-app.php b/wp-app.php
@@ -780,6 +780,7 @@ function get_file($postID) {
 		}
 
 		$location = get_post_meta($entry['ID'], '_wp_attached_file', true);
+		$location = get_option ('upload_path') . '/' . $location; 
 		$filetype = wp_check_filetype($location);
 
 		if(!isset($location) || 'attachment' != $entry['post_type'] || empty($filetype['ext']))
@@ -789,11 +790,19 @@ function get_file($postID) {
 		header('Content-Type: ' . $entry['post_mime_type']);
 		header('Connection: close');
 
-		$fp = fopen($location, "rb");
-		while(!feof($fp)) {
-			echo fread($fp, 4096);
+		if ($fp = fopen($location, "rb")) { 
+			status_header('200'); 
+			header('Content-Type: ' . $entry['post_mime_type']); 
+			header('Connection: close');
+
+			while(!feof($fp)) {
+				echo fread($fp, 4096);
+			}
+
+			fclose($fp);
+		} else {
+			status_header ('404');
 		}
-		fclose($fp);
 
 		log_app('function',"get_file($postID)");
 		exit;
