Fix MediaView extension handling as documented and expected.

cakephp · Aug 21, 2015 · f154a8f · f154a8f
1 parent 0df0f1a
commit f154a8f
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/lib/Cake/View/MediaView.php b/lib/Cake/View/MediaView.php
@@ -65,7 +65,7 @@ class MediaView extends View {
  * @return void
  */
 	public function render($view = null, $layout = null) {
-		$name = $download = $id = $modified = $path = $cache = $mimeType = $compress = null;
+		$name = $extension = $download = $id = $modified = $path = $cache = $mimeType = $compress = null;
 		extract($this->viewVars, EXTR_OVERWRITE);
 
 		$path = $path . $id;
@@ -86,7 +86,12 @@ public function render($view = null, $layout = null) {
 		}
 
 		if ($name !== null) {
-			$name .= '.' . pathinfo($id, PATHINFO_EXTENSION);
+			if (empty($extension)) {
+				$extension = pathinfo($id, PATHINFO_EXTENSION);
+			}
+			if (!empty($extension)) {
+				$name .= '.' . $extension;
+			}
 		}
 		$this->response->file($path, compact('name', 'download'));