cancel
Showing results for 
Search instead for 
Did you mean: 

Poor implementation of product methods

0 Kudos

Poor implementation of product methods

Feature request from SamTay, posted on GitHub Apr 13, 2016

  1. Product::isInStock is misleading. This does not check if product is in stock, but just checks if enabled.

  2. Product::getQty does not look for qty key in the correct place. Perhaps in some parts of the app, qty is set on the Product::_data array directly, but my screenshot shows that this is not the case in the product/price/amount/default.phtml template. Because this is the implementation of the SaleableInterface, this is a problem for abstraction. 2016-04-13-170441_979x1134_scrot