Thanks for the branch, Kyle. I've confirmed that with your patch, I can set a specific due time in the offline interface. That due time is then included in exported files of those transactions, and it seems that the due time makes it through to offline.pl and the open-ils.circ.checkout method (according to osrfsys.log). However, it gets lost somewhere in the circ code, and the entries in the action.circulation table all end up being at 23:59:59 instead of the user-specified time.
Also, it would be nice if the due date columns in the Offline Checkout screen, Pending Transactions tab, and Exception list used the egDueDate filter instead of the date filter. egDueDate is really clever at showing date formats for due dates and datetime formats when the user specified a certain time.
Thanks for the branch, Kyle. I've confirmed that with your patch, I can set a specific due time in the offline interface. That due time is then included in exported files of those transactions, and it seems that the due time makes it through to offline.pl and the open-ils. circ.checkout method (according to osrfsys.log). However, it gets lost somewhere in the circ code, and the entries in the action.circulation table all end up being at 23:59:59 instead of the user-specified time.
Also, it would be nice if the due date columns in the Offline Checkout screen, Pending Transactions tab, and Exception list used the egDueDate filter instead of the date filter. egDueDate is really clever at showing date formats for due dates and datetime formats when the user specified a certain time.