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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0051496
TypeCategorySeverityReproducibilityDate SubmittedLast Update
backport[POS2] POSmajorhave not tried2023-01-27 08:472023-02-02 17:16
ReporterAugustoMauchView Statuspublic 
Assigned ToAugustoMauch 
PrioritynormalResolutionfixedFixed in Version23Q1
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version23Q1
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

0051496: Errors when saving keymap editor changes if there are more than 300 records in a keymap

DescriptionWhen saving the changes on a keymap layout, there is some code that checks if it needs to create new instances of Keymap models by comparing the data stored in memory with the data returned by a IndexedDB query (i.e. https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/blob/master/web-jspack/org.openbravo.pos2/src/components/KeyMapEditor/steps/KeyMapEditorLayout/utils/prepareKeymapVersionForSaving.js#L126 [^]).

The problem is that those queries do not specify a limit, so a default limit is set to 300. In that case, the system will think that it needs to create some new records because it mistakenly thinks that they do not exist yet, when the real problem is that they were not returned by the query because of the size limit
Steps To Reproduce- Create a keymap version that has more than 300 manual products among all its keymap categories
- Open that keymap in the Keymap Editor window in POS2.
- Select its version and go to the edit layout screen
- Make any change (update the position of a product, change a color, etc)
- Save. There will be an Error While Importing in the backend, complaining about some constraint not being met
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
blocks defect 0051477 closedAugustoMauch Errors when saving keymap editor changes if there are more than 300 records in a keymap 

-  Notes
(0146185)
hgbot (developer)
2023-02-02 17:14

Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/1503 [^]
(0146186)
hgbot (developer)
2023-02-02 17:16

Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/1503 [^]
(0146187)
hgbot (developer)
2023-02-02 17:16

Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2 [^]
Changeset: 6a7e59efe6f4db5de40f0bf04c33cd9de0ee5cf4
Author: Guillermo Alvarez de Eulate <guillermo.alvarez@openbravo.com>
Date: 02-02-2023 17:13:07
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/commit/6a7e59efe6f4db5de40f0bf04c33cd9de0ee5cf4 [^]

fixed ISSUE-51496: Increases the limit of supported keymap products of a keymap version

---
A web-jspack/org.openbravo.pos2/src/components/KeyMapEditor/steps/KeyMapEditorLayout/utils/getMasterdataModelBy.js
M web-jspack/org.openbravo.pos2/src/components/KeyMapEditor/steps/KeyMapEditorLayout/utils/prepareKeymapCategoryForSaving.js
M web-jspack/org.openbravo.pos2/src/components/KeyMapEditor/steps/KeyMapEditorLayout/utils/prepareKeymapProductsForSaving.js
M web-jspack/org.openbravo.pos2/src/components/KeyMapEditor/steps/KeyMapEditorLayout/utils/prepareKeymapVersionForSaving.js
---

- Issue History
Date Modified Username Field Change
2023-01-30 11:11 guilleaer Type defect => backport
2023-01-30 11:11 guilleaer Target Version => 23Q1
2023-02-02 17:14 hgbot Note Added: 0146185
2023-02-02 17:16 hgbot Resolution open => fixed
2023-02-02 17:16 hgbot Status scheduled => closed
2023-02-02 17:16 hgbot Note Added: 0146186
2023-02-02 17:16 hgbot Fixed in Version => 23Q1
2023-02-02 17:16 hgbot Note Added: 0146187


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker