Booking pickup stops showing captured resources after first day
Bug #925169 reported by
Lebbeous Fogle-Weekley
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Evergreen |
Fix Released
|
Undecided
|
Unassigned | ||
2.0 |
Fix Released
|
Undecided
|
Unassigned | ||
2.1 |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
For evergreen master, and it should be backportable through rel_2_0.
Booking: make pick up interface show resources captured for reservation ...
... even beyond the first day of the reservation. I can think of no
reason why this limitation existed in the first place.
This comes about as a result of discussion on the open-ils-general mailing list. The thread starts here: http://
Here's the code:
Changed in evergreen: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
I agree with the assessment that the restriction is unnecessary, and I think I can shed some light on why it's there.
IIRC, the point was to show "hot" reservations, those that had events occur "today". Another reason may be to limit the number of reservations the function would return, but that's unlikely because the way it was 1) couldn't guarantee that and 2) would be a very heavy handed way to do it.
As a side note, I noticed an opportunity for a little refactoring. The naive_start_ of_day( ) sub could be replaced with the string 'today', which Postgres will interpret in the way that the sub intends.
Thanks, Lebbeous!