Checkout: further validation of due date override
authorDan Scott <dan@coffeecode.net>
Mon, 8 Oct 2012 19:08:47 +0000 (15:08 -0400)
committerDan Scott <dscott@laurentian.ca>
Tue, 9 Oct 2012 02:04:18 +0000 (22:04 -0400)
commit6022fa0855483b4a1280553c8f50e1a57f28f240
tree016d4127a00ff3d4102f878fc0172973ca9ea98a
parent5937a037a8a51206c7156608e047a4dbdbf9f7a5
Checkout: further validation of due date override

The check_past() function failed if given a date that was not strictly
in YYYY-mm-dd format; interestingly, a common transposition typo such
as "0212-10-20" results in "212-10-20" getting passed to check_past(),
and therefore generating an invalid date. Throw an exception in
check_past() rather than returning true, because we are not in fact
stating that the due date is in the past - and catch the exception and
flag the due date override box accordingly in the checkout screen.

We could bubble the exception up to the user, but hopefully highlighting
the checkout box as being in an invalid state will catch the attention
of the users.

Signed-off-by: Dan Scott <dan@coffeecode.net>
Open-ILS/xul/staff_client/chrome/content/util/date.js
Open-ILS/xul/staff_client/server/circ/checkout.js