[freeside-commits] freeside/FS/FS cust_main.pm, 1.271.2.42, 1.271.2.43

Ivan,,, ivan at wavetail.420.am
Fri Mar 14 13:11:50 PDT 2008


Update of /home/cvs/cvsroot/freeside/FS/FS
In directory wavetail.420.am:/tmp/cvs-serv25878

Modified Files:
      Tag: FREESIDE_1_7_BRANCH
	cust_main.pm 
Log Message:
eliminate some harmless uninitialized value warnings

Index: cust_main.pm
===================================================================
RCS file: /home/cvs/cvsroot/freeside/FS/FS/cust_main.pm,v
retrieving revision 1.271.2.42
retrieving revision 1.271.2.43
diff -u -d -r1.271.2.42 -r1.271.2.43
--- cust_main.pm	13 Feb 2008 23:45:42 -0000	1.271.2.42
+++ cust_main.pm	14 Mar 2008 20:11:47 -0000	1.271.2.43
@@ -2744,7 +2744,7 @@
     'gatewaynum' => ( $payment_gateway ? $payment_gateway->gatewaynum : '' ),
   };
   $cust_pay_pending->payunique( $options{payunique} )
-    if length($options{payunique});
+    if defined($options{payunique}) && length($options{payunique});
   my $cpp_new_err = $cust_pay_pending->insert; #mutex lost when this is inserted
   return $cpp_new_err if $cpp_new_err;
 
@@ -2883,7 +2883,8 @@
        'paydate'  => $paydate,
     } );
     #doesn't hurt to know, even though the dup check is in cust_pay_pending now
-    $cust_pay->payunique( $options{payunique} ) if length($options{payunique});
+    $cust_pay->payunique( $options{payunique} )
+      if defined($options{payunique}) && length($options{payunique});
 
     my $oldAutoCommit = $FS::UID::AutoCommit;
     local $FS::UID::AutoCommit = 0;



More information about the freeside-commits mailing list