MDL-23864 auth/cas Coding error: Undefined index REQUEST_URI in auth/…

…cas/CAS/CAS.php Patch our local copy (and record the change) while upstream fixes it (see https://issues.jasig.org/browse/PHPCAS-81 )
ralferlebach · Sep 3, 2010 · b44ff4e · b44ff4e
1 parent 9d97f08
commit b44ff4e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/auth/cas/CAS/CAS.php b/auth/cas/CAS/CAS.php
@@ -32,7 +32,7 @@
 //
 // hack by Vangelis Haniotakis to handle the absence of $_SERVER['REQUEST_URI'] in IIS
 //
-if (!$_SERVER['REQUEST_URI']) {
+if (!isset($_SERVER['REQUEST_URI'])) {
 	$_SERVER['REQUEST_URI'] = $_SERVER['SCRIPT_NAME'] . '?' . $_SERVER['QUERY_STRING'];
 }
 

diff --git a/auth/cas/CAS/readme_moodle.txt b/auth/cas/CAS/readme_moodle.txt
@@ -0,0 +1,8 @@
+Description of phpCAS 1.1.2 library import into Moodle
+
+Our changes:
+ * CAS.php - fix notice of $_SERVER['REQUEST_URI'] not being defined under IIS
+   (we can remove this change when we upgrade to a phpCAS version that has
+    https://issues.jasig.org/browse/PHPCAS-81 fixed).
+
+iarenaza