From e6fa943d43179a4fb2935f30efa946b102e291aa Mon Sep 17 00:00:00 2001 From: Matthew Wire Date: Wed, 2 Sep 2020 18:17:11 +0100 Subject: [PATCH] Fix #239 hide configuration fields that we do not use --- docs/releasenotes.md | 1 + stripe.php | 12 ++++++++++-- 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/docs/releasenotes.md b/docs/releasenotes.md index 4f428b6..c5adc22 100644 --- a/docs/releasenotes.md +++ b/docs/releasenotes.md @@ -28,6 +28,7 @@ Releases use the following numbering system: * Support translating text strings in javascript. * Fix issues with popup notifications not showing in some circumstances (eg. "Card declined"). * Disable logging for `civicrm_stripe_paymentintent` table. +* Fix [#239](https://lab.civicrm.org/extensions/stripe/-/issues/239): Hide configuration fields that we don't use. * Fix PHP notices: * When a checking for an extension dependency that is not yet downloaded or installed. diff --git a/stripe.php b/stripe.php index 8ec06c9..3b79204 100644 --- a/stripe.php +++ b/stripe.php @@ -184,6 +184,14 @@ function stripe_civicrm_buildForm($formName, &$form) { // Do nothing, we won't attempt further stripe processing } break; + + case 'CRM_Admin_Form_PaymentProcessor': + // Hide configuration fields that we don't use + foreach (['accept_credit_cards', 'url_site', 'url_recur', 'test_url_site', 'test_url_recur'] as $element) + if ($form->elementExists($element)) { + $form->removeElement($element); + } + break; } } @@ -204,14 +212,14 @@ function stripe_civicrm_check(&$messages) { * Implements hook_civicrm_navigationMenu(). */ function stripe_civicrm_navigationMenu(&$menu) { - _stripe_civix_insert_navigation_menu($menu, 'Administer/CiviContribute', array( + _stripe_civix_insert_navigation_menu($menu, 'Administer/CiviContribute', [ 'label' => E::ts('Stripe Settings'), 'name' => 'stripe_settings', 'url' => 'civicrm/admin/setting/stripe', 'permission' => 'administer CiviCRM', 'operator' => 'OR', 'separator' => 0, - )); + ]); _stripe_civix_navigationMenu($menu); }