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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0057216
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] Retail Configurationminorhave not tried2024-11-20 18:012024-11-20 18:20
ReportershuehnerView Statuspublic 
Assigned Toshuehner 
PrioritynormalResolutionopenFixed in Version
StatusnewFix 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

0057216: Check constraint renaming broken custom AD_MESSAGE associated to it

DescriptionBack in 2019 the check-constraint
EM_OBRETCO_AVAILABLECROSSS_CHK
was renamed to be
EM_OBRETCO_AVAILABLECROSS

https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.config/-/commit/caf5670d148cc660f507214f45c3c4c929159023#232157b5d7a7d690e3aa00678be197d1a23a0e3b_8_8 [^]

But we missed hidden api of having an AD_MESSAGE with VALUE matching name of the check constraint.
So since then the custom error message for it is no longer shown
https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.config/-/blob/master/src-db/database/sourcedata/AD_MESSAGE.xml?ref_type=heads#L44 [^]
Steps To ReproduceSearch for EM_OBRETCO_AVAILABLECROSSS_CHK and do not find any usage for it
Find the check constraint rename linked in description noticing it missed to rename the message also

functional case for reproducer:
- as system admin role
- Go to window Inventory status
- Insert a new row
  with Available=N

Error message in UI is:
Saving failed. There is a constraint defined that was not satisfied. Please check the data entered

and openbravo.log shows technical error to be:
ERROR: new row for relation "m_inventorystatus" violates check constraint "em_obretco_availablecross"

When message should be:
Saving failed. Inventory Status can't be Available for Cross Store if it is not Available
Proposed SolutionRename the ad_message.value to match the new check constraint.
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
blocks feature request 0057210 newTriage Platform Base Openbravo ERP Move content of org.openbravo.retail.config module to backoffice/core repo 

-  Notes
(0172366)
hgbot (developer)
2024-11-20 18:20

Merge Request created: https://gitlab.com/openbravo/ci/mobile-api/-/merge_requests/32 [^]
(0172367)
hgbot (developer)
2024-11-20 18:20

Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.config/-/merge_requests/34 [^]

- Issue History
Date Modified Username Field Change
2024-11-20 18:01 shuehner New Issue
2024-11-20 18:01 shuehner Assigned To => Retail
2024-11-20 18:01 shuehner Triggers an Emergency Pack => No
2024-11-20 18:01 shuehner Assigned To Retail => shuehner
2024-11-20 18:01 shuehner Relationship added blocks 0057210
2024-11-20 18:15 shuehner Steps to Reproduce Updated View Revisions
2024-11-20 18:16 shuehner Description Updated View Revisions
2024-11-20 18:20 hgbot Note Added: 0172366
2024-11-20 18:20 hgbot Note Added: 0172367


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker