Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for membership discounts not showing in 5.62 #26538

Merged
merged 1 commit into from
Jun 16, 2023

Conversation

eileenmcnaughton
Copy link
Contributor

@eileenmcnaughton eileenmcnaughton commented Jun 16, 2023

Overview

Fix for membership discounts not showing in 5.62

Before

A membership configured per the screen shot automatically is applied for a renewing member in 5.61 but not 5.62

image

After

Functionality is returned

Technical Details

The issue is due to #24765

In 5.61 membership price sets had a value of 3 in the extends column and in 5.62 (after the upgrade is run as @kewljuice cleverly spotted) the value is no longer 3 but 3 wrapped in custom separators & hence the match is not made, the form is not identifed to the hook as a 'membership' form and civi-discount does not leap into action.

I put the change in a function ONLY called by this code patch because it is the rc but it should move to a more generic spot in the BAO_Order class in master. Also note I renamed the context function on the form as it is doing the same thing as one differently named on similar forms. The function is newly added in 5.61

Comments

@civibot
Copy link

civibot bot commented Jun 16, 2023

(Standard links)

@seamuslee001
Copy link
Contributor

Jenkins re test this please

@composerjk
Copy link
Contributor

A quick test of the patch added to a 5.62.0 install properly showed the automatic discount on the contribution page and the confirmation page. Only did a simple quick test of those two pages. fyi.

@kewljuice
Copy link
Contributor

kewljuice commented Jun 16, 2023

I added the patch and did some test(s), everything works again!

@totten
Copy link
Member

totten commented Jun 16, 2023

Thanks @composerjk @kewljuice

@totten totten merged commit 60964d2 into civicrm:5.63 Jun 16, 2023
@totten totten deleted the member_fix_rc branch June 16, 2023 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants