Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0033702Retail ModulesWeb POSpublic2016-08-12 09:552016-08-18 13:35
Reportermtaal 
Assigned Tomtaal 
PrioritynormalSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR16Q2.2Fixed in VersionRR16Q2.2 
Merge Request Status
Review Assigned Tomigueldejuana
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0033702: NPE in POSImportEntryProcessor when the posterminal json property has an illegal value
DescriptionSee stacktrace [1], this causes the import entry from not being saved at all. Change the code [2] to set the status to error and a warning messages.

[1]
Caused by: org.openbravo.base.exception.OBException: java.lang.NullPointerException
        at org.openbravo.mobile.core.process.DataSynchronizationProcess.createImportEntry(DataSynchronizationProcess.java:89)
        at org.openbravo.mobile.core.process.DataSynchronizationProcess.executeCreateImportEntry(DataSynchronizationProcess.java:115)
        ... 41 more
Caused by: java.lang.NullPointerException
        at org.openbravo.retail.posterminal.importprocess.POSImportEntryProcessor.onSave(POSImportEntryProcessor.java:79)
        at sun.reflect.GeneratedMethodAccessor445.invoke(Unknown Source)

[2]
https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/file/060b566c4098/src/org/openbravo/retail/posterminal/importprocess/POSImportEntryProcessor.java [^]
Steps To ReproduceRun retail
Put breakpoint in java code in POSImportEntryProcessor
Login to webpos
Do some actions
At breakpoint change the json to set the posTerminalId to the value "null" (note use the string value "null" so not NULL or null)
Proposed Solutionlog warning
set status to Error if no posterminal can be found
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0033696RR16Q4 closed mtaal NPE in POSImportEntryProcessor when the posterminal json property has an illegal value 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-08-12 12:46OrekariaTypedefect => backport
2016-08-12 12:46OrekariaTarget VersionRR16Q4 => RR16Q2.2
2016-08-15 19:39hgbotCheckin
2016-08-15 19:39hgbotNote Added: 0089101
2016-08-15 19:39hgbotStatusscheduled => resolved
2016-08-15 19:39hgbotResolutionopen => fixed
2016-08-15 19:39hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/77435e4a7b40ba41925cf9d53712db38b242675c [^] => http://code.openbravo.com/retail/backports/3.0RR16Q2.2/org.openbravo.retail.posterminal/rev/0eadefd4c5c3e835e70a2873c04b6615e871509a [^]
2016-08-18 13:35guilleaerStatusresolved => closed
2016-08-18 13:35guilleaerFixed in Version => RR16Q2.2

Notes
(0089101)
hgbot   
2016-08-15 19:39   
Repository: retail/backports/3.0RR16Q2.2/org.openbravo.retail.posterminal
Changeset: 0eadefd4c5c3e835e70a2873c04b6615e871509a
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Mon Aug 15 19:39:08 2016 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR16Q2.2/org.openbravo.retail.posterminal/rev/0eadefd4c5c3e835e70a2873c04b6615e871509a [^]

Fixes issue 33702: NPE in POSImportEntryProcessor when the posterminal json property has an illegal value
If no posterminal id can be found then still create the import entry but with status error.
Also use static finals were appropriate.
The changeset for issue 33555 has been included in the commit as it seemed better to keep than to rework the
code to a new version.

---
M src/org/openbravo/retail/posterminal/importprocess/POSImportEntryProcessor.java
---