Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0055778
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] [Modules] Openbravo Business APIminoralways2024-06-14 19:372024-09-03 07:56
ReporterivazquezView Statuspublic 
Assigned Tosreehari 
PriorityhighResolutionfixedFixed in VersionRR24Q4
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned To
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0055778: Update Alternative UOM Product

DescriptionIf you try in a single request to deactivate an existing UOM and create another one that has the same settings but changes the name and rate, the resulting EDL request will have the following error:

ERROR org.openbravo.retail.api.mappings.product.RetailApiImportProductJavaPropertyMappingHandler - Error while updating AlternateUOM for product 457E17BB12C64F2994B036AD3CAE2383

Tested in this live build:

https://livebuilds.openbravo.com/context/retail_pos2_modules_pgsql/security/Login [^]

Using the following API:

/ws/org.openbravo.api.ImportService/Product
Steps To Reproduce-Search for a product (or create one) that does not have an Alternative UOM.
-Execute the API to create this alternative UOM and verify that it has been created correctly.
-Execute again the API but in the JSON of the alternativeUOM add the parameter "active": false
-Verify in the back office that it has been deactivated correctly.
-Change in the request the name and conversion rate of the alternativeUOM, in order to create a new one with a different name and rate but with the same parameters in the other fields (obviously removing the "active" parameter: false).
-You will see that nothing was updated and in the EDL Request window you will have the EDL generated by this request in error status.

Attached is a video demonstrating the problem
TagsNo tags attached.
Attached Files? file icon problemalternativeuomtworequests.mp4 [^] (2,054,975 bytes) 2024-06-14 19:37
? file icon Payload 2 [^] (1,351 bytes) 2024-06-25 07:34 [Show Content]

- Relationships Relation Graph ] Dependency Graph ]
related to defect 0053935 closedRajesh_18 Modules Update UOM product 

-  Notes
(0166220)
sreehari (developer)
2024-06-24 07:02

EDL generated by request is due to the not the uom that is not available in the list of UOM defined, when the defined UOM is sent in the request the issue no longer exists.
(0166326)
sreehari (developer)
2024-06-26 05:48

When sending request with multiple AUOM with first UOM active=false, then we have EDL error.
(0166363)
hgbot (developer)
2024-06-26 11:34

Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.api/-/merge_requests/100 [^]
(0168613)
ivazquez (developer)
2024-08-30 19:52

Hello Team,

Any News?

Regards,
Ignacio Vazquez
(0168664)
marvintm (manager)
2024-09-03 07:53

Merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.api/-/merge_requests/100 [^]
(0168666)
hgbot (developer)
2024-09-03 07:55

Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.api/-/merge_requests/100 [^]
(0168667)
hgbot (developer)
2024-09-03 07:55

Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.api [^]
Changeset: 57f75cf11b2c81aec375b6afb782be93a3322d9d
Author: Sreehari Venkataraman <sreehari.venkataraman.ext@openbravo.com>
Date: 03-09-2024 05:51:33
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.api/-/commit/57f75cf11b2c81aec375b6afb782be93a3322d9d [^]

Fixes ISSUE-55778: Added active and isNewOBObject case for multiAUOM
*Handled case when multiple alternate UOM is sent with active flag
*Optimized codes and created functions for validation and exeption

---
M src/org/openbravo/retail/api/mappings/product/RetailApiImportProductJavaPropertyMappingHandler.java
---
(0168668)
marvintm (manager)
2024-09-03 07:56

Checked that before fix, issue was reproducible (created product with alternate uom, then requested update with previous alternate uom disabled, and new uom -> failed)
Tested that initial creation of product with alternate works after fix
Tested that reproduced problem is no longer happening (requested update with previous alternate uom disabled, and new uom -> worked correctly)
Tested that duplicated alternate uoms still returns error

- Issue History
Date Modified Username Field Change
2024-06-14 19:37 ivazquez New Issue
2024-06-14 19:37 ivazquez Assigned To => Triage Platform Conn
2024-06-14 19:37 ivazquez File Added: problemalternativeuomtworequests.mp4
2024-06-14 19:37 ivazquez Relationship added depends on 0053935
2024-06-14 19:37 ivazquez Relationship deleted depends on 0053935
2024-06-14 19:38 ivazquez Relationship added related to 0053935
2024-06-19 07:34 marvintm Assigned To Triage Platform Conn => Retail
2024-06-19 07:34 marvintm Status new => acknowledged
2024-06-20 11:56 sreehari Assigned To Retail => sreehari
2024-06-20 11:56 sreehari Status acknowledged => scheduled
2024-06-24 07:02 sreehari Note Added: 0166220
2024-06-25 07:34 sreehari File Added: Payload 2
2024-06-25 08:38 egoitz Severity major => minor
2024-06-26 05:48 sreehari Note Added: 0166326
2024-06-26 11:34 hgbot Note Added: 0166363
2024-08-30 19:52 ivazquez Note Added: 0168613
2024-09-02 16:45 marvintm Project Modules => Retail Modules
2024-09-03 07:53 marvintm Note Added: 0168664
2024-09-03 07:55 hgbot Note Added: 0168666
2024-09-03 07:55 hgbot Resolution open => fixed
2024-09-03 07:55 hgbot Status scheduled => closed
2024-09-03 07:55 hgbot Note Added: 0168667
2024-09-03 07:56 marvintm Note Added: 0168668
2024-09-03 07:56 marvintm Status closed => resolved
2024-09-03 07:56 marvintm Fixed in Version => RR24Q4
2024-09-03 07:56 marvintm Status resolved => closed


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker