From 1b1931152a86683bd792c6103f9ac8fb0c0bb3ad Mon Sep 17 00:00:00 2001 From: eileenmcnaugton Date: Mon, 28 Sep 2015 10:42:58 +1300 Subject: [PATCH] CRM-17289 limit membership_type entity reference to active types --- api/v3/MembershipType.php | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) diff --git a/api/v3/MembershipType.php b/api/v3/MembershipType.php index a2312fd956..42d4f94001 100644 --- a/api/v3/MembershipType.php +++ b/api/v3/MembershipType.php @@ -95,6 +95,25 @@ function civicrm_api3_membership_type_get($params) { return $results; } +/** + * Adjust input for getlist action. + * + * We want to only return active membership types for getlist. It's a bit + * arguable whether this should be applied at the 'get' level but, since it's hard + * to unset we'll just do it here. + * + * The usage of getlist is entity-reference fields & the like + * so using only active ones makes sense. + * + * @param array $request + * Array of parameters determined by getfields. + */ +function _civicrm_api3_membership_type_getlist_params(&$request) { + if (!isset($request['params']['is_active'])) { + $request['params']['is_active'] = 1; + } +} + /** * Deletes an existing membership type. * -- 2.25.1