Opened 13 years ago
Closed 12 years ago
#1163 closed defect (fixed)
メンバーリストが正しく機能しなくなるプロフィール識別子がある
Reported by: | hondaboo | Owned by: | ShogoKawahara |
---|---|---|---|
Priority: | minor | Milestone: | OpenPNE2.12.8 |
Component: | その他 | Version: | 2.12.x & 2.14.x |
Keywords: | OpenPNE2.13.6 | Cc: |
Description (last modified by )
■現象
識別子が「a」のプロフィール項目を作成すると、メンバーリストのソートや絞り込みが使えなくなり、管理画面トップに遷移してしまう。
■原因
プロフィール識別子をそのままリクエストパラメータして利用しているため、「a」はアクションとして解釈されてしまう。つまり同じような理由で以下の識別子のプロフィール項目を利用したとき、ソートや絞り込みで正常な挙動をしない。
- a
- m
- page
- is_pc_address
- is_ktai_address
- c_member_id
- nickname
- access_date
- r_date
- c_member_id_invite
- image_filename
- birth
■修正内容
プロフィールの絞り込みをリクエストするときはprofileという配列を使ってリクエストするようにして、直接、識別子を使うことがないようにした。
また、ソートをするときプロフィール項目のソートの場合、最後に -p というサフィックスを付けることにより、ソートするのはプロフィールかどうか判別できるようにした。
■関連情報
報告元
http://sns.openpne.jp/?m=pc&a=page_fh_diary&target_c_diary_id=12628
Change History (28)
comment:1 Changed 13 years ago by
Priority: | minor → critical |
---|
comment:2 Changed 13 years ago by
Keywords: | 再現済み added; 再現待ち removed |
---|---|
Summary: | 識別子が「a」のプロフィール項目を作成するとメンバーリストが機能しなくなる → メンバーリストが正しく機能しなくなるプロフィール識別子がある |
comment:3 Changed 13 years ago by
Priority: | critical → minor |
---|
comment:4 Changed 13 years ago by
Keywords: | OpenPNE2.8.10 OpenPNE2.11.5 added; 再現済み removed |
---|---|
Milestone: | → OpenPNE2.10.5 |
Version: | 2.6.x & 2.8.x & 2.10.x → 2.8.x & 2.10.x & 2.11.x |
comment:5 Changed 13 years ago by
Keywords: | OpenPNE2.8.10 OpenPNE2.11.5 removed |
---|---|
Milestone: | OpenPNE2.10.5 |
どのプロフィール識別子を登録できないようにするかについて検討をする必要があります。
今回のリリース項目からは外します。
comment:6 Changed 13 years ago by
Version: | 2.8.x & 2.10.x & 2.11.x → 2.10.x & 2.12.x & 2.13.x |
---|
comment:7 Changed 12 years ago by
Version: | 2.10.x & 2.12.x & 2.13.x → 2.10.x & 2.12.x |
---|
comment:8 Changed 12 years ago by
Version: | 2.10.x & 2.12.x → 2.12.x & 2.13.x |
---|
comment:9 Changed 12 years ago by
Keywords: | OpenPNE2.13.6 added |
---|---|
Milestone: | → OpenPNE2.12.8 |
profile[] という配列のキーに識別子が入るようにして修正しましょう。
comment:10 Changed 12 years ago by
Owner: | changed from nobody to ebihara |
---|---|
Status: | new → assigned |
comment:11 Changed 12 years ago by
Keywords: | 確認中 added |
---|
comment:12 Changed 12 years ago by
Keywords: | テスト待ち added; 確認中 removed |
---|
comment:13 Changed 12 years ago by
Keywords: | 差し戻し added; テスト待ち removed |
---|
通常のプロフィールだけでなく、is_ktai_addressといった、プロフィール項目でない絞り込み条件でも同じ配列を利用しているため、「is_ktai_address」というプロフィール項目を作成して絞り込むとうまく動作しないようです。
差し戻します。
comment:14 Changed 12 years ago by
Owner: | changed from ebihara to ShogoKawahara |
---|---|
Status: | assigned → new |
引き受けます。
comment:15 Changed 12 years ago by
Status: | new → assigned |
---|
comment:16 Changed 12 years ago by
Keywords: | 確認中 added; 差し戻し removed |
---|
で修正しました。
プロフィール項目に関しては、profileという配列に分けることにより
- access_date
- is_pc_address
- is_ktai_address
といった、プロフィール識別子であっても絞り込みが可能になりました。
また、プロフィール識別子が
- c_member_id
- nickname
- access_date
- r_date
- c_member_id_invite
- image_filename
- birth
であっても、正常にソートができていない状態であったのを修正しました。
手法としてはプロフィール項目の場合は リクエストパラメータ order の最後に -p というサフィックスを付けるようにしています。
ご確認ください。
comment:17 Changed 12 years ago by
Description: | modified (diff) |
---|
comment:18 Changed 12 years ago by
Description: | modified (diff) |
---|
comment:19 Changed 12 years ago by
Description: | modified (diff) |
---|
comment:20 Changed 12 years ago by
Description: | modified (diff) |
---|
comment:21 Changed 12 years ago by
Keywords: | 差し戻し added; 確認中 removed |
---|
「絞り込んだメンバーにメッセージ/Eメールを送る」がプロフィール項目の絞り込みに対応していません。
comment:22 Changed 12 years ago by
Keywords: | 確認中 added; 差し戻し removed |
---|
comment:23 follow-up: 25 Changed 12 years ago by
動作に影響はなさそうなのですが質問させてください。メンバーリスト(admin_page_list_c_member)にて
- プロフィール「hoge」を選択項目「moge」絞り込み
- プロフィール「hoge」で絞り込まれている状態で「moge」を「▼選択」に戻し、絞り込みを解除する
という動作を行うと、URLに
&profile[hoge]=moge&profile[hoge]=
と、プロフィール「hoge」の絞り込みが2つ入りますが、これは問題ないでしょうか?2つ以上にはなりません。
comment:24 Changed 12 years ago by
現時点の修正は問題ないと思います。
comment:23 について修正するようでしたら、その修正分だけチェックします。とりあえず確認中のままにしておきます。
comment:25 Changed 12 years ago by
comment:26 Changed 12 years ago by
comment:23について
同じパラメタがあったとき後のほうのパラメタが優先されるので挙動的には問題ありませんが、コンボボックスによってリクエストするときに、現在の絞り込み設定値に、あたらしいパラメータを追加してリクエストするために起きているようです。
これを修正するには、パラメータを追加する際に、現在の設定値に同じ項目があったら削除するような処理が必要になりそうです。
あまり重要度も高くない問題であり、小さな修正できかない気がしますので、この修正を見送りたいです。
comment:28 Changed 12 years ago by
Keywords: | テスト待ち removed |
---|---|
Resolution: | → fixed |
Status: | assigned → closed |
再現バージョン
【再現手順】
識別子がaの場合
識別子がmの場合
また、識別子が「page」のプロフィール項目を作成した場合、ページャが正しく動作しなくなりました(「前へ」が使用できない、2ページ目から動かない、など)