Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0024305 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Retail Modules] Web POS | minor | have not tried | 2013-07-10 12:34 | 2013-09-13 11:49 | |||
Reporter | adrianromero | View Status | public | |||||
Assigned To | marvintm | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | ||||
Status | closed | Fix in branch | Fixed in SCM revision | f400287ff6cd | ||||
Projection | none | ETA | none | Target Version | RMP27 | |||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Merge Request Status | ||||||||
Review Assigned To | umartirena | |||||||
OBNetwork customer | OBPS | |||||||
Support ticket | 21900 | |||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0024305: Discretionary discounts may produce a discount applied not only to the receipt line. | |||||||
Description | For example in the following case 1 x Ale beer 9$ --> Discount/Promotion fixed 10$ applied 1 x Lager beer 8$ --> No discount applied The total is now 7$. This is not correct, the total should be 8$, if the discount applied to a line is greater than the total of the line, the total of the line should be 0 and not spread the excess to the rest of the order | |||||||
Steps To Reproduce | In description | |||||||
Proposed Solution | When applying a discount it is not completely correct to verify that the total of the receipt is greater or equal than 0, it should be verified that the total of the line is greater or equal than 0. | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
![]() |
||||||||
|
![]() |
|
(0060137) hgbot (developer) 2013-07-22 09:58 |
Repository: erp/pmods/org.openbravo.retail.discounts Changeset: f0bd2f349385c18d24f56fe65eac9cd889972c4a Author: Guillermo Alvarez de Eulate <guillermo.alvarez <at> openbravo.com> Date: Mon Jul 22 09:58:10 2013 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.discounts/rev/f0bd2f349385c18d24f56fe65eac9cd889972c4a [^] fixed issue 24305: The qty to be discounted cannot be greater than line gross --- M web/org.openbravo.retail.discounts/js/promotion-manual.js --- |
(0060289) hgbot (developer) 2013-07-24 19:07 |
Repository: erp/pmods/org.openbravo.retail.discounts Changeset: 04651ac9f07e69a24eb5d9d42794beb4f789736f Author: Guillermo Alvarez de Eulate <guillermo.alvarez <at> openbravo.com> Date: Wed Jul 24 19:07:32 2013 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.discounts/rev/04651ac9f07e69a24eb5d9d42794beb4f789736f [^] related to issue 24305: revert previous commit because is wrong --- M web/org.openbravo.retail.discounts/js/promotion-manual.js --- |
(0060290) guilleaer (viewer) 2013-07-24 19:09 |
wrong fix. http://code.openbravo.com/erp/pmods/org.openbravo.retail.discounts/rev/04651ac9f07e69a24eb5d9d42794beb4f789736f [^] When affected lines are executed in the second time, we can't know if other discounts are being applied. |
(0060520) hgbot (developer) 2013-08-07 08:53 |
Repository: erp/pmods/org.openbravo.retail.discounts-col Changeset: f0bd2f349385c18d24f56fe65eac9cd889972c4a Author: Guillermo Alvarez de Eulate <guillermo.alvarez <at> openbravo.com> Date: Mon Jul 22 09:58:10 2013 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.discounts-col/rev/f0bd2f349385c18d24f56fe65eac9cd889972c4a [^] fixed issue 24305: The qty to be discounted cannot be greater than line gross --- M web/org.openbravo.retail.discounts/js/promotion-manual.js --- |
(0060521) hgbot (developer) 2013-08-07 08:53 |
Repository: erp/pmods/org.openbravo.retail.discounts-col Changeset: 04651ac9f07e69a24eb5d9d42794beb4f789736f Author: Guillermo Alvarez de Eulate <guillermo.alvarez <at> openbravo.com> Date: Wed Jul 24 19:07:32 2013 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.discounts-col/rev/04651ac9f07e69a24eb5d9d42794beb4f789736f [^] related to issue 24305: revert previous commit because is wrong --- M web/org.openbravo.retail.discounts/js/promotion-manual.js --- |
(0060523) guilleaer (viewer) 2013-08-07 11:48 |
reopening |
(0061102) hgbot (developer) 2013-09-12 13:49 |
Repository: erp/pmods/org.openbravo.retail.discounts Changeset: f400287ff6cd6a321c4cb52cf360970c17b65ed6 Author: Antonio Moreno <antonio.moreno <at> openbravo.com> Date: Thu Sep 12 13:49:09 2013 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.discounts/rev/f400287ff6cd6a321c4cb52cf360970c17b65ed6 [^] Fixed issue 24305. If the discountedPrice of the line is lower than the discount, the discount will be set to the remaining total of the line. --- M web/org.openbravo.retail.discounts/js/promotion-discount.js M web/org.openbravo.retail.discounts/js/promotion-manual.js --- |
(0061134) umartirena (viewer) 2013-09-13 11:49 |
Successfully tested and closed. |
![]() |
|||
Date Modified | Username | Field | Change |
2013-07-10 12:34 | adrianromero | New Issue | |
2013-07-10 12:34 | adrianromero | Assigned To | => marvintm |
2013-07-10 12:34 | adrianromero | OBNetwork customer | => Yes |
2013-07-10 12:34 | adrianromero | Support ticket | => 21900 |
2013-07-10 12:40 | adrianromero | Relationship added | related to 0024145 |
2013-07-16 11:18 | anherzog | Issue Monitored: anherzog | |
2013-07-16 18:16 | jecharri | Resolution time | => 1378332000 |
2013-07-22 09:58 | hgbot | Checkin | |
2013-07-22 09:58 | hgbot | Note Added: 0060137 | |
2013-07-22 09:58 | hgbot | Status | new => resolved |
2013-07-22 09:58 | hgbot | Resolution | open => fixed |
2013-07-22 09:58 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.retail.discounts/rev/f0bd2f349385c18d24f56fe65eac9cd889972c4a [^] |
2013-07-24 19:07 | hgbot | Checkin | |
2013-07-24 19:07 | hgbot | Note Added: 0060289 | |
2013-07-24 19:09 | guilleaer | Note Added: 0060290 | |
2013-07-24 19:09 | guilleaer | Status | resolved => new |
2013-07-24 19:09 | guilleaer | Resolution | fixed => open |
2013-08-07 08:53 | hgbot | Checkin | |
2013-08-07 08:53 | hgbot | Note Added: 0060520 | |
2013-08-07 08:53 | hgbot | Status | new => resolved |
2013-08-07 08:53 | hgbot | Resolution | open => fixed |
2013-08-07 08:53 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/pmods/org.openbravo.retail.discounts/rev/f0bd2f349385c18d24f56fe65eac9cd889972c4a [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.discounts-col/rev/f0bd2f349385c18d24f56fe65eac9cd889972c4a [^] |
2013-08-07 08:53 | hgbot | Checkin | |
2013-08-07 08:53 | hgbot | Note Added: 0060521 | |
2013-08-07 11:48 | guilleaer | Note Added: 0060523 | |
2013-08-07 11:48 | guilleaer | Status | resolved => new |
2013-08-07 11:48 | guilleaer | Resolution | fixed => open |
2013-08-19 12:08 | marvintm | Triggers an Emergency Pack | => No |
2013-08-19 12:08 | marvintm | Target Version | RMP26 => RMP27 |
2013-09-12 13:49 | hgbot | Checkin | |
2013-09-12 13:49 | hgbot | Note Added: 0061102 | |
2013-09-12 13:49 | hgbot | Status | new => resolved |
2013-09-12 13:49 | hgbot | Resolution | open => fixed |
2013-09-12 13:49 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/pmods/org.openbravo.retail.discounts-col/rev/f0bd2f349385c18d24f56fe65eac9cd889972c4a [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.discounts/rev/f400287ff6cd6a321c4cb52cf360970c17b65ed6 [^] |
2013-09-13 11:49 | umartirena | Review Assigned To | => umartirena |
2013-09-13 11:49 | umartirena | Note Added: 0061134 | |
2013-09-13 11:49 | umartirena | Status | resolved => closed |
Copyright © 2000 - 2009 MantisBT Group |