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 fatal error when using the "Renew - Credit Card" form to renew a membership (only for certain membership setups) #14391

Closed
wants to merge 1 commit into from

Conversation

alifrumin
Copy link
Contributor

Overview

This fixes a fatal error when renewing a membership on the backend thru the "Renew - Credit Card" form

Before

Fatal error
erroronRenewCreditCard

After

Form works as expected

Technical Details

In some instances the receive date was not being set on the contribution which would lead to a fatal error when submitting the form. This ensures the receive date is set.

@civibot
Copy link

civibot bot commented May 31, 2019

(Standard links)

@civibot civibot bot added the master label May 31, 2019
@seamuslee001
Copy link
Contributor

@alifrumin what version was this causing problems on?

@alifrumin
Copy link
Contributor Author

alifrumin commented May 31, 2019

I was able to replicate it on 5.14.beta1, however only when renewing a kind of odd membership set up.

@alifrumin alifrumin changed the title Fix fatal error when using the "Renew - Credit Card" form to renew a membership Fix fatal error when using the "Renew - Credit Card" form to renew a membership (only for certain membership setups) May 31, 2019
@eileenmcnaughton
Copy link
Contributor

@alifrumin is this on master too - it looks similar to this fix #14316 that was merged to master 6 days ago (I never hit the bug in the UI - only in tests)

@seamuslee001
Copy link
Contributor

@alifrumin @eileenmcnaughton i ran into this bug on AUG systems and tested Eileen's patch and confirmed that fixed it in our case. @eileenmcnaughton wanna do a RC backport of 14316?

@eileenmcnaughton
Copy link
Contributor

Closing in favour of #14412

@alifrumin
Copy link
Contributor Author

Thanks @eileenmcnaughton and @seamuslee001 I just tested and #14412 fixes this for our use case too.

@alifrumin alifrumin closed this Jun 3, 2019
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.

3 participants