f56d6530bc
This splits off the delete capability from the create reports permission.
From a UI perspective there were CSS issues, that this patch set hackily
bypasses. Perhaps someone else can amend this enhancement with the required
changes so that the extra column at the beginning of the table can be
removed when the user does not have delete capability.
TEST PLAN
---------
1) back up db
2) apply patch
3) ./installer/data/mysql/updatedatabase.pl
-- should run without issue.
4) in mysql:
> drop database ...
> create database ...
-- totally blanks it for fresh web install
5) run web install
-- installing should have no issues
6) go to a patron
7) set permissions
8) expand the reports permission
-- should have delete reports now
9) click help and scroll down to
'Granular Reports Permissions' right at the bottom.
-- there should be a new delete_reports section
10) Head over to guided reports and build a few reports.
-- as system account user, delete stuff should all be visible.
11) Find a patron, set all permissions, except delete reports.
12) log out and then log in as the modified patron
13) Head over the save reports
-- none of the delete options should be available to the user.
14) run koha qa test tools
15) restore db
Followed test plan. Additionally tried to delete using params in URL
(not possible, OK)
Signed-off-by: Marc <veron@veron.ch>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
|
||
|---|---|---|
| .. | ||
| acquisitions_stats.pl | ||
| bor_issues_top.pl | ||
| borrowers_out.pl | ||
| borrowers_stats.pl | ||
| cat_issues_top.pl | ||
| catalogue_out.pl | ||
| catalogue_stats.pl | ||
| dictionary.pl | ||
| guided_reports.pl | ||
| issues_avg_stats.pl | ||
| issues_by_borrower_category.plugin | ||
| issues_stats.pl | ||
| itemslost.pl | ||
| itemtypes.plugin | ||
| manager.pl | ||
| orders_by_fund.pl | ||
| reports-home.pl | ||
| reserves_stats.pl | ||
| serials_stats.pl | ||
| stats.print.pl | ||
| stats.screen.pl | ||