Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: lib/synchronizer.js

Issue 29338611: Issue 3836 - Don't set DownloadableSubscription.upgradeRequired in Synchronizer (Closed)
Patch Set: Created March 18, 2016, 12:31 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/synchronizer.js
===================================================================
--- a/lib/synchronizer.js
+++ b/lib/synchronizer.js
@@ -234,25 +234,20 @@ var Synchronizer = exports.Synchronizer
expirationInterval = interval * MILLIS_IN_DAY;
}
}
let [softExpiration, hardExpiration] = downloader.processExpirationInterval(expirationInterval);
subscription.softExpiration = Math.round(softExpiration / MILLIS_IN_SECOND);
subscription.expires = Math.round(hardExpiration / MILLIS_IN_SECOND);
- delete subscription.requiredVersion;
- delete subscription.upgradeRequired;
if (minVersion)
- {
- let {addonVersion} = require("info");
subscription.requiredVersion = minVersion;
- if (Services.vc.compare(minVersion, addonVersion) > 0)
- subscription.upgradeRequired = true;
- }
+ else
+ delete subscription.requiredVersion;
// Process filters
lines.shift();
let filters = [];
for (let line of lines)
{
line = Filter.normalize(line);
if (line)
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld