Openbravo Issue Tracking System - Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0048458ModulesExternal Data Integrationpublic2022-01-21 08:502022-01-25 08:16
Reportercaristu 
Assigned Tocaristu 
PriorityhighSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
Regression date
Regression introduced by commit
Regression level
Review Assigned To
Support ticket
OBNetwork customer
Regression introduced in release
Summary0048458: EDL request line status should be error if the related IE is set as error
DescriptionAfter processing an EDL request line without errors the line is marked as Success, but if the related import entry eventually fails (for example when trying to mark itself as processed), then the import entry is marked as Error but the request line status is not updated as Error in this case, leaving it with an outdated status.
Steps To Reproduce1) Install the module https://gitlab.com/openbravo/demo/org.openbravo.externaldata.integration.examples [^]
2) Artificial test: apply the attached patch which changes the ImportEntryManager.setImportEntryProcessed method to throw an exception
3) Run the Import UOM Asynchronously process
4) In the EDL window note that a new EDL recorg has been created with two lines. Note that the two lines are in Success status which is different from the status of the related import entries that is Error.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
depends on defect 0048454 closed caristu Openbravo ERP When a node is demoted as cluster node IE can be processed twice under some circumstances 
Attached Filesdiff iemError.diff (851) 2022-01-25 08:16
https://issues.openbravo.com/file_download.php?file_id=16547&type=bug
Issue History
Date ModifiedUsernameFieldChange
2022-01-21 08:50caristuNew Issue
2022-01-21 08:50caristuAssigned To => Triage Platform Conn
2022-01-21 08:50caristuAssigned ToTriage Platform Conn => caristu
2022-01-21 08:50caristuRelationship addeddepends on 0048454
2022-01-21 08:54caristuNote Added: 0134451
2022-01-21 10:35hgbotNote Added: 0134467
2022-01-21 10:37caristuNote Deleted: 0134451
2022-01-21 10:39hgbotResolutionopen => fixed
2022-01-21 10:39hgbotStatusnew => closed
2022-01-21 10:39hgbotNote Added: 0134471
2022-01-21 10:39hgbotNote Added: 0134472
2022-01-25 08:16caristuFile Added: iemError.diff

Notes
(0134467)
hgbot   
2022-01-21 10:35   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.externaldata.integration/-/merge_requests/11 [^]
(0134471)
hgbot   
2022-01-21 10:39   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.externaldata.integration [^]
Changeset: 5a9f21dec3f0c510727cce2f1637c9af6dcb93d8
Author: Carlos Aristu <carlos.aristu@openbravo.com>
Date: 21-01-2022 08:51:50
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.externaldata.integration/-/commit/5a9f21dec3f0c510727cce2f1637c9af6dcb93d8 [^]

fixes ISSUE-48458: set EDL request line in error if IE is set as error

---
M src/org/openbravo/externaldata/integration/process/EDLImportEntryProcessor.java
---
(0134472)
hgbot   
2022-01-21 10:39   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.externaldata.integration/-/merge_requests/11 [^]