diff options
author | Ivan Kohler <ivan@freeside.biz> | 2014-07-17 06:45:02 -0700 |
---|---|---|
committer | Ivan Kohler <ivan@freeside.biz> | 2014-07-17 06:45:02 -0700 |
commit | 8c39dadbacacbceec1bc0b6c5fbf1468b0d3cf32 (patch) | |
tree | 97b7619688cdd5f417458e93ad1736e7352b902b /FS/FS | |
parent | 7943c96636596806b9fc99195c23b166728280c8 (diff) |
REST API, RT#28181
Diffstat (limited to 'FS/FS')
-rw-r--r-- | FS/FS/API.pm | 60 | ||||
-rw-r--r-- | FS/FS/Record.pm | 16 | ||||
-rw-r--r-- | FS/FS/cust_main/API.pm | 94 |
3 files changed, 115 insertions, 55 deletions
diff --git a/FS/FS/API.pm b/FS/FS/API.pm index a0f1dba2a..45f8f5ebe 100644 --- a/FS/FS/API.pm +++ b/FS/FS/API.pm @@ -1,5 +1,6 @@ package FS::API; +use strict; use FS::Conf; use FS::Record qw( qsearch qsearchs ); use FS::cust_main; @@ -406,66 +407,18 @@ Referring customer number sub new_customer { my( $class, %opt ) = @_; + my $conf = new FS::Conf; return { 'error' => 'Incorrect shared secret' } unless $opt{secret} eq $conf->config('api_shared_secret'); #default agentnum like signup_server-default_agentnum? + #$opt{agentnum} ||= $conf->config('signup_server-default_agentnum'); #same for refnum like signup_server-default_refnum + $opt{refnum} ||= $conf->config('signup_server-default_refnum'); - my $cust_main = new FS::cust_main ( { - 'agentnum' => $agentnum, - 'refnum' => $opt{refnum} - || $conf->config('signup_server-default_refnum'), - 'payby' => 'BILL', - - map { $_ => $opt{$_} } qw( - agentnum refnum agent_custid referral_custnum - last first company - daytime night fax mobile - payby payinfo paydate paycvv payname - ), - - } ); - - my @invoicing_list = $opt{'invoicing_list'} - ? split( /\s*\,\s*/, $opt{'invoicing_list'} ) - : (); - push @invoicing_list, 'POST' if $opt{'postal_invoicing'}; - - my ($bill_hash, $ship_hash); - foreach my $f (FS::cust_main->location_fields) { - # avoid having to change this in front-end code - $bill_hash->{$f} = $opt{"bill_$f"} || $opt{$f}; - $ship_hash->{$f} = $opt{"ship_$f"}; - } - - my $bill_location = FS::cust_location->new($bill_hash); - my $ship_location; - # we don't have an equivalent of the "same" checkbox in selfservice^Wthis API - # so is there a ship address, and if so, is it different from the billing - # address? - if ( length($ship_hash->{address1}) > 0 and - grep { $bill_hash->{$_} ne $ship_hash->{$_} } keys(%$ship_hash) - ) { - - $ship_location = FS::cust_location->new( $ship_hash ); - - } else { - $ship_location = $bill_location; - } - - $cust_main->set('bill_location' => $bill_location); - $cust_main->set('ship_location' => $ship_location); - - $error = $cust_main->insert( {}, \@invoicing_list ); - return { 'error' => $error } if $error; - - return { 'error' => '', - 'custnum' => $cust_main->custnum, - }; - + $class->API_insert( %opt ); } =back @@ -486,8 +439,7 @@ sub customer_info { or return { 'error' => 'Unknown custnum' }; $cust_main->API_getinfo; - -} +: =item location_info diff --git a/FS/FS/Record.pm b/FS/FS/Record.pm index b226e177b..24f2a25dd 100644 --- a/FS/FS/Record.pm +++ b/FS/FS/Record.pm @@ -1127,13 +1127,27 @@ sub hashref { $self->{'Hash'}; } -#fallback +#fallbacks/generics + sub API_getinfo { my $self = shift; +{ ( map { $_=>$self->$_ } $self->fields ), }; } +sub API_insert { + my( $class, %opt ) = @_; + my $table = $class->table; + my $self = $class->new( { map { $_ => $opt{$_} } fields($table) } ); + my $error = $self->insert; + return +{ 'error' => $error } if $error; + my $pkey = $self->pkey; + return +{ 'error' => '', + 'primary_key' => $pkey, + $pkey => $self->$pkey, + }; +} + =item modified Returns true if any of this object's values have been modified with set (or via diff --git a/FS/FS/cust_main/API.pm b/FS/FS/cust_main/API.pm index 2637c7eba..1e1723474 100644 --- a/FS/FS/cust_main/API.pm +++ b/FS/FS/cust_main/API.pm @@ -1,6 +1,11 @@ package FS::cust_main::API; use strict; +use FS::Conf; + +=item API_getinfo FIELD => VALUE, ... + +=cut #some false laziness w/ClientAPI::Myaccount customer_info/customer_info_short @@ -60,4 +65,93 @@ sub API_getinfo { } + +#or maybe all docs go in FS::API ? argh + +=item API_insert + +Class method (constructor). + +Example: + + use FS::cust_main; + FS::cust_main->API_insert( + 'agentnum' => 1, + 'refnum' => 1, + 'first' => 'Harvey', + 'last' => 'Black', + 'address1' => '5354 Pink Rabbit Lane', + 'city' => 'Farscape', + 'state' => 'CA', + 'zip' => '54144', + + 'invoicing_list' => 'harvey2@example.com', + ); + +=cut + +#certainly false laziness w/ClientAPI::Signup new_customer/new_customer_minimal +# but approaching this from a clean start / back-office perspective +# i.e. no package/service, no immediate credit card run, etc. + +sub API_insert { + my( $class, %opt ) = @_; + + my $conf = new FS::Conf; + + #default agentnum like signup_server-default_agentnum? + + #same for refnum like signup_server-default_refnum? + + my $cust_main = new FS::cust_main ( { # $class->new( { + 'payby' => 'BILL', + + map { $_ => $opt{$_} } qw( + agentnum refnum agent_custid referral_custnum + last first company + daytime night fax mobile + payby payinfo paydate paycvv payname + ), + + } ); + + my @invoicing_list = $opt{'invoicing_list'} + ? split( /\s*\,\s*/, $opt{'invoicing_list'} ) + : (); + push @invoicing_list, 'POST' if $opt{'postal_invoicing'}; + + my ($bill_hash, $ship_hash); + foreach my $f (FS::cust_main->location_fields) { + # avoid having to change this in front-end code + $bill_hash->{$f} = $opt{"bill_$f"} || $opt{$f}; + $ship_hash->{$f} = $opt{"ship_$f"}; + } + + my $bill_location = FS::cust_location->new($bill_hash); + my $ship_location; + # we don't have an equivalent of the "same" checkbox in selfservice^Wthis API + # so is there a ship address, and if so, is it different from the billing + # address? + if ( length($ship_hash->{address1}) > 0 and + grep { $bill_hash->{$_} ne $ship_hash->{$_} } keys(%$ship_hash) + ) { + + $ship_location = FS::cust_location->new( $ship_hash ); + + } else { + $ship_location = $bill_location; + } + + $cust_main->set('bill_location' => $bill_location); + $cust_main->set('ship_location' => $ship_location); + + my $error = $cust_main->insert( {}, \@invoicing_list ); + return { 'error' => $error } if $error; + + return { 'error' => '', + 'custnum' => $cust_main->custnum, + }; + +} + 1; |