View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0003092 | OXID eShop (all versions) | 1.01. Products (product, categories, manufacturer, promotions etc.) | public | 2011-07-28 09:02 | 2012-12-07 14:28 |
| Reporter | tjungcl | Assigned To | |||
| Priority | normal | Severity | minor | Reproducibility | always |
| Status | resolved | Resolution | suspended | ||
| Product Version | Past development | ||||
| Summary | 0003092: article: "not isBuyable()" does not equal "isNotBuyable()" | ||||
| Description | in productmain.tpl the basket-buttons are wrapped into an if block that checks: (!$oDetailsProduct->isNotBuyable()) There is also a function isBuyable() - however those functions behave differently with variant parents. As far as i can judge, the isBuyable() function is more correct because a notbuyableparent is not buyable. The isNotBuyable() function does not check the buyableparent property. | ||||
| Tags | Products | ||||
| Theme | Both | ||||
| Browser | All | ||||
| PHP Version | any | ||||
| Database Version | any | ||||
| related to | 0001655 | resolved | dainius.bigelis | Calling oxArticle::loadAmountPrice befor oxArticle::getBasketPrice leads to price = 0 |
|
|
@developers: check from source code side if such an issue exist |
|
|
Agree! Sadly it can't be fixed right now (must refactor source code and change many templates). In next major release we're going to refactor this class (others also) and improve getters which now returns only properties, by enabling them to do real job (calculations etc). Usage of "oxArticle::isBuyable()" is correct and recommended. |
|
|
see comment |
