From 9700812075da7bb9b0f8c84cb807409e9b68022f Mon Sep 17 00:00:00 2001 From: larssandergreen Date: Tue, 22 Aug 2023 12:12:38 -0600 Subject: [PATCH] Fix membership price sets regression --- CRM/Member/Form/MembershipBlock.php | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/CRM/Member/Form/MembershipBlock.php b/CRM/Member/Form/MembershipBlock.php index 9018e63bc481..7910ad9d4695 100644 --- a/CRM/Member/Form/MembershipBlock.php +++ b/CRM/Member/Form/MembershipBlock.php @@ -242,16 +242,16 @@ public static function formRule($params, $files, $contributionPageId = NULL) { } if (!empty($params['member_is_active'])) { // Don't allow Contribution price set w/ membership signup, CRM-5095. - $priceSetExtendsMembership = \Civi\Api4\PriceSetEntity::get(FALSE) + $priceSetNotExtendingMembership = \Civi\Api4\PriceSetEntity::get(FALSE) ->addSelect('id') ->addJoin('PriceSet AS price_set', 'LEFT', ['price_set_id', '=', 'price_set.id']) ->addWhere('entity_table', '=', 'civicrm_contribution_page') ->addWhere('entity_id', '=', $contributionPageId) - ->addWhere('price_set.extends:name', 'CONTAINS', 'CiviMember') + ->addWhere('price_set.extends:name', 'NOT CONTAINS', 'CiviMember') ->addWhere('price_set.is_quick_config', '=', 0) ->execute() ->first(); - if (!$priceSetExtendsMembership) { + if ($priceSetNotExtendingMembership) { $errors['member_is_active'] = ts('You cannot enable both Membership Signup and a Contribution Price Set on the same online contribution page.'); return $errors; }