freeside-upgrade - fix uninitalized value warnings
[freeside.git] / FS / FS / Upgrade.pm
1 package FS::Upgrade;
2
3 use strict;
4 use vars qw( @ISA @EXPORT_OK $DEBUG );
5 use Exporter;
6 use Tie::IxHash;
7 use File::Slurp;
8 use FS::UID qw( dbh driver_name );
9 use FS::Conf;
10 use FS::Record qw(qsearchs qsearch str2time_sql);
11 use FS::queue;
12 use FS::upgrade_journal;
13 use FS::Setup qw( enable_banned_pay_pad );
14
15 use FS::svc_domain;
16 $FS::svc_domain::whois_hack = 1;
17
18 @ISA = qw( Exporter );
19 @EXPORT_OK = qw( upgrade_schema upgrade_config upgrade upgrade_sqlradius );
20
21 $DEBUG = 1;
22
23 =head1 NAME
24
25 FS::Upgrade - Database upgrade routines
26
27 =head1 SYNOPSIS
28
29   use FS::Upgrade;
30
31 =head1 DESCRIPTION
32
33 Currently this module simply provides a place to store common subroutines for
34 database upgrades.
35
36 =head1 SUBROUTINES
37
38 =over 4
39
40 =item upgrade_config
41
42 =cut
43
44 #config upgrades
45 sub upgrade_config {
46   my %opt = @_;
47
48   my $conf = new FS::Conf;
49
50   # to simplify tokenization upgrades
51   die "Conf selfservice-payment_gateway no longer supported"
52     if $conf->config('selfservice-payment_gateway');
53
54   $conf->touch('payment_receipt')
55     if $conf->exists('payment_receipt_email')
56     || $conf->config('payment_receipt_msgnum');
57
58   $conf->touch('geocode-require_nw_coordinates')
59     if $conf->exists('svc_broadband-require-nw-coordinates');
60
61   unless ( $conf->config('echeck-country') ) {
62     if ( $conf->exists('cust_main-require-bank-branch') ) {
63       $conf->set('echeck-country', 'CA');
64     } elsif ( $conf->exists('echeck-nonus') ) {
65       $conf->set('echeck-country', 'XX');
66     } else {
67       $conf->set('echeck-country', 'US');
68     }
69   }
70
71   my @agents = qsearch('agent', {});
72
73   upgrade_overlimit_groups($conf);
74   map { upgrade_overlimit_groups($conf,$_->agentnum) } @agents;
75
76   upgrade_invoice_from($conf);
77   foreach my $agent (@agents) {
78     upgrade_invoice_from($conf,$agent->agentnum,1);
79   }
80
81   my $DIST_CONF = '/usr/local/etc/freeside/default_conf/';#DIST_CONF in Makefile
82   $conf->set($_, scalar(read_file( "$DIST_CONF/$_" )) )
83     foreach grep { ! $conf->exists($_) && -s "$DIST_CONF/$_" }
84       qw( quotation_html quotation_latex quotation_latexnotes );
85
86   # change 'fslongtable' to 'longtable'
87   # in invoice and quotation main templates, and also in all secondary 
88   # invoice templates
89   my @latex_confs =
90     qsearch('conf', { 'name' => {op=>'LIKE', value=>'%latex%'} });
91
92   foreach my $c (@latex_confs) {
93     my $value = $c->value;
94     if (length($value) and $value =~ /fslongtable/) {
95       $value =~ s/fslongtable/longtable/g;
96       $conf->set($c->name, $value, $c->agentnum);
97     }
98   }
99
100   # if there's a USPS tools login, assume that's the standardization method
101   # you want to use
102   $conf->set('address_standardize_method', 'usps')
103     if $conf->exists('usps_webtools-userid')
104     && length($conf->config('usps_webtools-userid')) > 0
105     && ! $conf->exists('address_standardize_method');
106
107   # this option has been renamed/expanded
108   if ( $conf->exists('cust_main-enable_spouse_birthdate') ) {
109     $conf->touch('cust_main-enable_spouse');
110     $conf->delete('cust_main-enable_spouse_birthdate');
111   }
112
113   # renamed/repurposed
114   if ( $conf->exists('cust_pkg-show_fcc_voice_grade_equivalent') ) {
115     $conf->touch('part_pkg-show_fcc_options');
116     $conf->delete('cust_pkg-show_fcc_voice_grade_equivalent');
117     warn "
118 You have FCC Form 477 package options enabled.
119
120 Starting with the October 2014 filing date, the FCC has redesigned 
121 Form 477 and introduced new service categories.  See bin/convert-477-options
122 to update your package configuration for the new report.
123
124 If you need to continue using the old Form 477 report, turn on the
125 'old_fcc_report' configuration option.
126 ";
127   }
128
129   # boolean invoice_sections_by_location option is now
130   # invoice_sections_method = 'location'
131   my @invoice_sections_confs =
132     qsearch('conf', { 'name' => { op=>'LIKE', value=>'%sections_by_location' } });
133   foreach my $c (@invoice_sections_confs) {
134     $c->name =~ /^(\w+)sections_by_location$/;
135     $conf->delete($c->name);
136     my $newname = $1.'sections_method';
137     $conf->set($newname, 'location');
138   }
139
140   # boolean enable_taxproducts is now tax_data_vendor = 'cch'
141   if ( $conf->exists('enable_taxproducts') ) {
142
143     $conf->delete('enable_taxproducts');
144     $conf->set('tax_data_vendor', 'cch');
145
146   }
147
148   # boolean tax-cust_exempt-groups-require_individual_nums is now -num_req all
149   if ( $conf->exists('tax-cust_exempt-groups-require_individual_nums') ) {
150     $conf->set('tax-cust_exempt-groups-num_req', 'all');
151     $conf->delete('tax-cust_exempt-groups-require_individual_nums');
152   }
153
154   # boolean+text previous_balance-exclude_from_total is now two separate options
155   my $total_new_charges = $conf->config('previous_balance-exclude_from_total');
156   if ( defined $total_new_charges && length($total_new_charges) > 0 ) {
157     $conf->set('previous_balance-text-total_new_charges', $total_new_charges);
158     $conf->set('previous_balance-exclude_from_total', '');
159   }
160
161   # switch from specifying an email address to boolean check
162   if ( $conf->exists('batch-errors_to') ) {
163     $conf->touch('batch-errors_not_fatal');
164     $conf->delete('batch-errors_to');
165   }
166
167   if ( $conf->exists('voip-cust_email_csv_cdr') ) {
168     $conf->set('voip_cdr_email_attach', 'csv');
169     $conf->delete('voip-cust_email_csv_cdr') ;
170   }
171
172   if ($conf->exists('unsuspendauto') && !$conf->config('unsuspend_balance')) {
173     $conf->set('unsuspend_balance','Zero');
174     $conf->delete('unsuspendauto');
175   }
176
177   my $cust_fields = $conf->config('cust-fields');
178   if ( defined $cust_fields && $cust_fields =~ / \| Payment Type/ ) {
179     # so we can potentially use 'Payment Types' or somesuch in the future
180     $cust_fields =~ s/ \| Payment Type( \|)/$1/;
181     $cust_fields =~ s/ \| Payment Type$//;
182     $conf->set('cust-fields',$cust_fields);
183   }
184
185   enable_banned_pay_pad() unless length($conf->config('banned_pay-pad'));
186
187   # if translate-auto-insert is enabled for a locale, ensure that invoice
188   # terms are in the msgcat (is there a better place for this?)
189   if (my $auto_locale = $conf->config('translate-auto-insert')) {
190     my $lh = FS::L10N->get_handle($auto_locale);
191     foreach (@FS::Conf::invoice_terms) {
192       $lh->maketext($_) if length($_);
193     }
194   }
195
196   unless ( FS::upgrade_journal->is_done('deprecate_unmask_ss') ) {
197     if ( $conf->config_bool( 'unmask_ss' )) {
198       warn "'unmask_ssn' deprecated from global configuration\n";
199       for my $access_group ( qsearch( access_group => {} )) {
200         $access_group->grant_access_right( 'Unmask customer SSN' );
201         warn " - 'Unmask customer SSN' access right granted to '" .
202              $access_group->groupname . "' employee group\n";
203       }
204     }
205     FS::upgrade_journal->set_done('deprecate_unmask_ss');
206   }
207
208 }
209
210 sub upgrade_overlimit_groups {
211     my $conf = shift;
212     my $agentnum = shift;
213     my @groups = $conf->config('overlimit_groups',$agentnum); 
214     if(scalar(@groups)) {
215         my $groups = join(',',@groups);
216         my @groupnums;
217         my $error = '';
218         if ( $groups !~ /^[\d,]+$/ ) {
219             foreach my $groupname ( @groups ) {
220                 my $g = qsearchs('radius_group', { 'groupname' => $groupname } );
221                 unless ( $g ) {
222                     $g = new FS::radius_group {
223                                     'groupname' => $groupname,
224                                     'description' => $groupname,
225                                     };
226                     $error = $g->insert;
227                     die $error if $error;
228                 }
229                 push @groupnums, $g->groupnum;
230             }
231             $conf->set('overlimit_groups',join("\n",@groupnums),$agentnum);
232         }
233     }
234 }
235
236 sub upgrade_invoice_from {
237   my ($conf, $agentnum, $agentonly) = @_;
238   if (
239           ! $conf->exists('invoice_from_name',$agentnum,$agentonly)
240        && $conf->exists('invoice_from',$agentnum,$agentonly)
241        && $conf->config('invoice_from',$agentnum,$agentonly) =~ /\<(.*)\>/
242   ) {
243     my $realemail = $1;
244     $realemail =~ s/^\s*//; # remove leading spaces
245     $realemail =~ s/\s*$//; # remove trailing spaces
246     my $realname = $conf->config('invoice_from',$agentnum);
247     $realname =~ s/\<.*\>//; # remove email address
248     $realname =~ s/^\s*//; # remove leading spaces
249     $realname =~ s/\s*$//; # remove trailing spaces
250     # properly quote names that contain punctuation
251     if (($realname =~ /[^[:alnum:][:space:]]/) && ($realname !~ /^\".*\"$/)) {
252       $realname = '"' . $realname . '"';
253     }
254     $conf->set('invoice_from_name', $realname, $agentnum);
255     $conf->set('invoice_from', $realemail, $agentnum);
256   }
257 }
258
259 =item upgrade
260
261 =cut
262
263 sub upgrade {
264   my %opt = @_;
265
266   my $data = upgrade_data(%opt);
267
268   my $oldAutoCommit = $FS::UID::AutoCommit;
269   local $FS::UID::AutoCommit = 0;
270   local $FS::UID::AutoCommit = 0;
271
272   local $FS::cust_pkg::upgrade = 1; #go away after setup+start dates cleaned up for old customers
273
274
275   foreach my $table ( keys %$data ) {
276
277     my $class = "FS::$table";
278     eval "use $class;";
279     die $@ if $@;
280
281     if ( $class->can('_upgrade_data') ) {
282       warn "Upgrading $table...\n";
283
284       my $start = time;
285
286       $class->_upgrade_data(%opt);
287
288       # New interface for async upgrades: a class can declare a 
289       # "queueable_upgrade" method, which will run as part of the normal 
290       # upgrade, but if the -j option is passed, will instead be run from 
291       # the job queue.
292       if ( $class->can('queueable_upgrade') ) {
293         my $jobname = $class . '::queueable_upgrade';
294         my $num_jobs = FS::queue->count("job = '$jobname' and status != 'failed'");
295         if ($num_jobs > 0) {
296           warn "$class upgrade already scheduled.\n";
297         } else {
298           if ( $opt{'queue'} ) {
299             warn "Scheduling $class upgrade.\n";
300             my $job = FS::queue->new({ job => $jobname });
301             $job->insert($class, %opt);
302           } else {
303             $class->queueable_upgrade(%opt);
304           }
305         } #$num_jobs == 0
306       }
307
308       if ( $oldAutoCommit ) {
309         warn "  committing\n";
310         dbh->commit or die dbh->errstr;
311       }
312       
313       #warn "\e[1K\rUpgrading $table... done in ". (time-$start). " seconds\n";
314       warn "  done in ". (time-$start). " seconds\n";
315
316     } else {
317       warn "WARNING: asked for upgrade of $table,".
318            " but FS::$table has no _upgrade_data method\n";
319     }
320
321 #    my @records = @{ $data->{$table} };
322 #
323 #    foreach my $record ( @records ) {
324 #      my $args = delete($record->{'_upgrade_args'}) || [];
325 #      my $object = $class->new( $record );
326 #      my $error = $object->insert( @$args );
327 #      die "error inserting record into $table: $error\n"
328 #        if $error;
329 #    }
330
331   }
332
333   local($FS::cust_main::ignore_expired_card) = 1;
334   #this is long-gone... would need to set an equivalent in cust_location #local($FS::cust_main::ignore_illegal_zip) = 1;
335   local($FS::cust_main::ignore_banned_card) = 1;
336   local($FS::cust_main::skip_fuzzyfiles) = 1;
337
338   local($FS::cust_payby::ignore_expired_card) = 1;
339   local($FS::cust_payby::ignore_banned_card) = 1;
340
341   # decrypt inadvertantly-encrypted payinfo where payby != CARD,DCRD,CHEK,DCHK
342   # kind of a weird spot for this, but it's better than duplicating
343   # all this code in each class...
344   my @decrypt_tables = qw( cust_main cust_pay_void cust_pay cust_refund cust_pay_pending );
345   foreach my $table ( @decrypt_tables ) {
346       my @objects = qsearch({
347         'table'     => $table,
348         'hashref'   => {},
349         'extra_sql' => "WHERE payby NOT IN ( 'CARD', 'DCRD', 'CHEK', 'DCHK' ) ".
350                        " AND LENGTH(payinfo) > 100",
351       });
352       foreach my $object ( @objects ) {
353           my $payinfo = $object->decrypt($object->payinfo);
354           if ( $payinfo eq $object->payinfo ) {
355             warn "error decrypting payinfo for $table: $payinfo\n";
356             next;
357           }
358           $object->payinfo($payinfo);
359           my $error = $object->replace;
360           die $error if $error;
361       }
362   }
363
364 }
365
366 =item upgrade_data
367
368 =cut
369
370 sub upgrade_data {
371   my %opt = @_;
372
373   tie my %hash, 'Tie::IxHash', 
374
375     #remap log levels
376         'log' => [],
377
378     #payby conditions to new ones
379     'part_event_condition' => [],
380
381     #payby actions to new ones
382     'part_event' => [],
383
384     #fix whitespace - before cust_main
385     'cust_location' => [],
386
387     # need before cust_main tokenization upgrade,
388     # blocks tokenization upgrade if deprecated features still in use
389     'agent_payment_gateway' => [],
390
391     #cust_main (tokenizes cards, remove paycvv from history, locations, cust_payby, etc)
392     # (handles payinfo encryption/tokenization across all relevant tables)
393     'cust_main' => [],
394
395     #contact -> cust_contact / prospect_contact
396     'contact' => [],
397
398     #msgcat
399     'msgcat' => [],
400
401     #reason type and reasons
402     'reason_type'     => [],
403     'cust_pkg_reason' => [],
404
405     #need part_pkg before cust_credit...
406     'part_pkg' => [],
407
408     #customer credits
409     'cust_credit' => [],
410
411     # reason / void_reason migration to reasonnum / void_reasonnum
412     'cust_credit_void' => [],
413     'cust_bill_void' => [],
414     # also fix some tax allocation records
415     'cust_bill_pkg_void' => [],
416
417     #duplicate history records
418     'h_cust_svc'  => [],
419
420     #populate cust_pay.otaker
421     'cust_pay'    => [],
422
423     #populate part_pkg_taxclass for starters
424     'part_pkg_taxclass' => [],
425
426     #remove bad pending records
427     'cust_pay_pending' => [],
428
429     #replace invnum and pkgnum with billpkgnum
430     'cust_bill_pkg_detail' => [],
431
432     #usage_classes if we have none
433     'usage_class' => [],
434
435     #phone_type if we have none
436     'phone_type' => [],
437
438     #fixup access rights
439     'access_right' => [],
440
441     #change recur_flat and enable_prorate
442     'part_pkg_option' => [],
443
444     #add weights to pkg_category
445     'pkg_category' => [],
446
447     #cdrbatch fixes
448     'cdr' => [],
449
450     #otaker->usernum
451     'cust_attachment' => [],
452     #'cust_credit' => [],
453     #'cust_main' => [],
454     'cust_main_note' => [],
455     #'cust_pay' => [],
456     'cust_pay_void' => [],
457     'cust_pkg' => [],
458     #'cust_pkg_reason' => [],
459     'cust_pkg_discount' => [],
460     'cust_refund' => [],
461     'banned_pay' => [],
462
463     #paycardtype
464     'cust_payby' => [],
465
466     #default namespace
467     'payment_gateway' => [],
468
469     #migrate to templates
470     'msg_template' => [],
471
472     #return unprovisioned numbers to availability
473     'phone_avail' => [],
474
475     #insert scripcondition
476     'TicketSystem' => [],
477     
478     #insert LATA data if not already present
479     'lata' => [],
480     
481     #insert MSA data if not already present
482     'msa' => [],
483
484     # migrate to radius_group and groupnum instead of groupname
485     'radius_usergroup' => [],
486     'part_svc'         => [],
487     'part_export'      => [],
488
489     #insert default tower_sector if not present
490     'tower' => [],
491
492     #repair improperly deleted services
493     'cust_svc' => [],
494
495     #routernum/blocknum
496     'svc_broadband' => [],
497
498     #set up payment gateways if needed
499     'pay_batch' => [],
500
501     #flag monthly tax exemptions
502     'cust_tax_exempt_pkg' => [],
503
504     #kick off tax location history upgrade
505     'cust_bill_pkg' => [],
506
507     #fix taxable line item links
508     'cust_bill_pkg_tax_location' => [],
509
510     #populate state FIPS codes if not already done
511     'state' => [],
512
513     #set default locations on quoted packages
514     'quotation_pkg' => [],
515
516     #populate tax statuses
517     'tax_status' => [],
518
519     #mark certain taxes as system-maintained,
520     # and fix whitespace
521     'cust_main_county' => [],
522
523     #'compliance solutions' -> 'compliance_solutions'
524     'tax_rate' => [],
525     'tax_rate_location' => [],
526
527     #upgrade part_event_condition_option agentnum to a multiple hash value
528     'part_event_condition_option' =>[],
529   ;
530
531   \%hash;
532
533 }
534
535 =item upgrade_schema
536
537 =cut
538
539 sub upgrade_schema {
540   my %opt = @_;
541
542   my $data = upgrade_schema_data(%opt);
543
544   my $oldAutoCommit = $FS::UID::AutoCommit;
545   local $FS::UID::AutoCommit = 0;
546   local $FS::UID::AutoCommit = 0;
547
548   foreach my $table ( keys %$data ) {
549
550     my $class = "FS::$table";
551     eval "use $class;";
552     die $@ if $@;
553
554     if ( $class->can('_upgrade_schema') ) {
555       warn "Upgrading $table schema...\n";
556
557       my $start = time;
558
559       $class->_upgrade_schema(%opt);
560
561       if ( $oldAutoCommit ) {
562         warn "  committing\n";
563         dbh->commit or die dbh->errstr;
564       }
565       
566       #warn "\e[1K\rUpgrading $table... done in ". (time-$start). " seconds\n";
567       warn "  done in ". (time-$start). " seconds\n";
568
569     } else {
570       warn "WARNING: asked for schema upgrade of $table,".
571            " but FS::$table has no _upgrade_schema method\n";
572     }
573
574   }
575
576 }
577
578 =item upgrade_schema_data
579
580 =cut
581
582 sub upgrade_schema_data {
583   my %opt = @_;
584
585   #auto-find tables/classes with an _update_schema method?
586
587   tie my %hash, 'Tie::IxHash', 
588
589     #fix classnum character(1)
590     'cust_bill_pkg_detail' => [],
591     #add necessary columns to RT schema
592     'TicketSystem' => [],
593     #remove h_access_user_log if it exists (since our regular auto schema
594     # upgrade doesn't have the drop tables flag turned on) 
595     'access_user_log' => [],
596     #remove possible dangling records
597     'password_history' => [],
598     'cust_pay_pending' => [],
599     #remove records referencing removed things with their FKs
600     'pkg_referral' => [],
601     'cust_bill_pkg_discount' => [],
602     'cust_msg' => [],
603     'cust_bill_pay_batch' => [],
604     'cust_event_fee' => [],
605     'radius_attr' => [],
606     'queue_depend' => [],
607     'cust_main_invoice' => [],
608     #update records referencing removed things with their FKs
609     'cust_pkg' => [],
610   ;
611
612   \%hash;
613
614 }
615
616 sub upgrade_sqlradius {
617   #my %opt = @_;
618
619   my $conf = new FS::Conf;
620
621   my @part_export = FS::part_export::sqlradius->all_sqlradius_withaccounting();
622
623   foreach my $part_export ( @part_export ) {
624
625     my $errmsg = 'Error adding FreesideStatus to '.
626                  $part_export->option('datasrc'). ': ';
627
628     my $dbh = DBI->connect(
629       ( map $part_export->option($_), qw ( datasrc username password ) ),
630       { PrintError => 0, PrintWarn => 0 }
631     ) or do {
632       warn $errmsg.$DBI::errstr;
633       next;
634     };
635
636     my $str2time = str2time_sql( $dbh->{Driver}->{Name} );
637     my $group = "UserName";
638     $group .= ",Realm"
639       if ref($part_export) =~ /withdomain/
640       || $dbh->{Driver}->{Name} =~ /^Pg/; #hmm
641
642     my $sth_alter = $dbh->prepare(
643       "ALTER TABLE radacct ADD COLUMN FreesideStatus varchar(32) NULL"
644     );
645     if ( $sth_alter ) {
646       if ( $sth_alter->execute ) {
647         my $sth_update = $dbh->prepare(
648          "UPDATE radacct SET FreesideStatus = 'done' WHERE FreesideStatus IS NULL"
649         ) or die $errmsg.$dbh->errstr;
650         $sth_update->execute or die $errmsg.$sth_update->errstr;
651       } else {
652         my $error = $sth_alter->errstr;
653         warn $errmsg.$error
654           unless $error =~ /Duplicate column name/i  #mysql
655               || $error =~ /already exists/i;        #Pg
656 ;
657       }
658     } else {
659       my $error = $dbh->errstr;
660       warn $errmsg.$error; #unless $error =~ /exists/i;
661     }
662
663     my $sth_index = $dbh->prepare(
664       "CREATE INDEX FreesideStatus ON radacct ( FreesideStatus )"
665     );
666     if ( $sth_index ) {
667       unless ( $sth_index->execute ) {
668         my $error = $sth_index->errstr;
669         warn $errmsg.$error
670           unless $error =~ /Duplicate key name/i #mysql
671               || $error =~ /already exists/i;    #Pg
672       }
673     } else {
674       my $error = $dbh->errstr;
675       warn $errmsg.$error. ' (preparing statement)';#unless $error =~ /exists/i;
676     }
677
678     my $times = ($dbh->{Driver}->{Name} =~ /^mysql/)
679       ? ' AcctStartTime != 0 AND AcctStopTime != 0 '
680       : ' AcctStartTime IS NOT NULL AND AcctStopTime IS NOT NULL ';
681
682     my $sth = $dbh->prepare("SELECT UserName,
683                                     Realm,
684                                     $str2time max(AcctStartTime)),
685                                     $str2time max(AcctStopTime))
686                               FROM radacct
687                               WHERE FreesideStatus = 'done'
688                                 AND $times
689                               GROUP BY $group
690                             ")
691       or die $errmsg.$dbh->errstr;
692     $sth->execute() or die $errmsg.$sth->errstr;
693   
694     while (my $row = $sth->fetchrow_arrayref ) {
695       my ($username, $realm, $start, $stop) = @$row;
696   
697       $username = lc($username) unless $conf->exists('username-uppercase');
698
699       my $exportnum = $part_export->exportnum;
700       my $extra_sql = " AND exportnum = $exportnum ".
701                       " AND exportsvcnum IS NOT NULL ";
702
703       if ( ref($part_export) =~ /withdomain/ ) {
704         $extra_sql = " AND '$realm' = ( SELECT domain FROM svc_domain
705                          WHERE svc_domain.svcnum = svc_acct.domsvc ) ";
706       }
707   
708       my $svc_acct = qsearchs({
709         'select'    => 'svc_acct.*',
710         'table'     => 'svc_acct',
711         'addl_from' => 'LEFT JOIN cust_svc   USING ( svcnum )'.
712                        'LEFT JOIN export_svc USING ( svcpart )',
713         'hashref'   => { 'username' => $username },
714         'extra_sql' => $extra_sql,
715       });
716
717       if ($svc_acct) {
718         $svc_acct->last_login($start)
719           if $start && (!$svc_acct->last_login || $start > $svc_acct->last_login);
720         $svc_acct->last_logout($stop)
721           if $stop && (!$svc_acct->last_logout || $stop > $svc_acct->last_logout);
722       }
723     }
724   }
725
726 }
727
728 =back
729
730 =head1 BUGS
731
732 Sure.
733
734 =head1 SEE ALSO
735
736 =cut
737
738 1;
739