Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0044120
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] Web POSmajoralways2020-05-20 14:292020-06-01 19:59
ReporterjarmendarizView Statuspublic 
Assigned Tojarmendariz 
PriorityhighResolutionfixedFixed in VersionRR20Q3
StatusclosedFix in branchFixed in SCM revision2b021a8ac28e
ProjectionnoneETAnoneTarget VersionRR20Q3
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Tomarvintm
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0044120: 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.
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
depends on backport 0044121RR20Q2 closedjarmendariz Messages synchronized with backoffice does not include the message ID 

-  Notes
(0120455)
hgbot (developer)
2020-05-29 10:59

Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 2b021a8ac28e455c5ba6c75380b9e12f58c3d039
Author: Javier Armendáriz <javier.armendariz <at> openbravo.com>
Date: Wed May 20 17:22:45 2020 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/2b021a8ac28e455c5ba6c75380b9e12f58c3d039 [^]

Fixed ISSUE-44120: No messageId field sent in SyncBuffer messages

Adding the MessageID to the Messages sent to the backoffice, and adding a log line
to be notified when a received Message does not have this field set

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

- Issue History
Date Modified Username Field Change
2020-05-20 14:29 jarmendariz New Issue
2020-05-20 14:29 jarmendariz Assigned To => jarmendariz
2020-05-20 14:29 jarmendariz Triggers an Emergency Pack => No
2020-05-20 14:29 jarmendariz Status new => scheduled
2020-05-29 10:59 hgbot Checkin
2020-05-29 10:59 hgbot Note Added: 0120455
2020-05-29 10:59 hgbot Status scheduled => resolved
2020-05-29 10:59 hgbot Resolution open => fixed
2020-05-29 10:59 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/2b021a8ac28e455c5ba6c75380b9e12f58c3d039 [^]
2020-06-01 19:59 marvintm Review Assigned To => marvintm
2020-06-01 19:59 marvintm Status resolved => closed
2020-06-01 19:59 marvintm Fixed in Version => RR20Q3


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker