X-Git-Url: http://git.freeside.biz/gitweb/?p=freeside.git;a=blobdiff_plain;f=FS%2FFS%2Fgeocode_Mixin.pm;h=0625b5ac351e13e59a0f5ef51806a50d390deb29;hp=498b7cf97c44edb3ffa1571d0df9b4e355d25918;hb=11a4787e386291574aca2357421b586e365252d9;hpb=528e7e1cb1a6d05f87847bf9fd9b7e0e9fc33f6f diff --git a/FS/FS/geocode_Mixin.pm b/FS/FS/geocode_Mixin.pm index 498b7cf97..0625b5ac3 100644 --- a/FS/FS/geocode_Mixin.pm +++ b/FS/FS/geocode_Mixin.pm @@ -3,7 +3,8 @@ package FS::geocode_Mixin; use strict; use vars qw( $DEBUG $me ); use Carp; -use Locale::Country; +use Locale::Country (); +use Geo::Coder::Googlev3; #compile time for now, until others are supported use FS::Record qw( qsearchs qsearch ); use FS::Conf; use FS::cust_pkg; @@ -96,19 +97,22 @@ sub location_label { $notfirst++; } - my %location_type; - if ( 1 ) { #ikano, switch on via config - { no warnings 'void'; - eval { 'use FS::part_export::ikano;' }; - die $@ if $@; + my $lt = $self->get($prefix.'location_type'); + if ( $lt ) { + my %location_type; + if ( 1 ) { #ikano, switch on via config + { no warnings 'void'; + eval { 'use FS::part_export::ikano;' }; + die $@ if $@; + } + %location_type = FS::part_export::ikano->location_types; + } else { + %location_type = (); #? } - %location_type = FS::part_export::ikano->location_types; - } else { - %location_type = (); #? + + $line .= ' '.&$escape( $location_type{$lt} || $lt ); } - $line .= ' '. &$escape( $location_type{ $self->get($prefix.'location_type') }) - if $self->get($prefix.'location_type'); $line .= ' '. &$escape($self->get($prefix.'location_number')) if $self->get($prefix.'location_number'); @@ -122,12 +126,69 @@ sub location_label { $notfirst++; } } - $line .= $separator. &$escape(code2country($self->country)) + $line .= $separator. &$escape($self->country_full) if $self->country ne $cydefault; $line; } +=item country_full + +Returns the full country name. + +=cut + +sub country_full { + my $self = shift; + $self->code2country($self->country); +} + +sub code2country { + my( $self, $country ) = @_; + + #a hash? not expecting an explosion of business from unrecognized countries.. + return 'KKTC' if $country eq 'XC'; + + Locale::Country::code2country($country); +} + +=item set_coord + +Look up the coordinates of the location using (currently) the Google Maps +API and set the 'latitude' and 'longitude' fields accordingly. + +=cut + +sub set_coord { + my $self = shift; + + #my $module = FS::Conf->new->config('geocode_module') || 'Geo::Coder::Googlev3'; + + my $geocoder = Geo::Coder::Googlev3->new; + + my $location = eval { + $geocoder->geocode( location => + $self->get('address1'). ','. + ( $self->get('address2') ? $self->get('address2').',' : '' ). + $self->get('city'). ','. + $self->get('state'). ','. + $self->country_full + ); + }; + if ( $@ ) { + warn "geocoding error: $@\n"; + return; + } + + my $geo_loc = $location->{'geometry'}{'location'} or return; + if ( $geo_loc->{'lat'} && $geo_loc->{'lng'} ) { + $self->set('latitude', $geo_loc->{'lat'} ); + $self->set('longitude', $geo_loc->{'lng'} ); + $self->set('coord_auto', 'Y'); + } + +} + =item geocode DATA_VENDOR Returns a value for the customer location as encoded by DATA_VENDOR. @@ -141,18 +202,24 @@ sub geocode { my $geocode = $self->get('geocode'); #XXX only one data_vendor for geocode return $geocode if $geocode; - my $prefix = - ( FS::Conf->new->exists('tax-ship_address') && $self->has_ship_address ) - ? 'ship_' - : ''; + if ( $self->isa('FS::cust_main') ) { + warn "WARNING: FS::cust_main->geocode deprecated"; + + # do the best we can + my $m = FS::Conf->new->exists('tax-ship_address') ? 'ship_location' + : 'bill_location'; + my $location = $self->$m or return ''; + return $location->geocode($data_vendor); + } - my($zip,$plus4) = split /-/, $self->get("${prefix}zip") + my($zip,$plus4) = split /-/, $self->get('zip') if $self->country eq 'US'; $zip ||= ''; $plus4 ||= ''; #CCH specific location stuff - my $extra_sql = "AND plus4lo <= '$plus4' AND plus4hi >= '$plus4'"; + my $extra_sql = $plus4 ? "AND plus4lo <= '$plus4' AND plus4hi >= '$plus4'" + : ''; my @cust_tax_location = qsearch( { @@ -165,50 +232,64 @@ sub geocode { $geocode = $cust_tax_location[0]->geocode if scalar(@cust_tax_location); + warn "WARNING: customer ". $self->custnum. + ": multiple locations for zip ". $self->get("zip"). + "; using arbitrary geocode $geocode\n" + if scalar(@cust_tax_location) > 1; + $geocode; } -=item alternize +=item process_district_update CLASS ID -Attempts to parse data for location_type and location_number from address1 -and address2. +Queueable function to update the tax district code using the selected method +(config 'tax_district_method'). CLASS is either 'FS::cust_main' or +'FS::cust_location'; ID is the key in one of those tables. =cut -sub alternize { - my $self = shift; - my $prefix = $self->has_ship_address ? 'ship_' : ''; - - return '' if $self->get($prefix.'location_type') - || $self->get($prefix.'location_number'); - - my %parse; - if ( 1 ) { #ikano, switch on via config - { no warnings 'void'; - eval { 'use FS::part_export::ikano;' }; - die $@ if $@; +sub process_district_update { + my $class = shift; + my $id = shift; + + local $DEBUG = 1; + + eval "use FS::Misc::Geo qw(get_district); use FS::Conf; use $class;"; + die $@ if $@; + die "$class has no location data" if !$class->can('location_hash'); + + my $conf = FS::Conf->new; + my $method = $conf->config('tax_district_method') + or return; #nothing to do if null + my $self = $class->by_key($id) or die "object $id not found"; + + # dies on error, fine + my $tax_info = get_district({ $self->location_hash }, $method); + + if ( $tax_info ) { + $self->set('district', $tax_info->{'district'} ); + my $error = $self->replace; + die $error if $error; + + my %hash = map { $_ => $tax_info->{$_} } + qw( district city county state country ); + $hash{'taxname'} = ''; + + my $old = qsearchs('cust_main_county', \%hash); + if ( $old ) { + my $new = new FS::cust_main_county { $old->hash, %$tax_info }; + warn "updating tax rate for district ".$tax_info->{'district'} if $DEBUG; + $error = $new->replace($old); } - %parse = FS::part_export::ikano->location_types_parse; - } else { - %parse = (); #? - } - - foreach my $from ('address1', 'address2') { - foreach my $parse ( keys %parse ) { - my $value = $self->get($prefix.$from); - if ( $value =~ s/(^|\W+)$parse\W+(\w+)\W*$//i ) { - $self->set($prefix.'location_type', $parse{$parse}); - $self->set($prefix.'location_number', $2); - $self->set($prefix.$from, $value); - return ''; - } + else { + my $new = new FS::cust_main_county $tax_info; + warn "creating tax rate for district ".$tax_info->{'district'} if $DEBUG; + $error = $new->insert; } - } + die $error if $error; - #nothing matched, no changes - $self->get($prefix.'address2') - ? "Can't parse unit type and number from ${prefix}address2" - : ''; + } + return; } =back