Tymoteusz Motylewski
Tymoteusz Motylewski
have you checked if n98-magerun has sth similar already?
what about group by and having ? should it also be taken into account?
This kind of change has a significant risk of introducing some side effects. Maybe we are missing some api or event which would make the whole architecture of your extension...
Nice idea! Please also provide which IM version you have tested it with (different IM versions have different quircks). Maybe a check in the code would also be useful? Do...
Hi @matteorossi-thespacesm Thanks for the PR! Can you please describe the steps I can take to reproduce the issue this PR is fixing? Thanks a lot!
@colinmollenhour should be better now :)
fixed issues reported by phpstan
If I see correctly Its not an no-brainer. The code has several side effects. $this->_preparePrice calls internally Mage_Catalog_Helper_Product_Type_Composite::preparePrice which calls $product->getFinalPrice() and Mage_Catalog_Helper_Product_Type_Composite::registerJsPrice and getFinalPrice sets 'final_price' field on product...
the same would be great for other labels like component, or evolution