Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0057824Retail ModulesRetail APIpublic2025-01-31 13:302025-03-10 18:09
Reporterjavietxe 
Assigned Tocaristu 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Statusapproved
Review Assigned To
OBNetwork customerGold
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0057824: Import API: Confusing error when a configuration is missing
DescriptionIf "Send Business API event" is not configured in the EDL Configuration, the error of the EDL Request is confusing, as it's responding that a configuration is duplicated while it doesn't exist.
Steps To ReproduceGo to the Backoffice
- EDL Configuration-> Create a configuration for "JSON Based Import Process"
- If there exist any other configuration, delete it.

Try to import a Distribution order via API.

Check that the EDL is in error with a not meaningfull error, while the problem is that "Send Business API Event" is not configured:

2025-01-30 10:35:42,160 [http-nio-127.0.0.1-8080-exec-5] ERROR org.hibernate.engine.jdbc.spi.SqlExceptionHelper - ERROR: duplicate key value violates unique constraint "obedl_conf_client_proc_index"
  Detail: Key (ad_client_id, obedl_process_id)=(37A2B61F68344FCD8F973023D5418E8B, 17B50CF4701240868D03BDF047F9B1F3) already exists.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2025-01-31 13:30javietxeNew Issue
2025-01-31 13:30javietxeAssigned To => Retail
2025-01-31 13:30javietxeOBNetwork customer => Gold
2025-01-31 13:30javietxeTriggers an Emergency Pack => No
2025-01-31 13:31javietxeIssue Monitored: javietxe
2025-02-01 07:48guillermogilAssigned ToRetail => Triage Platform Conn
2025-02-06 10:37adrianromeroAssigned ToTriage Platform Conn => caristu
2025-02-06 13:31adrianromeroAssigned Tocaristu => Triage Platform Conn
2025-02-17 13:14adrianromeroAssigned ToTriage Platform Conn => fermin_gascon
2025-02-21 10:24caristuAssigned Tofermin_gascon => caristu
2025-02-21 10:24caristuSeveritymajor => minor
2025-02-28 13:56hgbotMerge Request Status => open
2025-02-28 13:56hgbotNote Added: 0176305
2025-03-05 14:05hgbotMerge Request Statusopen => approved
2025-03-05 14:06hgbotNote Added: 0176479
2025-03-05 14:06hgbotResolutionopen => fixed
2025-03-05 14:06hgbotStatusnew => closed
2025-03-05 14:06hgbotNote Added: 0176480
2025-03-07 15:31hgbotNote Added: 0176563
2025-03-10 11:02hgbotNote Added: 0176592
2025-03-10 18:09hgbotNote Added: 0176630

Notes
(0176305)
hgbot   
2025-02-28 13:56   
Merge Request created: https://gitlab.com/orisha-group/bu-commerce/openbravo/product/pmods/org.openbravo.externaldata.integration/-/merge_requests/47 [^]
(0176479)
hgbot   
2025-03-05 14:06   
Merge request merged: https://gitlab.com/orisha-group/bu-commerce/openbravo/product/pmods/org.openbravo.externaldata.integration/-/merge_requests/47 [^]
(0176480)
hgbot   
2025-03-05 14:06   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/orisha-group/bu-commerce/openbravo/product/pmods/org.openbravo.externaldata.integration [^]
Changeset: 78d406af94e79a71dbf37a11a61987510136e495
Author: Carlos Aristu <c.aristu@orisha.com>
Date: 05-03-2025 14:06:33
URL: https://gitlab.com/orisha-group/bu-commerce/openbravo/product/pmods/org.openbravo.externaldata.integration/-/commit/78d406af94e79a71dbf37a11a61987510136e495 [^]

fixes BUG-57824: default EDL configuration creation is not thread safe

---
A src/org/openbravo/externaldata/integration/process/EDLConfigurationProvider.java
M src-test/org/openbravo/externaldata/integration/process/AddRequestTest.java
M src-test/org/openbravo/externaldata/integration/util/EDLTestUtils.java
M src/org/openbravo/externaldata/integration/process/ProcessRequest.java
---
(0176563)
hgbot   
2025-03-07 15:31   
Repository: https://gitlab.com/orisha-group/bu-commerce/openbravo/product/pmods/org.openbravo.externaldata.integration [^]
Changeset: 07d0a9f078eef8b2771884b2d72fbbd104b85150
Author: Carlos Aristu <c.aristu@orisha.com>
Date: 07-03-2025 15:31:05
URL: https://gitlab.com/orisha-group/bu-commerce/openbravo/product/pmods/org.openbravo.externaldata.integration/-/commit/07d0a9f078eef8b2771884b2d72fbbd104b85150 [^]

related to BUG-57824: disable temporarily test failing in CI

---
M src-test/org/openbravo/externaldata/integration/process/AddRequestTest.java
---
(0176592)
hgbot   
2025-03-10 11:02   
Repository: https://gitlab.com/orisha-group/bu-commerce/openbravo/product/pmods/org.openbravo.externaldata.integration [^]
Changeset: 8db93b3262d7c866d7b28da747545c9a5630afc2
Author: Carlos Aristu <c.aristu@orisha.com>
Date: 10-03-2025 11:02:31
URL: https://gitlab.com/orisha-group/bu-commerce/openbravo/product/pmods/org.openbravo.externaldata.integration/-/commit/8db93b3262d7c866d7b28da747545c9a5630afc2 [^]

related to FR-57824: add log + re-enable test

---
M src-test/org/openbravo/externaldata/integration/process/AddRequestTest.java
M src-test/org/openbravo/externaldata/integration/util/EDLTestUtils.java
---
(0176630)
hgbot   
2025-03-10 18:09   
Repository: https://gitlab.com/orisha-group/bu-commerce/openbravo/product/pmods/org.openbravo.externaldata.integration [^]
Changeset: cb1e174fd29f56a2b6ec30e2984b0f2200fccf47
Author: Carlos Aristu <c.aristu@orisha.com>
Date: 10-03-2025 17:57:01
URL: https://gitlab.com/orisha-group/bu-commerce/openbravo/product/pmods/org.openbravo.externaldata.integration/-/commit/cb1e174fd29f56a2b6ec30e2984b0f2200fccf47 [^]

related to FR-57824: reset cache on config removal

  Clearing the cache on endProcess() could cause problems if other
threads are executing that method concurrently in the first execution of
the EDL process, i.e., if there was no EDL configuration when the process
request is created.

  To avoid this problem, we now clear the cache if the EDL configuration
is deleted because in that case we are again in an scenario where the
EDL process is created without having an existing configuration. And, in
this case we cannot use the value kept in the cache because it is staled.

---
A src/org/openbravo/externaldata/integration/process/EDLConfigurationEventObserver.java
M src/org/openbravo/externaldata/integration/process/ProcessRequest.java
---