-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
Wrong price amount in opengraph #20010
Comments
Hi @jcourtei. Thank you for your report.
Please make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, please, add a comment to the issue:
where @jcourtei do you confirm that you was able to reproduce the issue on vanilla Magento instance following steps to reproduce?
|
Hi @milindsingh. Thank you for working on this issue.
|
@magento-engcom-team give me 2.3-develop instance |
Hi @milindsingh. Thank you for your request. I'm working on Magento 2.3-develop instance for you |
Hi @milindsingh, here is your Magento instance. |
@magento-engcom-team give me 2.2-develop instance |
Hi @milindsingh. Thank you for your request. I'm working on Magento 2.2-develop instance for you |
Hi @milindsingh, here is your Magento instance. |
@milindsingh Thank you for verifying the issue. Based on the provided information internal tickets |
Issue fix magento#20010 Wrong price amount in opengraph
@magento-engcom-team give me 2.3-develop instance |
Hi @sdzhepa. Thank you for your request. I'm working on Magento 2.3-develop instance for you |
Hi @sdzhepa, here is your Magento instance. |
Hi @jcourtei. Thank you for your report. The fix will be available with the upcoming release. |
Issue fix magento#20010 Wrong price amount in opengraph
Hi @jcourtei. Thank you for your report. The fix will be available with the upcoming 2.2.9 release. |
The open graph price tag is still incorrect in 2.3.2 when Minimum Advertised Price is enabled. The price used will be Price, and not Manufacturer's Suggested Retail Price. Search engines like Google will use the open graph tag to override the price displayed on the page in their search results, which will cause you to violate any normal IMAP agreement. |
This is still an issue with grouped products on version 2.4.0, I'll update to 2.4.4 but I don't see any fixes that will help. My store is setup in British Pounds with 20% tax The "Set of Sprite Yoga Straps" returns: This should be: |
Preconditions (*)
Version 2.2.6 (Community Edition).
Steps to reproduce (*)
Expected result (*)
Meta tag "product:price:amount" should contain the price converted in USD.
Actual result (*)
Meta tag "product:price:amount" contains the price in EUR.
Meta tag "product:price:currency" on the other hand is correct (USD).
In module-catalog/view/frontend/templates/product/view/opengraph/general.phtml the following line is wrong :
<?php if ($priceAmount = $block->getProduct()->getFinalPrice()):?>
It should be :
<?php if ($priceAmount = $block->getProduct()->getPriceInfo()->getPrice('final_price')->getValue()):?>
The text was updated successfully, but these errors were encountered: