Openbravo Issue Tracking System - POS2
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0050268POS2POSpublic2022-09-16 13:312022-09-16 14:08
ReporterAugustoMauch 
Assigned ToAugustoMauch 
PrioritynormalSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version22Q4 
Merge Request Statusapproved
Review Assigned To
OBNetwork customerNo
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0050268: Error when saving keymap changes done from the Keymap Editor in POS2
DescriptionWhen a user saves keymap changes done on the Keymap Editor in POS2, there is an error when trying to process those changes in the backend:

org.codehaus.jettison.json.JSONException: JSONObject["default"] not found.
    at org.codehaus.jettison.json.JSONObject.get(JSONObject.java:360)
    at org.codehaus.jettison.json.JSONObject.getBoolean(JSONObject.java:376)
    at org.openbravo.pos2.KeymapLoader.saveRecord(KeymapLoader.java:90)
    at org.openbravo.mobile.core.process.DataSynchronizationProcess.saveRecord(DataSynchronizationProcess.java:204)
    at org.openbravo.mobile.core.process.DataSynchronizationProcess.exec(DataSynchronizationProcess.java:156)
    at org.openbravo.mobile.core.process.DataSynchronizationProcess.exec(DataSynchronizationProcess.java:88)

Steps To ReproduceIn any POS2 environment:
- Open the Keymap Editor
- Select the Main App Keymap
- Select the single version it has
- Click on SET LAYOUT
- Change the color of a button and save
- Check that there is an error in Errors while importing
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
has duplicate defect 0050266 closed Retail Changes in the Keymap from the POS are not saved in the backoffice generating errors 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2022-09-16 13:31AugustoMauchNew Issue
2022-09-16 13:31AugustoMauchAssigned To => AugustoMauch
2022-09-16 13:31AugustoMauchOBNetwork customer => No
2022-09-16 13:31AugustoMauchTriggers an Emergency Pack => No
2022-09-16 13:41hgbotMerge Request Status => open
2022-09-16 13:41hgbotNote Added: 0141187
2022-09-16 13:48AugustoMauchStatusnew => scheduled
2022-09-16 13:49AugustoMauchRelationship addedhas duplicate 0050266
2022-09-16 14:07hgbotMerge Request Statusopen => approved
2022-09-16 14:08hgbotResolutionopen => fixed
2022-09-16 14:08hgbotStatusscheduled => closed
2022-09-16 14:08hgbotNote Added: 0141192
2022-09-16 14:08hgbotFixed in Version => 22Q4
2022-09-16 14:08hgbotNote Added: 0141193

Notes
(0141187)
hgbot   
2022-09-16 13:41   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/1212 [^]
(0141192)
hgbot   
2022-09-16 14:08   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/1212 [^]
(0141193)
hgbot   
2022-09-16 14:08   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2 [^]
Changeset: f3b7a4e9cab5c264985902b8ce184dc7ad7629dc
Author: Augusto Mauch <augusto.mauch@openbravo.com>
Date: 16-09-2022 12:08:00
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/commit/f3b7a4e9cab5c264985902b8ce184dc7ad7629dc [^]

Fixes ISSUE-50268: Takes into account that default parameter is not mandatory

---
M src/org/openbravo/pos2/KeymapLoader.java
---