summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorivan <ivan>2008-07-02 02:55:51 +0000
committerivan <ivan>2008-07-02 02:55:51 +0000
commit71186e09c298831fab69243c0459aef5b086951c (patch)
tree907e8c7bc0914d7ef943b281a7abcd682ce1d661
parent661c9b9300d2d0c06aa5e697019b729746b77a13 (diff)
fix errors paying with an on-file card through self-service
-rw-r--r--FS/FS/ClientAPI/MyAccount.pm10
1 files changed, 6 insertions, 4 deletions
diff --git a/FS/FS/ClientAPI/MyAccount.pm b/FS/FS/ClientAPI/MyAccount.pm
index f2340a1e8..696467538 100644
--- a/FS/FS/ClientAPI/MyAccount.pm
+++ b/FS/FS/ClientAPI/MyAccount.pm
@@ -371,6 +371,7 @@ sub process_payment {
or return { 'error' => "illegal_payby " . $p->{'payby'} };
my $payby = $1;
+ #false laziness w/process/payment.cgi
my $payinfo;
my $paycvv = '';
if ( $payby eq 'CHEK' || $payby eq 'DCHK' ) {
@@ -389,14 +390,15 @@ sub process_payment {
} elsif ( $payby eq 'CARD' || $payby eq 'DCRD' ) {
$payinfo = $p->{'payinfo'};
- $payinfo =~ s/[^\dx]//g;
- $payinfo =~ /^(\d{13,16})$/
- or return { 'error' => gettext('invalid_card') }; # . ": ". $self->payinfo
- $payinfo = $1;
$payinfo = $cust_main->payinfo
if $cust_main->paymask eq $payinfo;
+ $payinfo =~ s/\D//g;
+ $payinfo =~ /^(\d{13,16})$/
+ or return { 'error' => gettext('invalid_card') }; # . ": ". $self->payinfo
+ $payinfo = $1;
+
validate($payinfo)
or return { 'error' => gettext('invalid_card') }; # . ": ". $self->payinfo
return { 'error' => gettext('unknown_card_type') }