diff options
| author | Mark Wells <mark@freeside.biz> | 2014-01-17 16:48:21 -0800 |
|---|---|---|
| committer | Mark Wells <mark@freeside.biz> | 2014-01-17 16:48:21 -0800 |
| commit | 074ab75187e3575674941a72ea50205c2ec42595 (patch) | |
| tree | 18a95e45201983e2bfd93745817bda64d90cf4d9 | |
| parent | a292db1a4cf376464f01b5625da8680bcb7928da (diff) | |
show add-on package description in sales report, #25836
| -rw-r--r-- | httemplate/search/cust_bill_pkg.cgi | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/httemplate/search/cust_bill_pkg.cgi b/httemplate/search/cust_bill_pkg.cgi index 7476f4d8c..6b7a5e6e2 100644 --- a/httemplate/search/cust_bill_pkg.cgi +++ b/httemplate/search/cust_bill_pkg.cgi @@ -20,12 +20,10 @@ 'fields' => [ @pkgnum, sub { $_[0]->pkgnum > 0 - # possibly use override.pkg but i think this correct ? $_[0]->get('pkgpart') : '' }, sub { $_[0]->pkgnum > 0 - # possibly use override.pkg but i think this correct ? $_[0]->get('pkg') : $_[0]->get('itemdesc') }, @@ -246,7 +244,6 @@ if ( $conf->exists('enable_taxclasses') ) { push @post_desc, 'taxclass'; push @post_desc_null, ''; $post_desc_align .= 'l'; - push @select, 'part_pkg.taxclass'; # or should this use override? } # valid in both the tax and non-tax cases @@ -321,7 +318,8 @@ if ( $use_override ) { )"; $part_pkg = 'override'; } -push @select, 'part_pkg.pkgpart', 'part_pkg.pkg'; # or should this use override? +push @select, "$part_pkg.pkgpart", "$part_pkg.pkg"; +push @select, "$part_pkg.taxclass" if $conf->exists('enable_taxclasses'); # the non-tax case if ( $cgi->param('nottax') ) { |