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

Braintree payment method does not display on the checkout using php 7 or above. #3797

Closed
benathey opened this issue Mar 15, 2016 · 10 comments
Closed
Assignees

Comments

@benathey
Copy link

Magento 2.0.2

Braintree payment method does not display on the checkout using php 7 or above.

Tested on apache and nginx using php 5.6, 7.0.0, 7.0.3.

Braintree payment method works fine on php 5.6.

Magento successfully generates tokens using php 5.6. but does not for 7 and above.
Php 7 and above encrypts the keys sent to braintree, braintree is unable to decrypt the keys, the tokens cant generate and the payment method will then not show on the checkout.

This would need to be resolved really quite quickly a massive blocker for anyone using magento2, php7 and braintree. Please advise if there is a workaround.

Cheers

@rudik99
Copy link

rudik99 commented Mar 16, 2016

this is the same issue i had for #3473 but was masked different PHP versions.
I am having this issue again since moving to php 7.0.3

@YevSent
Copy link
Contributor

YevSent commented Mar 25, 2016

Hi, @benathey, your issue perhaps related to problems which we already had with php 7.x. This issue fixed on develop branch.

@benathey
Copy link
Author

Great thanks.

@CKUDG
Copy link

CKUDG commented Mar 31, 2016

updated today to Magento 2.0.3 and it's still happening..

@YevSent
Copy link
Contributor

YevSent commented Mar 31, 2016

@CKUDG do you have any errors in logs or browser console? Sounds strange, because we also tested Braintree with php 7.0.x

@Doability
Copy link

Hi @joni-jones ,

I am also facing the same issue, below is the error I am getting in browser console:

TypeError: this.messageContainer is undefined his.messageContainer.addErrorMessage({

Could you post the name of commit where you have placed related changes.

Thanks & Regards

@CKUDG
Copy link

CKUDG commented Apr 6, 2016

Hi @joni-jones @Doability , sorry for the late answer... The following workaround/fix solves the problem, but i think it's still not marched into the latest version 2.0.3, right?

32ca5c9

Best regards,
Chris

@YevSent
Copy link
Contributor

YevSent commented Apr 11, 2016

Hi @CKUDG, unfortunately this fix is not merged into 2.0.4 version.

@YevSent
Copy link
Contributor

YevSent commented Apr 11, 2016

Hi @Doability, what changes do you mean? Your issue is still reproducible on the latest mainline version?

@alena-marchenko
Copy link

Closing this issue due to no response, please feel free to reopen when you have an update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants