Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0037547 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Retail Modules] Web POS | major | always | 2017-12-26 13:18 | 2018-02-16 17:15 | |||
| Reporter | malsasua | View Status | public | |||||
| Assigned To | rqueralta | |||||||
| Priority | normal | Resolution | fixed | Fixed in Version | RR18Q2 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | 824c1b0ddcba | ||||
| Projection | none | ETA | none | Target Version | ||||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Merge Request Status | ||||||||
| Review Assigned To | marvintm | |||||||
| OBNetwork customer | OBPS | |||||||
| Support ticket | 49703 | |||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0037547: Cashup invoice is not including the layaways, but it is including the canceled layaways | |||||||
| Description | When a layaway is created, and it is not paid totally, the layaway is not included in the invoice created by cashup. If you cancel the layaway (using "cancel layaway" or "cancel & replace" functionality) the layaway that it is cancelling the original layaway is included in the invoice of cashup, but the original layaway is not included in it. | |||||||
| Steps To Reproduce | POS: . do a cashup (so we start with a clean cashup) . create a new layaway - L1 (total 100 - paid 20) . do a cashup BO: . no invoice is created in the cashup. L1 is not invoiced POS: . load previous layaway L1 . execute "cancel layaway" functionality > L2 is created with amount -100 (total paid -20) . do a cashup BO: . invoice is created with total amount -100, and L2 is related with the invoice, but L1 never is invoiced | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
||||||||||||||||||||||
|
||||||||||||||||||||||
 Relationships |
|
Notes |
|
|
(0101686) hgbot (developer) 2018-01-15 04:37 |
Repository: erp/pmods/org.openbravo.retail.posterminal Changeset: 824c1b0ddcba362a1dafb01aa8e9f7420a2d2913 Author: Rafael Queralta Pozo <rqueralta <at> nauta.cu> Date: Sat Jan 13 23:40:52 2018 -0500 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/824c1b0ddcba362a1dafb01aa8e9f7420a2d2913 [^] Fixed issue 37547: Cashup invoice is not including the layaways, but it is including the canceled layaways --- M src/org/openbravo/retail/posterminal/OrderGroupingProcessor.java --- |
|
(0101687) marvintm (viewer) 2018-01-15 09:34 |
The fix is not correct. Instead, the only thing that is needed is to set the cashupId properly in the original layaway if it is different from the cancelling layaway, so that the cashup process takes both of them into account. |
|
(0101708) hgbot (developer) 2018-01-15 17:36 |
Repository: erp/pmods/org.openbravo.retail.posterminal Changeset: 2ea70517ede6239eb46102db3650687c4b537a7a Author: Rafael Queralta Pozo <rqueralta <at> nauta.cu> Date: Mon Jan 15 11:33:54 2018 -0500 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/2ea70517ede6239eb46102db3650687c4b537a7a [^] Related to issue 37547: Cashup invoice is not including the layaways, but it is including the canceled layaways --- M src/org/openbravo/retail/posterminal/OrderGroupingProcessor.java --- |
|
(0101731) hgbot (developer) 2018-01-16 20:37 |
Repository: erp/pmods/org.openbravo.retail.posterminal Changeset: fea83e2f893f4bc1a1905d6dcad67d317c9c0f8b Author: Rafael Queralta Pozo <rqueralta <at> nauta.cu> Date: Mon Jan 15 16:25:11 2018 -0500 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/fea83e2f893f4bc1a1905d6dcad67d317c9c0f8b [^] Related to issue 37547: Cashup invoice is not including the layaways, but it is including the canceled layaways --- M src/org/openbravo/retail/posterminal/CancelLayawayLoader.java M src/org/openbravo/retail/posterminal/OrderGroupingProcessor.java M src/org/openbravo/retail/posterminal/OrderLoader.java --- |
|
(0102469) hgbot (developer) 2018-02-16 17:15 |
Repository: tools/automation/pi-mobile Changeset: ab37a2b36d17b0988edb657d1412a3f95b83a2da Author: Alejandro <alekosmp86 <at> gmail.com> Date: Thu Feb 15 17:35:15 2018 -0500 URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/ab37a2b36d17b0988edb657d1412a3f95b83a2da [^] Verifies issue 37547: Added automated test "I37547_CashupInvoiceIsNotIncludingTheLayaways" --- A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/cashup/I37547_CashupInvoiceIsNotIncludingTheLayaways.java --- |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2017-12-26 13:18 | malsasua | New Issue | |
| 2017-12-26 13:18 | malsasua | Assigned To | => Retail |
| 2017-12-26 13:18 | malsasua | OBNetwork customer | => Yes |
| 2017-12-26 13:18 | malsasua | Support ticket | => 49703 |
| 2017-12-26 13:18 | malsasua | Resolution time | => 1515884400 |
| 2017-12-26 13:18 | malsasua | Triggers an Emergency Pack | => No |
| 2017-12-26 18:12 | Practics | Issue Monitored: Practics | |
| 2018-01-03 21:53 | rqueralta | Assigned To | Retail => rqueralta |
| 2018-01-03 21:53 | rqueralta | Status | new => scheduled |
| 2018-01-15 04:37 | hgbot | Checkin | |
| 2018-01-15 04:37 | hgbot | Note Added: 0101686 | |
| 2018-01-15 04:37 | hgbot | Status | scheduled => resolved |
| 2018-01-15 04:37 | hgbot | Resolution | open => fixed |
| 2018-01-15 04:37 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/824c1b0ddcba362a1dafb01aa8e9f7420a2d2913 [^] |
| 2018-01-15 09:34 | marvintm | Note Added: 0101687 | |
| 2018-01-15 09:34 | marvintm | Status | resolved => new |
| 2018-01-15 09:34 | marvintm | Resolution | fixed => open |
| 2018-01-15 09:35 | marvintm | Status | new => scheduled |
| 2018-01-15 17:36 | hgbot | Checkin | |
| 2018-01-15 17:36 | hgbot | Note Added: 0101708 | |
| 2018-01-16 20:37 | hgbot | Checkin | |
| 2018-01-16 20:37 | hgbot | Note Added: 0101731 | |
| 2018-01-16 20:38 | rqueralta | Status | scheduled => resolved |
| 2018-01-16 20:38 | rqueralta | Resolution | open => fixed |
| 2018-01-18 18:49 | marvintm | Review Assigned To | => marvintm |
| 2018-01-18 18:49 | marvintm | Status | resolved => closed |
| 2018-01-18 18:49 | marvintm | Fixed in Version | => RR18Q2 |
| 2018-01-22 11:28 | marvintm | Status | closed => new |
| 2018-01-22 11:28 | marvintm | Resolution | fixed => open |
| 2018-01-22 11:28 | marvintm | Fixed in Version | RR18Q2 => |
| 2018-01-22 11:28 | marvintm | Status | new => scheduled |
| 2018-01-22 11:28 | marvintm | Status | scheduled => resolved |
| 2018-01-22 11:28 | marvintm | Fixed in Version | => RR18Q2 |
| 2018-01-22 11:28 | marvintm | Resolution | open => fixed |
| 2018-01-22 11:28 | marvintm | Status | resolved => closed |
| 2018-02-16 17:15 | hgbot | Checkin | |
| 2018-02-16 17:15 | hgbot | Note Added: 0102469 | |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |