diff options
| author | Matěj Cepl <mcepl@redhat.com> | 2010-09-14 22:47:47 +0200 |
|---|---|---|
| committer | Matěj Cepl <mcepl@redhat.com> | 2010-09-14 22:47:47 +0200 |
| commit | a2b9bc18e3a11ddc5b0c5dc1fff6edc57b9285b5 (patch) | |
| tree | 437e02783b11244f712b69695f09b891ab70b85e /lib/util.js | |
| parent | 175cdd6039700ee656d220d35854c6277f4a1203 (diff) | |
| download | bugzilla-triage-a2b9bc18e3a11ddc5b0c5dc1fff6edc57b9285b5.tar.gz | |
Clarify the situation with getBugNo.
* get rid off getBugId ... it was never used and silly
* improve javadoc of getBugNo and explain why it is in util.js,
although it would more belong to bzpage.js (hint: there are no
unit tests for bzpage.js)
* add a test showing that aliases are not convereted to proper
numerical bug numbers
Diffstat (limited to 'lib/util.js')
| -rw-r--r-- | lib/util.js | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/lib/util.js b/lib/util.js index 18a3c1b..0b1a79e 100644 --- a/lib/util.js +++ b/lib/util.js @@ -61,7 +61,12 @@ var getParamsFromURL = exports.getParamsFromURL = function getParamsFromURL (url }; /** - * FIXME this should go to bzpage.js ... no reason to keep it here + * Get a bug no from URL ... fails with aliases + * It should theoretically belong to bzpage.js, but we don't have + * unit tests there yet, so keeping here. + * + * @param url String with URL to be analyzed + * @return String with the bug ID (hopefully number, but not for aliases) */ exports.getBugNo = function getBugNo(url) { var params = getParamsFromURL(url); |