Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0057216 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Retail Modules] Retail Configuration | minor | have not tried | 2024-11-20 18:01 | 2024-11-21 12:14 | |||
Reporter | shuehner | View Status | public | |||||
Assigned To | shuehner | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | RR25Q1 | |||
Status | closed | 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 | 0057216: Check constraint renaming broken custom AD_MESSAGE associated to it | |||||||
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 [^] | |||||||
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 | |||||||
Proposed Solution | Rename the ad_message.value to match the new check constraint. | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | |||||||||
|
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 [^] |
(0172388) hgbot (developer) 2024-11-21 12:14 |
Merge request merged: https://gitlab.com/openbravo/ci/mobile-api/-/merge_requests/32 [^] |
(0172389) hgbot (developer) 2024-11-21 12:14 |
Repository: https://gitlab.com/openbravo/ci/mobile-api [^] Changeset: 4a9f5b499dd1405693f161840be1dc879c426748 Author: Stefan Huehner <s.huehner@orisha.com> Date: 20-11-2024 18:18:28 URL: https://gitlab.com/openbravo/ci/mobile-api/-/commit/4a9f5b499dd1405693f161840be1dc879c426748 [^] ISSUE-57216: Accept API change --- M model/modules/org.openbravo.retail.config/src-db/database/sourcedata/AD_MESSAGE.xml --- |
(0172390) hgbot (developer) 2024-11-21 12:14 |
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.config/-/merge_requests/34 [^] |
(0172391) hgbot (developer) 2024-11-21 12:14 |
Directly closing issue as related merge request is already approved. Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.config [^] Changeset: 5ad1f05e38d7203d73e79fe211eccd3a381087a9 Author: Stefan Huehner <s.huehner@orisha.com> Date: 20-11-2024 18:17:06 URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.config/-/commit/5ad1f05e38d7203d73e79fe211eccd3a381087a9 [^] Fixes ISSUE-57216: Fix ad_message.value after constraint renaming This commit from 2019 renamed a check constraint on m_inventorystatus table. But it missed to also rename the associated custom error message in AD_MESSAGE table (automatically used if name matches). Rename the message to match the constraint name again. --- M src-db/database/sourcedata/AD_MESSAGE.xml --- |
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 | |
2024-11-21 12:14 | hgbot | Note Added: 0172388 | |
2024-11-21 12:14 | hgbot | Note Added: 0172389 | |
2024-11-21 12:14 | hgbot | Resolution | open => fixed |
2024-11-21 12:14 | hgbot | Status | new => closed |
2024-11-21 12:14 | hgbot | Note Added: 0172390 | |
2024-11-21 12:14 | hgbot | Fixed in Version | => RR25Q1 |
2024-11-21 12:14 | hgbot | Note Added: 0172391 |
Copyright © 2000 - 2009 MantisBT Group |