Bug 551874 - Figure out if we can avoid getting into partial synced state [r=mconnor]

For not-mobile clients, sync everything by specifying Infinity to keep existing math/logic working. Don't add a limit ?query if it's Infinity.
This commit is contained in:
Edward Lee 2010-03-19 11:35:01 -07:00
parent fd097ce9f9
commit cf3fc65766
2 changed files with 2 additions and 2 deletions

View File

@ -75,7 +75,7 @@ Collection.prototype = {
args.push('sort=' + this.sort);
if (this.ids != null)
args.push("ids=" + this.ids);
if (this.limit > 0)
if (this.limit > 0 && this.limit != Infinity)
args.push("limit=" + this.limit);
this.uri.query = (args.length > 0)? '?' + args.join('&') : '';

View File

@ -406,7 +406,7 @@ SyncEngine.prototype = {
this._log.trace("Downloading & applying server changes");
// Figure out how many total items to fetch this sync; do less on mobile
let fetchNum = 1500;
let fetchNum = Infinity;
if (Svc.Prefs.get("client.type") == "mobile")
fetchNum = 50;