Make URL for JSON file configurable (fixes #9)0.4

2 files changed, 25 insertions, 6 deletions

diff --git a/lib/bzpage.js b/lib/bzpage.js
index 1a43191..eac54af 100644
--- a/lib/bzpage.js
+++ b/lib/bzpage.js
@@ -12,6 +12,7 @@ var TriagedDistro = 13;
 var NumberOfFrames = 7;
 var bugURL = "https://bugzilla.redhat.com/show_bug.cgi?id=";
 var BSTPrefNS = "bugzilla-triage.setting.";
+exports.BSTPrefNS = BSTPrefNS;
 
 // ====================================================================================
 // BZPage's methods
@@ -97,11 +98,16 @@ BZPage.prototype.getBugId = function getBugId () {
  */
 BZPage.prototype.getInstalledPackages = function getInstalledPackages(cfg) {
     var installedPackages = {};
+    var prefName = BSTPrefNS+"enabledPacks";
+    var enabledPackages = [];
 
     if (cfg.gJSONData && ("commentPackages" in cfg.gJSONData)) {
-        // TODO: Temporary hack ... should be replaced by a proper configuration
-        // when replacement for jetpack.storage.settings lands
-        var enabledPackages = cfg.gJSONData.configData.enabledPacks.split(/[, ]/);
+        if (preferences.isSet(prefName)) {
+            enabledPackages = preferences.get(prefName,"").split(/[, ]/);
+        } else {
+            preferences.set(prefName,"");
+        }
+
         enabledPackages.forEach(function (pkg, idx, arr) {
             if (pkg in cfg.gJSONData.commentPackages) {
                 installedPackages[pkg] = cfg.gJSONData.commentPackages[pkg];
@@ -744,4 +750,4 @@ BZPage.prototype.getCCList = function getCCList () {
 };
 
 // exports.BZPage = apiUtils.publicConstructor(BZPage);
-exports.BZPage = BZPage;
\ No newline at end of file
+exports.BZPage = BZPage;
diff --git a/lib/main.js b/lib/main.js
index 7cd9a9c..f40dcd1 100644
--- a/lib/main.js
+++ b/lib/main.js
@@ -16,7 +16,10 @@ var logger = require("logger");
 var myStorage = require("simple-storage").storage;
 var browser = require("tab-browser");
 var urlMod = require("url");
-var JSONURL = "http://matej.ceplovi.cz/progs/data/RH_Data-packages.json";
+var preferences = require("preferences-service");
+var BSTPrefNS = require("bzpage").BSTPrefNS;
+// Use my JSON for now before it is fixed for general public
+var JSONURLDefault = "http://matej.ceplovi.cz/progs/data/RH_Data-packages.json";
 
 var TriagedDistro = 13;
 var NumberOfFrames = 7;
@@ -91,7 +94,17 @@ function skipThisPage(doc) {
 }
 
 function initialize(callback) {
-    util.loadJSON(JSONURL, function(parsedData) {
+    var prefName = BSTPrefNS+"JSONURL";
+    var url = "";
+
+    if (preferences.isSet(prefName)) {
+        url = preferences.get(prefName);
+    } else {
+        url = JSONURLDefault;
+        preferences.set(prefName, JSONURLDefault);
+    }
+
+    util.loadJSON(url, function(parsedData) {
         config.gJSONData = parsedData;
 
         var keys = "", key = "";