Changeset 16424 for lang/javascript/vimperator-plugins/trunk

Timestamp:

07/25/08 20:43:26 (4 months ago)

Author:

send

Message:

avoid false recognition, comma separated , pri

Files:

• lang/javascript/vimperator-plugins/trunk/appendAnchor.js (modified) (2 diffs)

lang/javascript/vimperator-plugins/trunk/appendAnchor.js

@@ -4 +4 @@
  * @description   append anchors to texts look like url.
  * @author        SAKAI, Kazuaki
- * @version       0.01
+ * @version       0.02
  * == VimperatorPlugin==
  */
@@ -16 +16 @@
         '/descendant::*[not(contains(" TITLE STYLE SCRIPT TEXTAREA XMP A ", concat(" ", local-name(), " ")))]/child::text()'
       );
-      var regex =  new RegExp("h?(ttps?):/+([a-zA-Z0-9][-_.!~*'()a-zA-Z0-9;/?:@&=+$,%#]+)");
+      var regex =  new RegExp("h?(ttps?):/+([a-zA-Z0-9][-_.!~*'()a-zA-Z0-9;/?:@&=+$,%#]+[-_~*(a-zA-Z0-9;/?@&=+$%#])");
+      
       var range = doc.createRange();
       var last;
+      var href;
       for (var i = 0; i < nodes.snapshotLength; i++) {
         var node = nodes.snapshotItem(i);
         range.selectNode(node);
         while (node && (last = range.toString().search(regex)) > -1) {
-          range.setStart(node, last); 
+          range.setStart(node, last);
           range.setEnd(node, last + RegExp.lastMatch.length);
+          href = 'h' + RegExp.$1 + '://' + RegExp.$2;
           var anchor = doc.createElement('a');
-          var href = 'h' + RegExp.$1 + '://' + RegExp.$2;
           range.insertNode(anchor);
           anchor.setAttribute('href', href);