Browse Source

Merge pull request #1970 from jasonlfunk/master

Allow developers to set the Content-Disposition header in Response::download()
Taylor Otwell 11 years ago
parent
commit
3965429176
1 changed files with 7 additions and 2 deletions
  1. 7 2
      laravel/response.php

+ 7 - 2
laravel/response.php

@@ -202,9 +202,14 @@ class Response {
 		// off to the HttpFoundation and let it create the header text.
 		$response = new static(File::get($path), 200, $headers);
 
-		$d = $response->disposition($name);
+		// If the Content-Disposition header has already been set by the
+		// merge above, then do not override it with out generated one.
+		if (!isset($headers['Content-Disposition'])) {
+			$d = $response->disposition($name);
+			$response = $response->header('Content-Disposition', $d);
+		}
 
-		return $response->header('Content-Disposition', $d);
+		return $response;
 	}
 
 	/**