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

Update Data.php to fix BasePrice on config Product #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

falkone
Copy link

@falkone falkone commented Nov 3, 2017

Magneto deliver different prices for $product->getFinalPrice() or $product->getPriceInfo()->getPrice('final_price')->getValue();
The Problem is for config products on getFinalPrice the "Dummy" price of parent is delivert and not the minimum price of the children.
"Dummy" price means when you create an config product you have to set an price for the parent because it is an requiered field. Normaly the "Dummy" price is not shown on frontend.

Magneto deliver different prices for $product->getFinalPrice() or $product->getPriceInfo()->getPrice('final_price')->getValue();
The Problem is for config products on getFinalPrice the "Dummy" price of parent is delivert and not the minimum price of the children.
"Dummy" price means when you create an config product you have to set an price for the parent because it is an requiered field. Normaly the "Dummy" price  is not shown on frontend.
@ghost ghost mentioned this pull request Dec 5, 2017
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

Successfully merging this pull request may close these issues.

1 participant