MDL-38373 Use correct PEAR->loadExtension()
authorEloy Lafuente (stronk7) <stronk7@moodle.org>
Thu, 7 Mar 2013 14:38:40 +0000 (15:38 +0100)
committerEloy Lafuente (stronk7) <stronk7@moodle.org>
Sat, 16 Mar 2013 10:43:51 +0000 (11:43 +0100)
Current use is wrong (static) and leads to strict standards notice.

lib/pear/Auth/RADIUS.php
lib/pear/README_MOODLE.txt

index d40050d..34b2bf9 100644 (file)
@@ -46,7 +46,7 @@ require_once 'PEAR.php';
 * @version $Revision$
 */
 
-PEAR::loadExtension('radius');
+// PEAR::loadExtension('radius'); // Moodle commented. See MDL-38373.
 
 /**
  * class Auth_RADIUS
@@ -127,6 +127,7 @@ class Auth_RADIUS extends PEAR {
      */
     function Auth_RADIUS()
     {
+        $this->loadExtension('radius'); // Moodle added. See MDL-38373.
         $this->PEAR();
     }
 
index 9632e50..f804184 100644 (file)
@@ -1,6 +1,12 @@
 MOODLE-SPECIFIC PEAR MODIFICATIONS
 ==================================
 
+Auth/RADIUS
+===========
+
+1/ Changed static call to correct alternative (MDL-38373):
+    - From: PEAR::loadExtension('radius'); (in global scope)
+    - To: $this->loadExtension('radius'); (in constructor)
 
 Spreadsheet/Excel
 =================