Bug 25005

Summary:	Admin Rights issue for Suggestion to Purchase
Product:	Koha	Reporter:	Christofer Zorn <christofer.zorn>
Component:	System Administration	Assignee:	Jonathan Druart <jonathan.druart>
Status:	CLOSED FIXED	QA Contact:	Marcel de Rooy <m.de.rooy>
Severity:	normal
Priority:	P5 - low	CC:	1joynelson, aleisha, cbrannon, cgdavis, david.roberts, gmcharlt, jonathan.druart, m.de.rooy, victor
Version:	Main
Hardware:	All
OS:	All
Change sponsored?:	---	Patch complexity:	Trivial patch
Documentation contact:		Documentation submission:
Text to go in the release notes:		Version(s) released in:	20.05.00, 19.11.10
Circulation function:
Bug Depends on:	22868
Bug Blocks:
Attachments:	Bug 25005: (bug 22868 follow-up) Fix suggestions pending on main page Bug 25005: (bug 22868 follow-up) Fix suggestions pending on main page Bug 25005: (bug 22868 follow-up) Fix suggestions pending on main page [19.11.x version] Bug 25005: (bug 22868 follow-up) Fix suggestions pending on main page

Description Christofer Zorn 2020-03-27 17:31:41 UTC

Hello All! I hope everyone staying safe during this time!

Yesterday I was trying to give a staff member permission to view and approve the "Suggestions Pending Approval". I had given that user full rights to the 'Suggestion Management' permission, however the option did not appear until I also provided the user with 'Moderate patron comments' under tools.

However, to continue testing this I removed the 'Suggestion Management' option and it did remove it.

I think there is something wrong with the check of the suggestion management piece that is preventing it from showing unless moderate comments option is also selected.

Thanks for looking into this.

Christofer

Comment 1 Jonathan Druart 2020-04-07 11:32:30 UTC

One occurrence missing on commit ae53caa6818939274352fde85454c24112e6f984
    Bug 22868: Move suggestions_manage subperm out of acquisition perm

Comment 2 Jonathan Druart 2020-04-07 11:36:52 UTC

Created attachment 102510 [details] [review]
Bug 25005: (bug 22868 follow-up) Fix suggestions pending on main page

There is a "Suggestions pending approval" link on the main page that is
displayed if there are new suggestions and the logged in user has the
permission to manage them.

On bug bug 22868 the permission changed from
acquisition.suggestions_manage to suggestions.suggestions_manage

But in the template, one occurrence has not been replaced correctly
(certainly because it was already wrong actually).

Test plan:
Create a suggestion at the OPAC
Create a patron with the suggestions permission
Use this patron to login at the staff interface
=> Without this patch the link does not appear on the main page
=> With this patch applied the link appears

Comment 3 David Roberts 2020-04-07 20:08:40 UTC

Created attachment 102526 [details] [review]
Bug 25005: (bug 22868 follow-up) Fix suggestions pending on main page

There is a "Suggestions pending approval" link on the main page that is
displayed if there are new suggestions and the logged in user has the
permission to manage them.

On bug bug 22868 the permission changed from
acquisition.suggestions_manage to suggestions.suggestions_manage

But in the template, one occurrence has not been replaced correctly
(certainly because it was already wrong actually).

Test plan:
Create a suggestion at the OPAC
Create a patron with the suggestions permission
Use this patron to login at the staff interface
=> Without this patch the link does not appear on the main page
=> With this patch applied the link appears

Signed-off-by: David Roberts <david@koha-ptfs.co.uk>

Comment 4 Marcel de Rooy 2020-04-10 10:21:59 UTC

Created attachment 102694 [details] [review]
Bug 25005: (bug 22868 follow-up) Fix suggestions pending on main page

There is a "Suggestions pending approval" link on the main page that is
displayed if there are new suggestions and the logged in user has the
permission to manage them.

On bug bug 22868 the permission changed from
acquisition.suggestions_manage to suggestions.suggestions_manage

But in the template, one occurrence has not been replaced correctly
(certainly because it was already wrong actually).

Test plan:
Create a suggestion at the OPAC
Create a patron with the suggestions permission
Use this patron to login at the staff interface
=> Without this patch the link does not appear on the main page
=> With this patch applied the link appears

Signed-off-by: David Roberts <david@koha-ptfs.co.uk>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Comment 5 Martin Renvoize (ashimema) 2020-04-14 07:37:14 UTC

Nice work everyone!

Pushed to master for 20.05

Comment 6 Joy Nelson 2020-05-04 21:16:08 UTC

does not apply to 19.11.x cleanly.  please rebase if needed.

Comment 7 Jonathan Druart 2020-05-05 07:31:19 UTC

Created attachment 104350 [details] [review]
[19.11.x version] Bug 25005: (bug 22868 follow-up) Fix suggestions pending on main page

There is a "Suggestions pending approval" link on the main page that is
displayed if there are new suggestions and the logged in user has the
permission to manage them.

On bug bug 22868 the permission changed from
acquisition.suggestions_manage to suggestions.suggestions_manage

But in the template, one occurrence has not been replaced correctly
(certainly because it was already wrong actually).

Test plan:
Create a suggestion at the OPAC
Create a patron with the suggestions permission
Use this patron to login at the staff interface
=> Without this patch the link does not appear on the main page
=> With this patch applied the link appears

Signed-off-by: David Roberts <david@koha-ptfs.co.uk>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Comment 8 Christopher Brannon 2020-09-02 18:33:57 UTC

Can this be ported to 19.11?

Comment 9 Aleisha Amohia 2020-09-03 04:16:03 UTC

backported to 19.11.x for 19.11.10

Comment 10 Victor Grousset/tuxayo 2020-09-20 15:12:44 UTC

Not backported to oldoldstable (19.05.x). Feel free to ask if it's needed.