]> git.koha-community.org Git - koha.git/commit
Bug 33611: Ensure dt_from_string always returns a new object
authorNick Clemens <nick@bywatersolutions.com>
Thu, 27 Apr 2023 13:02:52 +0000 (13:02 +0000)
committerTomas Cohen Arazi <tomascohen@theke.io>
Tue, 9 May 2023 13:58:04 +0000 (10:58 -0300)
commitca784d740d8e5758196c523dbeb205942de7729c
tree97c6c564bc911f67abb05b915612bf3cfd656b67
parent3551ef41a040c6da91e9b54dd7a030a1d6a6d266
Bug 33611: Ensure dt_from_string always returns a new object

This patch alters dt_from_string to pass a cloned object if called
on an existing date time object

This resolves an issue where a holds reserve date was being altered when
the default expiration date was set

To test:
1 - Set DefaultHoldExpiration to Set
2 - Set DefaultHoldExpirationPeriod to 365
3 - Set DefaultHoldExpirationunitOfTimeToDays
4 - Place a hold
5 - Note reserve date is 1 year in the future, same as expiration
6 - Apply patch
7 - Restart all
8 - Place another hold
9 - Note reserve date is set to today

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Hinemoea Viault <hinemoea.viault@inlibro.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Koha/DateUtils.pm