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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0048153
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] E. Translationminoralways2021-11-24 12:482022-01-05 10:15
ReporterjetxarriView Statuspublic 
Assigned ToAugustoMauch 
PriorityhighResolutionfixedFixed in VersionPR22Q2
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionpiSCM revision 
Review Assigned To
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0048153: Error translating a discount and promotion

DescriptionError 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
TagsNOR
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
Powered by Mantis Bugtracker