Add an auditor table for acq.fund_debit

Bug #1920210 reported by Tiffany Little
20
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Wishlist
Unassigned

Bug Description

I would like to see an edit_time column added to the acq.fund_debit table.

Currently there is only one column in the acqfd table that gives you any date information--create_time. However, fund debits change all the time. Consider:

1. They change from encumbrance true to false.
2. Amount and origin_amount are (probably) the same when the fund debit is first created, but amount can be changed when an invoice entry is attached.
3. You might delete the invoice entry that it's attached to, and/or change it to a different one.
4. They're deleted*

*For the sake of this bug I'm not proposing to keep deleted fund debits around, but that is a change that's made.

There may be some other circumstances I'm not thinking of right now, too.

Our libraries frequently run recurring fund reports. If I need to track down what's going on with a fund balance difference between two reports, the shifting sands of what's encumbered and what's not and when that change happened, etc. is really tough to pin down.

description: updated
description: updated
summary: - Add an edit_time column to the acq.fund_debit table
+ Add an auditor table for acq.fund_debit
Revision history for this message
Jeff Davis (jdavis-sitka) wrote :

Based on the change to the bug title, it sounds like an auditor table fits the bill? If so, working branch user/jeffdavis/lp1920210-audit-fund-debit should take care of that:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/jeffdavis/lp1920210-audit-fund-debit

Changed in evergreen:
status: New → Confirmed
importance: Undecided → Wishlist
milestone: none → 3.next
tags: added: pullrequest
Revision history for this message
Galen Charlton (gmc) wrote :

I've pushed a signoff branch and a release notes entry to working/user/gmcharlt/lp1920210_signoff

Changed in evergreen:
milestone: 3.next → 3.8-beta
tags: added: signedoff
Revision history for this message
Galen Charlton (gmc) wrote :

As a month has passed without comment, I have pushed this to master for inclusion in 3.8. Thanks, Jeff!

Changed in evergreen:
status: Confirmed → Fix Committed
Changed in evergreen:
status: Fix Committed → Fix Released
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.