diff options
| author | Ivan Kohler <ivan@freeside.biz> | 2012-04-04 15:31:13 -0700 |
|---|---|---|
| committer | Ivan Kohler <ivan@freeside.biz> | 2012-04-04 15:31:13 -0700 |
| commit | af42c70c94eb8fbb363238fba056d98ee2204b8f (patch) | |
| tree | f9433868dfea7a2b1410f909315aca5c8a08b618 /httemplate | |
| parent | f84937b2a6cc6ba63cdab177866b1417c32b1028 (diff) | |
add "Summarize packages" ACL, RT#16534
Diffstat (limited to 'httemplate')
| -rw-r--r-- | httemplate/elements/menu.html | 6 | ||||
| -rw-r--r-- | httemplate/search/cust_pkg_summary.cgi | 2 | ||||
| -rw-r--r-- | httemplate/search/cust_pkg_summary.html | 4 | ||||
| -rw-r--r-- | httemplate/search/cust_pkg_susp.cgi | 2 | ||||
| -rw-r--r-- | httemplate/search/cust_pkg_susp.html | 4 |
5 files changed, 14 insertions, 4 deletions
diff --git a/httemplate/elements/menu.html b/httemplate/elements/menu.html index 3b0969f5c..24e43907a 100644 --- a/httemplate/elements/menu.html +++ b/httemplate/elements/menu.html @@ -238,9 +238,11 @@ if ( $curuser->access_right('Financial reports') ) { $report_packages{'separator2'} = ''; } $report_packages{'All customer packages'} = [ $fsurl.'search/cust_pkg.cgi?pkgnum', 'List all customer packages', ]; -$report_packages{'Package summary'} = [ $fsurl.'search/cust_pkg_summary.html', 'Show package sales summary', ]; +$report_packages{'Package summary'} = [ $fsurl.'search/cust_pkg_summary.html', 'Show package sales summary', ] + if $curuser->access_right('Summarize packages'); $report_packages{'Suspended customer packages'} = [ $fsurl.'search/cust_pkg.cgi?magic=suspended', 'List suspended packages' ]; -$report_packages{'Suspension summary'} = [ $fsurl.'search/cust_pkg_susp.html', 'Show suspension activity', ]; +$report_packages{'Suspension summary'} = [ $fsurl.'search/cust_pkg_susp.html', 'Show suspension activity', ] + if $curuser->access_right('Summarize packages'); $report_packages{'Customer packages with unconfigured services'} = [ $fsurl.'search/cust_pkg.cgi?APKG_pkgnum', 'List packages which have provisionable services' ]; $report_packages{'FCC Form 477 packages'} = [ $fsurl.'search/report_477.html', 'Summarize packages by census tract for particular types' ] if $conf->exists('cust_main-require_censustract'); diff --git a/httemplate/search/cust_pkg_summary.cgi b/httemplate/search/cust_pkg_summary.cgi index cea4cdcd2..fbeeb92ce 100644 --- a/httemplate/search/cust_pkg_summary.cgi +++ b/httemplate/search/cust_pkg_summary.cgi @@ -25,7 +25,7 @@ my $curuser = $FS::CurrentUser::CurrentUser; die "access denied" - unless $curuser->access_right('List packages'); + unless $curuser->access_right('Summarize packages'); my $title = 'Package Summary Report'; my ($begin, $end) = FS::UI::Web::parse_beginning_ending($cgi); diff --git a/httemplate/search/cust_pkg_summary.html b/httemplate/search/cust_pkg_summary.html index a0ef47210..f9adf044e 100644 --- a/httemplate/search/cust_pkg_summary.html +++ b/httemplate/search/cust_pkg_summary.html @@ -21,4 +21,8 @@ <% include('/elements/footer.html') %> <%init> + +die "access denied" + unless $curuser->access_right('Summarize packages'); + </%init> diff --git a/httemplate/search/cust_pkg_susp.cgi b/httemplate/search/cust_pkg_susp.cgi index 9ab5992d9..d6bbc43d6 100644 --- a/httemplate/search/cust_pkg_susp.cgi +++ b/httemplate/search/cust_pkg_susp.cgi @@ -25,7 +25,7 @@ my $curuser = $FS::CurrentUser::CurrentUser; die "access denied" - unless $curuser->access_right('List packages'); + unless $curuser->access_right('Summarize packages'); my $money_char = FS::Conf->new()->config('money_char') || '$'; diff --git a/httemplate/search/cust_pkg_susp.html b/httemplate/search/cust_pkg_susp.html index c59e6c158..2ac643260 100644 --- a/httemplate/search/cust_pkg_susp.html +++ b/httemplate/search/cust_pkg_susp.html @@ -21,4 +21,8 @@ <% include('/elements/footer.html') %> <%init> + +die "access denied" + unless $curuser->access_right('Summarize packages'); + </%init> |