View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0002111 | OXID eShop (all versions) | 1.03. Basket, checkout process | public | 2010-09-17 21:13 | 2012-12-10 14:13 |
| Reporter | vgoebbels | Assigned To | |||
| Priority | high | Severity | major | Reproducibility | always |
| Status | resolved | Resolution | no change required | ||
| Product Version | 4.4.2 revision 29492 | ||||
| Summary | 0002111: Delivery calculation for category restricted delivery rules doesn't work with variants | ||||
| Description | When a delivery rule is restricted to one or more categories this doesn't work with varaints. | ||||
| Steps To Reproduce | Create a delivery rule bound to a category which contains articles with variants (like the ECO fashion cat in the demo data set). Then add a jeans to the basket. The restricted rule will not be used at all. | ||||
| Additional Information | oxDelivery::isForBasket() creates an oxarticle object for each basket item to check against the categories. For each item it then checks via $oProduct->inCategory($sCatId) if the item is conatined in the category of the delivery rule. At this point oxArticle::inCategory() is called, NOT oxSimpleVariant::inCategory(). Since the OXID of the variant is _not_ containted in oxobject2category but the OXID of the parent article, this method always returns false. This will never work for variants :) | ||||
| Tags | No tags attached. | ||||
| Theme | |||||
| Browser | All | ||||
| PHP Version | any | ||||
| Database Version | any | ||||
|
|
Problem is not in variant/category bindings. If you would check source code, you would see, that oxArticle::inCategory() does parent check. I think the problem is in delivery and category check. Imagine situation: CatA |-SubCatB Product Z is in SubCatB Delivery X assigned to CatA Delivery Y assigned to SubCatB Basket contains: Product Z + Delivery Y costs, but no Delivery X costs Problem is that delivery category check does not include parent categories.. |
|
|
As a solution for this case - the new option is needed, which would define if assignment of categories should be inherited to sub-categories or not. This is registered in list of feature requests: http://oxid.uservoice.com/forums/31940-feature-requests/suggestions/1261865-option-for-inheriting-assignments-to-sub-categorie?ref=title So entry here is closed. Thank you for your report. |
