Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||||||
ID | ||||||||||||
0052799 | ||||||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||||||
design defect | [Retail Modules] Retail API | major | have not tried | 2023-06-20 15:17 | 2023-06-29 14:28 | |||||||
Reporter | azzeddine_meftoh | View Status | public | |||||||||
Assigned To | Triage Omni WMS | |||||||||||
Priority | urgent | Resolution | open | Fixed in Version | ||||||||
Status | new | Fix in branch | Fixed in SCM revision | |||||||||
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 | ||||||||||||
Regression level | ||||||||||||
Regression date | ||||||||||||
Regression introduced in release | ||||||||||||
Regression introduced by commit | ||||||||||||
Triggers an Emergency Pack | No | |||||||||||
Summary | 0052799: [RM-6914] Distribution order Import API - when error on import the remaining objects are not tried one by one | |||||||||||
Description | when importing Distribution orders through the API with a list of DO if there i an error on one object the remaining objects are not moved to a new edl request line Dev analysis : I see that this is a bug in the definition of the DistributionOrder entity mapping. It does not have any record marked as "Identifies Record Univocally" which is the mechanism used to check existing records https://openbravo.atlassian.net/browse/RM-6914 [^] | |||||||||||
Steps To Reproduce | import with Distribution order API a list of DO make sure there are errors (such us undefined warehouse) when the error occurs the object with the undefined warehouse is rejected. but the remaining objects are not processed | |||||||||||
Proposed Solution | This behavior is happening because the DO import process is internally committing the transaction at this point[1]. It would be feasible to avoid doing that commit at that point at least when importing DO through the API? [1] https://gitlab.com/openbravo/product/pmods/org.openbravo.warehouse.advancedwarehouseoperations/-/blob/master/src/org/openbravo/warehouse/advancedwarehouseoperations/verbosity/AWOVerbosityLogger.java#L133 [^] | |||||||||||
Tags | No tags attached. | |||||||||||
Attached Files | CURL-DOi.txt [^] (6,838 bytes) 2023-06-21 23:08 [Show Content] | |||||||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | |
Notes | |
(0151848) mtaal (manager) 2023-06-29 14:28 |
Snippet of email discussion: what you have found is something incredibly deep in the core of the AWO Engine... This is not something to change as the implications are quite important for all customers using AWO. Thinking about importing DOs and the flow of generating tasks afterwards (which is the reason for this flush), changing it would also require significant effort and some redefinition of it. So this is something that has no easy fix from either side and it is a topic to discuss with more time and think carefully. I will discuss first with Azzeddine and Rafa and then we can have a meeting later on to sync up and think about the possible options. |
Issue History | |||
Date Modified | Username | Field | Change |
2023-06-20 15:17 | azzeddine_meftoh | New Issue | |
2023-06-20 15:17 | azzeddine_meftoh | Assigned To | => Triage Platform Conn |
2023-06-20 15:17 | azzeddine_meftoh | Triggers an Emergency Pack | => No |
2023-06-20 16:42 | rafaroda | Priority | normal => urgent |
2023-06-20 17:23 | adrianromero | Assigned To | Triage Platform Conn => Triage Omni OMS |
2023-06-20 17:38 | aferraz | Assigned To | Triage Omni OMS => Triage Omni WMS |
2023-06-21 23:08 | mtaal | File Added: CURL-DOi.txt | |
2023-06-21 23:47 | mtaal | Assigned To | Triage Omni WMS => ruben_jimenez |
2023-06-22 15:16 | caristu | Assigned To | ruben_jimenez => Triage Platform Base |
2023-06-22 15:16 | caristu | Assigned To | Triage Platform Base => Triage Platform Conn |
2023-06-22 17:00 | caristu | Assigned To | Triage Platform Conn => Triage Omni WMS |
2023-06-22 17:01 | caristu | Proposed Solution updated | |
2023-06-22 17:01 | caristu | Proposed Solution updated | |
2023-06-26 10:41 | mtaal | Summary | Distribution order Import API - when error on import the remaining objects are not tried one by one => [RM-6914] Distribution order Import API - when error on import the remaining objects are not tried one by one |
2023-06-26 10:41 | mtaal | Description Updated | View Revisions |
2023-06-29 14:28 | mtaal | Note Added: 0151848 | |
2023-06-29 14:28 | mtaal | Type | defect => design defect |
Copyright © 2000 - 2009 MantisBT Group |