MDL-36138 auth_cas: Fix CAS Authentication Strict Standards warnings
authorGilles-Philippe Leblanc <gilles-philippe.leblanc@umontreal.ca>
Thu, 1 Nov 2012 14:44:23 +0000 (10:44 -0400)
committerGilles-Philippe Leblanc <gilles-philippe.leblanc@umontreal.ca>
Fri, 18 Jan 2013 15:02:37 +0000 (10:02 -0500)
auth/cas/CAS/CAS.php
auth/cas/auth.php

index e6bae3f..975c0e5 100644 (file)
@@ -339,7 +339,7 @@ class phpCAS {
         *\r
         * @return a newly created CASClient object\r
         */\r
-       function client($server_version, $server_hostname, $server_port, $server_uri, $start_session = true) {\r
+       public static function client($server_version, $server_hostname, $server_port, $server_uri, $start_session = true) {\r
                global $PHPCAS_CLIENT, $PHPCAS_INIT_CALL;\r
 \r
                phpCAS :: traceBegin();\r
@@ -388,7 +388,7 @@ class phpCAS {
         *\r
         * @return a newly created CASClient object\r
         */\r
-       function proxy($server_version, $server_hostname, $server_port, $server_uri, $start_session = true) {\r
+       public static function proxy($server_version, $server_hostname, $server_port, $server_uri, $start_session = true) {\r
                global $PHPCAS_CLIENT, $PHPCAS_INIT_CALL;\r
 \r
                phpCAS :: traceBegin();\r
@@ -438,7 +438,7 @@ class phpCAS {
         *\r
         * @param $filename the name of the file used for logging, or FALSE to stop debugging.\r
         */\r
-       function setDebug($filename = '') {\r
+       public static function setDebug($filename = '') {\r
                global $PHPCAS_DEBUG;\r
 \r
                if ($filename != FALSE && gettype($filename) != 'string') {\r
@@ -480,7 +480,7 @@ class phpCAS {
         * This method is a wrapper for debug_backtrace() that is not available \r
         * in all PHP versions (>= 4.3.0 only)\r
         */\r
-       function backtrace() {\r
+       public static function backtrace() {\r
                if (function_exists('debug_backtrace')) {\r
                        return debug_backtrace();\r
                } else {\r
@@ -496,7 +496,7 @@ class phpCAS {
         *\r
         * @private\r
         */\r
-       function log($str) {\r
+       public static function log($str) {\r
                $indent_str = ".";\r
                global $PHPCAS_DEBUG;\r
 \r
@@ -517,7 +517,7 @@ class phpCAS {
         *\r
         * @private\r
         */\r
-       function error($msg) {\r
+       public static function error($msg) {\r
                $dbg = phpCAS :: backtrace();\r
                $function = '?';\r
                $file = '?';\r
@@ -542,7 +542,7 @@ class phpCAS {
        /**\r
         * This method is used to log something in debug mode.\r
         */\r
-       function trace($str) {\r
+       public static function trace($str) {\r
                $dbg = phpCAS :: backtrace();\r
                phpCAS :: log($str . ' [' . basename($dbg[1]['file']) . ':' . $dbg[1]['line'] . ']');\r
        }\r
@@ -550,7 +550,7 @@ class phpCAS {
        /**\r
         * This method is used to indicate the start of the execution of a function in debug mode.\r
         */\r
-       function traceBegin() {\r
+       public static function traceBegin() {\r
                global $PHPCAS_DEBUG;\r
 \r
                $dbg = phpCAS :: backtrace();\r
@@ -577,7 +577,7 @@ class phpCAS {
         *\r
         * @param $res the result of the function\r
         */\r
-       function traceEnd($res = '') {\r
+       public static function traceEnd($res = '') {\r
                global $PHPCAS_DEBUG;\r
 \r
                $PHPCAS_DEBUG['indent']--;\r
@@ -590,7 +590,7 @@ class phpCAS {
        /**\r
         * This method is used to indicate the end of the execution of the program\r
         */\r
-       function traceExit() {\r
+       public static function traceExit() {\r
                global $PHPCAS_DEBUG;\r
 \r
                phpCAS :: log('exit()');\r
@@ -617,7 +617,7 @@ class phpCAS {
         *\r
         * @sa PHPCAS_LANG_FRENCH, PHPCAS_LANG_ENGLISH\r
         */\r
-       function setLang($lang) {\r
+       public static function setLang($lang) {\r
                global $PHPCAS_CLIENT;\r
                if (!is_object($PHPCAS_CLIENT)) {\r
                        phpCAS :: error('this method should not be called before ' . __CLASS__ . '::client() or ' . __CLASS__ . '::proxy()');\r
@@ -642,7 +642,7 @@ class phpCAS {
         *\r
         * @return the phpCAS version.\r
         */\r
-       function getVersion() {\r
+       public static function getVersion() {\r
                return PHPCAS_VERSION;\r
        }\r
 \r
@@ -660,7 +660,7 @@ class phpCAS {
         *\r
         * @param $header the HTML header.\r
         */\r
-       function setHTMLHeader($header) {\r
+       public static function setHTMLHeader($header) {\r
                global $PHPCAS_CLIENT;\r
                if (!is_object($PHPCAS_CLIENT)) {\r
                        phpCAS :: error('this method should not be called before ' . __CLASS__ . '::client() or ' . __CLASS__ . '::proxy()');\r
@@ -676,7 +676,7 @@ class phpCAS {
         *\r
         * @param $footer the HTML footer.\r
         */\r
-       function setHTMLFooter($footer) {\r
+       public static function setHTMLFooter($footer) {\r
                global $PHPCAS_CLIENT;\r
                if (!is_object($PHPCAS_CLIENT)) {\r
                        phpCAS :: error('this method should not be called before ' . __CLASS__ . '::client() or ' . __CLASS__ . '::proxy()');\r
@@ -703,7 +703,7 @@ class phpCAS {
         * @param $format the format used to store the PGT's (`plain' and `xml' allowed)\r
         * @param $path the path where the PGT's should be stored\r
         */\r
-       function setPGTStorageFile($format = '', $path = '') {\r
+       public static function setPGTStorageFile($format = '', $path = '') {\r
                global $PHPCAS_CLIENT, $PHPCAS_AUTH_CHECK_CALL;\r
 \r
                phpCAS :: traceBegin();\r
@@ -749,7 +749,7 @@ class phpCAS {
         * @return TRUE on success, FALSE otherwise (in this later case, $err_code\r
         * gives the reason why it failed and $output contains an error message).\r
         */\r
-       function serviceWeb($url, & $err_code, & $output) {\r
+       public static function serviceWeb($url, & $err_code, & $output) {\r
                global $PHPCAS_CLIENT, $PHPCAS_AUTH_CHECK_CALL;\r
 \r
                phpCAS :: traceBegin();\r
@@ -792,7 +792,7 @@ class phpCAS {
         * @return an IMAP stream on success, FALSE otherwise (in this later case, $err_code\r
         * gives the reason why it failed and $err_msg contains an error message).\r
         */\r
-       function serviceMail($url, $service, $flags, & $err_code, & $err_msg, & $pt) {\r
+       public static function serviceMail($url, $service, $flags, & $err_code, & $err_msg, & $pt) {\r
                global $PHPCAS_CLIENT, $PHPCAS_AUTH_CHECK_CALL;\r
 \r
                phpCAS :: traceBegin();\r
@@ -839,7 +839,7 @@ class phpCAS {
         *\r
         * @param $n an integer.\r
         */\r
-       function setCacheTimesForAuthRecheck($n) {\r
+       public static function setCacheTimesForAuthRecheck($n) {\r
                global $PHPCAS_CLIENT;\r
                if (!is_object($PHPCAS_CLIENT)) {\r
                        phpCAS :: error('this method should not be called before ' . __CLASS__ . '::client() or ' . __CLASS__ . '::proxy()');\r
@@ -854,7 +854,7 @@ class phpCAS {
         * This method is called to check if the user is authenticated (use the gateway feature).\r
         * @return TRUE when the user is authenticated; otherwise FALSE.\r
         */\r
-       function checkAuthentication() {\r
+       public static function checkAuthentication() {\r
                global $PHPCAS_CLIENT, $PHPCAS_AUTH_CHECK_CALL;\r
 \r
                phpCAS :: traceBegin();\r
@@ -882,7 +882,7 @@ class phpCAS {
         * authenticated. If the user is not authenticated, halt by redirecting to \r
         * the CAS server.\r
         */\r
-       function forceAuthentication() {\r
+       public static function forceAuthentication() {\r
                global $PHPCAS_CLIENT, $PHPCAS_AUTH_CHECK_CALL;\r
 \r
                phpCAS :: traceBegin();\r
@@ -916,7 +916,7 @@ class phpCAS {
        /**\r
         * This method is called to renew the authentication.\r
         **/\r
-       function renewAuthentication() {\r
+       public static function renewAuthentication() {\r
                global $PHPCAS_CLIENT, $PHPCAS_AUTH_CHECK_CALL;\r
 \r
                phpCAS :: traceBegin();\r
@@ -941,7 +941,7 @@ class phpCAS {
        /**\r
         * This method has been left from version 0.4.1 for compatibility reasons.\r
         */\r
-       function authenticate() {\r
+       public static function authenticate() {\r
                phpCAS :: error('this method is deprecated. You should use ' . __CLASS__ . '::forceAuthentication() instead');\r
        }\r
 \r
@@ -951,7 +951,7 @@ class phpCAS {
         *\r
         * @return TRUE when the user is authenticated.\r
         */\r
-       function isAuthenticated() {\r
+       public static function isAuthenticated() {\r
                global $PHPCAS_CLIENT, $PHPCAS_AUTH_CHECK_CALL;\r
 \r
                phpCAS :: traceBegin();\r
@@ -981,7 +981,7 @@ class phpCAS {
         * @return true if authenticated, false otherwise.\r
         * @since 0.4.22 by Brendan Arnold\r
         */\r
-       function isSessionAuthenticated() {\r
+       public static function isSessionAuthenticated() {\r
                global $PHPCAS_CLIENT;\r
                if (!is_object($PHPCAS_CLIENT)) {\r
                        phpCAS :: error('this method should not be called before ' . __CLASS__ . '::client() or ' . __CLASS__ . '::proxy()');\r
@@ -996,7 +996,7 @@ class phpCAS {
         *\r
         * @return the login name of the authenticated user\r
         */\r
-       function getUser() {\r
+       public static function getUser() {\r
                global $PHPCAS_CLIENT, $PHPCAS_AUTH_CHECK_CALL;\r
                if (!is_object($PHPCAS_CLIENT)) {\r
                        phpCAS :: error('this method should not be called before ' . __CLASS__ . '::client() or ' . __CLASS__ . '::proxy()');\r
@@ -1017,7 +1017,7 @@ class phpCAS {
         *\r
         * @return the login name of the authenticated user\r
         */\r
-       function getAttributes() {\r
+       public static function getAttributes() {\r
                global $PHPCAS_CLIENT, $PHPCAS_AUTH_CHECK_CALL;\r
                if (!is_object($PHPCAS_CLIENT)) {\r
                        phpCAS :: error('this method should not be called before ' . __CLASS__ . '::client() or ' . __CLASS__ . '::proxy()');\r
@@ -1033,7 +1033,7 @@ class phpCAS {
        /**\r
         * Handle logout requests.\r
         */\r
-       function handleLogoutRequests($check_client = true, $allowed_clients = false) {\r
+       public static function handleLogoutRequests($check_client = true, $allowed_clients = false) {\r
                global $PHPCAS_CLIENT;\r
                if (!is_object($PHPCAS_CLIENT)) {\r
                        phpCAS :: error('this method should not be called before ' . __CLASS__ . '::client() or ' . __CLASS__ . '::proxy()');\r
@@ -1047,7 +1047,7 @@ class phpCAS {
         *\r
         * @return the login name of the authenticated user\r
         */\r
-       function getServerLoginURL() {\r
+       public static function getServerLoginURL() {\r
                global $PHPCAS_CLIENT;\r
                if (!is_object($PHPCAS_CLIENT)) {\r
                        phpCAS :: error('this method should not be called before ' . __CLASS__ . '::client() or ' . __CLASS__ . '::proxy()');\r
@@ -1060,7 +1060,7 @@ class phpCAS {
         * @param $url the login URL\r
         * @since 0.4.21 by Wyman Chan\r
         */\r
-       function setServerLoginURL($url = '') {\r
+       public static function setServerLoginURL($url = '') {\r
                global $PHPCAS_CLIENT;\r
                phpCAS :: traceBegin();\r
                if (!is_object($PHPCAS_CLIENT)) {\r
@@ -1081,7 +1081,7 @@ class phpCAS {
         * @param $url the serviceValidate URL\r
         * @since 1.1.0 by Joachim Fritschi\r
         */\r
-       function setServerServiceValidateURL($url = '') {\r
+       public static function setServerServiceValidateURL($url = '') {\r
                global $PHPCAS_CLIENT;\r
                phpCAS :: traceBegin();\r
                if (!is_object($PHPCAS_CLIENT)) {\r
@@ -1102,7 +1102,7 @@ class phpCAS {
         * @param $url the proxyValidate URL\r
         * @since 1.1.0 by Joachim Fritschi\r
         */\r
-       function setServerProxyValidateURL($url = '') {\r
+       public static function setServerProxyValidateURL($url = '') {\r
                global $PHPCAS_CLIENT;\r
                phpCAS :: traceBegin();\r
                if (!is_object($PHPCAS_CLIENT)) {\r
@@ -1122,7 +1122,7 @@ class phpCAS {
         * @param $url the samlValidate URL\r
         * @since 1.1.0 by Joachim Fritschi\r
         */\r
-       function setServerSamlValidateURL($url = '') {\r
+       public static function setServerSamlValidateURL($url = '') {\r
                global $PHPCAS_CLIENT;\r
                phpCAS :: traceBegin();\r
                if (!is_object($PHPCAS_CLIENT)) {\r
@@ -1143,7 +1143,7 @@ class phpCAS {
         *\r
         * @return the login name of the authenticated user\r
         */\r
-       function getServerLogoutURL() {\r
+       public static function getServerLogoutURL() {\r
                global $PHPCAS_CLIENT;\r
                if (!is_object($PHPCAS_CLIENT)) {\r
                        phpCAS :: error('this method should not be called before ' . __CLASS__ . '::client() or ' . __CLASS__ . '::proxy()');\r
@@ -1156,7 +1156,7 @@ class phpCAS {
         * @param $url the logout URL\r
         * @since 0.4.21 by Wyman Chan\r
         */\r
-       function setServerLogoutURL($url = '') {\r
+       public static function setServerLogoutURL($url = '') {\r
                global $PHPCAS_CLIENT;\r
                phpCAS :: traceBegin();\r
                if (!is_object($PHPCAS_CLIENT)) {\r
@@ -1176,7 +1176,7 @@ class phpCAS {
         * @params $params an array that contains the optional url and service parameters that will be passed to the CAS server\r
         * @public\r
         */\r
-       function logout($params = "") {\r
+       public static function logout($params = "") {\r
                global $PHPCAS_CLIENT;\r
                phpCAS :: traceBegin();\r
                if (!is_object($PHPCAS_CLIENT)) {\r
@@ -1206,7 +1206,7 @@ class phpCAS {
         * This method is used to logout from CAS. Halts by redirecting to the CAS server.\r
         * @param $service a URL that will be transmitted to the CAS server\r
         */\r
-       function logoutWithRedirectService($service) {\r
+       public static function logoutWithRedirectService($service) {\r
                global $PHPCAS_CLIENT;\r
                phpCAS :: traceBegin();\r
                if (!is_object($PHPCAS_CLIENT)) {\r
@@ -1226,7 +1226,7 @@ class phpCAS {
         * This method is used to logout from CAS. Halts by redirecting to the CAS server.\r
         * @param $url a URL that will be transmitted to the CAS server\r
         */\r
-       function logoutWithUrl($url) {\r
+       public static function logoutWithUrl($url) {\r
                global $PHPCAS_CLIENT;\r
                phpCAS :: traceBegin();\r
                if (!is_object($PHPCAS_CLIENT)) {\r
@@ -1247,7 +1247,7 @@ class phpCAS {
         * @param $service a URL that will be transmitted to the CAS server\r
         * @param $url a URL that will be transmitted to the CAS server\r
         */\r
-       function logoutWithRedirectServiceAndUrl($service, $url) {\r
+       public static function logoutWithRedirectServiceAndUrl($service, $url) {\r
                global $PHPCAS_CLIENT;\r
                phpCAS :: traceBegin();\r
                if (!is_object($PHPCAS_CLIENT)) {\r
@@ -1273,7 +1273,7 @@ class phpCAS {
         *\r
         * @param $url the URL\r
         */\r
-       function setFixedCallbackURL($url = '') {\r
+       public static function setFixedCallbackURL($url = '') {\r
                global $PHPCAS_CLIENT;\r
                phpCAS :: traceBegin();\r
                if (!is_object($PHPCAS_CLIENT)) {\r
@@ -1295,7 +1295,7 @@ class phpCAS {
         *\r
         * @param $url the URL\r
         */\r
-       function setFixedServiceURL($url) {\r
+       public static function setFixedServiceURL($url) {\r
                global $PHPCAS_CLIENT;\r
                phpCAS :: traceBegin();\r
                if (!is_object($PHPCAS_CLIENT)) {\r
@@ -1311,7 +1311,7 @@ class phpCAS {
        /**\r
         * Get the URL that is set as the CAS service parameter.\r
         */\r
-       function getServiceURL() {\r
+       public static function getServiceURL() {\r
                global $PHPCAS_CLIENT;\r
                if (!is_object($PHPCAS_CLIENT)) {\r
                        phpCAS :: error('this method should only be called after ' . __CLASS__ . '::proxy()');\r
@@ -1322,7 +1322,7 @@ class phpCAS {
        /**\r
         * Retrieve a Proxy Ticket from the CAS server.\r
         */\r
-       function retrievePT($target_service, & $err_code, & $err_msg) {\r
+       public static function retrievePT($target_service, & $err_code, & $err_msg) {\r
                global $PHPCAS_CLIENT;\r
                if (!is_object($PHPCAS_CLIENT)) {\r
                        phpCAS :: error('this method should only be called after ' . __CLASS__ . '::proxy()');\r
@@ -1338,7 +1338,7 @@ class phpCAS {
         *\r
         * @param $cert the PEM certificate\r
         */\r
-       function setCasServerCert($cert) {\r
+       public static function setCasServerCert($cert) {\r
                global $PHPCAS_CLIENT;\r
                phpCAS :: traceBegin();\r
                if (!is_object($PHPCAS_CLIENT)) {\r
@@ -1356,7 +1356,7 @@ class phpCAS {
         *\r
         * @param $cert the CA certificate\r
         */\r
-       function setCasServerCACert($cert) {\r
+       public static function setCasServerCACert($cert) {\r
                global $PHPCAS_CLIENT;\r
                phpCAS :: traceBegin();\r
                if (!is_object($PHPCAS_CLIENT)) {\r
@@ -1372,7 +1372,7 @@ class phpCAS {
        /**\r
         * Set no SSL validation for the CAS server.\r
         */\r
-       function setNoCasServerValidation() {\r
+       public static function setNoCasServerValidation() {\r
                global $PHPCAS_CLIENT;\r
                phpCAS :: traceBegin();\r
                if (!is_object($PHPCAS_CLIENT)) {\r
@@ -1390,7 +1390,7 @@ class phpCAS {
         * @param $key the option key\r
         * @param $value the value to set\r
         */\r
-       function setExtraCurlOption($key, $value) {\r
+       public static function setExtraCurlOption($key, $value) {\r
                global $PHPCAS_CLIENT;\r
                phpCAS :: traceBegin();\r
                if (!is_object($PHPCAS_CLIENT)) {\r
index c0bcd50..ccd1021 100644 (file)
@@ -104,12 +104,14 @@ class auth_plugin_cas extends auth_plugin_ldap {
         $this->connectCAS();
 
         if (phpCAS::checkAuthentication()) {
+            $frm = new stdClass();
             $frm->username = phpCAS::getUser();
             $frm->password = 'passwdCas';
             return;
         }
 
         if (isset($_GET['loginguest']) && ($_GET['loginguest'] == true)) {
+            $frm = new stdClass();
             $frm->username = 'guest';
             $frm->password = 'guest';
             return;