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
2 Comments
apiuser
New Member

Comment from piotrekkaminski, posted on GitHub May 24, 2016

@antonkril can you take a look?

apiuser
New Member

Comment from antonkril, posted on GitHub Jul 15, 2016

In the mid term these methods will be moved from product interface.