プロジェクト

全般

プロフィール

Bug(バグ) #204

未完了

ページャーヘルパーの$query_string不具合

Kousuke Ebihara さんが約15年前に追加. ほぼ5年前に更新.

ステータス:
New(新規)
優先度:
Normal(通常)
担当者:
-
対象バージョン:
開始日:
2009-07-24
期日:
進捗率:

0%

予定工数:
3.6 で発生するか:
Yes
3.8 で発生するか:
Unknown (未調査)

説明

以前のチケットは http://trac.openpne.jp/ticket/4165 です

http://sns.openpne.jp/?m=pc&a=page_fh_diary&target_c_diary_id=21187より転記

{{{
ちょっとバタバタしててプラグイン開発(正確にはDoctrine対応)停滞してましたが、
久しぶりにPNE3ネタです。

ページャ作成ヘルパーのpager_navigation($pager, $link_to, $is_total = true, $query_string = '')
 で、$query_stringを指定して引き継げそうな設計なのですが、何故か以下のようになります。

op_include_pager_navigation($pager, $url.'?page=%d', true, $search_condition);
 *$search_condition = bookmark[title]=Doctrine&bookmark[url]=http
 ↓
OPENPNE_URL/index.php/bookmark/search?page=2?bookmark[title]=Doctrine&bookmark[url]=http

1. $query_stringを、2回エンコードしてる模様です。

2. $link_toの部分にpage=%dを入れないとページャ変換してもらえません。
 名前からして明白にページャ用のヘルパーなのだから、引数で指定しなくても暗黙でpage=%d入れるくらいでもいっそいいのでは?と思うのと、
 引数の入れ方として、ちょっと違和感と・・・・
 なにかの事情で、この形式がマストなのだったら、query_stringの前になぜ?がはいるのでしょう ^^;

ちなみに3.1でop_include_pager_navigation()が追加されてましたが・・・状態はまったく一緒です。
おそらく、ページャー用テンプレートでlink_toへ渡してる辺りに共通の問題があるようですが

  プラグイン側では本線ヘルパー直すわけには行きませんし(T-T)

こちら、3.1.1で直るのでしょうか?

* なお、本線(3.1.0)のコミュニティ検索では、ページャが検索ワードを引き継いでないようです・・・・

}}}

Kousuke Ebihara さんが約14年前に更新

  • 3.6 で発生するかYes にセット

Rimpei Ogawa さんが約14年前に更新

  • 優先度Low(低め) から Normal(通常) に変更

kaoru n さんがほぼ5年前に更新

  • 対象バージョンOpenPNE 3.10.x にセット

他の形式にエクスポート: Atom PDF