Ticket #2367 (closed バグ指摘: 修正済)

Opened 7 years ago

Last modified 6 years ago

レビュー登録のコメント欄のURLチェックで日本語ドメインだとエラーとならない

Reported by: h_yoshimoto Owned by: shutta
Priority: Milestone: EC-CUBE2.13.3
Component: フロント Version: 2.12.5
Keywords: Cc: undertree
修正済み: yes

Description

レビュー登録のコメント欄のURLチェックにて 「  http://www.yahoo.co.jp 」はエラーとなりますが、 「  http://日本語.net 」のような日本語ドメインだとエラーとなりません。

関連
#2271

Change History

comment:1 Changed 7 years ago by m_uehara

  • Milestone changed from EC-CUBE 2.13.1 to EC-CUBE 2.13.2

comment:2 Changed 6 years ago by h_yoshimoto

  • Milestone changed from EC-CUBE2.13.2 to EC-CUBE2.13.3

comment:3 Changed 6 years ago by shutta

  • Status changed from new to closed
  • Resolution set to 再現不能

現状のソースで試してみましたが、説明とは逆に、
 http://www.yahoo.co.jp 」はエラーとなりませんが、 「  http://日本語.net 」のような日本語ドメインだとエラーになりました。
 http://日本語.net 」の本当のURLである「  http://xn--wgv71a119e.net 」は、エラーになりませんでした。

入力チェックとしては正常に機能しているように思いますが如何でしょうか?

comment:4 Changed 6 years ago by shutta

  • Status changed from closed to reopened
  • Resolution 再現不能 deleted

再現不能にすると、ステータスがclosedになっちゃったので、戻しておきます。

comment:5 follow-up: ↓ 6 Changed 6 years ago by m_uehara

こちら現状のソースで動作させてみましたが、フロントのレビュー入力画面でチケット説明と同じ挙動となります。
また、コメントいただいている内容通りであるならば
URLチェックを行っているのにレビューにURLが入力できてしまうのは問題ではないでしょうか。

comment:6 in reply to: ↑ 5 Changed 6 years ago by shutta

m_uehara への返信

こちら現状のソースで動作させてみましたが、フロントのレビュー入力画面でチケット説明と同じ挙動となります。
また、コメントいただいている内容通りであるならば
URLチェックを行っているのにレビューにURLが入力できてしまうのは問題ではないでしょうか。

「コメント」欄での挙動なんですね。
流し読みしていたので「投稿者URL」での挙動と勘違いしておりました。
再度確認しますと、確かに説明と同じ挙動となりました。
すみませんでした。

comment:7 Changed 6 years ago by shutta

  • Owner changed from somebody to shutta
  • 修正済み set
  • Status changed from reopened to new

#2367 と #2271 どちらも成立するように、 #2271 の不具合を再修正。(r23514)

comment:8 Changed 6 years ago by shutta

  • Status changed from new to assigned

comment:9 Changed 6 years ago by shinichi_takahashi

  • Status changed from assigned to closed
  • Resolution set to 修正済

shutta様

ご対応ありがとうございます。 動作確認できました。クローズさせていただきます。

Note: See TracTickets for help on using tickets.