backdating checkins to the future should not be allowed

Bug #1016204 reported by Thomas Berezansky
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
2.5
Fix Released
Undecided
Unassigned

Bug Description

Instead of trying to teach SIP2 to not backdate checkins to the future the branch below teaches the circ module to treat them as not backdated at all.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/tsbere/no_future_checkin

Tags: pullrequest
Changed in evergreen:
status: New → Triaged
Ben Shum (bshum)
Changed in evergreen:
milestone: none → 2.4.0-rc
Ben Shum (bshum)
Changed in evergreen:
milestone: 2.4.0-rc → 2.5.0-alpha
Dan Wells (dbw2)
Changed in evergreen:
milestone: 2.5.0-m1 → 2.5.0-m2
Dan Wells (dbw2)
Changed in evergreen:
milestone: 2.5.0-m2 → 2.5.0-alpha1
Remington Steed (rjs7)
Changed in evergreen:
milestone: 2.5.0-alpha1 → 2.5.0-alpha2
Dan Wells (dbw2)
Changed in evergreen:
milestone: 2.5.0-alpha2 → 2.5.0-beta1
Dan Wells (dbw2)
Changed in evergreen:
milestone: 2.5.0-beta1 → 2.5.0-rc
Dan Wells (dbw2)
Changed in evergreen:
milestone: 2.5.0-rc → 2.5.1
Ben Shum (bshum)
Changed in evergreen:
milestone: 2.5.1 → 2.5.2
importance: Undecided → Medium
Dan Wells (dbw2)
Changed in evergreen:
milestone: 2.5.2 → 2.6.0-alpha1
Revision history for this message
Dan Wells (dbw2) wrote :

Changed the title, since this code isn't really SIP specific, and hopefully the change will cause it to get more eyes.

Changed in evergreen:
milestone: 2.6.0-alpha1 → 2.6.0-beta1
summary: - SIP2 can backdate to the future
+ backdating checkins to the future should not be allowed
Revision history for this message
Ben Shum (bshum) wrote :

When I tried to apply this change, I got a conflict in the file. Looks like when http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=7a7c229db5044712ad44ad5a75ca7a65cd6d9105 was merged, the code here was shuffled a tiny bit.

Marking as "incomplete" and assigning back to tsbere to do some light rebasing with the changes here.

Changed in evergreen:
status: Triaged → Incomplete
assignee: nobody → Thomas Berezansky (tsbere)
Revision history for this message
Thomas Berezansky (tsbere) wrote :

I pushed a rebased variant to correct the issue.

Changed in evergreen:
assignee: Thomas Berezansky (tsbere) → nobody
status: Incomplete → Triaged
Ben Shum (bshum)
Changed in evergreen:
status: Triaged → Fix Committed
Changed in evergreen:
status: Fix Committed → Fix Released
no longer affects: evergreen/2.4
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.