]> git.koha-community.org Git - koha.git/commit
Bug 10843: fix crash that can occur when confirming hold if ReservesMaxPickUpDelay...
authorJonathan Druart <jonathan.druart@biblibre.com>
Mon, 9 Sep 2013 12:12:05 +0000 (14:12 +0200)
committerGalen Charlton <gmc@esilibrary.com>
Fri, 18 Oct 2013 06:16:57 +0000 (06:16 +0000)
commita9b564678e184b44b4c54d72e21f66efc3810f4e
tree5a4e0f86db20813127d37463b75f78ff0a92c91d
parent5fe524cf3ab0ff78401e7b467823da4d62d9a537
Bug 10843: fix crash that can occur when confirming hold if ReservesMaxPickUpDelay is undefined

Test plan:
1) set an empty string for the ReservesMaxPickUpDelay pref
2) place a hold on an item
3) check in the item
4) click on "Print and confirm"
5) an error occurs
> The 'days' parameter (undef) to DateTime::Duration::new was an 'undef'
6) apply the patch
7) repeat steps 1 to 4
8) the error does not occur anymore.

Signed-off-by: Srdjan <srdjan@catalyst.net.nz>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
An empty string didn't do it for me, I had to set the
variable for the systempreference to NULL. I am not sure
if this can happen when editing from the interface, but
this change should not have any ill side effects and it has
unit tests!

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
C4/Letters.pm
t/Letters.t