Fix content-type header in cached views. Fixes issue #2358

lib/Cake/Routing/Filter/CacheDispatcher.php

@@ -60,6 +60,7 @@ public function beforeDispatch(CakeEvent $event) {
 		if (file_exists($filename)) {
 			$controller = null;
 			$view = new View($controller);
+			$view->response = $event->data['response'];
 			$result = $view->renderCache($filename, microtime(true));
 			if ($result !== false) {
 				$event->stopPropagation();

lib/Cake/View/Helper/CacheHelper.php

@@ -307,7 +307,7 @@ protected function _writeFile($content, $timestamp, $useCallbacks = false) {
 
 		$file .= '
 				$request = unserialize(base64_decode(\'' . base64_encode(serialize($this->request)) . '\'));
-				$response = new CakeResponse();
+				$response->type(\'' . $this->_View->response->type() . '\');
 				$controller = new ' . $this->_View->name . 'Controller($request, $response);
 				$controller->plugin = $this->plugin = \'' . $this->_View->plugin . '\';
 				$controller->helpers = $this->helpers = unserialize(base64_decode(\'' . base64_encode(serialize($this->_View->helpers)) . '\'));

lib/Cake/View/View.php

@@ -547,10 +547,12 @@ public function renderLayout($content, $layout = null) {
  * @return boolean Success of rendering the cached file.
  */
 	public function renderCache($filename, $timeStart) {
+		$response = $this->response;
 		ob_start();
 		include ($filename);
 
-		if (Configure::read('debug') > 0 && $this->layout !== 'xml') {
+		$type = $response->mapType($response->type());
+		if (Configure::read('debug') > 0 && $type === 'html') {
 			echo "<!-- Cached Render Time: " . round(microtime(true) - $timeStart, 4) . "s -->";
 		}
 		$out = ob_get_clean();
@@ -563,9 +565,6 @@ public function renderCache($filename, $timeStart) {
 				unset($out);
 				return false;
 			} else {
-				if ($this->layout === 'xml') {
-					header('Content-type: text/xml');
-				}
 				return substr($out, strlen($match[0]));
 			}
 		}