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

View Revisions: Issue #57216 Back to Issue ]
Summary 0057216: Check constraint renaming broken custom AD_MESSAGE associated to it
Revision 2024-11-20 18:16 by shuehner
Description Back 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 [^]
Revision 2024-11-20 18:15 by shuehner
Description Back in 2019 the check-constraint
EM_OBRETCO_AVAILABLECROSSS_CHK
ws renamed to be
EM_OBRETCO_AVAILABLECROSS_CHK
to fix the triple SSS wrong spelling.

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 [^]
Revision 2024-11-20 18:15 by shuehner
Steps To Reproduce Search 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
Revision 2024-11-20 18:01 by shuehner
Steps To Reproduce Search 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


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker