[Koha-bugs] [Bug 10888] Don't hide authorities module from users without permission to edit authorities
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10888 Tomás Cohen Arazi tomasco...@gmail.com changed: What|Removed |Added Status|Pushed to Master|Pushed to Stable CC||tomasco...@gmail.com --- Comment #16 from Tomás Cohen Arazi tomasco...@gmail.com --- This patch has been pushed to 3.12.x, will be in 3.12.7. Thanks Owen! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10888] Don't hide authorities module from users without permission to edit authorities
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10888 Galen Charlton gmcha...@gmail.com changed: What|Removed |Added CC||gmcha...@gmail.com --- Comment #14 from Galen Charlton gmcha...@gmail.com --- General reminder -- when signing off, please remember to mark 'obsolete' older versions of the patch when attaching the version with your signoff. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10888] Don't hide authorities module from users without permission to edit authorities
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10888 Galen Charlton gmcha...@gmail.com changed: What|Removed |Added Attachment #21226|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10888] Don't hide authorities module from users without permission to edit authorities
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10888 Galen Charlton gmcha...@gmail.com changed: What|Removed |Added Attachment #21143|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10888] Don't hide authorities module from users without permission to edit authorities
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10888 Galen Charlton gmcha...@gmail.com changed: What|Removed |Added Status|Passed QA |Pushed to Master --- Comment #15 from Galen Charlton gmcha...@gmail.com --- Pushed to master. Thanks, Owen! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10888] Don't hide authorities module from users without permission to edit authorities
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10888 Paola Rossi paola.ro...@cineca.it changed: What|Removed |Added Status|Needs Signoff |Signed Off Assignee|oleon...@myacpl.org |paola.ro...@cineca.it --- Comment #10 from Paola Rossi paola.ro...@cineca.it --- Created attachment 21226 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21226action=edit Bug 10888 Don t hide authorities module from users without permission to edit authorities -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10888] Don't hide authorities module from users without permission to edit authorities
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10888 --- Comment #11 from Paola Rossi paola.ro...@cineca.it --- I signed off the patch -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10888] Don't hide authorities module from users without permission to edit authorities
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10888 Jonathan Druart jonathan.dru...@biblibre.com changed: What|Removed |Added Status|Signed Off |Passed QA CC||jonathan.dru...@biblibre.co ||m QA Contact|testo...@bugs.koha-communit |jonathan.dru...@biblibre.co |y.org |m --- Comment #12 from Jonathan Druart jonathan.dru...@biblibre.com --- QA comment: Looks consistent to me. I don't think a new permission for searching authorities is relevant. Marked as Passed QA. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10888] Don't hide authorities module from users without permission to edit authorities
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10888 --- Comment #13 from Jonathan Druart jonathan.dru...@biblibre.com --- Created attachment 21229 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21229action=edit Bug 10888 - Don't hide authorities module from users without permission to edit authorities It should be possible for staff client users to search and view authority records even if they don't have permission to edit them. To test, apply the patch and view the staff client as a user with and without editauthorities permission. With editauthorities permission, authority search results should show both an edit and delete link. Viewing the details of an authority record, one should see a toolbar with edit/delete/new options. The detail view has been altered to use the term record in place of biblio (Used in X record(s)). Without editauthorities permission, authority search results should show no edit or delete link. Viewing the details of an authority, the only option shown in the toolbar should be Save. On the staff client home page and in the header's More menu the link to the authorities module should now appear with and without permission to edit authorities. Revision: Corrected permissions in export script to allow saving of authority records by users who do not have permission to edit. Revision: Corrected incorrect merge with 10600 (duplicate authority) Signed-off-by: Paola Rossi paola.ro...@cineca.it Signed-off-by: Jonathan Druart jonathan.dru...@biblibre.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10888] Don't hide authorities module from users without permission to edit authorities
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10888 Paola Rossi paola.ro...@cineca.it changed: What|Removed |Added Status|Needs Signoff |Failed QA --- Comment #7 from Paola Rossi paola.ro...@cineca.it --- I can't sign off the patch. With permission to edit authorities, all the options but Edit work fine. So, in the cgi searchresultlist.tt, I changed the line 89 from: a href=/cgi-bin/koha/authorities/authorities.pl?authid=[% authid %]Edit/a to: a href=/cgi-bin/koha/authorities/authorities.pl?authid=[% resul.authid %]Edit/a Then, with permission to edit authorities, all the options work fine. Without permission to edit authorities, the option save works fine. But the option duplicate is presented too. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10888] Don't hide authorities module from users without permission to edit authorities
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10888 Owen Leonard oleon...@myacpl.org changed: What|Removed |Added Attachment #21124|0 |1 is obsolete|| --- Comment #8 from Owen Leonard oleon...@myacpl.org --- Created attachment 21142 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21142action=edit Bug 10888 - Don't hide authorities module from users without permission to edit authorities It should be possible for staff client users to search and view authority records even if they don't have permission to edit them. To test, apply the patch and view the staff client as a user with and without editauthorities permission. With editauthorities permission, authority search results should show both an edit and delete link. Viewing the details of an authority record, one should see a toolbar with edit/delete/new options. The detail view has been altered to use the term record in place of biblio (Used in X record(s)). Without editauthorities permission, authority search results should show no edit or delete link. Viewing the details of an authority, the only option shown in the toolbar should be Save. On the staff client home page and in the header's More menu the link to the authorities module should now appear with and without permission to edit authorities. Revision: Corrected permissions in export script to allow saving of authority records by users who do not have permission to edit. Revision: Corrected incorrect merge with 10600 (duplicate authority) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10888] Don't hide authorities module from users without permission to edit authorities
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10888 Owen Leonard oleon...@myacpl.org changed: What|Removed |Added Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10888] Don't hide authorities module from users without permission to edit authorities
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10888 --- Comment #9 from Owen Leonard oleon...@myacpl.org --- Created attachment 21143 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21143action=edit Bug 10888 - Don't hide authorities module from users without permission to edit authorities It should be possible for staff client users to search and view authority records even if they don't have permission to edit them. To test, apply the patch and view the staff client as a user with and without editauthorities permission. With editauthorities permission, authority search results should show both an edit and delete link. Viewing the details of an authority record, one should see a toolbar with edit/delete/new options. The detail view has been altered to use the term record in place of biblio (Used in X record(s)). Without editauthorities permission, authority search results should show no edit or delete link. Viewing the details of an authority, the only option shown in the toolbar should be Save. On the staff client home page and in the header's More menu the link to the authorities module should now appear with and without permission to edit authorities. Revision: Corrected permissions in export script to allow saving of authority records by users who do not have permission to edit. Revision: Corrected incorrect merge with 10600 (duplicate authority) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10888] Don't hide authorities module from users without permission to edit authorities
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10888 Owen Leonard oleon...@myacpl.org changed: What|Removed |Added Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10888] Don't hide authorities module from users without permission to edit authorities
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10888 Owen Leonard oleon...@myacpl.org changed: What|Removed |Added Status|Needs Signoff |Failed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10888] Don't hide authorities module from users without permission to edit authorities
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10888 Owen Leonard oleon...@myacpl.org changed: What|Removed |Added Attachment #21142|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10888] Don't hide authorities module from users without permission to edit authorities
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10888 Owen Leonard oleon...@myacpl.org changed: What|Removed |Added Status|In Discussion |Failed QA --- Comment #5 from Owen Leonard oleon...@myacpl.org --- Thanks Paola, you're right that this needs more work. Setting this to in discussion is too polite. It should be Failed QA. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10888] Don't hide authorities module from users without permission to edit authorities
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10888 Paola Rossi paola.ro...@cineca.it changed: What|Removed |Added Status|Needs Signoff |In Discussion CC||paola.ro...@cineca.it --- Comment #4 from Paola Rossi paola.ro...@cineca.it --- I installed the patch. It works fine with the editauthorities permission. Without the editauthorities permission, the search works well, while the save option doesn't work and sends the session in you're not permitted webpage. The save option sends you to this link: /cgi-bin/koha/authorities/export.pl?format=madsop=exportauthid=1 The cgi export.pl, under authorities, is affected by editauthorities permission. So I can't sign off. In my opinion the patch is not complete; we have two options in the case of no editauthorities permission: 1) authority search results should show no edit or delete link or save link; 2) a new specific permission is required, to search and to save too. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10888] Don't hide authorities module from users without permission to edit authorities
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10888 Owen Leonard oleon...@myacpl.org changed: What|Removed |Added Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10888] Don't hide authorities module from users without permission to edit authorities
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10888 Owen Leonard oleon...@myacpl.org changed: What|Removed |Added Attachment #21098|0 |1 is obsolete|| --- Comment #6 from Owen Leonard oleon...@myacpl.org --- Created attachment 21124 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21124action=edit Bug 10888 - Don't hide authorities module from users without permission to edit authorities It should be possible for staff client users to search and view authority records even if they don't have permission to edit them. To test, apply the patch and view the staff client as a user with and without editauthorities permission. With editauthorities permission, authority search results should show both an edit and delete link. Viewing the details of an authority record, one should see a toolbar with edit/delete/new options. The detail view has been altered to use the term record in place of biblio (Used in X record(s)). Without editauthorities permission, authority search results should show no edit or delete link. Viewing the details of an authority, the only option shown in the toolbar should be Save. On the staff client home page and in the header's More menu the link to the authorities module should now appear with and without permission to edit authorities. Revision: Corrected permissions in export script to allow saving of authority records by users who do not have permission to edit. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10888] Don't hide authorities module from users without permission to edit authorities
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10888 Owen Leonard oleon...@myacpl.org changed: What|Removed |Added CC||mag...@enger.priv.no --- Comment #3 from Owen Leonard oleon...@myacpl.org --- *** Bug 9086 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10888] Don't hide authorities module from users without permission to edit authorities
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10888 Owen Leonard oleon...@myacpl.org changed: What|Removed |Added Status|ASSIGNED|Needs Signoff Patch complexity|--- |Small patch -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10888] Don't hide authorities module from users without permission to edit authorities
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10888 Owen Leonard oleon...@myacpl.org changed: What|Removed |Added CC||ch...@bigballofwax.co.nz --- Comment #2 from Owen Leonard oleon...@myacpl.org --- *** Bug 5916 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10888] Don't hide authorities module from users without permission to edit authorities
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10888 --- Comment #1 from Owen Leonard oleon...@myacpl.org --- Created attachment 21098 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21098action=edit Bug 10888 - Don't hide authorities module from users without permission to edit authorities It should be possible for staff client users to search and view authority records even if they don't have permission to edit them. To test, apply the patch and view the staff client as a user with and without editauthorities permission. With editauthorities permission, authority search results should show both an edit and delete link. Viewing the details of an authority record, one should see a toolbar with edit/delete/new options. The detail view has been altered to use the term record in place of biblio (Used in X record(s)). Without editauthorities permission, authority search results should show no edit or delete link. Viewing the details of an authority, the only option shown in the toolbar should be Save. On the staff client home page and in the header's More menu the link to the authorities module should now appear with and without permission to edit authorities. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/