https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30230

Katrin Fischer <katrin.fisc...@bsz-bw.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Passed QA                   |Failed QA

--- Comment #68 from Katrin Fischer <katrin.fisc...@bsz-bw.de> ---
I reviewed the patch more closely today and tested. We'll need some more small
fixes:

1) +    bug_number  => "BUG_30230",

This is wrong and should be just 30230.

2) Please add view to the permission descriptions in the SQL and template file:

+            "INSERT IGNORE INTO permissions (module_bit, code, description)
VALUES (4, 'list_borrowers', 'Search and list patrons')"

3) In the main navigation (not start page, but on top) the Patrons module is
missing with only catalogue and list_borrowers permission.


I quite like the way how this lets you limit patron search to your own library
in combination with library groups. When view_borrower_infos_from_any_libraries
is added, I can search them all. But I am not required to also give
edit_borrowers. This is nice. 

Please help to finish this! :)

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to