fix upgrade: clear bad cust_refund.source_paynum before encrypting, RT#81508
[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_payby 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     #fix whitespace - before cust_main
379     'cust_location' => [],
380
381     # need before cust_main tokenization upgrade,
382     # blocks tokenization upgrade if deprecated features still in use
383     'agent_payment_gateway' => [],
384
385     #remove bad source_paynum before cust_main
386     'cust_refund' => [],
387
388     #cust_main (tokenizes cards, remove paycvv from history, locations, cust_payby, etc)
389     # (handles payinfo encryption/tokenization across all relevant tables)
390     'cust_main' => [],
391
392     #contact -> cust_contact / prospect_contact
393     'contact' => [],
394
395     #msgcat
396     'msgcat' => [],
397
398     #reason type and reasons
399     'reason_type'     => [],
400     'cust_pkg_reason' => [],
401
402     #need part_pkg before cust_credit...
403     'part_pkg' => [],
404
405     #customer credits
406     'cust_credit' => [],
407
408     # reason / void_reason migration to reasonnum / void_reasonnum
409     'cust_credit_void' => [],
410     'cust_bill_void' => [],
411     # also fix some tax allocation records
412     'cust_bill_pkg_void' => [],
413
414     #duplicate history records
415     'h_cust_svc'  => [],
416
417     #populate cust_pay.otaker
418     'cust_pay'    => [],
419
420     #populate part_pkg_taxclass for starters
421     'part_pkg_taxclass' => [],
422
423     #remove bad pending records
424     'cust_pay_pending' => [],
425
426     #replace invnum and pkgnum with billpkgnum
427     'cust_bill_pkg_detail' => [],
428
429     #usage_classes if we have none
430     'usage_class' => [],
431
432     #phone_type if we have none
433     'phone_type' => [],
434
435     #fixup access rights
436     'access_right' => [],
437
438     #change recur_flat and enable_prorate
439     'part_pkg_option' => [],
440
441     #add weights to pkg_category
442     'pkg_category' => [],
443
444     #cdrbatch fixes
445     'cdr' => [],
446
447     #otaker->usernum
448     'cust_attachment' => [],
449     #'cust_credit' => [],
450     #'cust_main' => [],
451     'cust_main_note' => [],
452     #'cust_pay' => [],
453     'cust_pay_void' => [],
454     'cust_pkg' => [],
455     #'cust_pkg_reason' => [],
456     'cust_pkg_discount' => [],
457     #'cust_refund' => [],
458     'banned_pay' => [],
459
460     #paycardtype
461     'cust_payby' => [],
462
463     #default namespace
464     'payment_gateway' => [],
465
466     #migrate to templates
467     'msg_template' => [],
468
469     #return unprovisioned numbers to availability
470     'phone_avail' => [],
471
472     #insert scripcondition
473     'TicketSystem' => [],
474     
475     #insert LATA data if not already present
476     'lata' => [],
477     
478     #insert MSA data if not already present
479     'msa' => [],
480
481     # migrate to radius_group and groupnum instead of groupname
482     'radius_usergroup' => [],
483     'part_svc'         => [],
484     'part_export'      => [],
485
486     #insert default tower_sector if not present
487     'tower' => [],
488
489     #repair improperly deleted services
490     'cust_svc' => [],
491
492     #routernum/blocknum
493     'svc_broadband' => [],
494
495     #set up payment gateways if needed
496     'pay_batch' => [],
497
498     #flag monthly tax exemptions
499     'cust_tax_exempt_pkg' => [],
500
501     #kick off tax location history upgrade
502     'cust_bill_pkg' => [],
503
504     #fix taxable line item links
505     'cust_bill_pkg_tax_location' => [],
506
507     #populate state FIPS codes if not already done
508     'state' => [],
509
510     #set default locations on quoted packages
511     'quotation_pkg' => [],
512
513     #populate tax statuses
514     'tax_status' => [],
515
516     #mark certain taxes as system-maintained,
517     # and fix whitespace
518     'cust_main_county' => [],
519
520     #'compliance solutions' -> 'compliance_solutions'
521     'tax_rate' => [],
522     'tax_rate_location' => [],
523
524     #upgrade part_event_condition_option agentnum to a multiple hash value
525     'part_event_condition_option' =>[],
526
527     #fix ip format
528     'svc_circuit' => [],
529
530     #fix ip format
531     'svc_hardware' => [],
532
533     #fix ip format
534     'svc_pbx' => [],
535
536     #fix ip format
537     'tower_sector' => [],
538
539
540   ;
541
542   \%hash;
543
544 }
545
546 =item upgrade_schema
547
548 =cut
549
550 sub upgrade_schema {
551   my %opt = @_;
552
553   my $data = upgrade_schema_data(%opt);
554
555   my $oldAutoCommit = $FS::UID::AutoCommit;
556   local $FS::UID::AutoCommit = 0;
557   local $FS::UID::AutoCommit = 0;
558
559   foreach my $table ( keys %$data ) {
560
561     my $class = "FS::$table";
562     eval "use $class;";
563     die $@ if $@;
564
565     if ( $class->can('_upgrade_schema') ) {
566       warn "Upgrading $table schema...\n";
567
568       my $start = time;
569
570       $class->_upgrade_schema(%opt);
571
572       if ( $oldAutoCommit ) {
573         warn "  committing\n";
574         dbh->commit or die dbh->errstr;
575       }
576       
577       #warn "\e[1K\rUpgrading $table... done in ". (time-$start). " seconds\n";
578       warn "  done in ". (time-$start). " seconds\n";
579
580     } else {
581       warn "WARNING: asked for schema upgrade of $table,".
582            " but FS::$table has no _upgrade_schema method\n";
583     }
584
585   }
586
587 }
588
589 =item upgrade_schema_data
590
591 =cut
592
593 sub upgrade_schema_data {
594   my %opt = @_;
595
596   #auto-find tables/classes with an _update_schema method?
597
598   tie my %hash, 'Tie::IxHash', 
599
600     #fix classnum character(1)
601     'cust_bill_pkg_detail' => [],
602     #add necessary columns to RT schema
603     'TicketSystem' => [],
604     #remove h_access_user_log if it exists (since our regular auto schema
605     # upgrade doesn't have the drop tables flag turned on) 
606     'access_user_log' => [],
607     #remove possible dangling records
608     'password_history' => [],
609     'cust_pay_pending' => [],
610     #remove records referencing removed things with their FKs
611     'pkg_referral' => [],
612     'cust_bill_pkg_discount' => [],
613     'cust_msg' => [],
614     'cust_bill_pay_batch' => [],
615     'cust_event_fee' => [],
616     'radius_attr' => [],
617     'queue_depend' => [],
618     'cust_main_invoice' => [],
619     #update records referencing removed things with their FKs
620     'cust_pkg' => [],
621   ;
622
623   \%hash;
624
625 }
626
627 sub upgrade_sqlradius {
628   #my %opt = @_;
629
630   my $conf = new FS::Conf;
631
632   my @part_export = FS::part_export::sqlradius->all_sqlradius_withaccounting();
633
634   foreach my $part_export ( @part_export ) {
635
636     my $errmsg = 'Error adding FreesideStatus to '.
637                  $part_export->option('datasrc'). ': ';
638
639     my $dbh = DBI->connect(
640       ( map $part_export->option($_), qw ( datasrc username password ) ),
641       { PrintError => 0, PrintWarn => 0 }
642     ) or do {
643       warn $errmsg.$DBI::errstr;
644       next;
645     };
646
647     my $str2time = str2time_sql( $dbh->{Driver}->{Name} );
648     my $group = "UserName";
649     $group .= ",Realm"
650       if ref($part_export) =~ /withdomain/
651       || $dbh->{Driver}->{Name} =~ /^Pg/; #hmm
652
653     my $sth_alter = $dbh->prepare(
654       "ALTER TABLE radacct ADD COLUMN FreesideStatus varchar(32) NULL"
655     );
656     if ( $sth_alter ) {
657       if ( $sth_alter->execute ) {
658         my $sth_update = $dbh->prepare(
659          "UPDATE radacct SET FreesideStatus = 'done' WHERE FreesideStatus IS NULL"
660         ) or die $errmsg.$dbh->errstr;
661         $sth_update->execute or die $errmsg.$sth_update->errstr;
662       } else {
663         my $error = $sth_alter->errstr;
664         warn $errmsg.$error
665           unless $error =~ /Duplicate column name/i  #mysql
666               || $error =~ /already exists/i;        #Pg
667 ;
668       }
669     } else {
670       my $error = $dbh->errstr;
671       warn $errmsg.$error; #unless $error =~ /exists/i;
672     }
673
674     my $sth_index = $dbh->prepare(
675       "CREATE INDEX FreesideStatus ON radacct ( FreesideStatus )"
676     );
677     if ( $sth_index ) {
678       unless ( $sth_index->execute ) {
679         my $error = $sth_index->errstr;
680         warn $errmsg.$error
681           unless $error =~ /Duplicate key name/i #mysql
682               || $error =~ /already exists/i;    #Pg
683       }
684     } else {
685       my $error = $dbh->errstr;
686       warn $errmsg.$error. ' (preparing statement)';#unless $error =~ /exists/i;
687     }
688
689     my $times = ($dbh->{Driver}->{Name} =~ /^mysql/)
690       ? ' AcctStartTime != 0 AND AcctStopTime != 0 '
691       : ' AcctStartTime IS NOT NULL AND AcctStopTime IS NOT NULL ';
692
693     my $sth = $dbh->prepare("SELECT UserName,
694                                     Realm,
695                                     $str2time max(AcctStartTime)),
696                                     $str2time max(AcctStopTime))
697                               FROM radacct
698                               WHERE FreesideStatus = 'done'
699                                 AND $times
700                               GROUP BY $group
701                             ")
702       or die $errmsg.$dbh->errstr;
703     $sth->execute() or die $errmsg.$sth->errstr;
704   
705     while (my $row = $sth->fetchrow_arrayref ) {
706       my ($username, $realm, $start, $stop) = @$row;
707   
708       $username = lc($username) unless $conf->exists('username-uppercase');
709
710       my $exportnum = $part_export->exportnum;
711       my $extra_sql = " AND exportnum = $exportnum ".
712                       " AND exportsvcnum IS NOT NULL ";
713
714       if ( ref($part_export) =~ /withdomain/ ) {
715         $extra_sql = " AND '$realm' = ( SELECT domain FROM svc_domain
716                          WHERE svc_domain.svcnum = svc_acct.domsvc ) ";
717       }
718   
719       my $svc_acct = qsearchs({
720         'select'    => 'svc_acct.*',
721         'table'     => 'svc_acct',
722         'addl_from' => 'LEFT JOIN cust_svc   USING ( svcnum )'.
723                        'LEFT JOIN export_svc USING ( svcpart )',
724         'hashref'   => { 'username' => $username },
725         'extra_sql' => $extra_sql,
726       });
727
728       if ($svc_acct) {
729         $svc_acct->last_login($start)
730           if $start && (!$svc_acct->last_login || $start > $svc_acct->last_login);
731         $svc_acct->last_logout($stop)
732           if $stop && (!$svc_acct->last_logout || $stop > $svc_acct->last_logout);
733       }
734     }
735   }
736
737 }
738
739 =back
740
741 =head1 BUGS
742
743 Sure.
744
745 =head1 SEE ALSO
746
747 =cut
748
749 1;