Bug 24858

Summary: Incorrect labels on wording in ExcludeHolidaysFromMaxPickUpDelay system preference
Product: Koha Reporter: Andrew Fuerste-Henry <andrewfh>
Component: Staff interfaceAssignee: Lucas Gass <lucas>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: major    
Priority: P5 - low CC: 1joynelson, gmcharlt, hayleypelham, jonathan.druart
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
20.05.00, 19.11.05
Bug Depends on: 22381    
Bug Blocks: 25779    
Attachments: Bug 24858: Correct labels on wording in ExcludeHolidaysFromMaxPickUpDelay
Bug 24858: Correct labels on wording in ExcludeHolidaysFromMaxPickUpDelay
Bug 24858: Correct labels on wording in ExcludeHolidaysFromMaxPickUpDelay

Description Andrew Fuerste-Henry 2020-03-12 13:48:56 UTC
The wording of the two options in ExcludeHolidaysFromMaxPickUpDelay is reversed. This error was introduced in bug 22381.

- mark your library closed today and tomorrow
- set ReservesMaxPickUpDelay to 1 day
- set ExcludeHolidaysFromMaxPickUpDelay to "Ignore the calendar"
- place hold, capture hold
- hold expiration date is set to the day after tomorrow (skips closed days)

- Set ExcludeHolidaysFromMaxPickUpDelay to "Use the calendar"
- Place hold, capture hold
- hold expiration date is set to tomorrow (doesn't skip closed days)
Comment 1 Lucas Gass 2020-03-12 22:40:44 UTC
Created attachment 100667 [details] [review]
Bug 24858: Correct labels on wording in ExcludeHolidaysFromMaxPickUpDelay

TO TEST:
- Apply patch
- mark your library closed today and tomorrow
- set ReservesMaxPickUpDelay to 1 day
- set ExcludeHolidaysFromMaxPickUpDelay to "Ignore the calendar"
- place hold, capture hold
- hold expiration date is set to tomorrow (doesn't skip closed days)

- Set ExcludeHolidaysFromMaxPickUpDelay to "Use the calendar"
- Place hold, capture hold
- hold expiration date is set to the day after tomorrow (skips closed days)
Comment 2 David Nind 2020-03-13 08:41:04 UTC
Created attachment 100669 [details] [review]
Bug 24858: Correct labels on wording in ExcludeHolidaysFromMaxPickUpDelay

TO TEST:
- Apply patch
- mark your library closed today and tomorrow
- set ReservesMaxPickUpDelay to 1 day
- set ExcludeHolidaysFromMaxPickUpDelay to "Ignore the calendar"
- place hold, capture hold
- hold expiration date is set to tomorrow (doesn't skip closed days)

- Set ExcludeHolidaysFromMaxPickUpDelay to "Use the calendar"
- Place hold, capture hold
- hold expiration date is set to the day after tomorrow (skips closed days)

Signed-off-by: David Nind <david@davidnind.com>
Comment 3 Jonathan Druart 2020-03-13 10:02:58 UTC
Created attachment 100676 [details] [review]
Bug 24858: Correct labels on wording in ExcludeHolidaysFromMaxPickUpDelay

TO TEST:
- Apply patch
- mark your library closed today and tomorrow
- set ReservesMaxPickUpDelay to 1 day
- set ExcludeHolidaysFromMaxPickUpDelay to "Ignore the calendar"
- place hold, capture hold
- hold expiration date is set to tomorrow (doesn't skip closed days)

- Set ExcludeHolidaysFromMaxPickUpDelay to "Use the calendar"
- Place hold, capture hold
- hold expiration date is set to the day after tomorrow (skips closed days)

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 4 Jonathan Druart 2020-03-13 10:03:11 UTC
Thanks!
Comment 5 Martin Renvoize 2020-03-16 11:03:37 UTC
Nice work everyone!

Pushed to master for 20.05
Comment 6 Joy Nelson 2020-04-03 17:36:07 UTC
Backported to 19.11.x branch for 19.11.05
Comment 7 Lucas Gass 2020-04-10 16:12:47 UTC
backported to 19.05.x for 19.05.10
Comment 8 Hayley Pelham 2020-04-14 23:53:14 UTC
Dependency not in 18.11.x, not backporting to 18.11.x series
Comment 9 Lucas Gass 2020-04-15 22:58:23 UTC
reverted from 19.05.x, missing dependency that I overlooked