View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0000673 | OXID eShop (all versions) | 1.03. Basket, checkout process | public | 2009-03-06 11:35 | 2012-12-10 13:24 |
Reporter | andreas_ziethen | Assigned To | |||
Priority | normal | Severity | major | Reproducibility | always |
Status | resolved | Resolution | duplicate | ||
Product Version | 4.0.1.0 revision 15990 | ||||
Summary | 0000673: wrong amount calculation on basket update | ||||
Description | the logic behind amount calculation in basket is wrong: Do the following: create an article with oxstock = 4. Put this article into your basket in shop with amount = 4. Now try to update the amount to "2". This does not work, you stay with 4 and you are not able to change this amount. Reason: wrong logic in function setAmount() in class oxbasketitem line 252: $iOnStock = $oArticle->checkForStock( $dAmount + $this->_dAmount ); For the example above this gives you: $iOnStock = $oArticle->checkForStock( 2 + 4); which obviously is wrong. It should check for "2" not for "6". Solution: // checking for stock if($blOverride) $dSumToCheck = $dAmount; else $dSumToCheck = $dAmount + $this->_dAmount; $iOnStock = $oArticle->checkForStock( $dSumToCheck ); | ||||
Tags | Order | ||||
Theme | |||||
Browser | |||||
PHP Version | 5.2.6 | ||||
Database Version | 5.0.33 | ||||
duplicate of | 0000604 | resolved | vilma_liorensaityte | wrong behavior in order step1, when trying to buy more product items than there is available |
|
I cannot reproduce this case from the GUI: when updating article to "2" in described case - it updates successfuly. Can be fixed together with fix 0000604. Please check from the source code side, if the problem may exist and fix if needed. |