View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0003942||OXID eShop (all versions)||1.03. Basket, checkout process||public||2012-04-26 16:32||2012-12-10 13:22|
|Product Version||4.5.6 revision 40808|
|Target Version||Fixed in Version||4.6.6 revision 54646|
|Summary||0003942: Email notifying for low stock articles does not work for variants|
|Description||In a shop we have only parent articles with variants. There is the option set to remind the admin if the stock is low. It works for main products but does not work for variants. |
We discovered that the selectString() in oxArticleList::loadStockRemindProducts does not get any articles if they are variants.
can confirm this - please fix this officially
(Version CE 4.5.9)
||how could you fix that un-officially?|
The obvious problem is, that 'oxremindactive' is not set for variants in the oxarticles table. This value is intended to be "inherited" from parent, and therefore not set in the database for a variant. So the select-query cannot return any value different than '0'.
So a quick'n'dirty solution is to manually set 'oxremindactive' to 1 for all variants (as I did as an SQL-hack), or patch this officially:
1.) make the select-query more complex to look for parent-setting if article has a parentid. Also update-query, of course.
2.) Rethink parent-inheritage concerning Reminders completely and make this setting individual for each variant. (like the OXREMINDAMOUNT setting already is - would be my preferred solution if anyone'd ask me ;-))
3.) Update every Variant-Setting when parent-setting is updated in the database. (a little bit crappy solution ;-))
||Hi, we have added updating of variant article to remindactive of parent item on admin/stock save. Now if you would like to have functionality in updown 2) suggestion, just remove editval[oxarticles__oxremindactive] input disabled tag in article_stock.tpl and it should be fine.|