URL中に文字参照が入っていても許容するよう変更

ただし、>はアンカーと被るので弾く。
awef · Nov 12, 2012 · e65ceb2 · e65ceb2
1 parent 016eb2a
commit e65ceb2
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/src/test/UI.ThreadContent.spec.coffee b/src/test/UI.ThreadContent.spec.coffee
@@ -215,6 +215,15 @@ describe "UI.ThreadContent", ->
       hogehoge()
       return
 
+    it "URL中に文字参照が入っていても許容する", ->
+      example.a.data1.message = "test http://goo.gl/e?1&amp;1 test"
+      example.a.dom1.querySelector(".message").innerHTML = """
+        test <a href="http://goo.gl/e?1&amp;1" target="_blank">http://goo.gl/e?1&amp;1</a> test
+      """
+
+      hogehoge()
+      return
+
     it "URLは空白文字以外と隣接していても認識する", ->
       example.a.data1.message = "テストhttp://goo.gl/eテスト"
       example.a.dom1.querySelector(".message").innerHTML = """

diff --git a/src/ui/ThreadContent.coffee b/src/ui/ThreadContent.coffee
@@ -258,7 +258,7 @@ class UI.ThreadContent
             #タグ除去
             .replace(/<(?!(?:br|hr|\/?b)>).*?(?:>|$)/ig, "")
             #URLリンク
-            .replace(/(h)?(ttps?:\/\/(?:[a-hj-zA-HJ-Z\d_\-.!~*'();\/?:@=+$,%#]|\&(?!(?:#(\d+)|#x([\dA-Fa-f]+)|([\da-zA-Z]+));)|[iI](?![dD]:)+)+)/g,
+            .replace(/(h)?(ttps?:\/\/(?:[a-hj-zA-HJ-Z\d_\-.!~*'();\/?:@=+$,%#]|\&(?!gt;)|[iI](?![dD]:)+)+)/g,
               '<a href="h$2" target="_blank">$1$2</a>')
             #Beアイコン埋め込み表示
             .replace ///^\s*sssp://(img\.2ch\.net/ico/[\w\-_]+\.gif)\s*<br>///, ($0, $1) =>