[freeside-commits] branch master updated. e9bf7c82e204b29ddca9f7d3179a7e4c5473b767

[Ivan]

The branch, master has been updated
       via  e9bf7c82e204b29ddca9f7d3179a7e4c5473b767 (commit)
      from  9a963c59933af6813405d311d6de1cd5a4eb8597 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit e9bf7c82e204b29ddca9f7d3179a7e4c5473b767
Author: Ivan Kohler <ivan at freeside.biz>
Date:   Mon Jun 15 12:04:21 2015 -0700

    self-service API: allow changing CVV with edit_info, RT#36795

diff --git a/FS/FS/ClientAPI/MyAccount.pm b/FS/FS/ClientAPI/MyAccount.pm
index 36989db..85a96c7 100644
--- a/FS/FS/ClientAPI/MyAccount.pm
+++ b/FS/FS/ClientAPI/MyAccount.pm
@@ -822,16 +822,16 @@ sub edit_info {
 
     if ( $new->payinfo eq $cust_main->paymask ) {
       $new->payinfo($cust_main->payinfo);
+      $new->paycvv( $p->{'paycvv'} || $cust_main->paycvv );
     } else {
       $new->payinfo($p->{'payinfo'});
+      return { 'error' => 'CVV2 is required' }
+        if ! $p->{'paycvv'} && $conf->exists('selfservice-onfile_require_cvv');
+      $new->paycvv( $p->{'paycvv'} )
     }
 
     $new->set( 'payby' => $p->{'auto'} ? 'CARD' : 'DCRD' );
 
-    if ( $conf->exists('selfservice-onfile_require_cvv') ){
-      return { 'error' => 'CVV2 is required' } unless $p->{'paycvv'};
-    }
-
   } elsif ( $payby =~ /^(CHEK|DCHK)$/ ) {
 
     my $payinfo;

-----------------------------------------------------------------------

Summary of changes:
 FS/FS/ClientAPI/MyAccount.pm |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)