RequestFactory: script path is not detected from DOCUMENT_ROOT & SCRIPT_FILENAME (BC break)

see #31

Author: dg

src/Http/RequestFactory.php

@@ -88,17 +88,8 @@ public function createHttpRequest()
 		$url->setQuery(isset($tmp[1]) ? $tmp[1] : '');
 
 		// detect script path
-		if (isset($_SERVER['SCRIPT_NAME'])) {
-			$script = $_SERVER['SCRIPT_NAME'];
-		} elseif (isset($_SERVER['DOCUMENT_ROOT'], $_SERVER['SCRIPT_FILENAME'])
-			&& strncmp($_SERVER['DOCUMENT_ROOT'], $_SERVER['SCRIPT_FILENAME'], strlen($_SERVER['DOCUMENT_ROOT'])) === 0
-		) {
-			$script = '/' . ltrim(strtr(substr($_SERVER['SCRIPT_FILENAME'], strlen($_SERVER['DOCUMENT_ROOT'])), '\\', '/'), '/');
-		} else {
-			$script = '/';
-		}
-
 		$path = strtolower($url->getPath()) . '/';
+		$script = isset($_SERVER['SCRIPT_NAME']) ? $_SERVER['SCRIPT_NAME'] : '/';
 		$script = strtolower($script) . '/';
 		$max = min(strlen($path), strlen($script));
 		for ($i = 0; $i < $max; $i++) {

tests/Http/Request.request.phpt

@@ -19,7 +19,6 @@ $_SERVER = array(
 	'REMOTE_ADDR' => '192.168.188.66',
 	'REQUEST_METHOD' => 'GET',
 	'REQUEST_URI' => '/file.php?x param=val.&pa%%72am=val2&param3=v%20a%26l%3Du%2Be)',
-	'SCRIPT_FILENAME' => '/public_html/www/file.php',
 	'SCRIPT_NAME' => '/file.php',
 );
 

tests/Http/RequestFactory.scriptPath.phpt

@@ -16,7 +16,6 @@ $factory = new RequestFactory;
 test(function() use ($factory) {
 	$_SERVER = array(
 		'REQUEST_URI' => '/projects/modules-usage/www/',
-		'SCRIPT_FILENAME' => 'W:/projects/Modules-Usage/www/index.php',
 		'SCRIPT_NAME' => '/projects/modules-usage/www/index.php',
 	);
 
@@ -27,7 +26,6 @@ test(function() use ($factory) {
 test(function() use ($factory) {
 	$_SERVER = array(
 		'REQUEST_URI' => '/projects/modules-usage/www/default/add-item',
-		'SCRIPT_FILENAME' => 'W:/projects/Modules-Usage/www/index.php',
 		'SCRIPT_NAME' => '/projects/Modules-Usage/www/index.php',
 	);
 
@@ -38,7 +36,6 @@ test(function() use ($factory) {
 test(function() use ($factory) {
 	$_SERVER = array(
 		'REQUEST_URI' => '/www/index.php',
-		'SCRIPT_FILENAME' => 'w:\projects\modules-usage\www\index.php',
 		'SCRIPT_NAME' => '/www/index.php',
 	);
 
@@ -49,7 +46,6 @@ test(function() use ($factory) {
 test(function() use ($factory) {
 	$_SERVER = array(
 		'REQUEST_URI' => '/www/',
-		'SCRIPT_FILENAME' => 'w:\projects\modules-usage\www\index.php',
 		'SCRIPT_NAME' => '/www/',
 	);