Project

General

Profile

Bug(バグ) #204

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

Added by Kousuke Ebihara almost 13 years ago. Updated over 2 years ago.

Status:
New(新規)
Priority:
Normal(通常)
Assignee:
-
Target version:
Start date:
2009-07-24
Due date:
% Done:

0%

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

Description

以前のチケットは 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)のコミュニティ検索では、ページャが検索ワードを引き継いでないようです・・・・

}}}

History

#1 Updated by Kousuke Ebihara about 12 years ago

  • 3.6 で発生するか set to Yes

#2 Updated by Rimpei Ogawa about 12 years ago

  • Priority changed from Low(低め) to Normal(通常)

#3 Updated by kaoru n over 2 years ago

  • Target version set to OpenPNE 3.10.x

Also available in: Atom PDF