summaryrefslogtreecommitdiff
path: root/FS
diff options
context:
space:
mode:
authorIvan Kohler <ivan@freeside.biz>2014-09-05 18:22:47 -0700
committerIvan Kohler <ivan@freeside.biz>2014-09-05 18:22:47 -0700
commit43530fe13141377a688b498d72617cd02ea93fae (patch)
treee10dc8e951cceeb80546f1f08dbfa25872e01f4a /FS
parent5a76902cdaee8a0276c49af42352efd673c4e2d0 (diff)
add invnum to payment import, RT#29673
Diffstat (limited to 'FS')
-rw-r--r--FS/FS/cust_pay.pm13
1 files changed, 12 insertions, 1 deletions
diff --git a/FS/FS/cust_pay.pm b/FS/FS/cust_pay.pm
index 139837c3a..01736f78f 100644
--- a/FS/FS/cust_pay.pm
+++ b/FS/FS/cust_pay.pm
@@ -1183,7 +1183,7 @@ sub batch_import {
my @fields;
my $payby;
if ( $format eq 'simple' ) {
- @fields = qw( custnum agent_custid paid payinfo );
+ @fields = qw( custnum agent_custid paid payinfo invnum );
$payby = 'BILL';
} elsif ( $format eq 'extended' ) {
die "unimplemented\n";
@@ -1268,9 +1268,20 @@ sub batch_import {
$cust_pay{custnum} = $2;
}
+ my $custnum = $cust_pay{custnum};
+
my $cust_pay = new FS::cust_pay( \%cust_pay );
my $error = $cust_pay->insert;
+ if ( ! $error && $cust_pay->custnum != $custnum ) {
+ #invnum was defined, and ->insert set custnum to the customer for that
+ #invoice, but it wasn't the one the import specified.
+ $dbh->rollback if $oldAutoCommit;
+ $error = "specified invoice #". $cust_pay{invnum}.
+ " is for custnum ". $cust_pay->custnum.
+ ", not specified custnum $custnum";
+ }
+
if ( $error ) {
$dbh->rollback if $oldAutoCommit;
return "can't insert payment for $line: $error";