492094 - Weave posts warnings about integer error during sync in Preferences.js

Status: VERIFIED DUPLICATE of bug 493816

(Cloud Services :: General, defect)

Description

[Tanner M. Young [:tmyoung]]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b4) Gecko/20090423 Firefox/3.5b4
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b4) Gecko/20090423 Firefox/3.5b4

Weave 0.3.0 generates the following warnings in Firefox 3.5b4.  These warnings appear in Firebug and the Error Console during Weave syncs to a private Weave Server.  Each warning points to Preferences.js line 115.

Warning: setting the clients.lastSync pref to the non-integer number
1241805112.41 converted it to the integer number 1241805112; to retain
fractional precision, store non-integer numbers as strings. [Break on this
error] "numbers as strings.");

Warning: setting the bookmarks.lastSync pref to the non-integer number
1241805112.41 converted it to the integer number 1241805112; to retain
fractional precision, store non-integer numbers as strings. [Break on this
error] "numbers as strings.");

Warning: setting the passwords.lastSync pref to the non-integer number
1241805112.41 converted it to the integer number 1241805112; to retain
fractional precision, store non-integer numbers as strings. [Break on this
error] "numbers as strings.");

This is not a major issue, but it might indicate an problem with data type (int versus string) management in Weave or another issue.

Reproducible: Always

Steps to Reproduce:
1. Install Weave
2. Wait until second sync with server
3. Errors appear in Error Console or Firebug
Actual Results:  
Warnings are sent through the Cu.reportError() function in Preferences.js.

Expected Results:  
No warnings should occur

Comment 1

[Dan Mills [:thunder]]

!?

Seriously, the pref svc can't store numbers with fractions?

Comment 2

[Myk Melez [:myk] [@mykmelez]]

> Seriously, the pref svc can't store numbers with fractions?

srsly!

Also, sweet!  The warning I added to the Preferences module had an effect!

Comment 3

[Dan Mills [:thunder]]

omg I am shocked!

The pref svc does not rock.

Comment 4

[Ed Lee :Mardak]

Locally avoided with bug 493816 to store as a string. Preferences.js bug:
https://www.mozdev.org/bugs/show_bug.cgi?id=21060

Comment 5

[Tanner M. Young [:tmyoung]]

Verified duplicate.