We should show the expiration date for waiting holds in the holds datatable that appears in circulation.pl and moremember.pl
Created attachment 36949 [details] [review] Bug 13853 - Show waiting hold expiration date for waiting holds in holds ajax datatable We should show the expiration date for waiting holds in the holds datatable that appears in circulation.pl and moremember.pl Test Plan: 1) Ensure ReservesMaxPickUpDelay is set 2) Place a hold on a record 3) Place a second hold and set it to waiting by checking it in 4) Place a third hold with an expiration date 5) View the holds tab on circulation.pl and moremember.pl 6) Note the first hold has no expiration date 7) Note the second hold has an expiration date of today + ReservesMaxPickUpDelay days 8) Note the thrid hold has an expiration date of whatever you set it to
Created attachment 36984 [details] [review] Bug 13853 - Show waiting hold expiration date for waiting holds in holds ajax datatable We should show the expiration date for waiting holds in the holds datatable that appears in circulation.pl and moremember.pl Test Plan: 1) Ensure ReservesMaxPickUpDelay is set 2) Place a hold on a record 3) Place a second hold and set it to waiting by checking it in 4) Place a third hold with an expiration date 5) View the holds tab on circulation.pl and moremember.pl 6) Note the first hold has no expiration date 7) Note the second hold has an expiration date of today + ReservesMaxPickUpDelay days 8) Note the thrid hold has an expiration date of whatever you set it to
Created attachment 39748 [details] [review] Bug 13853 - Show waiting hold expiration date for waiting holds in holds ajax datatable We should show the expiration date for waiting holds in the holds datatable that appears in circulation.pl and moremember.pl Test Plan: 1) Ensure ReservesMaxPickUpDelay is set 2) Place a hold on a record 3) Place a second hold and set it to waiting by checking it in 4) Place a third hold with an expiration date 5) View the holds tab on circulation.pl and moremember.pl 6) Note the first hold has no expiration date 7) Note the second hold has an expiration date of today + ReservesMaxPickUpDelay days 8) Note the thrid hold has an expiration date of whatever you set it to
Created attachment 40026 [details] [review] Bug 13853 - Show waiting hold expiration date for waiting holds in holds ajax datatable We should show the expiration date for waiting holds in the holds datatable that appears in circulation.pl and moremember.pl Test Plan: 1) Ensure ReservesMaxPickUpDelay is set 2) Place a hold on a record 3) Place a second hold and set it to waiting by checking it in 4) Place a third hold with an expiration date 5) View the holds tab on circulation.pl and moremember.pl 6) Note the first hold has no expiration date 7) Note the second hold has an expiration date of today + ReservesMaxPickUpDelay days 8) Note the thrid hold has an expiration date of whatever you set it to
Created attachment 40037 [details] [review] [SIGNED-OFF] Bug 13853 - Show waiting hold expiration date for waiting holds in holds ajax datatable We should show the expiration date for waiting holds in the holds datatable that appears in circulation.pl and moremember.pl Test Plan: 1) Ensure ReservesMaxPickUpDelay is set 2) Place a hold on a record 3) Place a second hold and set it to waiting by checking it in 4) Place a third hold with an expiration date 5) View the holds tab on circulation.pl and moremember.pl 6) Note the first hold has no expiration date 7) Note the second hold has an expiration date of today + ReservesMaxPickUpDelay days 8) Note the thrid hold has an expiration date of whatever you set it to Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Cathi Wiggins <CWIGGINS@ci.arcadia.ca.us> Signed-off-by: Megan Wianecki <mwianecki@mplmain.mtpl.org>
Created attachment 40417 [details] [review] Bug 13853: Add tests for is_waiting
Kyle, Could you please provide tests for Koha::Item? Something else: On this page there are 2 places where the library is displayed. On top ("Holds waiting" block): Time (), by John D Hall Hold placed on 22/04/2015. Waiting at Le Cannet until 05/05/2015 Studio Ciné Live (000), by Philippe Boulnois [791] Hold placed on 18/06/2015. Waiting at Médiathèque Départementale de Prêt until 25/06/2015 In the table: Item is waiting here Item is waiting at Le Cannet Is it expected to have "here" in one case and the library name in the other case?
Created attachment 40673 [details] [review] Bug 13853 - Show waiting hold expiration date for waiting holds in holds ajax datatable We should show the expiration date for waiting holds in the holds datatable that appears in circulation.pl and moremember.pl Test Plan: 1) Ensure ReservesMaxPickUpDelay is set 2) Place a hold on a record 3) Place a second hold and set it to waiting by checking it in 4) Place a third hold with an expiration date 5) View the holds tab on circulation.pl and moremember.pl 6) Note the first hold has no expiration date 7) Note the second hold has an expiration date of today + ReservesMaxPickUpDelay days 8) Note the thrid hold has an expiration date of whatever you set it to Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Cathi Wiggins <CWIGGINS@ci.arcadia.ca.us> Signed-off-by: Megan Wianecki <mwianecki@mplmain.mtpl.org>
Created attachment 40674 [details] [review] Bug 13853: Add tests for is_waiting
Created attachment 40675 [details] [review] Bug 13853 [QA Followup] - Add unit tests for branch getting methods
> In the table: > Item is waiting here > Item is waiting at Le Cannet > > Is it expected to have "here" in one case and the library name in the other > case? Yes that is existing behavior and remains unchanged.
Created attachment 40755 [details] [review] Bug 13853 - Show waiting hold expiration date for waiting holds in holds ajax datatable We should show the expiration date for waiting holds in the holds datatable that appears in circulation.pl and moremember.pl Test Plan: 1) Ensure ReservesMaxPickUpDelay is set 2) Place a hold on a record 3) Place a second hold and set it to waiting by checking it in 4) Place a third hold with an expiration date 5) View the holds tab on circulation.pl and moremember.pl 6) Note the first hold has no expiration date 7) Note the second hold has an expiration date of today + ReservesMaxPickUpDelay days 8) Note the thrid hold has an expiration date of whatever you set it to Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Cathi Wiggins <CWIGGINS@ci.arcadia.ca.us> Signed-off-by: Megan Wianecki <mwianecki@mplmain.mtpl.org> Signed-off-by: Jonathan Druart <jonathan.druart@koha-community.org>
Created attachment 40756 [details] [review] Bug 13853: Add tests for is_waiting Signed-off-by: Jonathan Druart <jonathan.druart@koha-community.org>
Created attachment 40757 [details] [review] Bug 13853 [QA Followup] - Add unit tests for branch getting methods Signed-off-by: Jonathan Druart <jonathan.druart@koha-community.org>
Created attachment 40758 [details] [review] Bug 13853: (follow-up) unit tests Specify the reference, we should receive Koha::Object. Signed-off-by: Jonathan Druart <jonathan.druart@koha-community.org>
Patches pushed to master. Thanks Kyle!