RT# 30783 Improve speed of ip address auto-assignment
[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     #fix ip format
531     'svc_circuit' => [],
532
533     #fix ip format
534     'svc_hardware' => [],
535
536     #fix ip format
537     'svc_pbx' => [],
538
539     #fix ip format
540     'tower_sector' => [],
541
542
543   ;
544
545   \%hash;
546
547 }
548
549 =item upgrade_schema
550
551 =cut
552
553 sub upgrade_schema {
554   my %opt = @_;
555
556   my $data = upgrade_schema_data(%opt);
557
558   my $oldAutoCommit = $FS::UID::AutoCommit;
559   local $FS::UID::AutoCommit = 0;
560   local $FS::UID::AutoCommit = 0;
561
562   foreach my $table ( keys %$data ) {
563
564     my $class = "FS::$table";
565     eval "use $class;";
566     die $@ if $@;
567
568     if ( $class->can('_upgrade_schema') ) {
569       warn "Upgrading $table schema...\n";
570
571       my $start = time;
572
573       $class->_upgrade_schema(%opt);
574
575       if ( $oldAutoCommit ) {
576         warn "  committing\n";
577         dbh->commit or die dbh->errstr;
578       }
579       
580       #warn "\e[1K\rUpgrading $table... done in ". (time-$start). " seconds\n";
581       warn "  done in ". (time-$start). " seconds\n";
582
583     } else {
584       warn "WARNING: asked for schema upgrade of $table,".
585            " but FS::$table has no _upgrade_schema method\n";
586     }
587
588   }
589
590 }
591
592 =item upgrade_schema_data
593
594 =cut
595
596 sub upgrade_schema_data {
597   my %opt = @_;
598
599   #auto-find tables/classes with an _update_schema method?
600
601   tie my %hash, 'Tie::IxHash', 
602
603     #fix classnum character(1)
604     'cust_bill_pkg_detail' => [],
605     #add necessary columns to RT schema
606     'TicketSystem' => [],
607     #remove h_access_user_log if it exists (since our regular auto schema
608     # upgrade doesn't have the drop tables flag turned on) 
609     'access_user_log' => [],
610     #remove possible dangling records
611     'password_history' => [],
612     'cust_pay_pending' => [],
613     #remove records referencing removed things with their FKs
614     'pkg_referral' => [],
615     'cust_bill_pkg_discount' => [],
616     'cust_msg' => [],
617     'cust_bill_pay_batch' => [],
618     'cust_event_fee' => [],
619     'radius_attr' => [],
620     'queue_depend' => [],
621     'cust_main_invoice' => [],
622     #update records referencing removed things with their FKs
623     'cust_pkg' => [],
624   ;
625
626   \%hash;
627
628 }
629
630 sub upgrade_sqlradius {
631   #my %opt = @_;
632
633   my $conf = new FS::Conf;
634
635   my @part_export = FS::part_export::sqlradius->all_sqlradius_withaccounting();
636
637   foreach my $part_export ( @part_export ) {
638
639     my $errmsg = 'Error adding FreesideStatus to '.
640                  $part_export->option('datasrc'). ': ';
641
642     my $dbh = DBI->connect(
643       ( map $part_export->option($_), qw ( datasrc username password ) ),
644       { PrintError => 0, PrintWarn => 0 }
645     ) or do {
646       warn $errmsg.$DBI::errstr;
647       next;
648     };
649
650     my $str2time = str2time_sql( $dbh->{Driver}->{Name} );
651     my $group = "UserName";
652     $group .= ",Realm"
653       if ref($part_export) =~ /withdomain/
654       || $dbh->{Driver}->{Name} =~ /^Pg/; #hmm
655
656     my $sth_alter = $dbh->prepare(
657       "ALTER TABLE radacct ADD COLUMN FreesideStatus varchar(32) NULL"
658     );
659     if ( $sth_alter ) {
660       if ( $sth_alter->execute ) {
661         my $sth_update = $dbh->prepare(
662          "UPDATE radacct SET FreesideStatus = 'done' WHERE FreesideStatus IS NULL"
663         ) or die $errmsg.$dbh->errstr;
664         $sth_update->execute or die $errmsg.$sth_update->errstr;
665       } else {
666         my $error = $sth_alter->errstr;
667         warn $errmsg.$error
668           unless $error =~ /Duplicate column name/i  #mysql
669               || $error =~ /already exists/i;        #Pg
670 ;
671       }
672     } else {
673       my $error = $dbh->errstr;
674       warn $errmsg.$error; #unless $error =~ /exists/i;
675     }
676
677     my $sth_index = $dbh->prepare(
678       "CREATE INDEX FreesideStatus ON radacct ( FreesideStatus )"
679     );
680     if ( $sth_index ) {
681       unless ( $sth_index->execute ) {
682         my $error = $sth_index->errstr;
683         warn $errmsg.$error
684           unless $error =~ /Duplicate key name/i #mysql
685               || $error =~ /already exists/i;    #Pg
686       }
687     } else {
688       my $error = $dbh->errstr;
689       warn $errmsg.$error. ' (preparing statement)';#unless $error =~ /exists/i;
690     }
691
692     my $times = ($dbh->{Driver}->{Name} =~ /^mysql/)
693       ? ' AcctStartTime != 0 AND AcctStopTime != 0 '
694       : ' AcctStartTime IS NOT NULL AND AcctStopTime IS NOT NULL ';
695
696     my $sth = $dbh->prepare("SELECT UserName,
697                                     Realm,
698                                     $str2time max(AcctStartTime)),
699                                     $str2time max(AcctStopTime))
700                               FROM radacct
701                               WHERE FreesideStatus = 'done'
702                                 AND $times
703                               GROUP BY $group
704                             ")
705       or die $errmsg.$dbh->errstr;
706     $sth->execute() or die $errmsg.$sth->errstr;
707   
708     while (my $row = $sth->fetchrow_arrayref ) {
709       my ($username, $realm, $start, $stop) = @$row;
710   
711       $username = lc($username) unless $conf->exists('username-uppercase');
712
713       my $exportnum = $part_export->exportnum;
714       my $extra_sql = " AND exportnum = $exportnum ".
715                       " AND exportsvcnum IS NOT NULL ";
716
717       if ( ref($part_export) =~ /withdomain/ ) {
718         $extra_sql = " AND '$realm' = ( SELECT domain FROM svc_domain
719                          WHERE svc_domain.svcnum = svc_acct.domsvc ) ";
720       }
721   
722       my $svc_acct = qsearchs({
723         'select'    => 'svc_acct.*',
724         'table'     => 'svc_acct',
725         'addl_from' => 'LEFT JOIN cust_svc   USING ( svcnum )'.
726                        'LEFT JOIN export_svc USING ( svcpart )',
727         'hashref'   => { 'username' => $username },
728         'extra_sql' => $extra_sql,
729       });
730
731       if ($svc_acct) {
732         $svc_acct->last_login($start)
733           if $start && (!$svc_acct->last_login || $start > $svc_acct->last_login);
734         $svc_acct->last_logout($stop)
735           if $stop && (!$svc_acct->last_logout || $stop > $svc_acct->last_logout);
736       }
737     }
738   }
739
740 }
741
742 =back
743
744 =head1 BUGS
745
746 Sure.
747
748 =head1 SEE ALSO
749
750 =cut
751
752 1;