Project

General

Profile

Bug(バグ) #2323

半角空白区切りの連続した複数のURLが正常に自動リンクされない

Added by Shingo Yamada over 8 years ago. Updated over 4 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2011-07-29
Due date:
% Done:

100%

3.6 で発生するか:
Yes (はい)
3.8 で発生するか:
Unknown (未調査)

Description

Overview (現象)

半角空白区切りの連続した複数のURLが正常に自動リンクされない

詳細

日記本文で現象を確認

書き込み内容
http://example.com/ http://example.com/hoge http://example.com/fuga http://example.com/hoge
ブラウザ出力結果(HTML)
<div class="title">
<p class="heading">test</p>
</div>
<div class="body">
<a href="http://example.com/" target="_blank">http://example.com/</a> http://example.com/hoge <a href="http://example.com/fuga" target="_blank">http://example.com/fuga</a> http://example.com/hoge</div>
</dd>
</dl>

Causes (原因)

リンクを生成する際に URL を正規表現でマッチさせているが,これがプレフィックスとして空白文字を考慮していない.

lib/helper/opUtilHelper.php 313-317行目

 310 if (!defined('SF_AUTO_LINK_RE'))
 311 {
 312   define('SF_AUTO_LINK_RE', '~
 313     (                       # leading text
 314       <\w+.*?>|             #   leading HTML tag, or
 315       [^=!:\'"/]|           #   leading punctuation, or
 316       ^                     #   beginning of line, or
 317     )                             
 318     (                             
 319       (?:https?://)|        # protocol spec, or
 320       (?:www\.)             # www.*
 321     )                             
 322     (                     
 323       [-\w]+                   # subdomain or domain
 324       (?:\.[-\w]+)*            # remaining subdomains or domain
 325       (?::\d+)?                # port
 326       \/?                 
 327       [a-zA-Z0-9_\-\/.,:;\~\?@&=+$%#!()]*
 328     )                     
 329     ([[:punct:]]|\s|<|$)    # trailing text
 330    ~x');                  
 331 }                         

Way to fix (修正内容)

正規表現で URL にマッチさせる際に空白文字を考慮する.

diff --git a/lib/helper/opUtilHelper.php b/lib/helper/opUtilHelper.php
index 59d22bc..d918493 100644
--- a/lib/helper/opUtilHelper.php
+++ b/lib/helper/opUtilHelper.php
@@ -313,7 +313,8 @@ if (!defined('SF_AUTO_LINK_RE'))
     (                       # leading text
       <\w+.*?>|             #   leading HTML tag, or
       [^=!:\'"/]|           #   leading punctuation, or
-      ^                     #   beginning of line
+      ^|                    #   beginning of line, or
+      \s?                   #   leading whitespaces
     )
     (
       (?:https?://)|        # protocol spec, or

スクリーンショット(2011-07-29_18.16.26).png View (23.3 KB) Shingo Yamada, 2011-07-29 18:23

スクリーンショット.png View (35.5 KB) Yuya Watanabe, 2012-01-17 01:22

20120306134755.png View (17.6 KB) Shouta Kashiwagi, 2012-03-06 13:48


Related issues

Related to OpenPNE 3 - Backport(バックポート) #2733: 半角空白区切りの連続した複数のURLが正常に自動リンクされない Fixed(完了) 2011-07-29
Related to OpenPNE 3 - Backport(バックポート) #2734: 半角空白区切りの連続した複数のURLが正常に自動リンクされない Fixed(完了) 2011-07-29

Associated revisions

Revision a242107d (diff)
Added by Yuya Watanabe about 8 years ago

(fixes #2323) fixed to put links automatically when the text has whitespaces

Revision c400cefb (diff)
Added by Yuya Watanabe almost 8 years ago

(fixes #2323) fixed to put links automatically when the text has whitespaces

Revision 3ddfaa35 (diff)
Added by Yuya Watanabe almost 8 years ago

(fixes #2323) fixed to link correct url

Revision 0ce086c4 (diff)
Added by Yuya Watanabe almost 8 years ago

(refs #2323) fixed for cording standard

Revision 7697b6fd (diff)
Added by Yuya Watanabe almost 8 years ago

(fixes #2323) fixed to link correct url

Revision 454f44d9 (diff)
Added by Yuya Watanabe almost 8 years ago

(refs #2323) fixed for cording standard

History

#1 Updated by Yuma Sakata about 8 years ago

  • 3.6 で発生するか changed from Yes to Yes (はい)
  • 3.4 で発生するか set to Yes (はい)

再現確認

以下バージョンで再現確認できました。

  • 3.6.1
  • 3.4.18

#2 Updated by Yuya Watanabe about 8 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Yuya Watanabe

#3 Updated by Yuya Watanabe about 8 years ago

www.google.co.jpwww.amazon.co.jp など,小窓となりうるリンクが存在する場合に http://www.www.google.co.jphttp://www.www.amazon.co.jp などといったようなリンクが生成されるバグが存在することを確認.

!https://redmine.openpne.jp/attachments/397/スクリーンショット.png!

#4 Updated by Yuya Watanabe about 8 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 a242107d3d94c5b454036de6ac7be7fa23c47b93 で適用されました。

#5 Updated by Kousuke Ebihara almost 8 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#6 Updated by Shouta Kashiwagi almost 8 years ago

  • File 20120306134755.png View added
  • Status changed from Pending Testing(テスト待ち) to Rejected(差し戻し)
  • % Done changed from 70 to 50

テスト実施しましたが、修正が必要な点がありましたので確認お願いします。

以下のように、全角空白文字でURL区切りを行った場合にURLリンクが添付画像のように不完全となってしまいます。
(添付画像、「 http://example.com/detail.php?id=1 」の部分です)

携帯版でも同様の現象が再現できました。
修正の程、よろしくお願い致します。

#7 Updated by Shouta Kashiwagi almost 8 years ago

  • Target version changed from OpenPNE 3.7.0 to 252

#8 Updated by Yuya Watanabe almost 8 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)
  • % Done changed from 50 to 0

原因

URLの終了部分が正しく考慮されていなかったため trailing text にマッチする部分として '=' や '.' があったために note-6 のような挙動になったと言えます.

修正案

trailing text を記号([:punct:])だけでなく,URLに用いられる文字以外すべてとすることで問題を解決できると思います.

diff --git a/lib/helper/opUtilHelper.php b/lib/helper/opUtilHelper.php
index 5a17e99..4e2694c 100644
--- a/lib/helper/opUtilHelper.php
+++ b/lib/helper/opUtilHelper.php
@@ -327,8 +327,8 @@ if (!defined('SF_AUTO_LINK_RE'))
       \/?
       [a-zA-Z0-9_\-\/.,:;\~\?@&=+$%#!()]*
     )
-    ([[:punct:]]|\s|<|$)    # trailing text
-   ~x');
+    ([^a-zA-Z0-9_\-\/.,:;\~\?@&=+$%#!()]|\s|<|$)    # trailing text
+   ~xu');
 }

 function op_url_cmd($text)

その他

3.6.x および 3.4.x でも同じ修正を施しているため,これらのバージョンでも再現するものと思うため,それぞれ確認のうえでバグチケットの報告および修正が必要と思います.

参考

https://tools.ietf.org/html/rfc3986#section-3.3

#9 Updated by Yuma Sakata almost 8 years ago

  • Target version changed from 252 to OpenPNE 3.8beta1

#10 Updated by Yuya Watanabe almost 8 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 c400cefb14d56e07e3e9aa73d25343a601749daa で適用されました。

#11 Updated by Yuya Watanabe almost 8 years ago

  • Status changed from Pending Review(レビュー待ち) to Accepted(着手)
  • % Done changed from 50 to 0

note-8 の修正はまだ取り込んでいないためステータスを「Accepted」に戻します.

#12 Updated by Yuya Watanabe almost 8 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 3ddfaa35a811a2b7794a144c98a58146c2bf738a で適用されました。

#13 Updated by Yuya Watanabe almost 8 years ago

更新履歴 7697b6fdd9ca2c668192b5fd10475284906c2c4b で適用されました。

#14 Updated by Yuya Watanabe almost 8 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#15 Updated by Yuma Sakata almost 8 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

テストOKです。

#16 Updated by kaoru n over 4 years ago

  • 3.8 で発生するか set to Unknown (未調査)

Also available in: Atom PDF