Openbravo Issue Tracking System - POS2
View Issue Details
0057424POS2POSpublic2024-12-09 11:182024-12-12 09:18
wicherr 
aferraz 
urgentcriticalalways
closedfixed 
5
 
24Q4.1 
No
0057424: French fiscal : Cancel & replace makes EWI
When you do a cancel & replace of a ticket made from another till, an EWI is created :
org.openbravo.base.util.CheckException: Triggers were not disabled in this session, call disable before calling this method
    at org.openbravo.base.util.Check.isNotNull(Check.java:72)
    at org.openbravo.dal.core.TriggerHandler.enable(TriggerHandler.java:89)
    at org.openbravo.retail.posterminal.OrderLoader.saveRecord(OrderLoader.java:626)
    at org.openbravo.mobile.core.process.DataSynchronizationProcess.saveRecord(DataSynchronizationProcess.java:214)
    at org.openbravo.mobile.core.process.DataSynchronizationProcess.exec(DataSynchronizationProcess.java:165)
    at org.openbravo.mobile.core.process.DataSynchronizationProcess.exec(DataSynchronizationProcess.java:97)
    at org.openbravo.mobile.core.process.MobileImportEntryProcessorRunnable.processEntry(MobileImportEntryProcessorRunnable.java:55)
    at org.openbravo.retail.posterminal.importprocess.OrderImportEntryProcessor$OrderLoaderRunnable.processEntry(OrderImportEntryProcessor.java:64)
    at org.openbravo.service.importprocess.ImportEntryProcessor$ImportEntryProcessRunnable.doRunCycle(ImportEntryProcessor.java:447)
    at org.openbravo.service.importprocess.ImportEntryProcessor$ImportEntryProcessRunnable.run(ImportEntryProcessor.java:370)
    at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)
    at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
    at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
    at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
    at java.base/java.lang.Thread.run(Thread.java:829)


This is due to the fact that the terminal uses 2 seq number at a time. In the JSON, we can see that :
The tickets are declared from the first terminal with one of the last seqno of the terminal 2
Prerequisite : having 2 terminals from the same organisation using the french fiscal (with the hashcode and so on... full configuration done)

- Open terminal 1
- Make a sale, pay it and finalize it

- Open terminal 2
- Open the sale from the terminal 1
- Make a cancel & replace
- Validate it

Fix the seqno and maybe the "posTerminal$_identifier"
No tags attached.
blocks defect 0057385 closed aferraz French fiscal : Cancel & replace makes EWI 
Issue History
2024-12-11 12:00aferrazTypedefect => backport
2024-12-11 12:00aferrazTarget Version => 24Q4.1
2024-12-11 13:15hgbotNote Added: 0173177
2024-12-11 13:44hgbotNote Added: 0173178
2024-12-12 09:14hgbotResolutionopen => fixed
2024-12-12 09:14hgbotStatusscheduled => closed
2024-12-12 09:14hgbotFixed in Version => PR24Q4.1
2024-12-12 09:14hgbotNote Added: 0173216
2024-12-12 09:14hgbotNote Added: 0173217
2024-12-12 09:18hgbotNote Added: 0173222
2024-12-12 09:18hgbotNote Added: 0173223

Notes
(0173177)
hgbot   
2024-12-11 13:15   
Merge Request created: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/1471 [^]
(0173178)
hgbot   
2024-12-11 13:44   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.certification.france/-/merge_requests/21 [^]
(0173216)
hgbot   
2024-12-12 09:14   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/openbravo [^]
Changeset: 725b567d79f1651ddb9011d7e73252e32177f274
Author: Álvaro Ferraz <alvaro.ferraz@openbravo.com>
Date: 11-12-2024 13:14:19
URL: https://gitlab.com/openbravo/product/openbravo/-/commit/725b567d79f1651ddb9011d7e73252e32177f274 [^]

Fixes ISSUE-57424: Be able to define priority in CancelAndReplaceOrder hooks

---
M src/org/openbravo/erpCommon/businessUtility/CancelAndReplaceOrderHook.java
M src/org/openbravo/erpCommon/businessUtility/CancelAndReplaceOrderHookCaller.java
---
(0173217)
hgbot   
2024-12-12 09:14   
Merge request merged: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/1471 [^]
(0173222)
hgbot   
2024-12-12 09:18   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.certification.france [^]
Changeset: d5ad597a1de87474a7086be7fc755a86e878712d
Author: Álvaro Ferraz <a.ferraz@orisha.com>
Date: 12-12-2024 08:18:53
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.certification.france/-/commit/d5ad597a1de87474a7086be7fc755a86e878712d [^]

Fixes ISSUE-57424: TicketCancelReplaceHook should run after CancelAndReplaceHook

---
M src/org/openbravo/certification/france/ticket/TicketCancelReplaceHook.java
---
(0173223)
hgbot   
2024-12-12 09:18   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.certification.france/-/merge_requests/21 [^]