X-Git-Url: http://git.freeside.biz/gitweb/?p=freeside.git;a=blobdiff_plain;f=FS%2FFS%2FUpgrade.pm;h=263230b342b4840701c3f39bac57545286a44158;hp=bf3e2ad38f7ec0bdca3362a874b97214bc36b7af;hb=15a4e1674694b76ecc2af87de479aabe370ac03d;hpb=7bfd0175b05d52b648e283fca4343aef79e851d7 diff --git a/FS/FS/Upgrade.pm b/FS/FS/Upgrade.pm index bf3e2ad38..263230b34 100644 --- a/FS/FS/Upgrade.pm +++ b/FS/FS/Upgrade.pm @@ -8,7 +8,9 @@ use File::Slurp; use FS::UID qw( dbh driver_name ); use FS::Conf; use FS::Record qw(qsearchs qsearch str2time_sql); +use FS::queue; use FS::upgrade_journal; +use FS::Setup qw( enable_banned_pay_pad ); use FS::svc_domain; $FS::svc_domain::whois_hack = 1; @@ -62,8 +64,15 @@ sub upgrade_config { } } + my @agents = qsearch('agent', {}); + upgrade_overlimit_groups($conf); - map { upgrade_overlimit_groups($conf,$_->agentnum) } qsearch('agent', {}); + map { upgrade_overlimit_groups($conf,$_->agentnum) } @agents; + + upgrade_invoice_from($conf); + foreach my $agent (@agents) { + upgrade_invoice_from($conf,$agent->agentnum,1); + } my $DIST_CONF = '/usr/local/etc/freeside/default_conf/';#DIST_CONF in Makefile $conf->set($_, scalar(read_file( "$DIST_CONF/$_" )) ) @@ -96,6 +105,63 @@ sub upgrade_config { $conf->touch('cust_main-enable_spouse'); $conf->delete('cust_main-enable_spouse_birthdate'); } + + # renamed/repurposed + if ( $conf->exists('cust_pkg-show_fcc_voice_grade_equivalent') ) { + $conf->touch('part_pkg-show_fcc_options'); + $conf->delete('cust_pkg-show_fcc_voice_grade_equivalent'); + warn " +You have FCC Form 477 package options enabled. + +Starting with the October 2014 filing date, the FCC has redesigned +Form 477 and introduced new service categories. See bin/convert-477-options +to update your package configuration for the new report. + +If you need to continue using the old Form 477 report, turn on the +'old_fcc_report' configuration option. +"; + } + + # boolean invoice_sections_by_location option is now + # invoice_sections_method = 'location' + my @invoice_sections_confs = + qsearch('conf', { 'name' => { op=>'LIKE', value=>'%sections_by_location' } }); + foreach my $c (@invoice_sections_confs) { + $c->name =~ /^(\w+)sections_by_location$/; + $conf->delete($c->name); + my $newname = $1.'sections_method'; + $conf->set($newname, 'location'); + } + + # boolean enable_taxproducts is now tax_data_vendor = 'cch' + if ( $conf->exists('enable_taxproducts') ) { + + $conf->delete('enable_taxproducts'); + $conf->set('tax_data_vendor', 'cch'); + + } + + # boolean tax-cust_exempt-groups-require_individual_nums is now -num_req all + if ( $conf->exists('tax-cust_exempt-groups-require_individual_nums') ) { + $conf->set('tax-cust_exempt-groups-num_req', 'all'); + $conf->delete('tax-cust_exempt-groups-require_individual_nums'); + } + + # boolean+text previous_balance-exclude_from_total is now two separate options + my $total_new_charges = $conf->config('previous_balance-exclude_from_total'); + if (length($total_new_charges) > 0) { + $conf->set('previous_balance-text-total_new_charges', $total_new_charges); + $conf->set('previous_balance-exclude_from_total', ''); + } + + # switch from specifying an email address to boolean check + if ( $conf->exists('batch-errors_to') ) { + $conf->touch('batch-errors_not_fatal'); + $conf->delete('batch-errors_to'); + } + + enable_banned_pay_pad() unless length($conf->config('banned_pay-pad')); + } sub upgrade_overlimit_groups { @@ -124,6 +190,28 @@ sub upgrade_overlimit_groups { } } +sub upgrade_invoice_from { + my ($conf, $agentnum, $agentonly) = @_; + if ( + (!$conf->exists('invoice_from_name',$agentnum,$agentonly)) && + ($conf->config('invoice_from',$agentnum,$agentonly) =~ /\<(.*)\>/) + ) { + my $realemail = $1; + $realemail =~ s/^\s*//; # remove leading spaces + $realemail =~ s/\s*$//; # remove trailing spaces + my $realname = $conf->config('invoice_from',$agentnum); + $realname =~ s/\<.*\>//; # remove email address + $realname =~ s/^\s*//; # remove leading spaces + $realname =~ s/\s*$//; # remove trailing spaces + # properly quote names that contain punctuation + if (($realname =~ /[^[:alnum:][:space:]]/) && ($realname !~ /^\".*\"$/)) { + $realname = '"' . $realname . '"'; + } + $conf->set('invoice_from_name', $realname, $agentnum); + $conf->set('invoice_from', $realemail, $agentnum); + } +} + =item upgrade =cut @@ -153,6 +241,26 @@ sub upgrade { $class->_upgrade_data(%opt); + # New interface for async upgrades: a class can declare a + # "queueable_upgrade" method, which will run as part of the normal + # upgrade, but if the -j option is passed, will instead be run from + # the job queue. + if ( $class->can('queueable_upgrade') ) { + my $jobname = $class . '::queueable_upgrade'; + my $num_jobs = FS::queue->count("job = '$jobname' and status != 'failed'"); + if ($num_jobs > 0) { + warn "$class upgrade already scheduled.\n"; + } else { + if ( $opt{'queue'} ) { + warn "Scheduling $class upgrade.\n"; + my $job = FS::queue->new({ job => $jobname }); + $job->insert($class, %opt); + } else { + $class->queueable_upgrade(%opt); + } + } #$num_jobs == 0 + } + if ( $oldAutoCommit ) { warn " committing\n"; dbh->commit or die dbh->errstr; @@ -179,7 +287,7 @@ sub upgrade { } local($FS::cust_main::ignore_expired_card) = 1; - local($FS::cust_main::ignore_illegal_zip) = 1; + #this is long-gone... would need to set an equivalent in cust_location #local($FS::cust_main::ignore_illegal_zip) = 1; local($FS::cust_main::ignore_banned_card) = 1; local($FS::cust_main::skip_fuzzyfiles) = 1; @@ -217,9 +325,18 @@ sub upgrade_data { tie my %hash, 'Tie::IxHash', - #cust_main (remove paycvv from history) + #payby conditions to new ones + 'part_event_condition' => [], + + #payby actions to new ones + 'part_event' => [], + + #cust_main (remove paycvv from history, locations, cust_payby, etc) 'cust_main' => [], + #contact -> cust_contact / prospect_contact + 'contact' => [], + #msgcat 'msgcat' => [], @@ -322,6 +439,15 @@ sub upgrade_data { #fix taxable line item links 'cust_bill_pkg_tax_location' => [], + + #populate state FIPS codes if not already done + 'state' => [], + + #set default locations on quoted packages + 'quotation_pkg' => [], + + #populate tax statuses + 'tax_status' => [], ; \%hash;