diff options
| author | Matěj Cepl <mcepl@redhat.com> | 2011-05-02 21:59:55 +0200 |
|---|---|---|
| committer | Matěj Cepl <mcepl@redhat.com> | 2011-05-02 22:15:17 +0200 |
| commit | bfb84b19e36f06c8f21ff54bda9a104af89cea79 (patch) | |
| tree | 4680259accec8966c74275e20faa5587c6bcd3fb | |
| parent | 4619c44e1c0f79523466934747ec599c45db52ce (diff) | |
| download | bugzilla-triage-bfb84b19e36f06c8f21ff54bda9a104af89cea79.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 |