Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0048153 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] E. Translation | minor | always | 2021-11-24 12:48 | 2022-01-05 10:15 | |||
Reporter | jetxarri | View Status | public | |||||
Assigned To | AugustoMauch | |||||||
Priority | high | Resolution | fixed | Fixed in Version | PR22Q2 | |||
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 | pi | SCM revision | ||||||
Review Assigned To | ||||||||
Web browser | ||||||||
Modules | Core | |||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0048153: Error translating a discount and promotion | |||||||
Description | Error translating a discount and promotion after editing in grid view | |||||||
Steps To Reproduce | -Go to discount and promotions -Create a translation in form view -Go to grid view -Edit and remove the language -Press save --> you will see red rectangle with the error -Press Edit -You will see Albanian as selected -Press save The error appears again You need to select manually the lang to avoid the error | |||||||
Tags | NOR | |||||||
Attached Files | ||||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | |
Notes | |
(0133987) hgbot (developer) 2021-12-27 14:14 |
Merge Request created: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/483 [^] |
(0134003) hgbot (developer) 2021-12-28 10:54 |
Merge request merged: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/483 [^] |
(0134004) hgbot (developer) 2021-12-28 10:54 |
Directly closing issue as related merge request is already approved. Repository: https://gitlab.com/openbravo/product/openbravo [^] Changeset: 990fdf0637d46addd3d8bfb8874df4d590d496fa Author: Augusto Mauch <augusto.mauch@openbravo.com> Date: 28-12-2021 10:53:54 URL: https://gitlab.com/openbravo/product/openbravo/-/commit/990fdf0637d46addd3d8bfb8874df4d590d496fa [^] Fixes ISSUE-48153: Validating error cleared when no mandatory fields are empty To reproduce the issue, the user had to: - create a record, filling in all mandatory fields - empty one of the mandatory fields and save, an expected validation error was raised - edit the line in grid view, mandatory field is given a default value (the FIC does it), but after trying to save the validation error does not go away and the new value is not saved The problem was that even though the FIC was providing a valid default value for the empty mandatory field, the save process was not trying to save the record because it mistakingly thought that the edit form had not been changed. To fix this, when processing the FIC return the setHasChanged function is invoked if the FIC has provided a not-null value for a field that used to be empty. --- M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/ob-view-form.js --- |
(0134127) hgbot (developer) 2022-01-05 09:30 |
Directly closing issue as related merge request is already approved. Repository: https://gitlab.com/openbravo/product/openbravo [^] Changeset: 4aa9945c68b21f31b6ecacd55cf63a1aa5ec7ed4 Author: Augusto Mauch <augusto.mauch@openbravo.com> Date: 05-01-2022 09:24:27 URL: https://gitlab.com/openbravo/product/openbravo/-/commit/4aa9945c68b21f31b6ecacd55cf63a1aa5ec7ed4 [^] Revert "Fixes ISSUE-48153: Validating error cleared when no mandatory fields are empty" This reverts commit 990fdf0637d46addd3d8bfb8874df4d590d496fa. Fix is suspect of breaking CI, will be backed out until the cause is clear --- M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/ob-view-form.js --- |
Issue History | |||
Date Modified | Username | Field | Change |
2021-11-24 12:48 | jetxarri | New Issue | |
2021-11-24 12:48 | jetxarri | Assigned To | => platform |
2021-11-24 12:48 | jetxarri | Modules | => Core |
2021-11-24 12:48 | jetxarri | Triggers an Emergency Pack | => No |
2021-11-24 12:48 | jetxarri | Tag Attached: NOR | |
2021-12-14 10:32 | sebastien_liron | Issue Monitored: sebastien_liron | |
2021-12-27 10:58 | AugustoMauch | Assigned To | platform => AugustoMauch |
2021-12-27 14:14 | hgbot | Note Added: 0133987 | |
2021-12-27 16:58 | AugustoMauch | Status | new => scheduled |
2021-12-28 10:54 | hgbot | Resolution | open => fixed |
2021-12-28 10:54 | hgbot | Status | scheduled => closed |
2021-12-28 10:54 | hgbot | Note Added: 0134003 | |
2021-12-28 10:54 | hgbot | Fixed in Version | => PR22Q1 |
2021-12-28 10:54 | hgbot | Note Added: 0134004 | |
2022-01-05 09:30 | hgbot | Fixed in Version | PR22Q1 => PR22Q2 |
2022-01-05 09:30 | hgbot | Note Added: 0134127 | |
2022-01-05 10:15 | AugustoMauch | Issue cloned | 0048354 |
Copyright © 2000 - 2009 MantisBT Group |