Magento2: Link at checkout not working for "What Is PayPal?"

Created on 2 Sep 2016  路  9Comments  路  Source: magento/magento2

At the checkout the built-in link What is PayPal opens an _empty_ pop-up window. This is bad for customers as the link is supposed to gain their confidence. Using IE11 the window shows the 404 page but is completely blank using Firefox and Chrome.

Screenshot of checkout page:

magento-paypal1

Screenshot of pop-up after clicking on "What is PayPal?":

magento-paypal2

Config: Magento 2.1.0
Theme: Luma (default, unedited)
Payment method: PayPal Website Payments Standard

How can this be fixed please?

Payment Format is not valid Ready for Work bug report

Most helpful comment

Why is it still broken in 2.1.8 if it was merged to develop in April?

All 9 comments

@gazzaf
Thank you for reporting this issue!
Please, format this issue according to the Issue reporting guidelines.

I've uninstalled Magento 2.1.x because of the enormous number of bugs and slowness that make it unfit for production and "upgraded" to 1.9.x. Sorry.

Hi @gazzaf! Thank you for your report, we've create internal ticket MAGETWO-58327 to fix this issue.

Quick fix:

In:

  • vendor/magento/module-paypal/view/frontend/web/template/payment/payflow-express.html
  • vendor/magento/module-paypal/view/frontend/web/template/payment/paypal-express-in-context.html
  • vendor/magento/module-paypal/view/frontend/web/template/payment/paypal-express.html

Replace
<a data-bind="attr: {href: getPaymentAcceptanceMarkHref()}, click: showAcceptanceWindow"

With
<a data-bind="attr: {href: getPaymentAcceptanceMarkHref()}" target="_blank"

Run:

Please rewirte paypal-express-abstract.js

app/design/frontend/Magento/theme/Magento_Paypal/web/js/view/payment/method-renderer/paypal-express-abstract.js
in line 43:
change
$(event.target).attr('href') 
to
$(event.target).parent().attr('href')

You can resolve it.

Correct fix should be
app/design/frontend/Magento/theme/Magento_Paypal/web/js/view/payment/method-renderer/paypal-express-abstract.js in line 43: change $(event.target).attr('href') to $(event.currentTarget).attr('href')

Hi @gazzaf

Fix for MAGETWO-58327 is merged to develop branch, closing the issue.
Thank you!

Why is it still broken in 2.1.8 if it was merged to develop in April?

@alena-marchenko Which version of 2.1 will this fix finally be included in?

Was this page helpful?
0 / 5 - 0 ratings