Openbravo Issue Tracking System - Modules
View Issue Details
0054498ModulesOpenbravo Business APIpublic2024-01-30 10:372024-01-30 11:28
vmromanos 
Retail 
normalmajoralways
newopen 
5
 
 
0054498: Cross dependency to org.openbravo.retail.discounts
In Promotion entity mapping Inwards, the xUnits property mapping instance is included in the Business API module. However, this property belongs org.openbravo.retail.discounts so it shouldn't be added to Business API module.


In fact the Retail API module does already include this property, so I think we just need to delete it
This is how I detected the problem

In a minimum environment, without retail stuff, try to access swagger. An error is thrown.

After removing the xUnits property mapping instance, you can access to swagger.

This is my environment:
 * openbravo
 * modules/org.openbravo.api
 * modules/org.openbravo.events.core:
 * modules/org.openbravo.externaldata.integration
 * modules/org.openbravo.service.external.integration
 * modules/org.openbravo.service.openapi
Remove the xUnits property mapping instance. Attached proposed file

Nice to have: Try to automate this check at compilation time to raise the problem in CI.
No tags attached.
related to feature request 0054499 acknowledged Triage Platform Conn Mechanism to avoid property mapping definitions causing incorrect module dependencies 
diff xunits-org.openbravo.api.diff (2,218) 2024-01-30 10:37
https://issues.openbravo.com/file_download.php?file_id=19401&type=bug
png 54498-Inward-WRONG.png (107,780) 2024-01-30 10:39
https://issues.openbravo.com/file_download.php?file_id=19402&type=bug
png

png 54498-Outward-OK.png (102,129) 2024-01-30 10:40
https://issues.openbravo.com/file_download.php?file_id=19403&type=bug
png
Issue History
2024-01-30 10:37vmromanosNew Issue
2024-01-30 10:37vmromanosAssigned To => Triage Platform Conn
2024-01-30 10:37vmromanosFile Added: xunits-org.openbravo.api.diff
2024-01-30 10:38vmromanosSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=27417#r27417
2024-01-30 10:39vmromanosFile Added: 54498-Inward-WRONG.png
2024-01-30 10:40vmromanosFile Added: 54498-Outward-OK.png
2024-01-30 11:22caristuAssigned ToTriage Platform Conn => Retail
2024-01-30 11:28caristuIssue cloned0054499
2024-01-30 11:28caristuRelationship addedrelated to 0054499

There are no notes attached to this issue.