Bad logic for reccuring processors
authorLisa Marie Maginnis <lisam@fsf.org>
Mon, 21 Sep 2015 17:22:33 +0000 (13:22 -0400)
committerLisa Marie Maginnis <lisam@fsf.org>
Mon, 21 Sep 2015 17:22:33 +0000 (13:22 -0400)
CRM/Member/Form/MembershipBlock.php

index 33717a6efc0a39d842ab920f68c7d3d75f933af6..eab30567f4c58bccfdbe72ed2c281aa4d7100901 100644 (file)
@@ -230,10 +230,11 @@ class CRM_Member_Form_MembershipBlock extends CRM_Contribute_Form_ContributionPa
                 $pid, 'payment_processor_type_id'
                 );
             }
-            if (!($paymentProcessorTypeId == CRM_Utils_Array::key('PayPal', $paymentProcessorType) ||
-              ($paymentProcessorTypeId == CRM_Utils_Array::key('AuthNet', $paymentProcessorType)))) {
-              $errors['member_price_set_id'] = ts('The membership price set associated with this online contribution allows a user to select BOTH an auto-renew AND a non-auto-renew membership. This requires submitting multiple processor transactions, and is not supported for one or more of the payment processors enabled under the Fees tab.');
-            }
+           // THIS LOGIC MAKES NO SENSE - davet 11/24/2014
+            // if (!($paymentProcessorTypeId == CRM_Utils_Array::key('PayPal', $paymentProcessorType) ||
+            //   ($paymentProcessorTypeId == CRM_Utils_Array::key('AuthNet', $paymentProcessorType)))) {
+            //   $errors['member_price_set_id'] = ts('The membership price set associated with this online contribution allows a user to select BOTH an auto-renew AND a non-auto-renew membership. This requires submitting multiple processor transactions, and is not supported for one or more of the payment processors enabled under the Fees tab.');
+            // }
 
           }
         }