Changeset 2071 for trunk/include

Timestamp:

Jul 24, 2007, 9:14:11 PM (17 years ago)

Author:

rub

Message:

Resolved 0000726: script_basename returns bad result

With specific server configuration, script_basename returns bad result.

File:

• trunk/include/functions.inc.php (modified) (2 diffs)

trunk/include/functions.inc.php

@@ -1287 +1287 @@
 
 /**
- * Return is "file_name" argument is candidate to
- * compute script_basename value
- *
- * @param string file_name
- *
- * @return boolean is candidate or not?
- */
-function is_script_basename($file_name)
-{
-  $file_name = basename($file_name);
-  return !empty($file_name);
-}
-
-/**
  * Return basename of the current script
  * Lower case convertion is applied on return value
@@ -1311 +1297 @@
 function script_basename()
 {
-  if (is_script_basename($_SERVER['SCRIPT_NAME']))
-  {
-    $file_name = $_SERVER['SCRIPT_NAME'];
-  }
-  else if (is_script_basename($_SERVER['SCRIPT_FILENAME']))
-  {
-    $file_name = $_SERVER['SCRIPT_FILENAME'];
-  }
-  else
-  {
-    $file_name = '';
-  }
-
-  // $_SERVER return lower string following var and systems
-  return basename(strtolower($file_name), '.php');
+  global $conf;
+
+  foreach (array('SCRIPT_NAME', 'SCRIPT_FILENAME', 'PHP_SELF') as $value)
+  {
+    $continue = !empty($_SERVER[$value]);
+    if ($continue)
+    {
+      $filename = strtolower($_SERVER[$value]);
+
+      if ($conf['php_extension_in_urls'])
+      {
+        $continue = get_extension($filename) ===  'php';
+      }
+
+      if ($continue)
+      {
+        $basename = basename($filename, '.php');
+        $continue = !empty($basename);
+      }
+
+      if ($continue)
+      {
+        return $basename;
+      }
+    }
+  }
+
+  return '';
 }