1242439 - Sync engine messing around with record ids

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Sync, defect)

Description

[Francisco Jordano [:arcturus] [:francisco]]

When creating the adapter for the password manager I've come with an extrange result.

I was using record.id to identify the different records comming from the engine in the |update| function invoked in the adapter, and went through the following case:

- Update once and have all the passwords already synced.
- Modiy a password in firefox desktop.
- Sync firefox desktop.
- Sync firefoxos.

Expected:
- The updated record (with id XYZ) is modified in the list of records received in the update function.

Error:
- The updated record has a new id, and the id XYZ is being reused in a different record.

This is applied to passwords, but I guess that will be the same for other collections right?

Comment 1

[Francisco Jordano [:arcturus] [:francisco]]

Michiel is this familiar to you?

Comment 2

[Michiel de Jong [:michielbdejong]]

Investigating

Comment 3

[Michiel de Jong [:michielbdejong]]

Could not replicate the problem... :( I saved one password to my test account, and was able to sync it to FxOS. When I updated the password in FxDesktop, synced FxDesktop, then synced FxOS, I saw the changed password come in to the 'List of modifications ---------->> ' console.log statement, as expected.

These are the changes I made while debugging on the arcturus/bug-1206010 branch, although that's probably not related:
https://github.com/arcturus/gaia/compare/bug-1206010...michielbdejong:passwords-adapter?expand=1

Comment 4

[Francisco Jordano [:arcturus] [:francisco]]

Thanks Michiel, will give it a try tomorrow again to check if happens.

Comment 5

[BMO Automation]

Firefox OS is not being worked on