ACHEFT only disables the billing info update, not the general schedul…

…e editing or amount
JMAConsulting · Mar 18, 2021 · a250198 · a250198
1 parent 5a1a4e2
commit a250198
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/CRM/Core/Payment/iATSService.php b/CRM/Core/Payment/iATSService.php
@@ -120,10 +120,7 @@ public function isSupported($method) {
    * @return bool
    */
   protected function allowSelfService($action) {
-    if ('CRM_Core_Payment_iATSServiceACHEFT' == CRM_Utils_System::getClassName($this)) {
-      return FALSE;
-    }
-    elseif (!CRM_Core_Permission::check('access CiviContribute')) {
+    if (!CRM_Core_Permission::check('access CiviContribute')) {
       // disable self-service 'action' if the admin has not allowed it
       if (FALSE == $this->getSettings('enable_'.$action)) {
 	return FALSE;
@@ -149,6 +146,10 @@ public function supportsFutureRecurStartDate() {
    */
 
   public function supportsUpdateSubscriptionBillingInfo() {
+    // the CiviCRM internal form for updating billing info for ACHEFT is inadequate
+    if ('CRM_Core_Payment_iATSServiceACHEFT' == CRM_Utils_System::getClassName($this)) {
+      return FALSE;
+    }
     return $this->allowSelfService('update_subscription_billing_info');
   }