2.0.11JP RC1の検索の全てを表示のときの次のページリンク

• Reporter: toshimitsu
• Owner: (None)
• Type: Bugs
• Status: Closed
• Component: (None)
• MileStone: (None)
• Priority: 5 - Medium
• Severity: 5 - Medium
• Resolution: Fixed
• File: 1

Details Reply

2.0.11JP RC1の検索の

全てを表示のときの

次のページ
前のページ
リンクが

2.0.11RC1で、'&' --> '&amp' に変更したのに
対してhtmlspecialcharsしているため
不正なリンクになっています

Ticket History (3/4 Histories)

2005-06-30 21:30 Updated by: okuhiki

Comment Reply

Logged In: YES
user_id=13441

ごめんなさい。
私が、表示の確認もせずに、該当行の文字列のみを見てフォーラム
に書いたからかも。
196行以外は、不必要でした。

197行も他の行と同じようにhtmlspecialchars通過させると、196も
不要でしょうかね。

2005-07-01 13:54 Updated by: toshimitsu

Comment Reply

Logged In: YES
user_id=6033

okuhikiさんへ、
チェックするためのものがRCなのですから気にする必要はないです
すぐにコードをおえなかったのでかけなかったのですが
&に戻すか、htmlspecialcharsをしない変更なのか？のどちらかで

この場合、query文字列はhtmlspecialchars済みなので、
すでにhtmlspecialcharsを無駄２回かけていることになるようです
&-->&にするのは正解で
下記も直す必要があったのだと思います

line 267
--------------------------------------------------
echo '<a href="'.htmlspecialchars
($search_url_prev).'">'._SR_PREVIOUS.'</a></td>
--------------------------------------------------
to
--------------------------------------------------
echo '<a
href="'.$search_url_prev.'">'._SR_PREVIOUS.'</a></td>
--------------------------------------------------

line 275
--------------------------------------------------
echo '<td align="right"><a href="'.htmlspecialchars
($search_url_next).'">'._SR_NEXT.'</a></td>
--------------------------------------------------
to
--------------------------------------------------
echo '<td align="right"><a
href="'.$search_url_next.'">'._SR_NEXT.'</a></td>
--------------------------------------------------

2005-07-01 13:55 Updated by: toshimitsu

• File 1340: search.php is attached

2005-07-02 23:42 Updated by: onokazu

• Ticket Close date is changed to 2005-07-02 23:42
• Resolution Update from None to Fixed
• Status Update from Open to Closed

Comment Reply

Logged In: YES
user_id=754

CVSにて修正いたしました。