Bug 28680 - Staff without edit_borrower permission still see patron information
Summary: Staff without edit_borrower permission still see patron information
Status: NEW
Alias: None
Product: Koha
Classification: Unclassified
Component: Patrons (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low normal
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-07-07 20:14 UTC by Donna
Modified: 2024-08-13 19:23 UTC (History)
8 users (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 Donna 2021-07-07 20:14:58 UTC
The permission edit_borrowers specifies:  Add, modify and view patron information.  However, when that permission is not enabled, staff can view patron information in a number of places, including holds queue, circulation.pl, waitingreserves.pl, and pendingreserves.pl.  

In the Holds Queue staff can see patron name and other info for patrons at their library, but patrons from other libraries show "patron from central branch" etc.  When looking at a bib record, if an item is checked out, only the borrower number is displayed.  

This should be consistent, where staff without that permission only see a borrower number and nothing else.

To replicate, create a staff member with only permissions catalogue ( to log in) and circulate_remaining_permissions.  Check out to a patron, see the holds queue, etc and see that the patron information is displayed.
Comment 1 David Cook 2021-07-09 00:55:47 UTC
I don't think that this really makes sense.

Arguably "edit_borrowers" could be changed to just "add and modify" patron information, but if we want to restrict viewing patron information, then there should probably be a new permission to control that. Of course, that wouldn't be backwards compatible, so that probably wouldn't happen.

Maybe a system preference in addition to a new permission I suppose...
Comment 2 George Williams (NEKLS) 2021-07-13 14:10:46 UTC
I agree with Donna that, since the edit_borrowers permission says "Add, modify and view patron information" it should actually work that way.  If, as David suggests, the wording in the description were changed, then I would want a separate permission that controls "view patron information" because we have staff at 50+ libraries that are currently restricted from viewing borrower address information that we need to continue to restrict from viewing address information.
Comment 3 David Cook 2021-07-13 23:18:01 UTC
For what it's worth, off the top of my head, it would be a massive long expensive development with little chance of success to hide the borrower information as desired.

I could be wrong, but that's my first instinct. I can also think of a few holes (like reports) that would probably be impossible to plug without completely rewriting entire modules in Koha, which would then break many workflows and expectations. 

But that's just my two cents. Someone else might have a good idea.