chore(views): Cache assets by viewtype/view, not md5(viewtype/view)

Fixes Elgg#8381
ewinslow · Jun 19, 2015 · e7ee012 · e7ee012
1 parent 2b431ba
commit e7ee012
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/engine/classes/Elgg/Application/CacheHandler.php b/engine/classes/Elgg/Application/CacheHandler.php
@@ -83,7 +83,7 @@ public function handleRequest($path) {
 			exit;
 		}
 
-		$filename = $config->getVolatile('dataroot') . 'views_simplecache/' . md5("$viewtype|$view");
+		$filename = $config->getVolatile('dataroot') . "views_simplecache/$viewtype/$view";
 		if (file_exists($filename)) {
 			$this->sendCacheHeaders($etag);
 			readfile($filename);
@@ -104,9 +104,9 @@ public function handleRequest($path) {
 
 			$content = $this->getProcessedView($view, $viewtype);
 
-			$dir_name = $config->getDataPath() . 'views_simplecache/';
+			$dir_name = dirname($filename);
 			if (!is_dir($dir_name)) {
-				mkdir($dir_name, 0700);
+				mkdir($dir_name, 0700, true);
 			}
 
 			file_put_contents($filename, $content);