From 208add41d52153ccff338fe0f1eafbc0c462a2ed Mon Sep 17 00:00:00 2001 From: ivan Date: Mon, 25 May 2009 22:42:21 +0000 Subject: fix total links on sales graph when a package class is specified, RT#5449 --- httemplate/graph/cust_bill_pkg.cgi | 33 ++++++++++++++++++++++++++------- 1 file changed, 26 insertions(+), 7 deletions(-) (limited to 'httemplate/graph') diff --git a/httemplate/graph/cust_bill_pkg.cgi b/httemplate/graph/cust_bill_pkg.cgi index d7cae8055..a96eed779 100644 --- a/httemplate/graph/cust_bill_pkg.cgi +++ b/httemplate/graph/cust_bill_pkg.cgi @@ -9,7 +9,7 @@ 'links' => \@links, 'remove_empty' => 1, 'bottom_total' => 1, - 'bottom_link' => "$link;", + 'bottom_link' => $bottom_link, 'agentnum' => $agentnum, ) %> @@ -27,21 +27,42 @@ if ( $cgi->param('agentnum') =~ /^(\d+)$/ ) { } my $title = $sel_agent ? $sel_agent->agent.' ' : ''; -#false lazinessish w/search/cust_pkg.cgi +my $link = "${p}search/cust_bill_pkg.cgi?nottax=1;include_comp_cust=1"; +my $bottom_link = "$link;"; + +#classnum (here) +# 0: all classes +# not specified: empty class +# N: classnum +#classnum (link) +# not specified: all classes +# 0: empty class +# N: classnum + +#false lazinessish w/FS::cust_pkg::search_sql (previously search/cust_pkg.cgi) my $classnum = 0; my @pkg_class = (); if ( $cgi->param('classnum') =~ /^(\d*)$/ ) { $classnum = $1; - if ( $classnum ) { + + 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.' '; - } elsif ( $classnum eq '' ) { + $bottom_link .= "classnum=$classnum;"; + + } elsif ( $classnum eq '' ) { #the empty class + $title .= 'Empty class '; @pkg_class = ( '(empty class)' ); - } elsif ( $classnum eq '0' ) { + $bottom_link .= "classnum=0;"; + + } elsif ( $classnum eq '0' ) { #all classes + @pkg_class = qsearch('pkg_class', {} ); # { 'disabled' => '' } ); push @pkg_class, '(empty class)'; + } } #eslaf @@ -57,8 +78,6 @@ my @labels = (); my @colors = (); my @links = (); -my $link = "${p}search/cust_bill_pkg.cgi?nottax=1;include_comp_cust=1"; - foreach my $agent ( $sel_agent || qsearch('agent', { 'disabled' => '' } ) ) { my $col_scheme = Color::Scheme->new -- cgit v1.2.1 From 52feaa06fcb7a06e253adcb18d87854498ad218e Mon Sep 17 00:00:00 2001 From: ivan Date: Wed, 24 Jun 2009 01:40:12 +0000 Subject: fix total links on agent-specific sales report, RT#5449 --- httemplate/graph/cust_bill_pkg.cgi | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'httemplate/graph') diff --git a/httemplate/graph/cust_bill_pkg.cgi b/httemplate/graph/cust_bill_pkg.cgi index a96eed779..b655f1f01 100644 --- a/httemplate/graph/cust_bill_pkg.cgi +++ b/httemplate/graph/cust_bill_pkg.cgi @@ -18,18 +18,19 @@ die "access denied" unless $FS::CurrentUser::CurrentUser->access_right('Financial reports'); +my $link = "${p}search/cust_bill_pkg.cgi?nottax=1;include_comp_cust=1"; +my $bottom_link = "$link;"; + #XXX or virtual my( $agentnum, $sel_agent ) = ('', ''); if ( $cgi->param('agentnum') =~ /^(\d+)$/ ) { $agentnum = $1; + $bottom_link .= "agentnum=$agentnum;"; $sel_agent = qsearchs('agent', { 'agentnum' => $agentnum } ); die "agentnum $agentnum not found!" unless $sel_agent; } my $title = $sel_agent ? $sel_agent->agent.' ' : ''; -my $link = "${p}search/cust_bill_pkg.cgi?nottax=1;include_comp_cust=1"; -my $bottom_link = "$link;"; - #classnum (here) # 0: all classes # not specified: empty class @@ -94,7 +95,6 @@ foreach my $agent ( $sel_agent || qsearch('agent', { 'disabled' => '' } ) ) { push @items, 'cust_bill_pkg'; - push @labels, ( $sel_agent ? '' : $agent->agent.' ' ). ( $classnum eq '0' -- cgit v1.2.1 From 2e1d2a91264e8441bc31583be7352ae1b3c31bb8 Mon Sep 17 00:00:00 2001 From: jeff Date: Wed, 19 Aug 2009 06:15:14 +0000 Subject: option to count subpackages outside packages in sales report #5588 --- httemplate/graph/cust_bill_pkg.cgi | 11 ++++++++--- httemplate/graph/report_cust_bill_pkg.html | 5 +++++ 2 files changed, 13 insertions(+), 3 deletions(-) (limited to 'httemplate/graph') diff --git a/httemplate/graph/cust_bill_pkg.cgi b/httemplate/graph/cust_bill_pkg.cgi index b655f1f01..021189abb 100644 --- a/httemplate/graph/cust_bill_pkg.cgi +++ b/httemplate/graph/cust_bill_pkg.cgi @@ -68,6 +68,9 @@ if ( $cgi->param('classnum') =~ /^(\d*)$/ ) { } #eslaf +my $use_override = 0; +$use_override = 1 if ( $cgi->param('use_override') ); + my $hue = 0; #my $hue_increment = 170; #my $hue_increment = 145; @@ -104,11 +107,13 @@ foreach my $agent ( $sel_agent || qsearch('agent', { 'disabled' => '' } ) ) { my $row_classnum = ref($pkg_class) ? $pkg_class->classnum : 0; my $row_agentnum = $agent->agentnum; - push @params, [ 'classnum' => $row_classnum, - 'agentnum' => $row_agentnum, + push @params, [ 'classnum' => $row_classnum, + 'agentnum' => $row_agentnum, + 'use_override' => $use_override, ]; - push @links, "$link;agentnum=$row_agentnum;classnum=$row_classnum;"; + push @links, "$link;agentnum=$row_agentnum;classnum=$row_classnum;". + "use_override=$use_override;"; @recur_colors = ($col_scheme->colors)[0,4,8,1,5,9] unless @recur_colors; diff --git a/httemplate/graph/report_cust_bill_pkg.html b/httemplate/graph/report_cust_bill_pkg.html index 5193bf4a3..073f86773 100644 --- a/httemplate/graph/report_cust_bill_pkg.html +++ b/httemplate/graph/report_cust_bill_pkg.html @@ -25,6 +25,11 @@ --> + + + Separate sub-packages from parents + +
-- cgit v1.2.1 From b53214c810513eda313fb7aea5405fb02a3b94c5 Mon Sep 17 00:00:00 2001 From: jeff Date: Mon, 24 Aug 2009 05:09:27 +0000 Subject: add rated call sales report and option to sales report to count usage separately from recurring #5588 --- httemplate/graph/cust_bill_pkg.cgi | 60 ++++++---- httemplate/graph/cust_bill_pkg_detail.cgi | 137 ++++++++++++++++++++++ httemplate/graph/report_cust_bill_pkg.html | 6 + httemplate/graph/report_cust_bill_pkg_detail.html | 48 ++++++++ 4 files changed, 225 insertions(+), 26 deletions(-) create mode 100644 httemplate/graph/cust_bill_pkg_detail.cgi create mode 100644 httemplate/graph/report_cust_bill_pkg_detail.html (limited to 'httemplate/graph') diff --git a/httemplate/graph/cust_bill_pkg.cgi b/httemplate/graph/cust_bill_pkg.cgi index 021189abb..832660ffd 100644 --- a/httemplate/graph/cust_bill_pkg.cgi +++ b/httemplate/graph/cust_bill_pkg.cgi @@ -71,6 +71,9 @@ if ( $cgi->param('classnum') =~ /^(\d*)$/ ) { my $use_override = 0; $use_override = 1 if ( $cgi->param('use_override') ); +my $use_usage = 0; +$use_usage = 1 if ( $cgi->param('use_usage') ); + my $hue = 0; #my $hue_increment = 170; #my $hue_increment = 145; @@ -92,35 +95,40 @@ foreach my $agent ( $sel_agent || qsearch('agent', { 'disabled' => '' } ) ) { my @onetime_colors = (); ### fixup the color handling for package classes... + ### and usage my $n = 0; foreach my $pkg_class ( @pkg_class ) { - - push @items, 'cust_bill_pkg'; - - push @labels, - ( $sel_agent ? '' : $agent->agent.' ' ). - ( $classnum eq '0' - ? ( ref($pkg_class) ? $pkg_class->classname : $pkg_class ) - : '' - ); - - my $row_classnum = ref($pkg_class) ? $pkg_class->classnum : 0; - my $row_agentnum = $agent->agentnum; - push @params, [ 'classnum' => $row_classnum, - 'agentnum' => $row_agentnum, - 'use_override' => $use_override, - ]; - - push @links, "$link;agentnum=$row_agentnum;classnum=$row_classnum;". - "use_override=$use_override;"; - - @recur_colors = ($col_scheme->colors)[0,4,8,1,5,9] - unless @recur_colors; - @onetime_colors = ($col_scheme->colors)[2,6,10,3,7,11] - unless @onetime_colors; - push @colors, shift @recur_colors; - + foreach my $component ( $use_usage ? ('recurring', 'usage') : ('') ) { + + push @items, 'cust_bill_pkg'; + + push @labels, + ( $sel_agent ? '' : $agent->agent.' ' ). + ( $classnum eq '0' + ? ( ref($pkg_class) ? $pkg_class->classname : $pkg_class ) + : '' + ). + " $component"; + + my $row_classnum = ref($pkg_class) ? $pkg_class->classnum : 0; + my $row_agentnum = $agent->agentnum; + push @params, [ 'classnum' => $row_classnum, + 'agentnum' => $row_agentnum, + 'use_override' => $use_override, + 'use_usage' => $component, + ]; + + push @links, "$link;agentnum=$row_agentnum;classnum=$row_classnum;". + "use_override=$use_override;use_usage=$component;"; + + @recur_colors = ($col_scheme->colors)[0,4,8,1,5,9] + unless @recur_colors; + @onetime_colors = ($col_scheme->colors)[2,6,10,3,7,11] + unless @onetime_colors; + push @colors, shift @recur_colors; + + } } $hue += $hue_increment; diff --git a/httemplate/graph/cust_bill_pkg_detail.cgi b/httemplate/graph/cust_bill_pkg_detail.cgi new file mode 100644 index 000000000..642a9ecf3 --- /dev/null +++ b/httemplate/graph/cust_bill_pkg_detail.cgi @@ -0,0 +1,137 @@ +<% include('elements/monthly.html', + 'title' => $title. 'Rated Call Sales Report (Gross)', + 'graph_type' => 'Mountain', + 'items' => \@items, + 'params' => \@params, + 'labels' => \@labels, + 'graph_labels' => \@labels, + 'colors' => \@colors, + 'remove_empty' => 1, + 'bottom_total' => 1, + 'agentnum' => $agentnum, + ) +%> +<%init> + +die "access denied" + unless $FS::CurrentUser::CurrentUser->access_right('Financial reports'); + +#XXX or virtual +my( $agentnum, $sel_agent ) = ('', ''); +if ( $cgi->param('agentnum') =~ /^(\d+)$/ ) { + $agentnum = $1; + $sel_agent = qsearchs('agent', { 'agentnum' => $agentnum } ); + die "agentnum $agentnum not found!" unless $sel_agent; +} +my $title = $sel_agent ? $sel_agent->agent.' ' : ''; + +#false lazinessish w/FS::cust_pkg::search_sql (previously search/cust_pkg.cgi) +my $classnum = ''; +if ( $cgi->param('classnum') =~ /^(\d*)$/ ) { + $classnum = $1; + + if ( $classnum ) { #a specific class + + my $pkg_class = ( qsearchs('pkg_class', { 'classnum' => $classnum } ) ); + die "classnum $classnum not found!" unless $pkg_class; + $title .= $pkg_class->classname.' '; + + } elsif ( $classnum eq '' ) { #the empty class + + $title .= 'Empty class '; + # FS::Report::Table::Monthly.pm has the converse view + $classnum = 0; + + } elsif ( $classnum eq '0' ) { #all classes + + # FS::Report::Table::Monthly.pm has the converse view + $classnum = ''; + } +} +#eslaf + +my $use_override = 0; +$use_override = 1 if ( $cgi->param('use_override') ); + +my $usageclass = 0; +my @usage_class = (); +if ( $cgi->param('usageclass') =~ /^(\d*)$/ ) { + $usageclass = $1; + + if ( $usageclass ) { #a specific class + + @usage_class = ( qsearchs('usage_class', { 'classnum' => $usageclass } ) ); + die "usage class $usageclass not found!" unless $usage_class[0]; + $title .= $usage_class[0]->classname.' '; + + } elsif ( $usageclass eq '' ) { #the empty class -- legacy + + $title .= 'Empty usage class '; + @usage_class = ( '(empty usage class)' ); + + } elsif ( $usageclass eq '0' ) { #all classes + + @usage_class = qsearch('usage_class', {} ); # { 'disabled' => '' } ); + push @usage_class, '(empty usage class)'; + + } +} +#eslaf + +my $hue = 0; +#my $hue_increment = 170; +#my $hue_increment = 145; +my $hue_increment = 125; + +my @items = (); +my @params = (); +my @labels = (); +my @colors = (); + +foreach my $agent ( $sel_agent || qsearch('agent', { 'disabled' => '' } ) ) { + + my $col_scheme = Color::Scheme->new + ->from_hue($hue) #->from_hex($agent->color) + ->scheme('analogic') + ; + my @recur_colors = (); + my @onetime_colors = (); + + ### fixup the color handling for usage classes... + my $n = 0; + + foreach my $usage_class ( @usage_class ) { + + push @items, 'cust_bill_pkg_detail'; + + push @labels, + ( $sel_agent ? '' : $agent->agent.' ' ). + ( $usageclass eq '0' + ? ( ref($usage_class) ? $usage_class->classname : $usage_class ) + : '' + ); + + my $row_classnum = ref($usage_class) ? $usage_class->classnum : 0; + my $row_agentnum = $agent->agentnum; + push @params, [ 'usageclass' => $row_classnum, + 'agentnum' => $row_agentnum, + 'use_override' => $use_override, + 'classnum' => $classnum, + ]; + + @recur_colors = ($col_scheme->colors)[0,4,8,1,5,9] + unless @recur_colors; + @onetime_colors = ($col_scheme->colors)[2,6,10,3,7,11] + unless @onetime_colors; + push @colors, shift @recur_colors; + + } + + $hue += $hue_increment; + +} + +#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 073f86773..51655a925 100644 --- a/httemplate/graph/report_cust_bill_pkg.html +++ b/httemplate/graph/report_cust_bill_pkg.html @@ -30,6 +30,12 @@ Separate sub-packages from parents + + + Separate rated usage from recurring fees + + +
diff --git a/httemplate/graph/report_cust_bill_pkg_detail.html b/httemplate/graph/report_cust_bill_pkg_detail.html new file mode 100644 index 000000000..3b85d521c --- /dev/null +++ b/httemplate/graph/report_cust_bill_pkg_detail.html @@ -0,0 +1,48 @@ +<% include('/elements/header.html', 'Usage Sales Report' ) %> + +
+ + + +<% include('/elements/tr-select-from_to.html' ) %> + +<% include('/elements/tr-select-agent.html', + 'label' => 'For agent: ', + 'disable_empty' => 0, + ) +%> + +<% include('/elements/tr-select-pkg_class.html', + 'pre_options' => [ '0' => 'all' ], + 'empty_label' => '(empty class)', + ) +%> + + + + + + +<% include('/elements/tr-select-table.html', + 'label' => 'Usage class: ', + 'element_name' => 'usageclass', + 'table' => 'usage_class', + 'name_col' => 'classname', + 'hashref' => { 'disabled' => '' }, + 'pre_options' => [ '0' => 'all' ], + 'empty_label' => '(empty class)', + ) +%> + +
Separate sub-packages from parents
+ +
+
+ +<% include('/elements/footer.html') %> +<%init> + +die "access denied" + unless $FS::CurrentUser::CurrentUser->access_right('Financial reports'); + + -- cgit v1.2.1