From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Date: Wed, 20 Jan 2016 08:45:48 +0000 (+0000)
Subject: Bug 15607: batch patron mod - do not update dates if not given
X-Git-Url: http://git.rot13.org/?a=commitdiff_plain;h=2505a3d840c15e51c7967a576d53a416b5f92bfb;p=koha.git

Bug 15607: batch patron mod - do not update dates if not given

Bug 15332 fixed a bug but introduced a bigger one. Even if dateenrolled and
dateexpiry are not modified, they will be updated to today.
Indeed, dt_from_string returns today without parameters.

Test plan:
- Set dateenrolled and expirydate for a patron
- Using the batch patron mod tool, update any field but dateenrolled and
expirydate.
=> Without this patch, the 2 date fields should be set to today
=> With this patch applied, the 2 date fields should not have been
modified.

Signed-off-by: Karam Qubsi <karamqubsi@gmail.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Jesse Weaver <jweaver@bywatersolutions.com>
---

diff --git a/tools/modborrowers.pl b/tools/modborrowers.pl
index f88581fbd2..95d26de398 100755
--- a/tools/modborrowers.pl
+++ b/tools/modborrowers.pl
@@ -266,7 +266,7 @@ if ( $op eq 'do' ) {
     }
 
     for my $field ( qw( dateenrolled dateexpiry ) ) {
-        $infos->{$field} = dt_from_string($infos->{$field});
+        $infos->{$field} = dt_from_string($infos->{$field}) if $infos->{$field};
     }
 
     my @attributes = $input->param('patron_attributes');