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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0044121
TypeCategorySeverityReproducibilityDate SubmittedLast Update
backport[Retail Modules] Web POSmajoralways2020-05-20 14:292020-06-01 19:57
ReporterjarmendarizView Statuspublic 
Assigned Tojarmendariz 
PriorityhighResolutionfixedFixed in VersionRR20Q2
StatusclosedFix in branchFixed in SCM revision40ff9336a742
ProjectionnoneETAnoneTarget VersionRR20Q2
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

0044121: 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 ]
blocks defect 0044120RR20Q3 closedjarmendariz Messages synchronized with backoffice does not include the message ID 

-  Notes
(0120470)
hgbot (developer)
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
---

- Issue History
Date Modified Username Field Change
2020-05-20 14:29 jarmendariz Type defect => backport
2020-05-20 14:29 jarmendariz Target Version RR20Q3 => RR20Q2
2020-05-29 12:35 hgbot Checkin
2020-05-29 12:35 hgbot Note Added: 0120470
2020-05-29 12:35 hgbot Status scheduled => resolved
2020-05-29 12:35 hgbot Resolution open => fixed
2020-05-29 12:35 hgbot Fixed in SCM revision => http://code.openbravo.com/retail/backports/3.0RR20Q2/org.openbravo.mobile.core/rev/40ff9336a742cbb23ab03c79e853d7657fe82c88 [^]
2020-06-01 19:57 marvintm Review Assigned To => marvintm
2020-06-01 19:57 marvintm Status resolved => closed
2020-06-01 19:57 marvintm Fixed in Version => RR20Q2


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker