0050438: Keymap loader should be more ressilient to not fail when some properties are not received

Description

Keymap loader can fail if some properties are not present.

We should make code more resilient to avoid errors like this one

org.openbravo.base.exception.OBException: org.openbravo.pos2.OBPOS2KeymapProduct product is a mandatory field to create a new org.openbravo.pos2.OBPOS2KeymapProduct from Web Pos
    at org.openbravo.pos2.KeymapLoader.checkMandatoryFields(KeymapLoader.java:257)
    at org.openbravo.pos2.KeymapLoader.createEntity(KeymapLoader.java:201)
    at org.openbravo.pos2.KeymapLoader.processEntity(KeymapLoader.java:240)
    at org.openbravo.pos2.KeymapLoader.saveRecord(KeymapLoader.java:87)
    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)
    at org.openbravo.mobile.core.process.MobileImportEntryProcessorRunnable.processEntry(MobileImportEntryProcessorRunnable.java:57)
    at org.openbravo.service.importprocess.ImportEntryProcessor$ImportEntryProcessRunnable.doRunCycle(ImportEntryProcessor.java:421)
    at org.openbravo.service.importprocess.ImportEntryProcessor$ImportEntryProcessRunnable.run(ImportEntryProcessor.java:338)
    at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)
    at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
    at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
    at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
    at java.base/java.lang.Thread.run(Thread.java:829)

Steps To Reproduce

N/A

Tags

No tags attached.

Attached Files

Relationships [ Relation Graph ] [ Dependency Graph ]

blocks

defect

0050436

closed

AugustoMauch

Keymap loader should be more ressilient to not fail when some properties are not received

Notes
(0141739) hgbot (developer) 2022-10-06 14:39	Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/1257 [^]

(0141741) hgbot (developer) 2022-10-06 14:45	Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/1257 [^]

Issue History
Date Modified	Username	Field	Change
2022-10-06 13:40	guilleaer	Type	defect => backport
2022-10-06 13:40	guilleaer	Target Version	=> 22Q3.2
2022-10-06 14:39	hgbot	Note Added: 0141739
2022-10-06 14:45	hgbot	Note Added: 0141741
2022-11-03 11:38	AugustoMauch	Status	scheduled => resolved
2022-11-03 11:38	AugustoMauch	Resolution	open => fixed
2022-11-03 11:38	AugustoMauch	Status	resolved => closed