X-Git-Url: http://git.freeside.biz/gitweb/?p=freeside.git;a=blobdiff_plain;f=FS%2FFS%2Fprospect_main.pm;h=81f71a99679bb27d8dcb7eceab129dbb51e47912;hp=bace1f040d7140e35225c5190673288b42329b1f;hb=167dbdad01e2c1b62fd9be43cc05212e8c874a02;hpb=fced2ed0147049fded427862a3517c172f86513e diff --git a/FS/FS/prospect_main.pm b/FS/FS/prospect_main.pm index bace1f040..81f71a996 100644 --- a/FS/FS/prospect_main.pm +++ b/FS/FS/prospect_main.pm @@ -1,16 +1,53 @@ package FS::prospect_main; +use base qw( FS::Quotable_Mixin FS::o2m_Common FS::Record ); use strict; -use base qw( FS::o2m_Common FS::Record ); -use vars qw( $DEBUG ); +use vars qw( $DEBUG @location_fields ); use Scalar::Util qw( blessed ); -use FS::Record qw( dbh qsearch ); #qsearchs ); -use FS::agent; +use FS::Conf; +use FS::Record qw( dbh qsearch ); # qsearchs ); use FS::cust_location; -use FS::contact; +use FS::cust_main; $DEBUG = 0; +#started as false laziness w/cust_main/Location.pm + +use Carp qw(carp); + +my $init = 0; +BEGIN { + # set up accessors for location fields + if (!$init) { + no strict 'refs'; + @location_fields = + qw( address1 address2 city county state zip country district + latitude longitude coord_auto censustract censusyear geocode + addr_clean ); + + foreach my $f (@location_fields) { + *{"FS::prospect_main::$f"} = sub { + carp "WARNING: tried to set cust_main.$f with accessor" if (@_ > 1); + my @cust_location = shift->cust_location or return ''; + #arbitrarily picking the first because the UI only lets you add one + $cust_location[0]->$f + }; + } + $init++; + } +} + +#debugging shim--probably a performance hit, so remove this at some point +sub get { + my $self = shift; + my $field = shift; + if ( $DEBUG and grep { $_ eq $field } @location_fields ) { + carp "WARNING: tried to get() location field $field"; + $self->$field; + } + $self->FS::Record::get($field); +} + =head1 NAME FS::prospect_main - Object methods for prospect_main records @@ -43,7 +80,11 @@ primary key =item agentnum -Agent +Agent (see L) + +=item refnum + +Referral (see L) =item company @@ -202,25 +243,42 @@ sub check { my $error = $self->ut_numbern('prospectnum') - || $self->ut_foreign_key('agentnum', 'agent', 'agentnum' ) - || $self->ut_text('company') + || $self->ut_foreign_key( 'agentnum', 'agent', 'agentnum' ) + || $self->ut_foreign_keyn( 'refnum', 'part_referral', 'refnum' ) + || $self->ut_textn('company') ; return $error if $error; + my $company = $self->company; + $company =~ s/^\s+//; + $company =~ s/\s+$//; + $company =~ s/\s+/ /g; + $self->company($company); + $self->SUPER::check; } -=item contact +=item name -Returns the contacts (see L) associated with this prospect. +Returns a name for this prospect, as a string (company name for commercial +prospects, contact name for residential prospects). =cut -sub contact { +sub name { my $self = shift; - qsearch( 'contact', { 'prospectnum' => $self->prospectnum } ); + return $self->company if $self->company; + + my $contact = ($self->prospect_contact)[0]->contact; #first contact? good enough for now + return $contact->line if $contact; + + 'Prospect #'. $self->prospectnum; } +=item contact + +Returns the contacts (see L) associated with this prospect. + =item cust_location Returns the locations (see L) associated with this prospect. @@ -229,7 +287,66 @@ Returns the locations (see L) associated with this prospect. sub cust_location { my $self = shift; - qsearch( 'cust_location', { 'prospectnum' => $self->prospectnum } ); + qsearch( 'cust_location', { 'prospectnum' => $self->prospectnum, + 'custnum' => '' } ); +} + +=item qual + +Returns the qualifications (see L) associated with this prospect. + +=item agent + +Returns the agent (see L) for this customer. + +=item convert_cust_main + +Converts this prospect to a customer. + +If there is an error, returns an error message, otherwise, returns the +newly-created FS::cust_main object. + +=cut + +sub convert_cust_main { + my $self = shift; + + my @cust_location = $self->cust_location; + #the interface only allows one, so we're just gonna go with that for now + + my @contact = map $_->contact, $self->prospect_contact; + + #XXX define one contact type as "billing", then we could pick just that one + my @invoicing_list = map $_->emailaddress, map $_->contact_email, @contact; + + #XXX i'm not compatible with cust_main-require_phone (which is kind of a + # pre-contact thing anyway) + + my $cust_main = new FS::cust_main { + 'bill_location' => $cust_location[0], + 'ship_location' => $cust_location[0], + ( map { $_ => $self->$_ } qw( agentnum refnum company ) ), + }; + + $cust_main->refnum( FS::Conf->new->config('referraldefault') || 1 ) + unless $cust_main->refnum; + + #XXX again, arbitrary, if one contact was "billing", that would be better + if ( $contact[0] ) { + $cust_main->set($_, $contact[0]->get($_)) foreach qw( first last ); + } else { + $cust_main->set('first', 'Unknown'); + $cust_main->set('last', 'Unknown'); + } + + #v3 payby + $cust_main->payby('BILL'); + $cust_main->paydate('12/2037'); + + $cust_main->insert( {}, \@invoicing_list, + 'prospectnum' => $self->prospectnum, + ) + or $cust_main; } =item search HASHREF @@ -288,6 +405,11 @@ sub search { } +# stub this so that calling ->cust_bill doesn't return an empty string +sub cust_bill { + return; +} + =back =head1 BUGS