Skip to content

[-] MO: Blocklayered, wrong price in category list #1317

Closed
wants to merge 1 commit into from

3 participants

@IceEyz
IceEyz commented Jan 31, 2014

Previous change adding the id_product_attribute into the select, caused wrong price in category list. By selecting the highest id, category shows price for that id, instead of the default one.

Now, if a product has combinations, only the default attribute combination is selected. As such, that price will be shown.

@IceEyz IceEyz [-] MO: Blocklayered, wrong price in category list
Previous change adding the id_product_attribute into the select, caused wrong price in category list. By selecting the highest id, category shows price for that id, instead of the default one.

Now, if a product has combinations, only the default attribute combination is selected. As such, that price will be shown.
3cbbd12
@dkarvounaris

I can confirm this patch works and is what should be the default behaviour.
I needed to fix the same issue, because my customer wanted not the price of some random combination, but the price of the default combination in product listings shown.

Relates to issues PNM-2049 and PNM-1189 too.

@IceEyz
IceEyz commented Jul 2, 2014

This has been fixed in blocklayered around line 2005 with a better commit than proposed.

@IceEyz IceEyz closed this Jul 2, 2014
@gRoussac
gRoussac commented Jul 2, 2014

Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.