Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0038862 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Retail Modules] Web POS | major | always | 2018-06-29 09:25 | 2018-10-16 18:08 | |||
Reporter | marvintm | View Status | public | |||||
Assigned To | migueldejuana | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | RR18Q4 | |||
Status | closed | Fix in branch | Fixed in SCM revision | a3ae464ccea7 | ||||
Projection | none | ETA | none | Target Version | ||||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Review Assigned To | marvintm | |||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0038862: Ticket completion should be executed fully inside the same database transaction | |||||||
Description | 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. | |||||||
Steps To Reproduce | . | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | ||||||||||||||||||||||||||||||||||||
|
Notes | |
(0106698) hgbot (developer) 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 (developer) 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 (developer) 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 (developer) 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 (developer) 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 (developer) 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 (developer) 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 (developer) 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 (developer) 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 (developer) 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 (developer) 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 --- |
Issue History | |||
Date Modified | Username | Field | Change |
2018-06-29 09:25 | marvintm | New Issue | |
2018-06-29 09:25 | marvintm | Assigned To | => Retail |
2018-06-29 09:25 | marvintm | Triggers an Emergency Pack | => No |
2018-06-29 09:25 | marvintm | Relationship added | related to 0038676 |
2018-06-29 09:26 | marvintm | Resolution time | => 1532815200 |
2018-07-26 11:17 | marvintm | Resolution time | 1532815200 => 1533852000 |
2018-08-01 14:38 | rqueralta | Status | new => scheduled |
2018-08-10 13:31 | marvintm | Resolution time | 1533852000 => 1534543200 |
2018-08-13 16:30 | marvintm | Resolution time | 1534543200 => 1535148000 |
2018-08-20 07:55 | ranjith_qualiantech_com | Relationship added | related to 0035745 |
2018-08-22 10:44 | marvintm | Resolution time | 1535148000 => 1536530400 |
2018-08-23 13:30 | marvintm | Resolution time | 1536530400 => 1536962400 |
2018-08-23 13:30 | marvintm | Assigned To | Retail => migueldejuana |
2018-09-05 13:40 | ngarcia | Issue Monitored: ngarcia | |
2018-09-05 13:45 | rafaroda | Issue Monitored: rafaroda | |
2018-09-05 13:46 | rafaroda | Relationship added | related to 0039228 |
2018-09-12 13:55 | hgbot | Checkin | |
2018-09-12 13:55 | hgbot | Note Added: 0106698 | |
2018-09-12 13:55 | hgbot | Status | scheduled => resolved |
2018-09-12 13:55 | hgbot | Resolution | open => fixed |
2018-09-12 13:55 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.certification.france/rev/fe341ce2af80ba021cd41c1659c97c70bf7052f1 [^] |
2018-09-12 13:55 | hgbot | Checkin | |
2018-09-12 13:55 | hgbot | Note Added: 0106699 | |
2018-09-12 13:55 | hgbot | Fixed in SCM revision | http://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:00 | hgbot | Checkin | |
2018-09-12 17:00 | hgbot | Note Added: 0106716 | |
2018-09-12 17:00 | hgbot | Fixed in SCM revision | http://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:06 | hgbot | Checkin | |
2018-09-12 17:06 | hgbot | Note Added: 0106717 | |
2018-09-12 17:06 | hgbot | Fixed in SCM revision | http://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:07 | hgbot | Checkin | |
2018-09-13 12:07 | hgbot | Note Added: 0106746 | |
2018-09-13 12:07 | hgbot | Fixed in SCM revision | http://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:51 | hgbot | Checkin | |
2018-09-14 08:51 | hgbot | Note Added: 0106765 | |
2018-09-14 08:51 | hgbot | Fixed in SCM revision | http://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-14 11:51 | marvintm | Resolution time | 1536962400 => 1537308000 |
2018-09-17 10:12 | hgbot | Checkin | |
2018-09-17 10:12 | hgbot | Note Added: 0106798 | |
2018-09-17 10:12 | hgbot | Fixed in SCM revision | http://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:29 | hgbot | Checkin | |
2018-09-17 17:29 | hgbot | Note Added: 0106816 | |
2018-09-17 17:29 | hgbot | Fixed in SCM revision | http://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:05 | hgbot | Checkin | |
2018-09-19 08:05 | hgbot | Note Added: 0106850 | |
2018-09-19 08:05 | hgbot | Fixed in SCM revision | http://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:05 | hgbot | Checkin | |
2018-09-19 08:05 | hgbot | Note Added: 0106851 | |
2018-09-19 08:05 | hgbot | Fixed in SCM revision | http://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:44 | hgbot | Checkin | |
2018-09-19 13:44 | hgbot | Note Added: 0106859 | |
2018-09-19 13:44 | hgbot | Fixed in SCM revision | http://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:57 | marvintm | Status | resolved => new |
2018-09-21 11:57 | marvintm | Resolution | fixed => open |
2018-09-21 11:57 | marvintm | Status | new => scheduled |
2018-09-21 11:57 | marvintm | Status | scheduled => resolved |
2018-09-21 11:57 | marvintm | Fixed in Version | => RR18Q4 |
2018-09-21 11:57 | marvintm | Resolution | open => fixed |
2018-09-21 12:44 | marvintm | Review Assigned To | => marvintm |
2018-09-21 12:44 | marvintm | Status | resolved => closed |
2018-10-16 18:07 | marvintm | Status | closed => new |
2018-10-16 18:07 | marvintm | Resolution | fixed => open |
2018-10-16 18:07 | marvintm | Fixed in Version | RR18Q4 => |
2018-10-16 18:07 | marvintm | Status | new => scheduled |
2018-10-16 18:08 | marvintm | Status | scheduled => resolved |
2018-10-16 18:08 | marvintm | Fixed in Version | => RR18Q4 |
2018-10-16 18:08 | marvintm | Resolution | open => fixed |
2018-10-16 18:08 | marvintm | Status | resolved => closed |
Copyright © 2000 - 2009 MantisBT Group |