RT# 74435 - removed excess cust_pay code not needed in V3
authorChristopher Burger <burgerc@freeside.biz>
Thu, 8 Nov 2018 21:10:09 +0000 (16:10 -0500)
committerChristopher Burger <burgerc@freeside.biz>
Thu, 8 Nov 2018 21:10:09 +0000 (16:10 -0500)
httemplate/edit/process/cust_refund.cgi

index 2b6f9cd..33bc886 100755 (executable)
@@ -66,122 +66,6 @@ if ( $cgi->param('paynum') > 0) {
   });
 }
 
-if ( $cgi->param('paynum') > 0) {
-
-  $payinfo = $cust_pay->payinfo;
-  $payname = $cust_pay->payname;
-
-} else {
-
-  ##
-  # use new info
-  ##
-
-  $cgi->param('year') =~ /^(\d+)$/
-    or errorpage("illegal year ". $cgi->param('year'));
-  $year = $1;
-
-  $cgi->param('month') =~ /^(\d+)$/
-    or errorpage("illegal month ". $cgi->param('month'));
-  $month = $1;
-
-  $cgi->param('payname') =~ /^([\w \,\.\-\']+)$/
-    or errorpage(gettext('illegal_name'). " payname: ". $cgi->param('payname'));
-  $payname = $1;
-
-  if ( $payby eq 'CHEK' ) {
-
-    $cgi->param('payinfo1') =~ /^(\d+)$/
-      or errorpage("Illegal account number ". $cgi->param('payinfo1'));
-    my $payinfo1 = $1;
-    $cgi->param('payinfo2') =~ /^(\d+)$/
-      or errorpage("Illegal ABA/routing number ". $cgi->param('payinfo2'));
-    my $payinfo2 = $1;
-    if ( $conf->config('echeck-country') eq 'CA' ) {
-      $cgi->param('payinfo3') =~ /^(\d{5})$/
-        or errorpage("Illegal branch number ". $cgi->param('payinfo2'));
-      $payinfo2 = "$1.$payinfo2";
-    }
-    $payinfo = $payinfo1 . '@'. $payinfo2;
-
-  } elsif ( $payby eq 'CARD' ) {
-
-    $payinfo = $cgi->param('payinfo');
-
-    $payinfo =~ s/\D//g;
-    $payinfo =~ /^(\d{13,19}|\d{8,9})$/
-      or errorpage(gettext('invalid_card'));
-    $payinfo = $1;
-    validate($payinfo)
-      or errorpage(gettext('invalid_card'));
-
-    unless ( $cust_main->tokenized($payinfo) ) { #token
-
-      my $cardtype = cardtype($payinfo);
-
-      errorpage(gettext('unknown_card_type'))
-        if $cardtype eq "Unknown";
-
-      my %bop_card_types = map { $_=>1 } values %{ card_types() };
-      errorpage("$cardtype not accepted") unless $bop_card_types{$cardtype};
-
-    }
-
-    if ( length($cgi->param('paycvv') ) ) {
-      if ( cardtype($payinfo) eq 'American Express card' ) {
-        $cgi->param('paycvv') =~ /^(\d{4})$/
-          or errorpage("CVV2 (CID) for American Express cards is four digits.");
-        $paycvv = $1;
-      } else {
-        $cgi->param('paycvv') =~ /^(\d{3})$/
-          or errorpage("CVV2 (CVC2/CID) is three digits.");
-        $paycvv = $1;
-      }
-    } elsif ( $conf->exists('backoffice-require_cvv') ){
-      errorpage("CVV2 is required");
-    }
-
-  } else {
-    die "unknown payby $payby";
-  }
-
-  # save first, for proper tokenization
-  if ( $cgi->param('save') ) {
-
-    my %saveopt;
-    if ( $payby eq 'CARD' ) {
-      my $bill_location = FS::cust_location->new;
-      $bill_location->set( $_ => scalar($cgi->param($_)) )
-        foreach @{$payby2fields{$payby}};
-      $saveopt{'bill_location'} = $bill_location;
-      $saveopt{'paycvv'} = $paycvv; # save_cust_payby contains conf logic for when to use this
-      $saveopt{'paydate'} = "$year-$month-01";
-    } else {
-      # ss/stateid/stateid_state won't be saved, but should be harmless to pass
-      %saveopt = map { $_ => scalar($cgi->param($_)) } @{$payby2fields{$payby}};
-    }
-
-    my $error = $cust_main->save_cust_payby(
-      'saved_cust_payby' => \$cust_payby,
-      'payment_payby' => $payby,
-      'auto'          => scalar($cgi->param('auto')),
-      'weight'        => scalar($cgi->param('weight')),
-      'payinfo'       => $payinfo,
-      'payname'       => $payname,
-      %saveopt
-    );
-
-    errorpage("error saving info, payment not processed: $error")
-      if $error;
-
-  } elsif ( $payby eq 'CARD' ) { # not saving
-
-    $paymask = FS::payinfo_Mixin->mask_payinfo('CARD',$payinfo); # for untokenized but tokenizable payinfo
-
-  }
-
-}
-
 ##
 # now run the refund
 ##