Bug 37258 - Locked records can still be modified/deleted by an unauthorized librarian with merge and in advanced editor
Summary: Locked records can still be modified/deleted by an unauthorized librarian wit...
Status: NEW
Alias: None
Product: Koha
Classification: Unclassified
Component: Cataloging (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low minor
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on: 31791
Blocks:
  Show dependency treegraph
 
Reported: 2024-07-04 18:45 UTC by Janusz Kaczmarek
Modified: 2024-07-22 09:48 UTC (History)
1 user (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Janusz Kaczmarek 2024-07-04 18:45:24 UTC
Locked records can still be modified/deleted by an unauthorized librarian with merge and in advanced editor.

Although the Edit record action is grayed out from the regular Edit pulldown, and also an attempt to open directly the basic editor (.../cgi-bin/koha/cataloguing/addbiblio.pl?biblionumber=<biblionumber>) results with Error 403 page, an unauthorized user is still able to modify/delete a locked record in several ways, including:

1. merge operation: if a locked record has been chosen as the destination (ref) record, fields can be inserted/deleted from it as a result of a merge and so modifying the locked record;

2. merge operation: if a locked record has NOT been chosen as the destination (ref) record, the locked record will be deleted;

3. a user cat launch directly the advanced editor (wit URL .../cgi-bin/koha/cataloguing/editor.pl#catalog/<biblionumber>) and save the modified version, with no respect for the lock status and edit_locked_records permission.

Theoretically, such an a user could be restricted from using advanced editor, but this does not seem as a right way of solving this issue.  And the merge path would still remain open.