[Bps-public-commit] rt-authen-externalauth branch, master, updated. 0.09_03-1-g5f1526d

Kevin Falcone falcone at bestpractical.com
Thu Feb 16 12:25:12 EST 2012


The branch, master has been updated
       via  5f1526dcefe2700cd013dd27846de4922b9a09e5 (commit)
      from  95f2ff2d9c9045a00f362cdf24187c3d04c5f46f (commit)

Summary of changes:
 lib/RT/Authen/ExternalAuth.pm |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

- Log -----------------------------------------------------------------
commit 5f1526dcefe2700cd013dd27846de4922b9a09e5
Author: Kevin Falcone <falcone at bestpractical.com>
Date:   Thu Feb 16 12:24:21 2012 -0500

    Avoid looping once over undef when we wanted an empty list
    
    This really only caused log messages, but they were confusing, and
    CanonicalizeEmailAddress is run also.

diff --git a/lib/RT/Authen/ExternalAuth.pm b/lib/RT/Authen/ExternalAuth.pm
index 44355c1..788f9de 100644
--- a/lib/RT/Authen/ExternalAuth.pm
+++ b/lib/RT/Authen/ExternalAuth.pm
@@ -484,7 +484,7 @@ sub UserDisabled {
     my $username = shift;
     my $user_disabled = 0;
     
-    my @info_services = $RT::ExternalInfoPriority ? @{$RT::ExternalInfoPriority} : undef;
+    my @info_services = $RT::ExternalInfoPriority ? @{$RT::ExternalInfoPriority} : ();
 
     # For each named service in the list
     # Check to see if the user is found in the external service

-----------------------------------------------------------------------



More information about the Bps-public-commit mailing list