deleted column on biblio.record_note not completely implemented
Bug #2055187 reported by
Galen Charlton
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Evergreen |
New
|
Undecided
|
Unassigned |
Bug Description
The changes in bug 1929242 included adding a 'deleted' flag to biblio.record_note.
However, nothing that can be done via the staff interface would cause that flag to ever be set to true. There is no rule or trigger to set that flag field rather than actually deleting a biblio.record_note row, nor does the UI include the deleted flag in the record note editor. Deleting a record note via the interface deletes the row.
Neither asset.copy_note nor asset.call_
Is it in fact desired to have the ability to retain deleted record notes and only logically delete them?
IMO, as long as there is a separate permission for deleting record notes, I think we can drop the deleted BOOL column. Local sites could add an auditor if they want to time travel.
If that's the consensus, we should probably remove that column ASAP to avoid report dependencies from creeping in.