Bug 22461: (follow-up) Use `exists` not `defined`

Prior to this patch there is a regression in the LDAP replication
functionality such that clearing a field become impossible. This patch
restores that functionality.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>

diff --git a/C4/Auth_with_ldap.pm b/C4/Auth_with_ldap.pm
index 589dbcc..6f3b9a3 100644 (file)
--- a/C4/Auth_with_ldap.pm
+++ b/C4/Auth_with_ldap.pm
@@ -224,7 +224,7 @@ sub checkpw_ldap {
         my @columns = Koha::Patrons->columns;
         my $patron = Koha::Patron->new(
             {
-                map { defined( $borrower{$_} ) ? ( $_ => $borrower{$_} ) : () } @columns
+                map { exists( $borrower{$_} ) ? ( $_ => $borrower{$_} ) : () } @columns
             }
         )->store;
         die "Insert of new patron failed" unless $patron;