refine the use of $debug_mode
authorDenis Ovsienko <denis@ovsienko.info>
Tue, 16 May 2017 17:16:57 +0000 (18:16 +0100)
committerDenis Ovsienko <denis@ovsienko.info>
Tue, 16 May 2017 17:16:57 +0000 (18:16 +0100)
* authenticated_via_saml(): remove an unused global
* EntityNotFoundException::dispatch(): check isset() first

wwwroot/inc/auth.php
wwwroot/inc/exceptions.php

index df78de499c0d9cf8eb548528dfe29b1e5e16ccaa..9e92d18539f5994a6544bd9fc66e762478f844bc 100644 (file)
@@ -269,7 +269,7 @@ function processAdjustmentSentence ($modlist, &$chain)
 // a wrapper for SAML auth method
 function authenticated_via_saml (&$saml_username = NULL, &$saml_displayname = NULL)
 {
-       global $SAML_options, $debug_mode, $auto_tags;
+       global $SAML_options, $auto_tags;
        if (! file_exists ($SAML_options['simplesamlphp_basedir'] . '/lib/_autoload.php'))
                throw new RackTablesError ('Configured for SAML authentication, but simplesaml is not found.', RackTablesError::MISCONFIGURED);
        require_once ($SAML_options['simplesamlphp_basedir'] . '/lib/_autoload.php');
index bbf2115ee26dac45dc8254f10c3e07210879c567..a26a70c74fe960d42f0899fb349c4e93d5b75b50 100644 (file)
@@ -159,7 +159,7 @@ class EntityNotFoundException extends RackTablesError
        public function dispatch()
        {
                global $debug_mode;
-               if ($debug_mode)
+               if (isset ($debug_mode) && $debug_mode)
                {
                        printGenericException ($this);
                        return;