Bug 27868: Add expiration date to holds awaiting pickup
authorLucas Gass <lucas@bywatersolutions.com>
Tue, 7 Dec 2021 22:14:07 +0000 (22:14 +0000)
committerFridolin Somers <fridolin.somers@biblibre.com>
Fri, 28 Jan 2022 21:09:07 +0000 (11:09 -1000)
commit28771ad8b724f28065fc412b9a8a317c0c0850a2
treee76b310e17062ea45d71a434277c7558f431a637
parent3645d2b26b65dc41efa8cebbd9156333e845e4e6
Bug 27868: Add expiration date to holds awaiting pickup

To test:
1. Have some holds awaiting pickup, also have some holds that have been waiting longer than the ReservesMaxPickUpDelay is set to.
2. Go to /cgi-bin/koha/circ/waitingreserves.pl
3. There is no expiration date
4. Apply patch and restart_all
5. Now there should be a Expiration date column, make sure you can see it in both Holds waiting and Holds waiting over X days
6. Make sure all the columns Waiting Since, Date hold placed, and expiration date still sort correctly
6. Go to Table settings, and make sure all the columns hide correctly for both of the  holds_awaiting_pickup tables ( holdst & holdso )

Note: This patch also corrects 2 other small problems

1. Corrects a problem where the table settings did not account for the checkbox column at the beginnning of each table
2. Removes the title-string class and uses data-order to sort Waiting Since, Date hold placed, and expiration date ( see Bug 27934 )

Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
admin/columns_settings.yml
koha-tmpl/intranet-tmpl/prog/en/includes/waiting_holds.inc