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 PayPalImpl Contribution Status PseudoConstant for non-English #15164

Merged
merged 1 commit into from
Aug 29, 2019

Conversation

mlutfy
Copy link
Member

@mlutfy mlutfy commented Aug 29, 2019

i.e. use the name property, not label.

Overview

When using PayPal Express, contributions were left pending when CiviCRM is not in English (or more specifically, when the label of the Contribution Status = Completed was not "Completed", ex: "Terminé"). It worked fine in English, since the name and label are the same.

Comments

I understand this Payment Processor is on its way out, to be replaced by PayPal Checkout, but we haven't finished testing yet, and had a client running into this bug.

@civibot
Copy link

civibot bot commented Aug 29, 2019

(Standard links)

@civibot civibot bot added the master label Aug 29, 2019
@mattwire
Copy link
Contributor

These just keep popping up!

@mlutfy
Copy link
Member Author

mlutfy commented Aug 29, 2019

yep, I think next time we add more Option Values, we should really make sure not to use name=label. At the very least, use lowercase for the name. That way the bugs will surface in English too.

@eileenmcnaughton
Copy link
Contributor

unrelated fail

@eileenmcnaughton eileenmcnaughton merged commit 5d8e2d9 into civicrm:master Aug 29, 2019
@eileenmcnaughton eileenmcnaughton deleted the ppIpnTranslation branch August 29, 2019 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants