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>
This commit is contained in:
parent
b100fb1609
commit
3e02dee64f
|
|
@ -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;
|
||||
|
|
|
|||
Loading…
Reference in New Issue