Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0044121Retail ModulesWeb POSpublic2020-05-20 14:292020-06-01 19:57
Reporterjarmendariz 
Assigned Tojarmendariz 
PriorityhighSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR20Q2Fixed in VersionRR20Q2 
Merge Request Status
Review Assigned Tomarvintm
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0044121: Messages synchronized with backoffice does not include the message ID
DescriptionWith the current implementation of the SynchronizationBuffer, the message ID is not sent to the backoffice so other fields such as the data.id are used to identify the message and used as an identifier for the Import Entry. This situation can be problematic in the future as the synchronization could fail for not having a valid identifier or the identifier provided already exists.
Steps To ReproduceLogin to WebPOS and open the developer tools in the Network tab:

- Push some buttons in order to create log events
- Eventually, a request to org.openbravo.mobile.core.TerminalLogLoader will be sent to the backoffice
- Check the request payload and verify that the payload object does contain fields like client, csrfToken, data... but messageId is not present.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0044120RR20Q3 closed jarmendariz Messages synchronized with backoffice does not include the message ID 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2020-05-20 14:29jarmendarizTypedefect => backport
2020-05-20 14:29jarmendarizTarget VersionRR20Q3 => RR20Q2
2020-05-29 12:35hgbotCheckin
2020-05-29 12:35hgbotNote Added: 0120470
2020-05-29 12:35hgbotStatusscheduled => resolved
2020-05-29 12:35hgbotResolutionopen => fixed
2020-05-29 12:35hgbotFixed in SCM revision => http://code.openbravo.com/retail/backports/3.0RR20Q2/org.openbravo.mobile.core/rev/40ff9336a742cbb23ab03c79e853d7657fe82c88 [^]
2020-06-01 19:57marvintmReview Assigned To => marvintm
2020-06-01 19:57marvintmStatusresolved => closed
2020-06-01 19:57marvintmFixed in Version => RR20Q2

Notes
(0120470)
hgbot   
2020-05-29 12:35   
Repository: retail/backports/3.0RR20Q2/org.openbravo.mobile.core
Changeset: 40ff9336a742cbb23ab03c79e853d7657fe82c88
Author: Javier Armendáriz <javier.armendariz <at> openbravo.com>
Date: Fri May 29 12:34:58 2020 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR20Q2/org.openbravo.mobile.core/rev/40ff9336a742cbb23ab03c79e853d7657fe82c88 [^]

Fixed ISSUE-44121: Messages synchronized does not include messageId

---
M src/org/openbravo/mobile/core/process/DataSynchronizationProcess.java
M web-test/model/synchronization-buffer/SynchronizationBuffer.test.js
M web/org.openbravo.mobile.core/app/model/synchronization-buffer/SynchronizationBuffer.js
---