Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0036803 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Retail Modules] Web POS | major | have not tried | 2017-09-07 10:47 | 2017-09-08 15:08 | |||
| Reporter | marvintm | View Status | public | |||||
| Assigned To | ranjith_qualiantech_com | |||||||
| Priority | normal | Resolution | fixed | Fixed in Version | RR17Q4 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | c390dfed1e4f | ||||
| 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 | ||||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0036803: Pay Open Tickets doesn't handle correctly the case of a receipt failing some check in the PreOrderSave hook | |||||||
| Description | There is a problem when using Pay Open Tickets, if one of the tickets fails in the middle of completion, when using synchronized mode. The problem can be reproduced when using Digital Coupons, but most likely can also happen with any other module that uses the PreOrderSave hook and cancels the process. | |||||||
| Steps To Reproduce | - Install digital coupons, and configure Synchronized mode - Log in the Web POS, and create three layaways. - Execute Pay Open Tickets, and select the three layaways. - Add the same digital coupon twice (can be achieved by using the scanner to scan the same coupon) - Add the rest of the payment in cash - Click on DONE. The process will start, and at some point you will get an error message explaining that the same coupon was added twice. - Click on DONE again, without doing any changes. Instead of seeing the same error message, some of the orders will be synchronized to the backend (the ones paid using cash), and the Web POS will navigate to the main page, without showing any order. <- This is wrong. - If you then refresh the page, you will see the missing ticket, with its payments intact, but still not synchronized to the backend. | |||||||
| Proposed Solution | The cause of this problem is that in dataordersave.js, inside the multiOrdersFunction flow, the multiorder variable me.ordersToSend is incremented when initially the process is executed, but then when it fails it's not reset to 0, so the next time the process is called, instead of starting in 0, it starts in a different position, which causes the runSyncProcess call to be executed too early. The solution is just to reset this variable when the process is started again. | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
|
 Relationships |
|
Notes |
|
|
(0098888) hgbot (developer) 2017-09-07 14:00 |
Repository: erp/pmods/org.openbravo.retail.posterminal Changeset: c390dfed1e4f5638c6b5035d407da4a595bd1171 Author: Ranjith S R <ranjith <at> qualiantech.com> Date: Thu Sep 07 17:29:54 2017 +0530 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/c390dfed1e4f5638c6b5035d407da4a595bd1171 [^] Fixed issue 36803 : Reset ordersToSend while Completing Multi Orders --- M web/org.openbravo.retail.posterminal/js/data/dataordersave.js --- |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2017-09-07 10:47 | marvintm | New Issue | |
| 2017-09-07 10:47 | marvintm | Assigned To | => Retail |
| 2017-09-07 10:47 | marvintm | OBNetwork customer | => No |
| 2017-09-07 10:47 | marvintm | Triggers an Emergency Pack | => No |
| 2017-09-07 10:48 | marvintm | OBNetwork customer | No => Yes |
| 2017-09-07 10:48 | marvintm | Resolution time | => 1505944800 |
| 2017-09-07 10:54 | marvintm | Proposed Solution updated | |
| 2017-09-07 11:31 | ranjith_qualiantech_com | Assigned To | Retail => ranjith_qualiantech_com |
| 2017-09-07 11:31 | ranjith_qualiantech_com | Status | new => scheduled |
| 2017-09-07 14:00 | hgbot | Checkin | |
| 2017-09-07 14:00 | hgbot | Note Added: 0098888 | |
| 2017-09-07 14:00 | hgbot | Status | scheduled => resolved |
| 2017-09-07 14:00 | hgbot | Resolution | open => fixed |
| 2017-09-07 14:00 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/c390dfed1e4f5638c6b5035d407da4a595bd1171 [^] |
| 2017-09-08 15:08 | marvintm | Review Assigned To | => marvintm |
| 2017-09-08 15:08 | marvintm | Status | resolved => closed |
| 2017-09-08 15:08 | marvintm | Fixed in Version | => RR17Q4 |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |