Bug 12026: Fix small errors
authorMirko Tietgen <mirko@abunchofthings.net>
Tue, 15 Nov 2016 10:36:33 +0000 (11:36 +0100)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 31 Mar 2017 14:28:17 +0000 (14:28 +0000)
When running tests I got two errors. Do these changes make sense?

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
C4/Auth_with_shibboleth.pm

index 1a68c96..6a9a1cf 100644 (file)
@@ -108,7 +108,7 @@ sub checkpw_shib {
         return _autocreate( $config, $match );
     } else {
         # If we reach this point, the user is not a valid koha user
-        $debug and warn "User $userid is not a valid Koha user";
+         $debug and warn "User with $config->{matchpoint} of $match is not a valid Koha user";
         return 0;
     }
 }
@@ -116,7 +116,7 @@ sub checkpw_shib {
 sub _autocreate {
     my ( $config, $match ) = @_;
 
-    my %borrower = ( $shibbolethMatchField => $match );
+    my %borrower = ( shibbolethMatchField => $match );
 
     while ( my ( $key, $entry ) = each %{$config->{'mapping'}} ) {
         $borrower{$key} = ( $entry->{'is'} && $ENV{ $entry->{'is'} } ) || $entry->{'content'} || '';