Edit Due Date should do more bounds checking of the date

Bug #2048386 reported by Galen Charlton
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Confirmed
Undecided
Unassigned

Bug Description

I ran across a couple cases where a loan had been made in December 2023 with a due date in December, then subsequently updated by circ staff to move the due date to January.

Likely enough occurrence during the holiday season, but with a catch: the due date was changed to January _2023_. I speculate that the staffers had edited the date in the text input rather than using the datepicker and forgot to bump up the year.

Some bounds checking both at the API layer and the UI layer is in order. Maybe something like:

- Warning in the Edit Due Date modal if the date that's supplied is early than the original due date. (Not necessarily a full block, though, as I can imagine circumstances where the due date should legitimately be earlier than the date supplied by the policy)
- Blocking both by the modal and the API if the new due date is earlier than loan's xact_start, as that seems like poor customer service to patrons who lack time machines.
- Ditto if the new due is in the past.

The last two may be more debatable (or suggest a need for an additional permission) if there are circumstances where libraries have to resort to using Edit Due Date as part of entering (old) back loans.

Galen Charlton (gmc)
tags: added: circ-checkout circulation
Galen Charlton (gmc)
description: updated
Revision history for this message
Elizabeth Davis (elidavis) wrote :

Confirming that we experienced this on 3.11.1.

Changed in evergreen:
status: New → Confirmed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.