Bug 21966

Summary: Fix descriptions of acquisition permissions to be more clear (again)
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: AcquisitionsAssignee: Katrin Fischer <katrin.fischer>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: fridolin.somers, lucas, martin.renvoize, nick, pierre-marc.thibault
Version: MainKeywords: regression
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: String patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 3849    
Bug Blocks:    
Attachments: Bug 21966: Fix descriptions of acquisition permissions to be more clear (again)
Bug 21966: Fix descriptions of acquisition permissions to be more clear (again)
Bug 21966: Fix descriptions of acquisition permissions to be more clear (again)

Description Katrin Fischer 2018-12-06 14:10:19 UTC
This was already done and pushed on bug 3849, but some of it got undone by a later patch (conflict I guess):

http://git.koha-community.org/gitweb/?p=koha.git;a=commitdiff;h=b8a2365a3455a70ef8bdcb99e74b419bc4572b96

koha-tmpl/intranet-tmpl/prog/en/includes/permissions.inc


-    [%- CASE 'budget_add_del' -%]<span>Add and delete funds (but can't modify funds)</span>
-    [%- CASE 'budget_manage' -%]<span>Manage funds</span>
-    [%- CASE 'budget_manage_all' -%]<span>Manage all funds</span>
-    [%- CASE 'budget_modify' -%]<span>Modify funds (can't create lines, but can modify existing ones)</span>
+    [%- CASE 'suggestions_manage' -%]<span>Manage purchase suggestions</span>
+    [%- CASE 'budget_add_del' -%]<span>Add and delete budgets (but can't modify budgets)</span>
+    [%- CASE 'budget_manage' -%]<span>Manage budgets</span>
+    [%- CASE 'budget_manage_all' -%]<span>Manage all budgets</span>
+    [%- CASE 'budget_modify' -%]<span>Modify budget (can't create lines, but can modify existing ones)</span>
Comment 1 Katrin Fischer 2018-12-06 14:21:07 UTC
Created attachment 82909 [details] [review]
Bug 21966: Fix descriptions of acquisition permissions to be more clear (again)

This was already done and pushed on bug 3849, but some of it got
undone accidentally by a later patch.

To test:
- Check the acquisition permissions
- The *budget* permissions should clearly describe their
  use for 'funds'.
Comment 2 Pierre-Marc Thibault 2018-12-07 20:58:14 UTC
Created attachment 83002 [details] [review]
Bug 21966: Fix descriptions of acquisition permissions to be more clear (again)

This was already done and pushed on bug 3849, but some of it got
undone accidentally by a later patch.

To test:
- Check the acquisition permissions
- The *budget* permissions should clearly describe their
  use for 'funds'.

Signed-off-by: Pierre-Marc Thibault <pierre-marc.thibault@inLibro.com>
Comment 3 Martin Renvoize 2019-02-08 12:59:19 UTC
Created attachment 84863 [details] [review]
Bug 21966: Fix descriptions of acquisition permissions to be more clear (again)

This was already done and pushed on bug 3849, but some of it got
undone accidentally by a later patch.

To test:
- Check the acquisition permissions
- The *budget* permissions should clearly describe their
  use for 'funds'.

Signed-off-by: Pierre-Marc Thibault <pierre-marc.thibault@inLibro.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 4 Martin Renvoize 2019-02-08 12:59:47 UTC
Trivial patch, no regressions, passes qa scripts.

Passing QA
Comment 5 Nick Clemens (kidclamp) 2019-02-19 14:32:59 UTC
Awesome work all!

Pushed to master for 19.05
Comment 6 Martin Renvoize 2019-02-26 08:59:40 UTC
Pushed to 18.11.x for 18.11.04
Comment 7 Lucas Gass 2019-03-06 18:02:44 UTC
backported to 18.05.x for 18.05.11