Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0033694Retail ModulesWeb POSpublic2016-08-11 23:492016-08-18 09:19
Reportermtaal 
Assigned Tomtaal 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR16Q4Fixed in Version 
Merge Request Status
Review Assigned Tomigueldejuana
OBNetwork customerNo
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0033694: POSImportEntryProcessor: log the json in case not all relevant information can be extracted from it
DescriptionIn the POSImportEntryProcessor important information is obtained from the json [1] and [2]. It is an error/warning if this information can't be found resulting in unexpected situations in later code.

Log the error/warning conditions which are interesting to know about for checking the json and repairing the cause.

[1]
https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/file/8b8fcaec12a9/src/org/openbravo/retail/posterminal/importprocess/POSImportEntryProcessor.java#l71 [^]

[2]
https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/file/8b8fcaec12a9/src/org/openbravo/retail/posterminal/importprocess/POSImportEntryProcessor.java#l99 [^]
Steps To ReproduceSee source code
Proposed SolutionSee description.
Additional Information
TagsNo tags attached.
Relationships
related to design defect 0033555RR16Q4 closed mtaal Prevent query for posterminal in POSImportEntryProcessor 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-08-11 23:49mtaalNew Issue
2016-08-11 23:49mtaalAssigned To => mtaal
2016-08-11 23:49mtaalOBNetwork customer => No
2016-08-11 23:49mtaalTriggers an Emergency Pack => No
2016-08-12 00:43mtaalSummaryPOSImportEntryProcessor: log the json in case not all information can be found from it => POSImportEntryProcessor: log the json in case not all relevant information can be extracted from it
2016-08-12 09:19hgbotCheckin
2016-08-12 09:19hgbotNote Added: 0089076
2016-08-12 09:19hgbotStatusnew => resolved
2016-08-12 09:19hgbotResolutionopen => fixed
2016-08-12 09:19hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/060b566c4098cd63051519ba78c2c04ef6284aec [^]
2016-08-12 09:47shuehnerRelationship addedrelated to 0033555
2016-08-16 10:47mtaalReview Assigned To => migueldejuana
2016-08-18 09:19migueldejuanaNote Added: 0089201
2016-08-18 09:19migueldejuanaStatusresolved => closed

Notes
(0089076)
hgbot   
2016-08-12 09:19   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 060b566c4098cd63051519ba78c2c04ef6284aec
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Fri Aug 12 09:19:28 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/060b566c4098cd63051519ba78c2c04ef6284aec [^]

Fixes issue 33694: POSImportEntryProcessor: log the json in case not all information can be found from it
Added 2 log.warn statements

---
M src/org/openbravo/retail/posterminal/importprocess/POSImportEntryProcessor.java
---
(0089201)
migueldejuana   
2016-08-18 09:19   
Verified