diff options
| author | Matěj Cepl <mcepl@redhat.com> | 2011-05-02 21:59:55 +0200 |
|---|---|---|
| committer | Matěj Cepl <mcepl@redhat.com> | 2011-06-05 14:51:41 +0200 |
| commit | 92421b51663ce34939fee936ddcbeec91a67bced (patch) | |
| tree | d5acb6b36369d7ed586038941ce790c37bf22193 | |
| parent | 08194f2c04e1831a8a6c50b8290615889a5080a3 (diff) | |
| download | bugzilla-triage-92421b51663ce34939fee936ddcbeec91a67bced.tar.gz | |
"all" keyword in enabledPackages configuration didn't work.
Bad use of Array.slice.
Fixes #89
| -rw-r--r-- | lib/libbugzilla.js | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/lib/libbugzilla.js b/lib/libbugzilla.js index aa2d0b7..a1792ea 100644 --- a/lib/libbugzilla.js +++ b/lib/libbugzilla.js @@ -171,11 +171,10 @@ exports.getInstalledPackages = function getInstalledPackages(locationLoginObj, c } } - var allIdx = null; if ((allIdx = enabledPackages.indexOf("all")) != -1) { - enabledPackages = enabledPackages.splice(allIdx, - Object.keys(config.gJSONData.commentPackages)); + enabledPackages.splice(allIdx, 1); + enabledPackages = enabledPackages.concat(Object.keys(config.gJSONData.commentPackages)); } // TODO To be decided, whether we cannot just eliminate packages in |