Ticket #6251

2.0.11JP RC1の検索の全てを表示のときの次のページリンク
Eröffnet am: 2005-06-30 12:42 Letztes Update: 2005-07-02 23:42

Auswertung:
Verantwortlicher:
(Keine)
Typ:
Status:
Geschlossen
Komponente:
(Keine)
Meilenstein:
(Keine)
Priorität:
5 - Mittel
Schweregrad:
5 - Mittel
Lösung:
Gefixt
Datei:
1

Details

2.0.11JP RC1の検索の

全てを表示のときの

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

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

Ticket-Verlauf (3/4 Historien)

2005-06-30 21:30 Aktualisiert von: okuhiki
Kommentar
Logged In: YES
user_id=13441

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

197行も他の行と同じようにhtmlspecialchars通過させると、196も
不要でしょうかね。
2005-07-01 13:54 Aktualisiert von: toshimitsu
Kommentar
Logged In: YES
user_id=6033

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

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

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 Aktualisiert von: toshimitsu
  • File 1340: search.php is attached
2005-07-02 23:42 Aktualisiert von: onokazu
  • Ticket Close date is changed to 2005-07-02 23:42
  • Lösung Update from Keine to Gefixt
  • Status Update from Offen to Geschlossen
Kommentar
Logged In: YES
user_id=754

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

Dateianhangliste

Bearbeiten

You are not logged in. I you are not logged in, your comment will be treated as an anonymous post. » Anmelden