diff options
author | Gaëtan Gilbert <gaetan.gilbert@skyskimmer.net> | 2017-10-18 19:13:39 +0200 |
---|---|---|
committer | Gaëtan Gilbert <gaetan.gilbert@skyskimmer.net> | 2017-10-18 19:26:00 +0200 |
commit | fd391e6755f6552c9786bd285cf55eb506e0b056 (patch) | |
tree | cac955a8903382fb85a84f52c86484989f118794 /dev/Coq_Bugzilla_autolink.user.js | |
parent | 6bda57bd75efe55fe1f7774f932e9ef5a65aeaaf (diff) |
Bugzilla autolink: avoid linking inside links (fix #5974).
Diffstat (limited to 'dev/Coq_Bugzilla_autolink.user.js')
-rw-r--r-- | dev/Coq_Bugzilla_autolink.user.js | 46 |
1 files changed, 40 insertions, 6 deletions
diff --git a/dev/Coq_Bugzilla_autolink.user.js b/dev/Coq_Bugzilla_autolink.user.js index 371c5adc0..5ff618a83 100644 --- a/dev/Coq_Bugzilla_autolink.user.js +++ b/dev/Coq_Bugzilla_autolink.user.js @@ -10,25 +10,59 @@ var regex = /BZ#(\d+)/g; var substr = '<a href="https://coq.inria.fr/bugs/show_bug.cgi?id=$1">$&</a>'; -function doNode(node) +function doTitle(node) { node.innerHTML = node.innerHTML.replace(regex,substr); } +function filter(node) +{ + if (node.nodeName == '#text') + { + return NodeFilter.FILTER_ACCEPT; + } + else if(node.nodeName == 'A') + { + return NodeFilter.FILTER_REJECT; + } + return NodeFilter.FILTER_SKIP; +} var comments = document.getElementsByClassName("comment-body"); -for(var i=0; i<comments.length; i++) +function doNode(parent) { - var pars = comments[i].getElementsByTagName("p"); - for(var j=0; j<pars.length; j++) + var nodes = document.createTreeWalker(parent,NodeFilter.SHOW_ALL,{ acceptNode : filter },false); + var node; + while(node=nodes.nextNode()) { - doNode(pars[j]); + var content = node.textContent; + var matches = regex.exec(content); + + if(matches && matches.length > 1) + { + var range = document.createRange(); + var start = content.search(regex); + var end = start + matches[0].length; + range.setStart(node, start); + range.setEnd(node, end); + var linkNode = document.createElement("a"); + linkNode.href = "https://coq.inria.fr/bugs/show_bug.cgi?id=" + matches[1]; + range.surroundContents(linkNode); + + //handle multiple matches in one text node + doNode(linkNode.parentNode); + } } } +for(var i=0; i<comments.length; i++) +{ + doNode(comments[i]); +} + // usually 1 or 0 titles... var titles = document.getElementsByClassName("js-issue-title"); for(var i=0; i<titles.length; i++) { - doNode(titles[i]); + doTitle(titles[i]); } |