diff options
| author | Ivan Kohler <ivan@freeside.biz> | 2012-06-20 16:37:46 -0700 |
|---|---|---|
| committer | Ivan Kohler <ivan@freeside.biz> | 2012-06-20 16:37:46 -0700 |
| commit | f5881f03875aa1460e76947cb5a45f96bf0144e4 (patch) | |
| tree | e8747c9f9fbb2cf5906c8977ab653646e7bac9d2 /httemplate/elements/menu.html | |
| parent | bccfbf5a97b3e5572fd6e68f1a9ed74ceebe39c4 (diff) | |
ACLs for advanced customer and service searching, RT#18157
Diffstat (limited to 'httemplate/elements/menu.html')
| -rw-r--r-- | httemplate/elements/menu.html | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/httemplate/elements/menu.html b/httemplate/elements/menu.html index 14f8cd2f7..892345133 100644 --- a/httemplate/elements/menu.html +++ b/httemplate/elements/menu.html @@ -110,8 +110,7 @@ $report_customers{'List customers'} = [ \%report_customers_lists, 'List customer $report_customers{'Zip code distribution'} = [ $fsurl. 'search/report_cust_main-zip.html', 'Zip codes by number of customers' ]; $report_customers{'Customer signup report'} = [ $fsurl. 'graph/report_cust_signup.html', 'New customer signups by date' ], $report_customers{'Advanced customer reports'} = [ $fsurl. 'search/report_cust_main.html', 'by status, signup date, agent, etc.' ] - if $curuser->access_right('List customers') - && $curuser->access_right('List packages'); + if $curuser->access_right('Advanced customer search'); tie my %report_invoices_open, 'Tie::IxHash', 'All open invoices' => [ $fsurl.'search/cust_bill.html?OPEN_date', 'All invoices with an unpaid balance' ], @@ -201,10 +200,10 @@ foreach my $svcdb ( FS::part_svc->svc_tables() ) { ]; } - if ( $svcdb =~ /^svc_(acct|broadband|hardware)$/ ) { $report_svc{"Advanced $lcsname reports"} = - [ $fsurl."search/report_$svcdb.html", '' ]; - } + [ $fsurl."search/report_$svcdb.html", '' ] + if $svcdb =~ /^svc_(acct|broadband|hardware)$/ + && $curuser->access_right("Services: $name: Advanced search"); if ( $svcdb eq 'svc_phone' ) { @@ -221,8 +220,8 @@ foreach my $svcdb ( FS::part_svc->svc_tables() ) { } - $report_services{$name} = [ \%report_svc, $longname ] if - $curuser->access_right("Services: $name"); + $report_services{$name} = [ \%report_svc, $longname ] + if $curuser->access_right("Services: $name"); } |