There are many parts of Koha that are new hiding hours by default. This is rarely helpful. KohaDates should display hours by default unless the time is 23:59, in which case the hours should be not be displayed.
Created attachment 36095 [details] [review] Bug 13739 - KohaDates should display hours if they exists by default unless specified There are many parts of Koha that are now hiding hours by default. This is rarely helpful. KohaDates should display hours by default unless the time is 23:59, in which case the hours should be not be displayed. Test Plan: 1) Check out two items, one due at at an hour, one not 2) View the patron's circ history, note you don't see the hours 3) Apply this patch 4) View the circ history again, note the hours are listed for date/time where the time is not 23:59.
Created attachment 36312 [details] [review] Bug 13739 - KohaDates should display hours if they exists by default unless specified There are many parts of Koha that are now hiding hours by default. This is rarely helpful. KohaDates should display hours by default unless the time is 23:59, in which case the hours should be not be displayed. Test Plan: 1) Check out two items, one due at at an hour, one not 2) View the patron's circ history, note you don't see the hours 3) Apply this patch 4) View the circ history again, note the hours are listed for date/time where the time is not 23:59.
Created attachment 36322 [details] [review] [SIGNED OFF] Bug 13739 - KohaDates should display hours if they exists by default unless specified There are many parts of Koha that are now hiding hours by default. This is rarely helpful. KohaDates should display hours by default unless the time is 23:59, in which case the hours should be not be displayed. Test Plan: 1) Check out two items, one due at at an hour, one not 2) View the patron's circ history, note you don't see the hours 3) Apply this patch 4) View the circ history again, note the hours are listed for date/time where the time is not 23:59. Signed-off-by: Jesse Maseto <jesse@bywatersolutions.com>
Kyle, Before this patch, the plugin manages the as_due_date parameter. It's not with your patch. Marked as Failed QA.
(In reply to Jonathan Druart from comment #4) > Kyle, > Before this patch, the plugin manages the as_due_date parameter. It's not > with your patch. > > Marked as Failed QA. I'm not sure I'm following you. Can you explain what you mean a bit more and what you'd like to seem me change to get this to pass qa? Thanks!
Kyle, Before your patch, [% my_date | $KohaDates as_due_date => 1 %] worked to display a date as a due date. After your patch, this parameter is not checked to display the date as requested. And you should also have a look at t/db_dependent/Koha_template_plugin_KohaDates.t, the tests don't pass with your patch.