From: mark Date: Fri, 21 May 2010 19:09:04 +0000 (+0000) Subject: RT#8504: option to aggregate agents on sales report X-Git-Tag: root_of_svc_elec_features~248 X-Git-Url: http://git.freeside.biz/gitweb/?p=freeside.git;a=commitdiff_plain;h=f360579dd091f1adf9cea1062aa06afdf5bff42b RT#8504: option to aggregate agents on sales report --- diff --git a/FS/FS/Report/Table/Monthly.pm b/FS/FS/Report/Table/Monthly.pm index 4b866c4dc..620af081e 100644 --- a/FS/FS/Report/Table/Monthly.pm +++ b/FS/FS/Report/Table/Monthly.pm @@ -1,13 +1,14 @@ package FS::Report::Table::Monthly; use strict; -use vars qw( @ISA ); +use vars qw( @ISA $DEBUG ); use Time::Local; use FS::UID qw( dbh ); use FS::Report::Table; use FS::CurrentUser; @ISA = qw( FS::Report::Table ); +$DEBUG = 0; # turning this on will trace all SQL statements, VERY noisy =head1 NAME @@ -378,12 +379,12 @@ sub cust_bill_pkg { } if ( $opt{'use_override'} ) { - $where = "( + $where = "AND ( part_pkg.classnum $comparison AND pkgpart_override IS NULL OR override.classnum $comparison AND pkgpart_override IS NOT NULL )"; } else { - $where = "part_pkg.classnum $comparison"; + $where = "AND part_pkg.classnum $comparison"; } } @@ -404,7 +405,7 @@ sub cust_bill_pkg { LEFT JOIN part_pkg USING ( pkgpart ) LEFT JOIN part_pkg AS override ON pkgpart_override = override.pkgpart WHERE pkgnum != 0 - AND $where + $where AND ". $self->in_time_period_and_agent($speriod, $eperiod, $agentnum); if ($opt{use_usage} && $opt{use_usage} eq 'recurring') { @@ -569,6 +570,7 @@ sub in_time_period_and_agent { sub scalar_sql { my( $self, $sql ) = ( shift, shift ); my $sth = dbh->prepare($sql) or die dbh->errstr; + warn "FS::Report::Table::Monthly\n$sql\n" if $DEBUG; $sth->execute or die "Unexpected error executing statement $sql: ". $sth->errstr; $sth->fetchrow_arrayref->[0] || 0; diff --git a/httemplate/graph/cust_bill_pkg.cgi b/httemplate/graph/cust_bill_pkg.cgi index ccd41b8a9..1fad5e1cd 100644 --- a/httemplate/graph/cust_bill_pkg.cgi +++ b/httemplate/graph/cust_bill_pkg.cgi @@ -1,4 +1,5 @@ <% include('elements/monthly.html', + #Dumper( 'title' => $title, 'graph_type' => 'Mountain', 'items' => \@items, @@ -26,8 +27,12 @@ my $use_usage = $cgi->param('use_usage') ? 1 : 0; my $average_per_cust_pkg = $cgi->param('average_per_cust_pkg') ? 1 : 0; #XXX or virtual -my( $agentnum, $sel_agent ) = ('', ''); -if ( $cgi->param('agentnum') =~ /^(\d+)$/ ) { +my( $agentnum, $sel_agent, $all_agent ) = ('', '', ''); +if ( $cgi->param('agentnum') eq 'all' ) { + $agentnum = 0; + $all_agent = 'ALL'; +} +elsif ( $cgi->param('agentnum') =~ /^(\d+)$/ ) { $agentnum = $1; $bottom_link .= "agentnum=$agentnum;"; $sel_agent = qsearchs('agent', { 'agentnum' => $agentnum } ); @@ -49,14 +54,18 @@ $title .= ', average per customer package' if $average_per_cust_pkg; #false lazinessish w/FS::cust_pkg::search_sql (previously search/cust_pkg.cgi) my $classnum = 0; my @pkg_class = (); -if ( $cgi->param('classnum') =~ /^(\d*)$/ ) { +my $all_class = ''; +if ( $cgi->param('classnum') eq 'all' ) { + $all_class = 'ALL'; + @pkg_class = (''); +} +elsif ( $cgi->param('classnum') =~ /^(\d*)$/ ) { $classnum = $1; - if ( $classnum ) { #a specific class @pkg_class = ( qsearchs('pkg_class', { 'classnum' => $classnum } ) ); die "classnum $classnum not found!" unless $pkg_class[0]; - $title .= $pkg_class[0]->classname.' '; + $title .= ' '.$pkg_class[0]->classname.' '; $bottom_link .= "classnum=$classnum;"; } elsif ( $classnum eq '' ) { #the empty class @@ -85,7 +94,7 @@ my @labels = (); my @colors = (); my @links = (); -foreach my $agent ( $sel_agent || qsearch('agent', { 'disabled' => '' } ) ) { +foreach my $agent ( $all_agent || $sel_agent || qsearch('agent', { 'disabled' => '' } ) ) { my $col_scheme = Color::Scheme->new ->from_hue($hue) #->from_hex($agent->color) @@ -104,7 +113,7 @@ foreach my $agent ( $sel_agent || qsearch('agent', { 'disabled' => '' } ) ) { push @items, 'cust_bill_pkg'; push @labels, - ( $sel_agent ? '' : $agent->agent.' ' ). + ( $all_agent || $sel_agent ? '' : $agent->agent.' ' ). ( $classnum eq '0' ? ( ref($pkg_class) ? $pkg_class->classname : $pkg_class ) : '' @@ -112,15 +121,16 @@ foreach my $agent ( $sel_agent || qsearch('agent', { 'disabled' => '' } ) ) { " $component"; my $row_classnum = ref($pkg_class) ? $pkg_class->classnum : 0; - my $row_agentnum = $agent->agentnum; - push @params, [ 'classnum' => $row_classnum, - 'agentnum' => $row_agentnum, + my $row_agentnum = $all_agent || $agent->agentnum; + push @params, [ ($all_class ? () : ('classnum' => $row_classnum) ), + ($all_agent ? () : ('agentnum' => $row_agentnum) ), 'use_override' => $use_override, 'use_usage' => $component, 'average_per_cust_pkg' => $average_per_cust_pkg, ]; - push @links, "$link;agentnum=$row_agentnum;classnum=$row_classnum;". + push @links, "$link;".($all_agent ? '' : "agentnum=$row_agentnum;"). + ($all_class ? '' : "classnum=$row_classnum;"). "use_override=$use_override;use_usage=$component;"; @recur_colors = ($col_scheme->colors)[0,4,8,1,5,9] @@ -137,6 +147,5 @@ foreach my $agent ( $sel_agent || qsearch('agent', { 'disabled' => '' } ) ) { } #use Data::Dumper; -#warn Dumper(\@items); diff --git a/httemplate/graph/report_cust_bill_pkg.html b/httemplate/graph/report_cust_bill_pkg.html index 4a6433272..348746514 100644 --- a/httemplate/graph/report_cust_bill_pkg.html +++ b/httemplate/graph/report_cust_bill_pkg.html @@ -9,11 +9,14 @@ <% include('/elements/tr-select-agent.html', 'label' => 'For agent: ', 'disable_empty' => 0, + 'pre_options' => [ 'all' => 'all (aggregate)' ], + 'empty_label' => 'all (breakdown)', ) %> <% include('/elements/tr-select-pkg_class.html', - 'pre_options' => [ '0' => 'all' ], + 'pre_options' => [ 'all' => 'all (aggregate)', + '0' => 'all (breakdown)' ], 'empty_label' => '(empty class)', ) %>