Openbravo Issue Tracking System - Modules |
| View Issue Details |
|
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0053908 | Modules | External Data Integration | public | 2023-10-31 13:32 | 2023-11-13 08:35 |
|
| Reporter | malsasua | |
| Assigned To | caristu | |
| Priority | high | Severity | major | Reproducibility | always |
| Status | closed | Resolution | fixed | |
| Platform | | OS | 5 | OS Version | |
| Product Version | | |
| Target Version | | Fixed in Version | | |
| Merge Request Status | |
| Regression date | |
| Regression introduced by commit | |
| Regression level | |
| Review Assigned To | cberner |
| Support ticket | |
| OBNetwork customer | |
| Regression introduced in release | |
|
| Summary | 0053908: EDL process is duplicating request lines |
| Description | In EDL process, when there are several item ids by line, the process of one item raises an error, and after that, the export of the items raise another error, the first item ID is added in a new request line, but it is not removed from the original request line. |
| Steps To Reproduce | . install edl example module:
https://gitlab.com/openbravo/demo/org.openbravo.externaldata.integration.examples [^]
. apply patch1 (attached) to force errors in some lines in the Import UOM Asynchronously process: it forces a processing error in one individual item and a general error when executing the OutputDataProcessor.
. execute Import UOM Asynchronously process. The generated EDL request have:
--> One line in error with the item whose processing failed. This is correct.
--> Another line in error (because the OutputDataProcessor failed) with the correctly processed items plus the item whose processing failed. This is wrong, the item whose processing failed should not be present in this line |
| Proposed Solution | attached possible fix (patch2) |
| Additional Information | |
| Tags | No tags attached. |
| Relationships | | blocks | defect | 0053816 | | closed | caristu | EDL process is duplicating request lines |
|
| Attached Files | |
|
| Issue History |
| Date Modified | Username | Field | Change |
| 2023-11-10 08:49 | caristu | Type | defect => backport |
| 2023-11-10 08:49 | caristu | Target Version | => PR23Q4 |
| 2023-11-10 08:50 | caristu | Project | Openbravo ERP => Modules |
| 2023-11-10 08:51 | caristu | Target Version | PR23Q4 => |
| 2023-11-10 09:05 | hgbot | Note Added: 0157008 | |
| 2023-11-10 09:06 | caristu | Note Added: 0157010 | |
| 2023-11-13 08:34 | hgbot | Note Added: 0157060 | |
| 2023-11-13 08:34 | hgbot | Note Added: 0157061 | |
| 2023-11-13 08:34 | hgbot | Note Added: 0157062 | |
| 2023-11-13 08:35 | caristu | Status | scheduled => resolved |
| 2023-11-13 08:35 | caristu | Resolution | open => fixed |
| 2023-11-13 08:35 | caristu | Review Assigned To | => cberner |
| 2023-11-13 08:35 | caristu | Status | resolved => closed |
|
Notes |
|
|
(0157008)
|
|
hgbot
|
|
2023-11-10 09:05
|
|
|
|
|
|
Note that this backport contains two commits:
1- One commit to fix some parts of the error management that were not working in the same way as it was before the changes introduced with 0052614. This is the part that can be considered a regression.
2- The specific fix for the issue, which is not a regression but we think it is worth to backport it (it is not very risky) to have it also fixed in 23Q4 release. |
|
|
|
(0157060)
|
|
hgbot
|
|
2023-11-13 08:34
|
|
|
|
|
(0157061)
|
|
hgbot
|
|
2023-11-13 08:34
|
|
|
|
|
(0157062)
|
|
hgbot
|
|
2023-11-13 08:34
|
|
|