[freeside-commits] branch master updated. c3d0d1fba9dfc9988f148cbf7bd07de33d5025d8

[Ivan]

The branch, master has been updated
       via  c3d0d1fba9dfc9988f148cbf7bd07de33d5025d8 (commit)
      from  4055cf6d3af3a16318a54e374cc04ae065f0dc14 (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 c3d0d1fba9dfc9988f148cbf7bd07de33d5025d8
Author: Ivan Kohler <ivan at freeside.biz>
Date:   Fri Mar 22 20:25:02 2013 -0700

    don't call realtime_collect for prepaid cards, fixes signup error in 3.x, RT#22151

diff --git a/FS/FS/ClientAPI/Signup.pm b/FS/FS/ClientAPI/Signup.pm
index 2fcc4b1..1dbb20b 100644
--- a/FS/FS/ClientAPI/Signup.pm
+++ b/FS/FS/ClientAPI/Signup.pm
@@ -773,13 +773,15 @@ sub new_customer {
     #     " new customer: $bill_error"
     #  if $bill_error;
 
-    $bill_error = $cust_main->realtime_collect(
-       method        => FS::payby->payby2bop( $packet->{payby} ),
-       depend_jobnum => $placeholder->jobnum,
-       selfservice   => 1,
-    );
-    #warn "$me error collecting from new customer: $bill_error"
-    #  if $bill_error;
+    unless ( $packet->{payby} eq 'PREPAY' ) {
+      $bill_error = $cust_main->realtime_collect(
+         method        => FS::payby->payby2bop( $packet->{payby} ),
+         depend_jobnum => $placeholder->jobnum,
+         selfservice   => 1,
+      );
+      #warn "$me error collecting from new customer: $bill_error"
+      #  if $bill_error;
+    }
 
     if ($bill_error && ref($bill_error) eq 'HASH') {
       return { 'error' => '_collect',

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

Summary of changes:
 FS/FS/ClientAPI/Signup.pm |   16 +++++++++-------
 1 files changed, 9 insertions(+), 7 deletions(-)