Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0034058 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Retail Modules] Web POS | major | always | 2016-09-22 14:04 | 2016-11-24 12:52 | |||
Reporter | guillermogil | View Status | public | |||||
Assigned To | jorge-garcia | |||||||
Priority | immediate | Resolution | fixed | Fixed in Version | ||||
Status | closed | Fix in branch | Fixed in SCM revision | 7c08268959f6 | ||||
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 | migueldejuana | |||||||
OBNetwork customer | OBPS | |||||||
Support ticket | ||||||||
Regression level | Production - Confirmed Stable | |||||||
Regression date | ||||||||
Regression introduced in release | RR16Q2 | |||||||
Regression introduced by commit | https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/09dc9563011a [^] | |||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0034058: If deleteCurrent function forces to create a new order the document Sequence is wrongly generated | |||||||
Description | If deleteCurrent function (OB.MobileApp.model.orderlist.deleteCurrent(true)) forces to create a new order the document Sequence is wrongly generated. It is not taking into account the new orderList order when it is adding the new order to the orderList. | |||||||
Steps To Reproduce | Login on WebPOS Add a product New ticket Add a product New ticket Add a product Set the following command on the console: OB.MobileApp.model.orderList.deleteCurrent(true); Set the following command on the console: OB.MobileApp.model.orderList.deleteCurrent(true); You will see that on the orderList are several document numbers wrong | |||||||
Proposed Solution | See attached diff | |||||||
Tags | No tags attached. | |||||||
Attached Files | ![]() | |||||||
![]() |
|||||||||||||||
|
![]() |
|
(0090343) hgbot (developer) 2016-09-28 11:50 |
Repository: erp/pmods/org.openbravo.retail.posterminal Changeset: b57385efc7794027cd79ad0f954a0e4525265e20 Author: Jorge Garcia <jorge.garcia <at> openbravo.com> Date: Fri Sep 23 11:17:23 2016 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/b57385efc7794027cd79ad0f954a0e4525265e20 [^] Fixed issue 34058: If deleteCurrent function forces to create a new order the document Sequence is wrongly generated If the creation of a new order is forced, the sequence number must be updated correctly. --- M web/org.openbravo.retail.posterminal/js/model/order.js --- |
(0090344) hgbot (developer) 2016-09-28 11:50 |
Repository: tools/automation/pi-mobile Changeset: fe18f0fdae2acf4fd56120ccdbade18f301ddd4a Author: Jorge Garcia <jorge.garcia <at> openbravo.com> Date: Tue Sep 27 11:39:57 2016 +0200 URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/fe18f0fdae2acf4fd56120ccdbade18f301ddd4a [^] Verifies issue 34058: If deleteCurrent function forces to create a new order the document Sequence is wrongly generated Added test I34058_WrongDocSequenceWithDeleteCurrent. --- M src-test/org/openbravo/test/mobile/retail/pack/selenium/TestIdPack.java A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/documentno/I34058_WrongDocSequenceWithDeleteCurrent.java --- |
(0091352) hgbot (developer) 2016-11-11 08:49 |
Repository: retail/backports/3.0RR16Q2.4/org.openbravo.retail.posterminal Changeset: fda289a0a3c41e7061cad2610c9c93e9f8e677dd Author: Jorge Garcia <jorge.garcia <at> openbravo.com> Date: Fri Sep 23 11:17:23 2016 +0200 URL: http://code.openbravo.com/retail/backports/3.0RR16Q2.4/org.openbravo.retail.posterminal/rev/fda289a0a3c41e7061cad2610c9c93e9f8e677dd [^] Fixed issue 34070, backport of issue 34058: If deleteCurrent function forces to create a new order the document Sequence is wrongly generated If the creation of a new order is forced, the sequence number must be updated correctly. --- M web/org.openbravo.retail.posterminal/js/model/order.js --- |
(0091360) hgbot (developer) 2016-11-11 08:50 |
Repository: retail/backports/3.0RR16Q2.4/pi-mobile Changeset: f9f5fe09439c19e9f338839687697e42bf69050e Author: Jorge Garcia <jorge.garcia <at> openbravo.com> Date: Tue Sep 27 11:39:57 2016 +0200 URL: http://code.openbravo.com/retail/backports/3.0RR16Q2.4/pi-mobile/rev/f9f5fe09439c19e9f338839687697e42bf69050e [^] Verifies issue 34058: If deleteCurrent function forces to create a new order the document Sequence is wrongly generated Added test I34058_WrongDocSequenceWithDeleteCurrent. --- M src-test/org/openbravo/test/mobile/common/selenium/javascript/TestId.java A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/documentno/I34058_WrongDocSequenceWithDeleteCurrent.java --- |
(0091364) hgbot (developer) 2016-11-11 08:51 |
Repository: retail/backports/3.0RR16Q3.3/org.openbravo.retail.posterminal Changeset: 7c08268959f6bc11a90cd19a82689e3c5af5940d Author: Jorge Garcia <jorge.garcia <at> openbravo.com> Date: Fri Sep 23 11:17:23 2016 +0200 URL: http://code.openbravo.com/retail/backports/3.0RR16Q3.3/org.openbravo.retail.posterminal/rev/7c08268959f6bc11a90cd19a82689e3c5af5940d [^] Fixed issue 34058, backport of issue 34069: If deleteCurrent function forces to create a new order the document Sequence is wrongly generated If the creation of a new order is forced, the sequence number must be updated correctly. --- M web/org.openbravo.retail.posterminal/js/model/order.js --- |
(0091369) hgbot (developer) 2016-11-11 08:52 |
Repository: retail/backports/3.0RR16Q3.3/pi-mobile Changeset: fde1171803762f3453b8f29712dc99ea0ce65c3b Author: Jorge Garcia <jorge.garcia <at> openbravo.com> Date: Tue Sep 27 11:39:57 2016 +0200 URL: http://code.openbravo.com/retail/backports/3.0RR16Q3.3/pi-mobile/rev/fde1171803762f3453b8f29712dc99ea0ce65c3b [^] Verifies issue 34058: If deleteCurrent function forces to create a new order the document Sequence is wrongly generated Added test I34058_WrongDocSequenceWithDeleteCurrent. --- M src-test/org/openbravo/test/mobile/retail/pack/selenium/TestIdPack.java A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/documentno/I34058_WrongDocSequenceWithDeleteCurrent.java --- |
(0091401) migueldejuana (viewer) 2016-11-11 14:24 |
Tested and reviewed |
(0091710) hgbot (developer) 2016-11-21 16:30 |
Repository: retail/backports/3.0RR16Q2.4/pi-mobile Changeset: f9f5fe09439c19e9f338839687697e42bf69050e Author: Jorge Garcia <jorge.garcia <at> openbravo.com> Date: Tue Sep 27 11:39:57 2016 +0200 URL: http://code.openbravo.com/retail/backports/3.0RR16Q2.4/pi-mobile/rev/f9f5fe09439c19e9f338839687697e42bf69050e [^] Verifies issue 34058: If deleteCurrent function forces to create a new order the document Sequence is wrongly generated Added test I34058_WrongDocSequenceWithDeleteCurrent. --- M src-test/org/openbravo/test/mobile/common/selenium/javascript/TestId.java A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/documentno/I34058_WrongDocSequenceWithDeleteCurrent.java --- |
(0091721) hgbot (developer) 2016-11-21 19:15 |
Repository: tools/automation/pi-mobile Changeset: f9f5fe09439c19e9f338839687697e42bf69050e Author: Jorge Garcia <jorge.garcia <at> openbravo.com> Date: Tue Sep 27 11:39:57 2016 +0200 URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/f9f5fe09439c19e9f338839687697e42bf69050e [^] Verifies issue 34058: If deleteCurrent function forces to create a new order the document Sequence is wrongly generated Added test I34058_WrongDocSequenceWithDeleteCurrent. --- M src-test/org/openbravo/test/mobile/common/selenium/javascript/TestId.java A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/documentno/I34058_WrongDocSequenceWithDeleteCurrent.java --- |
(0091778) hgbot (developer) 2016-11-24 12:52 |
Repository: tools/automation/pi-mobile Changeset: fde1171803762f3453b8f29712dc99ea0ce65c3b Author: Jorge Garcia <jorge.garcia <at> openbravo.com> Date: Tue Sep 27 11:39:57 2016 +0200 URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/fde1171803762f3453b8f29712dc99ea0ce65c3b [^] Verifies issue 34058: If deleteCurrent function forces to create a new order the document Sequence is wrongly generated Added test I34058_WrongDocSequenceWithDeleteCurrent. --- M src-test/org/openbravo/test/mobile/retail/pack/selenium/TestIdPack.java A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/documentno/I34058_WrongDocSequenceWithDeleteCurrent.java --- |
![]() |
|||
Date Modified | Username | Field | Change |
2016-09-22 14:04 | guillermogil | New Issue | |
2016-09-22 14:04 | guillermogil | Assigned To | => Retail |
2016-09-22 14:04 | guillermogil | OBNetwork customer | => Yes |
2016-09-22 14:04 | guillermogil | Resolution time | => 1474840800 |
2016-09-22 14:04 | guillermogil | Regression level | => Production - Confirmed Stable |
2016-09-22 14:04 | guillermogil | Regression introduced in release | => RR16Q2 |
2016-09-22 14:04 | guillermogil | Regression introduced by commit | => https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/09dc9563011a [^] |
2016-09-22 14:04 | guillermogil | Triggers an Emergency Pack | => No |
2016-09-22 14:05 | guillermogil | File Added: forceNewTicketCreation.diff | |
2016-09-22 14:36 | guillermogil | Steps to Reproduce Updated | View Revisions |
2016-09-22 14:55 | lorenzofidalgo | Priority | urgent => immediate |
2016-09-23 10:47 | jorge-garcia | Status | new => scheduled |
2016-09-23 10:47 | jorge-garcia | Assigned To | Retail => jorge-garcia |
2016-09-28 11:50 | hgbot | Checkin | |
2016-09-28 11:50 | hgbot | Note Added: 0090343 | |
2016-09-28 11:50 | hgbot | Status | scheduled => resolved |
2016-09-28 11:50 | hgbot | Resolution | open => fixed |
2016-09-28 11:50 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/b57385efc7794027cd79ad0f954a0e4525265e20 [^] |
2016-09-28 11:50 | hgbot | Checkin | |
2016-09-28 11:50 | hgbot | Note Added: 0090344 | |
2016-09-30 13:16 | marvintm | Review Assigned To | => marvintm |
2016-09-30 13:16 | marvintm | Status | resolved => closed |
2016-11-11 08:49 | hgbot | Checkin | |
2016-11-11 08:49 | hgbot | Note Added: 0091352 | |
2016-11-11 08:50 | hgbot | Checkin | |
2016-11-11 08:50 | hgbot | Note Added: 0091360 | |
2016-11-11 08:51 | hgbot | Checkin | |
2016-11-11 08:51 | hgbot | Note Added: 0091364 | |
2016-11-11 08:51 | hgbot | Status | closed => resolved |
2016-11-11 08:51 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/b57385efc7794027cd79ad0f954a0e4525265e20 [^] => http://code.openbravo.com/retail/backports/3.0RR16Q3.3/org.openbravo.retail.posterminal/rev/7c08268959f6bc11a90cd19a82689e3c5af5940d [^] |
2016-11-11 08:52 | hgbot | Checkin | |
2016-11-11 08:52 | hgbot | Note Added: 0091369 | |
2016-11-11 14:24 | migueldejuana | Review Assigned To | marvintm => migueldejuana |
2016-11-11 14:24 | migueldejuana | Note Added: 0091401 | |
2016-11-11 14:24 | migueldejuana | Status | resolved => closed |
2016-11-16 09:36 | lorenzofidalgo | Steps to Reproduce Updated | View Revisions |
2016-11-21 16:30 | hgbot | Checkin | |
2016-11-21 16:30 | hgbot | Note Added: 0091710 | |
2016-11-21 19:15 | hgbot | Checkin | |
2016-11-21 19:15 | hgbot | Note Added: 0091721 | |
2016-11-24 12:52 | hgbot | Checkin | |
2016-11-24 12:52 | hgbot | Note Added: 0091778 |
Copyright © 2000 - 2009 MantisBT Group |