Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0026469 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Retail Modules] Web POS | critical | always | 2014-05-06 11:08 | 2014-10-13 13:20 | |||
| Reporter | jecharri | View Status | public | |||||
| Assigned To | migueldejuana | |||||||
| Priority | immediate | Resolution | fixed | Fixed in Version | RR14Q2.1 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | |||||
| Projection | none | ETA | none | Target Version | RR14Q2.1 | |||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Review Assigned To | marvintm | |||||||
| Regression level | Production - QA Approved | |||||||
| Regression date | ||||||||
| Regression introduced in release | RR14Q2 | |||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0026469: Cash up is not working fine if you have a payment method with checkbox automated movement unchecked | |||||||
| Description | Cash up is not working fine if you have a payment method with checkbox automated movement unchecked | |||||||
| Steps To Reproduce | -With out sample data create a terminal to org Vall Blanca store -add credit card payment type -go to payment method and uncheck "Automated movement to other account" after that: -go to web pos -make a sale -go to cash up and execute You will see that Web POS is keeping all the money in the financial account when it should move all the money to back office financial account | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
||||||||
|
||||||||
 Relationships |
|
Notes |
|
|
(0067032) mtaal (manager) 2014-05-08 21:30 |
Remark by MAL: I have checking the issue. It was generated by a synchronization problem. In one function was added a called to OB.Dal.find function, then this function was changed to asynchronous The OB.Dal.find was added In the line https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/file/4e0e6c31d0a1/web/org.openbravo.retail.posterminal/js/closecash/model/cashup-model.js#l53 [^] and then, the line https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/file/4e0e6c31d0a1/web/org.openbravo.retail.posterminal/js/closecash/model/cashup-model.js#l102 [^] is executed before than the variable this.get('paymentList') is populated (in the line https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/file/4e0e6c31d0a1/web/org.openbravo.retail.posterminal/js/closecash/model/cashup-model.js#l87 [^]) and the setIgnoreStep3 function uses this variable, and it was empty. The fix is to call to setIgnoreStep3 function after that the variable is populated. |
|
(0067156) migueldejuana (developer) 2014-05-15 16:08 |
Repository: erp/pmods/org.openbravo.retail.posterminal Changeset: 007e96442b7388fbb0bc9c32c05ee1dac489dbd8 Author: Miguel de Juana <miguel.dejuana <at> openbravo.com> Date: Thu May 15 16:01:28 2014 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/007e96442b7388fbb0bc9c32c05ee1dac489dbd8 [^] [^] Fixed issue 0026470: The exchange of the sum of units is different from the sum of the individually exchanged units --- M web/org.openbravo.retail.posterminal/js/closecash/model/cashup-model.js --- |
|
(0070904) hgbot (developer) 2014-10-13 13:20 |
Repository: erp/pmods/org.openbravo.retail.posterminal Changeset: d2f5112566804a30c92fd0961a587b868f65758e Author: Miguel de Juana <miguel.dejuana <at> openbravo.com> Date: Mon Oct 13 13:19:48 2014 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/d2f5112566804a30c92fd0961a587b868f65758e [^] Related to issue 26469 and related to issue 26470: It seems we include removed and unnecesary code in one merge. We remove calls to setIgnoreStep3 and convertExpected because we already do this in the callback. Those calls need to be in the callback because after cashup refactor(saving cashup info in localStorage) we do an asynchronius call. --- M web/org.openbravo.retail.posterminal/js/closecash/model/cashup-model.js --- |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2014-05-06 11:08 | jecharri | New Issue | |
| 2014-05-06 11:08 | jecharri | Assigned To | => malsasua |
| 2014-05-06 11:08 | jecharri | Triggers an Emergency Pack | => No |
| 2014-05-06 11:10 | jecharri | Regression level | => Packaging and release |
| 2014-05-06 11:10 | jecharri | Steps to Reproduce Updated | View Revisions |
| 2014-05-07 14:45 | shuehner | Target Version | RR14Q2 => RR14Q2.1 |
| 2014-05-08 19:25 | dmitry_mezentsev | Relationship added | related to 0025569 |
| 2014-05-08 21:30 | mtaal | Note Added: 0067032 | |
| 2014-05-09 11:30 | malsasua | Assigned To | malsasua => Orekaria |
| 2014-05-15 16:08 | migueldejuana | Status | new => scheduled |
| 2014-05-15 16:08 | migueldejuana | Assigned To | Orekaria => migueldejuana |
| 2014-05-15 16:08 | migueldejuana | fix_in_branch | => pi |
| 2014-05-15 16:08 | migueldejuana | Note Added: 0067156 | |
| 2014-05-15 16:08 | migueldejuana | Status | scheduled => resolved |
| 2014-05-15 16:08 | migueldejuana | Fixed in Version | => RR14Q2.1 |
| 2014-05-15 16:08 | migueldejuana | Resolution | open => fixed |
| 2014-05-16 11:56 | marvintm | Review Assigned To | => marvintm |
| 2014-05-16 11:56 | marvintm | Status | resolved => closed |
| 2014-10-01 16:32 | plujan | Regression level | Packaging and release => Production - QA Approved |
| 2014-10-01 16:32 | plujan | Regression introduced in release | => RR14Q2 |
| 2014-10-01 16:32 | plujan | fix_in_branch | pi => |
| 2014-10-13 13:20 | hgbot | Checkin | |
| 2014-10-13 13:20 | hgbot | Note Added: 0070904 | |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |