Skip to content
Snippets Groups Projects
Commit e6fa943d authored by mattwire's avatar mattwire
Browse files

Fix #239 hide configuration fields that we do not use

parent 41e0579d
No related branches found
No related tags found
1 merge request!1246.5
......@@ -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.
......
......@@ -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);
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment