Skip to content

Conversation

vshatylo
Copy link
Contributor

Description (*)

In html markup meta 'product:price:amount' has wrong price

Fixed Issues (if relevant)

  1. Product model in frontend does not contain price including tax #13957
  2. ...

Manual testing scenarios (*)

  1. update config:
  • sales -> tax -> calculation settings -> catalog prices set value 'excluding tax'
  • sales -> tax -> price display settings -> display product prices in catalog set value 'including tax'
  1. configure tax rule (e.g. 21%)
  2. visit product view page
  3. compare price displayed on frontend with price amount in html meta tag 'product:price:amount'

Contribution checklist (*)

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)

…uding tax ,display final price using pricing
@magento-engcom-team
Copy link
Contributor

Hi @vshatylo. Thank you for your contribution
Here is some useful tips how you can test your changes using Magento test environment.
Add the comment under your pull request to deploy test or vanilla Magento instance:

  • @magento-engcom-team give me test instance - deploy test instance based on PR changes
  • @magento-engcom-team give me 2.3-develop instance - deploy vanilla Magento instance

For more details, please, review the Magento Contributor Assistant documentation

@magento-engcom-team magento-engcom-team added Area: Frontend Component: Catalog Release Line: 2.3 Partner: ISM eCompany Pull Request is created by partner ISM eCompany partners-contribution Pull Request is created by Magento Partner labels Jan 30, 2019
@VladimirZaets VladimirZaets self-assigned this Jan 30, 2019
@magento-engcom-team
Copy link
Contributor

Hi @VladimirZaets, thank you for the review.
ENGCOM-4059 has been created to process this Pull Request

@milindsingh
Copy link
Member

Duplicate #20011

@tdgroot
Copy link
Member

tdgroot commented Jan 31, 2019

@vshatylo this fixes one part on the front-end, but I question if this is the right solution for the problem that I stated in issue #13957: product model prices are excluding tax when Magento is configured to enter prices excluding tag, but show them including tax on the frontend.

@vshatylo
Copy link
Contributor Author

@tdgroot getFinalPrice method in Product model left from old Magento version for compatibility.
New approach is to use pricing models for getting specific one e.g. final price, tier price

@irajneeshgupta
Copy link
Member

irajneeshgupta commented Jan 31, 2019

@vshatylo I think a PR is already created for this issue fix. #20011
This would be the correct fix.
$block->getProduct()->getPriceInfo()->getPrice(\Magento\Catalog\Pricing\Price\FinalPrice::PRICE_CODE)->getAmount()

@sidolov
Copy link
Contributor

sidolov commented Feb 5, 2019

Hi @vshatylo fix for such issue already provided in PR: #20011
I'm closing this PR as duplicate.
Thank you for the collaboration!

@sidolov sidolov closed this Feb 5, 2019
@ghost
Copy link

ghost commented Feb 5, 2019

Hi @vshatylo, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Frontend Component: Catalog duplicate Partner: ISM eCompany Pull Request is created by partner ISM eCompany partners-contribution Pull Request is created by Magento Partner Release Line: 2.3
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants