Summary: | Label creator requires catalogue permission instead of label_creator permission from tools | ||
---|---|---|---|
Product: | Koha | Reporter: | Katrin Fischer <katrin.fischer> |
Component: | Label/patron card printing | Assignee: | Chris Nighswonger <cnighswonger> |
Status: | NEW --- | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | caroline.cyr-la-rose, dcook |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: |
Description
Katrin Fischer
2020-01-20 21:31:16 UTC
Can you clarify this one? It looks to me like the template for ./koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/cataloging-home.tt uses the correct tools label creator sub-permission. However, it looks like the Perl scripts (e.g. ./labels/label-home.pl) are using the "catalogue" permission instead. Actually running 'grep "flagsrequired" ./labels/*' shows either "catalogue => 1" or "tools => 'label_creator'" or "borrowers => 'edit_borrowers'" I wonder if the Perl scripts should have both the "label_creator" sub permission and the "catalogue" permission... It seems correct to update all the label-*.pl script to require "label_creator" permission. My only hesitation is about the quick spine label print option. If you have SpineLabelShowPrintOnBibDetails enabled the bibliographic detail page's holdings table has a column with "Print label" buttons. There is currently no permissions check on that. Should we add a "label_creator" permission check to that button and the corresponding page, or should we leave that one available to users with only "catalogue" permission? I feel like it should be separate? It's not necessarily the same people who print patron cards and who print item labels. Especially now that the item labels have been moved in cataloging instead of tools. |