[freeside-commits] branch FREESIDE_3_BRANCH updated. e50f2d69776ccaa4162c59cc4789ddd566be6df2

[Ivan]

The branch, FREESIDE_3_BRANCH has been updated
       via  e50f2d69776ccaa4162c59cc4789ddd566be6df2 (commit)
      from  e00d8a556b5d2acfc932b4fb2a0007bc186ddbe6 (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 e50f2d69776ccaa4162c59cc4789ddd566be6df2
Author: Ivan Kohler <ivan at freeside.biz>
Date:   Mon Jun 15 12:04:24 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 18e45a7..b0ea736 100644
--- a/FS/FS/ClientAPI/MyAccount.pm
+++ b/FS/FS/ClientAPI/MyAccount.pm
@@ -787,16 +787,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(-)