Openbravo Issue Tracking System - Retail Modules
View Issue Details
0038862Retail ModulesWeb POSpublic2018-06-29 09:252018-10-16 18:08
marvintm 
migueldejuana 
normalmajoralways
closedfixed 
5
 
RR18Q4 
marvintm
No
0038862: Ticket completion should be executed fully inside the same database transaction
After related issue https://issues.openbravo.com/view.php?id=38676 [^] some changes have been done to try to execute the whole ticket synchronization process in the same database transaction. However, there is one part which is still missing: the hook execution and the last save for the ticket which sets it to hasbeenpaid='Y' should also be part of the same database transaction, and currently they are not.
.
No tags attached.
related to defect 0038676 closed ranjith_qualiantech_com Ticket completion should be executed fully inside the same database transaction, and currently that's not the case 
related to design defect 0039228 closed migueldejuana Receipt object is global and therefore can be modified after the user has confirmed that the ticket should be processed 
depends on backport 0039337RR18Q3.2 closed migueldejuana Ticket completion should be executed fully inside the same database transaction 
depends on backport 0039464RR18Q2.4 closed migueldejuana Ticket completion should be executed fully inside the same database transaction 
related to defect 0035745 closed ranjith_qualiantech_com Add a transaction where it is needed 
Issue History
2018-06-29 09:25marvintmNew Issue
2018-06-29 09:25marvintmAssigned To => Retail
2018-06-29 09:25marvintmTriggers an Emergency Pack => No
2018-06-29 09:25marvintmRelationship addedrelated to 0038676
2018-08-01 14:38rqueraltaStatusnew => scheduled
2018-08-20 07:55ranjith_qualiantech_comRelationship addedrelated to 0035745
2018-08-23 13:30marvintmAssigned ToRetail => migueldejuana
2018-09-05 13:40ngarciaIssue Monitored: ngarcia
2018-09-05 13:45rafarodaIssue Monitored: rafaroda
2018-09-05 13:46rafarodaRelationship addedrelated to 0039228
2018-09-12 13:55hgbotCheckin
2018-09-12 13:55hgbotNote Added: 0106698
2018-09-12 13:55hgbotStatusscheduled => resolved
2018-09-12 13:55hgbotResolutionopen => fixed
2018-09-12 13:55hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/fe341ce2af80ba021cd41c1659c97c70bf7052f1 [^]
2018-09-12 13:55hgbotCheckin
2018-09-12 13:55hgbotNote Added: 0106699
2018-09-12 13:55hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/fe341ce2af80ba021cd41c1659c97c70bf7052f1 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/25a21535cb7144532969600a667800e8d6dcc41d [^]
2018-09-12 17:00hgbotCheckin
2018-09-12 17:00hgbotNote Added: 0106716
2018-09-12 17:00hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/25a21535cb7144532969600a667800e8d6dcc41d [^] => http://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/060582076a8dfc2c59782cc8f690a9f4ed41acb7 [^]
2018-09-12 17:06hgbotCheckin
2018-09-12 17:06hgbotNote Added: 0106717
2018-09-12 17:06hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/060582076a8dfc2c59782cc8f690a9f4ed41acb7 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/4a7264e636a053bdabee6903abc5bc53c5c3daac [^]
2018-09-13 12:07hgbotCheckin
2018-09-13 12:07hgbotNote Added: 0106746
2018-09-13 12:07hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/4a7264e636a053bdabee6903abc5bc53c5c3daac [^] => http://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/ec0c2009413c5afac1c9ffcd17c7d326c3d9a445 [^]
2018-09-14 08:51hgbotCheckin
2018-09-14 08:51hgbotNote Added: 0106765
2018-09-14 08:51hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/ec0c2009413c5afac1c9ffcd17c7d326c3d9a445 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/2ce1230fedb5a8866891abef12989b512ce81b98 [^]
2018-09-17 10:12hgbotCheckin
2018-09-17 10:12hgbotNote Added: 0106798
2018-09-17 10:12hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/2ce1230fedb5a8866891abef12989b512ce81b98 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/940b478745cbd721c9a5c2d0b57982499e0f8253 [^]
2018-09-17 17:29hgbotCheckin
2018-09-17 17:29hgbotNote Added: 0106816
2018-09-17 17:29hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/940b478745cbd721c9a5c2d0b57982499e0f8253 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/64b5648a059026708693bd7b4198704a3eeddcf8 [^]
2018-09-19 08:05hgbotCheckin
2018-09-19 08:05hgbotNote Added: 0106850
2018-09-19 08:05hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/64b5648a059026708693bd7b4198704a3eeddcf8 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/3142284690ff05d9d97d97eb532c5daa11b293d5 [^]
2018-09-19 08:05hgbotCheckin
2018-09-19 08:05hgbotNote Added: 0106851
2018-09-19 08:05hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/3142284690ff05d9d97d97eb532c5daa11b293d5 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/2e7962b28a3512ac402483e140ca197900e17fef [^]
2018-09-19 13:44hgbotCheckin
2018-09-19 13:44hgbotNote Added: 0106859
2018-09-19 13:44hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/2e7962b28a3512ac402483e140ca197900e17fef [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/a3ae464ccea7b7727300427f943a50fdd8c3c091 [^]
2018-09-21 11:57marvintmStatusresolved => new
2018-09-21 11:57marvintmResolutionfixed => open
2018-09-21 11:57marvintmStatusnew => scheduled
2018-09-21 11:57marvintmStatusscheduled => resolved
2018-09-21 11:57marvintmFixed in Version => RR18Q4
2018-09-21 11:57marvintmResolutionopen => fixed
2018-09-21 12:44marvintmReview Assigned To => marvintm
2018-09-21 12:44marvintmStatusresolved => closed
2018-10-16 18:07marvintmStatusclosed => new
2018-10-16 18:07marvintmResolutionfixed => open
2018-10-16 18:07marvintmFixed in VersionRR18Q4 =>
2018-10-16 18:07marvintmStatusnew => scheduled
2018-10-16 18:08marvintmStatusscheduled => resolved
2018-10-16 18:08marvintmFixed in Version => RR18Q4
2018-10-16 18:08marvintmResolutionopen => fixed
2018-10-16 18:08marvintmStatusresolved => closed

Notes
(0106698)
hgbot   
2018-09-12 13:55   
Repository: erp/pmods/org.openbravo.certification.france
Changeset: fe341ce2af80ba021cd41c1659c97c70bf7052f1
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Mon Sep 10 17:12:41 2018 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/fe341ce2af80ba021cd41c1659c97c70bf7052f1 [^]

Fixed issue 0038862: Ticket completion should be executed fully inside the same database transaction

- Use transaction to find and save TicketInformation. If tx doesn't exist, we will do a regular find and save.

---
M web/org.openbravo.certification.france/js/ticket/preSyncReceiptHook.js
---
(0106699)
hgbot   
2018-09-12 13:55   
Repository: erp/pmods/org.openbravo.certification.france
Changeset: 25a21535cb7144532969600a667800e8d6dcc41d
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Wed Sep 12 13:54:49 2018 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/25a21535cb7144532969600a667800e8d6dcc41d [^]

Fixed issue 0038862: Ticket completion should be executed fully inside the same database transaction

- Create canceled order in preOrderSaveHook instead of preSync
- Use tx transaction in the hook to mantain the consistency

---
M src-db/database/sourcedata/AD_MODULE.xml
M src/org/openbravo/certification/france/OBCFRApplicationProvider.java
M web/org.openbravo.certification.france/js/ticket/preSyncReceiptHook.js
A web/org.openbravo.certification.france/js/ticket/preOrderSaveHook.js
---
(0106716)
hgbot   
2018-09-12 17:00   
Repository: erp/pmods/org.openbravo.certification.france
Changeset: 060582076a8dfc2c59782cc8f690a9f4ed41acb7
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Wed Sep 12 17:00:39 2018 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/060582076a8dfc2c59782cc8f690a9f4ed41acb7 [^]

Fixed issue 0038862: Ticket completion should be executed fully inside the same database transaction

- Pass tx in a missing place

---
M web/org.openbravo.certification.france/js/ticket/preSyncReceiptHook.js
---
(0106717)
hgbot   
2018-09-12 17:06   
Repository: erp/pmods/org.openbravo.certification.france
Changeset: 4a7264e636a053bdabee6903abc5bc53c5c3daac
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Wed Sep 12 17:06:41 2018 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/4a7264e636a053bdabee6903abc5bc53c5c3daac [^]

Fixed issue 0038862: Ticket completion should be executed fully inside the same database transaction

. Module hash

---
M src-db/database/sourcedata/AD_MODULE.xml
---
(0106746)
hgbot   
2018-09-13 12:07   
Repository: erp/pmods/org.openbravo.certification.france
Changeset: ec0c2009413c5afac1c9ffcd17c7d326c3d9a445
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Thu Sep 13 12:04:28 2018 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/ec0c2009413c5afac1c9ffcd17c7d326c3d9a445 [^]

Fixed issue 0038862: Ticket completion should be executed fully inside the same database transaction

- Add a new hook better to avoid wrong implementations from other modules

---
M src-db/database/sourcedata/AD_MODULE.xml
M src/org/openbravo/certification/france/OBCFRApplicationProvider.java
A web/org.openbravo.certification.france/js/ticket/preOrderSaveFinishCancelLayawayHook.js
R web/org.openbravo.certification.france/js/ticket/preOrderSaveHook.js
---
(0106765)
hgbot   
2018-09-14 08:51   
Repository: erp/pmods/org.openbravo.certification.france
Changeset: 2ce1230fedb5a8866891abef12989b512ce81b98
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Fri Sep 14 08:51:28 2018 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/2ce1230fedb5a8866891abef12989b512ce81b98 [^]

Fixed issue 0038862: Ticket completion should be executed fully inside the same database transaction

- Do not cancelJson until the receipt is blockChainned

---
M src-db/database/sourcedata/AD_MODULE.xml
M web/org.openbravo.certification.france/js/ticket/preOrderSaveFinishCancelLayawayHook.js
---
(0106798)
hgbot   
2018-09-17 10:12   
Repository: erp/pmods/org.openbravo.certification.france
Changeset: 940b478745cbd721c9a5c2d0b57982499e0f8253
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Mon Sep 17 10:12:15 2018 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/940b478745cbd721c9a5c2d0b57982499e0f8253 [^]

Fixed issue 0038862: Ticket completion should be executed fully inside the same database transaction

- Remove has data of canceledorder

---
M src-db/database/sourcedata/AD_MODULE.xml
M web/org.openbravo.certification.france/js/ticket/preSyncReceiptHook.js
---
(0106816)
hgbot   
2018-09-17 17:29   
Repository: erp/pmods/org.openbravo.certification.france
Changeset: 64b5648a059026708693bd7b4198704a3eeddcf8
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Mon Sep 17 17:29:14 2018 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/64b5648a059026708693bd7b4198704a3eeddcf8 [^]

Fixed issue 0038862: Ticket completion should be executed fully inside the same database transaction

- Save canceledorder outside de main order to avoid differences

---
M src-db/database/sourcedata/AD_MODULE.xml
M web/org.openbravo.certification.france/js/ticket/preOrderSaveFinishCancelLayawayHook.js
M web/org.openbravo.certification.france/js/ticket/preSyncReceiptHook.js
---
(0106850)
hgbot   
2018-09-19 08:05   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 3142284690ff05d9d97d97eb532c5daa11b293d5
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Mon Sep 10 17:09:45 2018 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/3142284690ff05d9d97d97eb532c5daa11b293d5 [^]

Fixed issue 0038862: Ticket completion should be executed fully inside the same database transaction

- Send transaction to OBPOS_PreSyncReceipt hook to be able to use it

---
M web/org.openbravo.retail.posterminal/js/data/dataordersave.js
---
(0106851)
hgbot   
2018-09-19 08:05   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 2e7962b28a3512ac402483e140ca197900e17fef
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Thu Sep 13 12:07:24 2018 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/2e7962b28a3512ac402483e140ca197900e17fef [^]

Fixed issue 0038862: Ticket completion should be executed fully inside the same database transaction

-- Add a new hook better to avoid wrong implementations from other modules

---
M web/org.openbravo.retail.posterminal/js/pointofsale/model/pointofsale-model.js
---
(0106859)
hgbot   
2018-09-19 13:44   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: a3ae464ccea7b7727300427f943a50fdd8c3c091
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Wed Sep 19 13:43:34 2018 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/a3ae464ccea7b7727300427f943a50fdd8c3c091 [^]

Fixed issue 0038862: Ticket completion should be executed fully inside the same database transaction

- JSBeautify

---
M web/org.openbravo.retail.posterminal/js/pointofsale/model/pointofsale-model.js
---