1 package FS::part_pkg::voip_cdr;
2 use base qw( FS::part_pkg::recur_Common );
5 use vars qw( $DEBUG %info );
11 use FS::Record qw(qsearchs qsearch);
13 use FS::detail_format;
17 tie my %cdr_svc_method, 'Tie::IxHash',
18 'svc_phone.phonenum' => 'Phone numbers (svc_phone.phonenum)',
19 'svc_pbx.title' => 'PBX name (svc_pbx.title)',
20 'svc_pbx.svcnum' => 'Freeside service # (svc_pbx.svcnum)',
21 'svc_pbx.ip.src' => 'PBX name to source IP address',
22 'svc_pbx.ip.dst' => 'PBX name to destination IP address',
23 'svc_acct.username' => 'Username (svc_acct.username)',
26 tie my %rating_method, 'Tie::IxHash',
27 'prefix' => 'Rate calls by using destination prefix to look up a region and rate according to the internal prefix and rate tables',
28 # 'upstream' => 'Rate calls based on upstream data: If the call type is "1", map the upstream rate ID directly to an internal rate (rate_detail), otherwise, pass the upstream price through directly.',
29 'upstream_simple' => 'Simply pass through and charge the "upstream_price" amount (ignoring all options to skip calls).',
30 'single_price' => 'A single price per minute for all calls (ignoring all options to skip calls).',
33 tie my %rounding, 'Tie::IxHash',
34 '2' => 'Two decimal places (cent)',
35 '4' => 'Four decimal places (100th of a cent)',
38 #tie my %cdr_location, 'Tie::IxHash',
39 # 'internal' => 'Internal: CDR records imported into the internal CDR table',
40 # 'external' => 'External: CDR records queried directly from an external '.
41 # 'Asterisk (or other?) CDR table',
44 tie my %temporalities, 'Tie::IxHash',
45 'upcoming' => "Upcoming (future)",
46 'preceding' => "Preceding (past)",
49 tie my %granularity, 'Tie::IxHash', FS::rate_detail::granularities();
51 # previously "1" was "ignore"
52 tie my %unrateable_opts, 'Tie::IxHash',
53 '' => 'Exit with a fatal error',
54 1 => 'Ignore and continue',
55 2 => 'Flag for later review',
58 tie my %detail_formats, 'Tie::IxHash',
60 FS::cdr::invoice_formats()
63 tie my %accountcode_tollfree_field, 'Tie::IxHash',
64 'dst' => 'Destination (dst)',
65 'src' => 'Source (src)',
69 'name' => 'VoIP rating by plan of CDR records',
70 'shortname' => 'VoIP/telco CDR rating (standard)',
71 'inherit_fields' => [ 'prorate_Mixin', 'global_Mixin' ],
73 'suspend_bill' => { 'name' => 'Continue recurring billing while suspended',
76 #false laziness w/flat.pm
77 'recur_temporality' => { 'name' => 'Charge recurring fee for period',
79 'select_options' => \%temporalities,
82 'cutoff_day' => { 'name' => 'Billing Day (1 - 28) for prorating or '.
86 'recur_method' => { 'name' => 'Recurring fee method',
88 #'options' => \%recur_method,
90 'select_options' => \%FS::part_pkg::recur_Common::recur_method,
93 'cdr_svc_method' => { 'name' => 'CDR service matching method',
96 'select_options' => \%cdr_svc_method,
99 'rating_method' => { 'name' => 'Rating method',
101 'options' => \%rating_method,
104 'rounding' => { 'name' => 'Rounding for destination prefix rating',
106 'select_options' => \%rounding,
109 'ratenum' => { 'name' => 'Rate plan',
110 'type' => 'select-rate',
113 'intrastate_ratenum' => { 'name' => 'Optional alternate intrastate rate plan',
114 'type' => 'select-rate',
115 'disable_empty' => 0,
116 'empty_label' => ' ',
119 'calls_included' => { 'name' => 'Number of calls included at no usage charge', },
121 'min_included' => { 'name' => 'Minutes included when using the "single price per minute" or "prefix" rating method',
124 'min_charge' => { 'name' => 'Charge per minute when using "single price per minute" rating method',
127 'sec_granularity' => { 'name' => 'Granularity when using "single price per minute" rating method',
129 'select_options' => \%granularity,
132 'ignore_unrateable' => { 'name' => 'Handling of calls without a rate in the rate table',
134 'select_options' => \%unrateable_opts,
137 'default_prefix' => { 'name' => 'Default prefix optionally prepended to customer DID numbers when searching for CDR records',
141 'disable_src' => { 'name' => 'Disable rating of CDR records based on the "src" field in addition to "charged_party"',
145 'domestic_prefix' => { 'name' => 'Destination prefix for domestic CDR records',
149 # 'domestic_prefix_required' => { 'name' => 'Require explicit destination prefix for domestic CDR records',
150 # 'type' => 'checkbox',
153 'international_prefix' => { 'name' => 'Destination prefix for international CDR records (or "none" for no prefix)',
157 'disable_tollfree' => { 'name' => 'Disable automatic toll-free processing',
158 'type' => 'checkbox',
161 'use_amaflags' => { 'name' => 'Only charge for CDRs where the amaflags field is set to "2" ("BILL"/"BILLING").',
162 'type' => 'checkbox',
165 'use_carrierid' => { 'name' => 'Only charge for CDRs where the Carrier ID is set to any of these (comma-separated) values: ',
168 'use_cdrtypenum' => { 'name' => 'Only charge for CDRs where the CDR Type is set to this cdrtypenum: ',
169 'validate' => \&FS::part_pkg::validate_number_blank,
170 'js_validate' => 'digits: true',
173 'ignore_cdrtypenum' => { 'name' => 'Do not charge for CDRs where the CDR Type is set to this cdrtypenum: ',
174 'validate' => \&FS::part_pkg::validate_number_blank,
175 'js_validate' => 'digits: true',
178 'use_calltypenum' => { 'name' => 'Only charge for CDRs where the CDR Call Type is set to this calltypenum: ',
181 'ignore_calltypenum' => { 'name' => 'Do not charge for CDRs where the CDR Call Type is set to this calltypenum: ',
184 'ignore_disposition' => { 'name' => 'Do not charge for CDRs where the Disposition is set to any of these (comma-separated) values: ',
187 'disposition_in' => { 'name' => 'Only charge for CDRs where the Disposition is set to any of these (comma-separated) values: ',
190 'disposition_prefix' => { 'name' => 'Only charge for CDRs where the Disposition starts with: ',
193 'skip_dst_prefix' => { 'name' => 'Do not charge for CDRs where the destination number starts with any of these (comma-separated) values: ',
196 'skip_dcontext' => { 'name' => 'Do not charge for CDRs where dcontext is set to any of these (comma-separated) values: ',
199 'skip_dcontext_prefix' => { 'name' => 'Do not charge for CDRs where dcontext starts with: ',
202 'skip_dcontext_suffix' => { 'name' => 'Do not charge for CDRs where dcontext ends with: ',
205 'skip_dstchannel_prefix' => { 'name' => 'Do not charge for CDRs where the dstchannel starts with:',
208 'skip_src_length_more' => { 'name' => 'Do not charge for CDRs where the source is more than this many digits:',
211 'noskip_src_length_accountcode_tollfree' => { 'name' => 'Do charge for CDRs where source is equal or greater than the specified digits, when accountcode is toll free',
212 'type' => 'checkbox',
215 'accountcode_tollfree_ratenum' => {
216 'name' => 'Optional alternate rate plan when accountcode is toll free: ',
218 'select_table' => 'rate',
219 'select_key' => 'ratenum',
220 'select_label' => 'ratename',
221 'disable_empty' => 0,
225 'accountcode_tollfree_field' => {
226 'name' => 'When using an alternate rate plan for toll-free accountcodes, the CDR field to use in rating calculations',
228 'select_options' => \%accountcode_tollfree_field,
231 'skip_dst_length_less' => { 'name' => 'Do not charge for CDRs where the destination is less than this many digits:',
234 'noskip_dst_length_accountcode_tollfree' => { 'name' => 'Do charge for CDRs where dst is less than the specified digits, when accountcode is toll free',
235 'type' => 'checkbox',
238 'skip_lastapp' => { 'name' => 'Do not charge for CDRs where the lastapp matches this value: ',
241 'skip_max_callers' => { 'name' => 'Do not charge for CDRs where max_callers is less than or equal to this value: ',
244 'skip_same_customer' => {
245 'name' => 'Do not charge for calls between numbers belonging to the same customer',
246 'type' => 'checkbox',
249 'use_duration' => { 'name' => 'Calculate usage based on the duration field instead of the billsec field',
250 'type' => 'checkbox',
253 '411_rewrite' => { 'name' => 'Rewrite these (comma-separated) destination numbers to 411 for rating purposes (also ignore any carrierid check): ',
256 #false laziness w/cdr_termination.pm
257 'output_format' => { 'name' => 'CDR display format for invoices',
259 'select_options' => \%detail_formats,
260 'default' => 'default', #XXX test
263 'selfservice_format' =>
264 { 'name' => 'CDR display format for selfservice',
266 'select_options' => \%detail_formats,
267 'default' => 'default'
269 'selfservice_inbound_format' =>
270 { 'name' => 'Inbound CDR display format for selfservice',
272 'select_options' => \%detail_formats,
276 'usage_section' => { 'name' => 'Section in which to place usage charges (whether separated or not): ',
279 'summarize_usage' => { 'name' => 'Include usage summary with recurring charges when usage is in separate section',
280 'type' => 'checkbox',
283 'usage_mandate' => { 'name' => 'Always put usage details in separate section. The section is defined in the next option.',
284 'type' => 'checkbox',
288 'usage_showzero' => { 'name' => 'Show details for included / no-charge calls.',
289 'type' => 'checkbox',
292 'bill_every_call' => { 'name' => 'Generate an invoice immediately for every call (as well any setup fee, upon first payment). Useful for prepaid.',
293 'type' => 'checkbox',
296 'bill_inactive_svcs' => { 'name' => 'Bill for all phone numbers that were active during the billing period',
297 'type' => 'checkbox',
299 'bill_only_pkg_dates' => { 'name' => 'Only bill CDRs with a date during the package billing period',
300 'type' => 'checkbox',
302 'skip_old' => { 'name' => 'Do not charge for CDRs older than: ',
306 #XXX also have option for an external db? these days we suck them into ours
307 # 'cdr_location' => { 'name' => 'CDR database location'
308 # 'type' => 'select',
309 # 'select_options' => \%cdr_location,
310 # 'select_callback' => {
312 # 'enable' => [ 'datasrc', 'username', 'password' ],
315 # 'disable' => [ 'datasrc', 'username', 'password' ],
319 # 'datasrc' => { 'name' => 'DBI data source for external CDR table',
322 # 'username' => { 'name' => 'External database username',
325 # 'password' => { 'name' => 'External database password',
332 recur_method cutoff_day ),
333 FS::part_pkg::prorate_Mixin::fieldorder,
336 rating_method rounding ratenum intrastate_ratenum
338 min_charge min_included
343 domestic_prefix international_prefix
347 use_cdrtypenum ignore_cdrtypenum
348 use_calltypenum ignore_calltypenum
349 ignore_disposition disposition_in disposition_prefix
350 skip_dcontext skip_dcontext_prefix skip_dcontext_suffix
352 skip_dstchannel_prefix skip_src_length_more
353 noskip_src_length_accountcode_tollfree
354 accountcode_tollfree_ratenum accountcode_tollfree_field
356 noskip_dst_length_accountcode_tollfree
363 selfservice_format selfservice_inbound_format
364 usage_mandate usage_section summarize_usage
365 usage_showzero bill_every_call bill_inactive_svcs
366 bill_only_pkg_dates skip_old
372 # Validation function for FS::part_pkg::check_options()
374 return "Please choose a Rate Plan for use with selected Rating Method"
375 if $options->{rating_method} eq 'prefix' && !$options->{ratenum};
382 my $str = $self->SUPER::price_info(@_);
383 $str .= " plus usage" if $str;
389 my($cust_pkg, $sdate, $details, $param ) = @_;
393 $charges += $self->calc_usage(@_);
394 $charges += ($cust_pkg->quantity || 1) * $self->calc_recur_Common(@_);
403 # my($cust_pkg, $sdate, $details, $param ) = @_;
405 # $self->calc_usage(@_);
408 #false laziness w/voip_sqlradacct calc_recur resolve it if that one ever gets used again
412 my($cust_pkg, $sdate, $details, $param ) = @_;
414 #my $last_bill = $cust_pkg->last_bill;
415 my $last_bill = $cust_pkg->get('last_bill'); #->last_bill falls back to setup
418 if $self->recur_temporality eq 'preceding'
419 && ( $last_bill eq '' || $last_bill == 0 );
423 my $included_min_total = ($self->option('min_included', 1) || 0)
424 * ($cust_pkg->quantity || 1);
427 my $included_min_left = $included_min_total;
429 my $included_calls = $self->option('calls_included', 1) || 0;
430 $included_calls *= ($cust_pkg->quantity || 1);
432 my $cdr_svc_method = $self->option('cdr_svc_method',1)||'svc_phone.phonenum';
433 my $rating_method = $self->option('rating_method') || 'prefix';
434 my %detail_included_min = ();
436 my $output_format = $self->option('output_format', 'Hush!')
437 || ( $rating_method eq 'upstream_simple'
442 my $usage_showzero = $self->option('usage_showzero', 1);
444 my $formatter = FS::detail_format->new($output_format,
446 locale => $cust_pkg->cust_main->locale,
447 rounding => ($self->option_cacheable('rounding') || 2),
450 my $use_duration = $self->option('use_duration');
452 my($svc_table, $svc_field, $by_ip_addr) = split('\.', $cdr_svc_method);
455 if( $self->option('bill_inactive_svcs',1) ) {
456 #XXX in this mode do we need to restrict the set of CDRs by date also?
457 @cust_svc = $cust_pkg->h_cust_svc($$sdate, $last_bill);
460 @cust_svc = $cust_pkg->cust_svc;
462 @cust_svc = grep { $_->part_svc->svcdb eq $svc_table } @cust_svc;
464 foreach my $cust_svc (@cust_svc) {
467 if( $self->option('bill_inactive_svcs',1) ) {
468 $svc_x = $cust_svc->h_svc_x($$sdate, $last_bill);
471 $svc_x = $cust_svc->svc_x;
475 my $h = $self->option('bill_inactive_svcs',1) ? 'h_' : '';
476 warn "WARNING: no $h$svc_table for svcnum ". $cust_svc->svcnum. "\n";
480 'disable_src' => $self->option('disable_src',1),
481 'default_prefix' => $self->option('default_prefix',1),
482 'cdrtypenum' => $self->option('use_cdrtypenum',1),
483 'calltypenum' => $self->option('use_calltypenum',1),
487 if ( $self->option('bill_only_pkg_dates') ) {
488 $options{'begin'} = $last_bill;
489 $options{'end'} = $$sdate;
491 if ( $svc_field eq 'svcnum' ) {
492 $options{'by_svcnum'} = 1;
493 } elsif ($svc_table eq 'svc_pbx' and $svc_field eq 'ip') {
494 $options{'by_ip_addr'} = $by_ip_addr;
497 #my @invoice_details_sort;
499 # for tagging invoice details
500 # (unfortunate; should be a svc_x class method or table_info item or
503 if ( $svc_table eq 'svc_phone' ) {
504 $phonenum = $svc_x->phonenum;
505 } elsif ( $svc_table eq 'svc_pbx' ) {
506 $phonenum = $svc_x->title;
507 } elsif ( $svc_table eq 'svc_acct' ) {
508 $phonenum = $svc_x->username;
510 $formatter->phonenum($phonenum);
512 #first rate any outstanding CDRs not yet rated
513 # use FS::Cursor for this starting in 4.x
514 my $cdr_search = $svc_x->psearch_cdrs(%options);
515 $cdr_search->limit(1000);
516 $cdr_search->increment(0); # because we're changing their status as we go
517 while ( my $cdr = $cdr_search->fetch ) {
519 my $error = $cdr->rate(
521 'cust_pkg' => $cust_pkg,
522 'svcnum' => $svc_x->svcnum,
523 'plan_included_min' => \$included_min_left,
524 'detail_included_min_hashref' => \%detail_included_min,
526 die $error if $error; #??
528 $cdr_search->adjust(1) if $cdr->freesidestatus eq '';
529 # it was skipped without changing status, so increment the
530 # offset so that we don't re-fetch it on refill
534 #then add details to invoices & get a total
535 $options{'status'} = 'rated';
537 $cdr_search = $svc_x->psearch_cdrs(%options);
538 $cdr_search->limit(1000);
539 $cdr_search->increment(0);
540 while ( my $cdr = $cdr_search->fetch ) {
542 # at this point we officially Do Not Care about the rating method
543 if ( $included_calls > 0 ) {
545 #$charges += 0, obviously
546 #but don't set the rated price to zero--there should be a record
547 $error = $cdr->set_status('no-charge');
550 $charges += $cdr->rated_price;
551 $error = $cdr->set_status('done');
553 die $error if $error;
554 $formatter->append($cdr)
555 unless $cdr->rated_price == 0 and not $usage_showzero;
557 $cdr_search->adjust(1) if $cdr->freesidestatus eq 'rated';
561 $formatter->finish; #writes into $details
562 unshift @$details, $formatter->header if @$details;
564 if ( $included_min_total > 0 ) {
566 my $min_detail = sprintf('%d / %d ',
567 $included_min_total - $included_min_left,
569 ) . $cust_pkg->mt('included minutes used');
570 unshift @$details, $min_detail;
576 #returns a reason why not to rate this CDR, or false if the CDR is chargeable
577 # lots of false laziness w/voip_inbound
578 sub check_chargable {
579 my( $self, $cdr, %flags ) = @_;
581 return 'amaflags != 2'
582 if $self->option_cacheable('use_amaflags') && $cdr->amaflags != 2;
584 return "disposition NOT IN ( ". $self->option_cacheable('disposition_in')." )"
585 if $self->option_cacheable('disposition_in') =~ /\S/
586 && !grep { $cdr->disposition eq $_ } split(/\s*,\s*/, $self->option_cacheable('disposition_in'));
588 my $disposition_prefix = $self->option_cacheable('disposition_prefix');
589 my $len_dis_prefix = length($disposition_prefix);
590 return "disposition does not start with $disposition_prefix"
592 && substr($cdr->disposition, 0, $len_dis_prefix) ne $disposition_prefix;
594 return "disposition IN ( ". $self->option_cacheable('ignore_disposition')." )"
595 if $self->option_cacheable('ignore_disposition') =~ /\S/
596 && grep { $cdr->disposition eq $_ } split(/\s*,\s*/, $self->option_cacheable('ignore_disposition'));
598 foreach(split(/\s*,\s*/, $self->option_cacheable('skip_dst_prefix'))) {
599 return "dst starts with '$_'"
600 if length($_) && substr($cdr->dst,0,length($_)) eq $_;
603 return "carrierid NOT IN ( ". $self->option_cacheable('use_carrierid'). " )"
604 if $self->option_cacheable('use_carrierid') =~ /\S/
605 && ! $flags{'da_rewrote'} #why?
606 && !grep { $cdr->carrierid eq $_ } split(/\s*,\s*/, $self->option_cacheable('use_carrierid')); #eq otherwise 0 matches ''
608 # unlike everything else, use_cdrtypenum is applied in FS::svc_x::get_cdrs.
609 return "cdrtypenum != ". $self->option_cacheable('use_cdrtypenum')
610 if length($self->option_cacheable('use_cdrtypenum'))
611 && $cdr->cdrtypenum ne $self->option_cacheable('use_cdrtypenum'); #ne otherwise 0 matches ''
613 return "cdrtypenum == ". $self->option_cacheable('ignore_cdrtypenum')
614 if length($self->option_cacheable('ignore_cdrtypenum'))
615 && $cdr->cdrtypenum eq $self->option_cacheable('ignore_cdrtypenum'); #eq otherwise 0 matches ''
617 # unlike everything else, use_calltypenum is applied in FS::svc_x::get_cdrs.
618 return "calltypenum != ". $self->option_cacheable('use_calltypenum')
619 if length($self->option_cacheable('use_calltypenum'))
620 && $cdr->calltypenum ne $self->option_cacheable('use_calltypenum'); #ne otherwise 0 matches ''
622 return "calltypenum == ". $self->option_cacheable('ignore_calltypenum')
623 if length($self->option_cacheable('ignore_calltypenum'))
624 && $cdr->calltypenum eq $self->option_cacheable('ignore_calltypenum'); #eq otherwise 0 matches ''
626 return "dcontext IN ( ". $self->option_cacheable('skip_dcontext'). " )"
627 if $self->option_cacheable('skip_dcontext') =~ /\S/
628 && grep { $cdr->dcontext eq $_ } split(/\s*,\s*/, $self->option_cacheable('skip_dcontext'));
630 my $len_dcontext_prefix =
631 length($self->option_cacheable('skip_dcontext_prefix'));
632 return "dcontext starts with ". $self->option_cacheable('skip_dcontext_prefix')
633 if $len_dcontext_prefix
634 && substr($cdr->dcontext,0,$len_dcontext_prefix) eq $self->option_cacheable('skip_dcontext_prefix');
636 my $len_suffix = length($self->option_cacheable('skip_dcontext_suffix'));
637 return "dcontext ends with ". $self->option_cacheable('skip_dcontext_suffix')
639 && substr($cdr->dcontext,-$len_suffix,$len_suffix) eq $self->option_cacheable('skip_dcontext_suffix');
641 my $len_prefix = length($self->option_cacheable('skip_dstchannel_prefix'));
642 return "dstchannel starts with ". $self->option_cacheable('skip_dstchannel_prefix')
644 && substr($cdr->dstchannel,0,$len_prefix) eq $self->option_cacheable('skip_dstchannel_prefix');
646 my $dst_length = $self->option_cacheable('skip_dst_length_less');
647 return "destination less than $dst_length digits"
648 if $dst_length && length($cdr->dst) < $dst_length
649 && ! ( $self->option_cacheable('noskip_dst_length_accountcode_tollfree')
650 && $cdr->is_tollfree('accountcode')
653 return "lastapp is ". $self->option_cacheable('skip_lastapp')
654 if length($self->option_cacheable('skip_lastapp')) && $cdr->lastapp eq $self->option_cacheable('skip_lastapp');
656 my $src_length = $self->option_cacheable('skip_src_length_more');
659 if ( $self->option_cacheable('noskip_src_length_accountcode_tollfree') ) {
661 if ( $cdr->is_tollfree('accountcode') ) {
662 return "source less than or equal to $src_length digits"
663 if length($cdr->src) <= $src_length;
665 return "source more than $src_length digits"
666 if length($cdr->src) > $src_length;
670 return "source more than $src_length digits"
671 if length($cdr->src) > $src_length;
676 return "max_callers <= ". $self->option_cacheable('skip_max_callers')
677 if length($self->option_cacheable('skip_max_callers'))
678 and length($cdr->max_callers)
679 and $cdr->max_callers <= $self->option_cacheable('skip_max_callers');
681 return "calldate < ". $self->option_cacheable('skip_old')
682 if length($self->option_cacheable('skip_old'))
683 && $cdr->calldate_unix < str2time($self->option_cacheable('skip_old'));
685 #all right then, rate it
694 my ($self, $cust_pkg, %opt) = @_;
695 my @part_pkg_usage = $self->part_pkg_usage or return '';
696 warn " resetting usage minutes\n" if $opt{debug};
697 my %cust_pkg_usage = map { $_->pkgusagepart, $_ } $cust_pkg->cust_pkg_usage;
698 foreach my $part_pkg_usage (@part_pkg_usage) {
699 my $part = $part_pkg_usage->pkgusagepart;
700 my $usage = $cust_pkg_usage{$part} ||
701 FS::cust_pkg_usage->new({
702 'pkgnum' => $cust_pkg->pkgnum,
703 'pkgusagepart' => $part,
704 'minutes' => $part_pkg_usage->minutes *
705 ($cust_pkg->quantity || 1),
707 foreach my $cdr_usage (
708 qsearch('cdr_cust_pkg_usage', {'cdrusagenum' => $usage->cdrusagenum})
710 my $error = $cdr_usage->delete;
711 warn " error resetting CDR usage: $error\n";
714 if ( $usage->pkgusagenum ) {
715 if ( $part_pkg_usage->rollover ) {
716 $usage->set('minutes', $part_pkg_usage->minutes + $usage->minutes);
718 $usage->set('minutes', $part_pkg_usage->minutes);
720 my $error = $usage->replace;
721 warn " error resetting usage minutes: $error\n" if $error;
723 my $error = $usage->insert;
724 warn " error resetting usage minutes: $error\n" if $error;
726 } #foreach $part_pkg_usage
729 # tells whether cust_bill_pkg_detail should return a single line for
731 # i think this is currently unused?
734 $self->option('output_format') =~ /^sum_/;
737 # and whether cust_bill should show a detail line for the service label
738 # (separate from usage details)
739 sub hide_svc_detail {
741 $self->option('output_format') =~ /^sum_/;