Comment 4 for bug 827356

Revision history for this message
Rogan Hamby (rogan-hamby) wrote :

patch at https://git.evergreen-ils.org/?p=working/Evergreen.git;a=commit;h=ee1e7279543790435c7de453b02c76804803ad1c

What this patch does:

* adds rules to biblio.record_entry id -1, asset.copy_location id 1, asset.call_number id -1 so that edits are rejected for those ids on those tables

* asset.merge_record_assets has a condition at the top that returns 0 if either record is -1 to protect against db merges

* the staff client marc editor will hide the delete button if it is id -1 and disable the save button

I did not disable the save button for asset.copy_location id 1 because I didn't feel that those kinds of changes to the fm-editor component were worth it for such a niche case and probably should be handled more comprehensively than one off exceptions if we want to think about special exceptions that the component may handle and it is protected on the db level anyway