From 047fe3a410624f34d617e4577669c4c7db52613b Mon Sep 17 00:00:00 2001
From: JorisDebonnet <joris@funkytime.com>
Date: Thu, 12 Feb 2015 02:20:54 +0100
Subject: [PATCH] BUG Include php version in default cache folder name Update
 CoreTest.php

---
 core/TempPath.php       |  5 +++--
 tests/core/CoreTest.php | 28 ++++++++++++++++------------
 2 files changed, 19 insertions(+), 14 deletions(-)

diff --git a/core/TempPath.php b/core/TempPath.php
index b64f2609d..93ae274db 100644
--- a/core/TempPath.php
+++ b/core/TempPath.php
@@ -64,8 +64,9 @@ function getTempParentFolder($base = null) {
 	}
 
 	// failing the above, try finding a namespaced silverstripe-cache dir in the system temp
-	$cacheFolder = DIRECTORY_SEPARATOR . 'silverstripe-cache' . str_replace(array(' ', '/', ':', '\\'), '-', $base);
-	$tempPath = sys_get_temp_dir() . $cacheFolder;
+	$tempPath = sys_get_temp_dir() . DIRECTORY_SEPARATOR . 
+		'silverstripe-cache-php' . preg_replace('/[^\w-\.+]+/', '-', PHP_VERSION) . 
+		str_replace(array(' ', '/', ':', '\\'), '-', $base);
 	if(!@file_exists($tempPath)) {
 		$oldUMask = umask(0);
 		$worked = @mkdir($tempPath, 0777);
diff --git a/tests/core/CoreTest.php b/tests/core/CoreTest.php
index a6e6df99c..3fa928cbb 100644
--- a/tests/core/CoreTest.php
+++ b/tests/core/CoreTest.php
@@ -22,20 +22,22 @@ class CoreTest extends SapphireTest {
 			$this->assertEquals(getTempFolder(BASE_PATH), $this->tempPath . DIRECTORY_SEPARATOR . $user);
 		} else {
 			$user = getTempFolderUsername();
+			$base = sys_get_temp_dir() . DIRECTORY_SEPARATOR . 'silverstripe-cache-php' . 
+				preg_replace('/[^\w-\.+]+/', '-', PHP_VERSION);
 
 			// A typical Windows location for where sites are stored on IIS
-			$this->assertEquals(sys_get_temp_dir() . DIRECTORY_SEPARATOR .
-				'silverstripe-cacheC--inetpub-wwwroot-silverstripe-test-project' . DIRECTORY_SEPARATOR . $user,
+			$this->assertEquals(
+				$base . 'C--inetpub-wwwroot-silverstripe-test-project' . DIRECTORY_SEPARATOR . $user,
 				getTempFolder('C:\\inetpub\\wwwroot\\silverstripe-test-project'));
 
 			// A typical Mac OS X location for where sites are stored
-			$this->assertEquals(sys_get_temp_dir() . DIRECTORY_SEPARATOR . 
-				'silverstripe-cache-Users-joebloggs-Sites-silverstripe-test-project' . DIRECTORY_SEPARATOR . $user,
+			$this->assertEquals(
+				$base . '-Users-joebloggs-Sites-silverstripe-test-project' . DIRECTORY_SEPARATOR . $user,
 				getTempFolder('/Users/joebloggs/Sites/silverstripe-test-project'));
 
 			// A typical Linux location for where sites are stored
-			$this->assertEquals(sys_get_temp_dir() . DIRECTORY_SEPARATOR .
-				'silverstripe-cache-var-www-silverstripe-test-project' . DIRECTORY_SEPARATOR . $user,
+			$this->assertEquals(
+				$base . '-var-www-silverstripe-test-project' . DIRECTORY_SEPARATOR . $user,
 				getTempFolder('/var/www/silverstripe-test-project'));
 		}
 	}
@@ -43,17 +45,19 @@ class CoreTest extends SapphireTest {
 	public function tearDown() {
 		parent::tearDown();
 		$user = getTempFolderUsername();
+		$base = sys_get_temp_dir() . DIRECTORY_SEPARATOR . 'silverstripe-cache-php' . 
+			preg_replace('/[^\w-\.+]+/', '-', PHP_VERSION);
 		foreach(array(
-			'silverstripe-cacheC--inetpub-wwwroot-silverstripe-test-project',
-			'silverstripe-cache-Users-joebloggs-Sites-silverstripe-test-project',
-			'silverstripe-cache-var-www-silverstripe-test-project'
+			'C--inetpub-wwwroot-silverstripe-test-project',
+			'-Users-joebloggs-Sites-silverstripe-test-project',
+			'-cache-var-www-silverstripe-test-project'
 		) as $dir) {
-			$path = sys_get_temp_dir().DIRECTORY_SEPARATOR.$dir;
+			$path = $base . $dir;
 			if(file_exists($path)) {
 				rmdir($path . DIRECTORY_SEPARATOR . $user);
 				rmdir($path);
+			}
 		}
-		}
-		}
+	}
 
 }