fix "Customer has automatic payment information" condition when migrated from v3

[freeside.git] / FS / FS / part_event / Condition / has_cust_payby_auto.pm

diff --git a/FS/FS/part_event/Condition/has_cust_payby_auto.pm b/FS/FS/part_event/Condition/has_cust_payby_auto.pm
index dce97df..f13b639 100644 (file)
--- a/FS/FS/part_event/Condition/has_cust_payby_auto.pm
+++ b/FS/FS/part_event/Condition/has_cust_payby_auto.pm
@@ -1,11 +1,10 @@
 package FS::part_event::Condition::has_cust_payby_auto;
+use base qw( FS::part_event::Condition );
 
 use strict;
 use Tie::IxHash;
 use FS::payby;
-use FS::Record qw(qsearch);
-
-use base qw( FS::part_event::Condition );
+use FS::Record qw( qsearch dbh );
 
 sub description {
   'Customer has automatic payment information';
@@ -31,11 +30,24 @@ sub condition {
 
   my $cust_main = $self->cust_main($object);
 
+  #handle multiple (HASH) type options migrated from a v3 payby.pm condition
+  # (and maybe we should be a select-multiple or checkbox-multiple too?)
+  my @payby = ();
+  my $payby = $self->option('payby');
+  if ( ref($payby) ) {
+    @payby = keys %$payby;
+  } elsif ( $payby ) {
+    @payby = ( $payby );
+  }
+
   scalar( qsearch({ 
     'table'     => 'cust_payby',
     'hashref'   => { 'custnum' => $cust_main->custnum,
-                     'payby'   => $self->option('payby')
+                     #'payby'   => $self->option('payby')
                    },
+    'extra_sql' => 'AND payby IN ( '.
+                     join(',', map dbh->quote($_), @payby).
+                   ' ) ',
     'order_by'  => 'LIMIT 1',
   }) );