https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17387 --- Comment #151 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 194247 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=194247&action=edit Bug 17387: (follow-up) address review feedback - Fix permission description to explicitly mention bibliographic records and items (not just 'records') - Fix date filter to use correct DB column per table: deletedbiblio uses 'timestamp', deleteditems uses 'deleted_on' - Remove leftover CAN_user_editcatalogue_edit_catalogue condition from the Batch editing section in cat-menu.inc; a user with only edit_catalogue would see an empty section heading - Wrap biblio and item restore operations in DB transactions so that a failed store cannot leave a record inserted without being removed from the deleted table (or vice-versa) - Catch Koha::Exceptions::Object::DuplicateID in the item restore controller and return a human-readable 409 instead of a generic 500 - When 'Restore bibliographic record' is clicked in the deleted-bib warning modal, fetch the biblio and open the full restore modal with the originally clicked item pre-selected, rather than restoring immediately without item selection Signed-off-by: Michaela <michaela.sieber@kit.edu> Sponsored-by: ByWater Solutions Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.