メンバーリストが正しく機能しなくなるプロフィール識別子がある

[hondaboo]

■現象

識別子が「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

[kiwa]

再現バージョン

• 2.6.10, 2.8.4, 2,10beta1

---

【再現手順】
識別子がaの場合

1. 識別子がaのプロフィール項目を作成する
2. メンバーリスト（page_list_c_member）を表示
3. 以下のいずれかを実行
   • プロフィール項目の並べ替え
   • 表示件数変更
   • ページャ移動
   • 「絞り込んだメンバーにメッセージ／Eメール送信」を選択
   • プロフィール項目の絞込み
4. 管理画面トップに遷移してしまう

識別子がmの場合

1. 識別子がaのプロフィール項目を作成する
2. メンバーリスト（page_list_c_member）を表示
3. 以下のいずれかを実行
   • プロフィール項目の並べ替え
   • 表示件数変更
   • ページャ移動
   • 「絞り込んだメンバーにメッセージ／Eメール送信」を選択
   • プロフィール項目の絞込み
4. メンテナンス画面に遷移
   • 「アクションファイルが見つかりません」と表示される

---

また、識別子が「page」のプロフィール項目を作成した場合、ページャが正しく動作しなくなりました（「前へ」が使用できない、２ページ目から動かない、など）

[ebihara]

どのプロフィール識別子を登録できないようにするかについて検討をする必要があります。

今回のリリース項目からは外します。

[ebihara]

profile[] という配列のキーに識別子が入るようにして修正しましょう。

[ebihara]

r10614, r10615 で修正しました。 comment:9 には「profile」という配列に値が入るように修正する旨を書きましたが、絞り込み条件に使われるのはプロフィール項目だけではないので、元々利用していた cond というリクエストパラメータに、文字列ではなく配列が入るようにして修正しました。

[ShogoKawahara]

通常のプロフィールだけでなく、is_ktai_addressといった、プロフィール項目でない絞り込み条件でも同じ配列を利用しているため、「is_ktai_address」というプロフィール項目を作成して絞り込むとうまく動作しないようです。

差し戻します。

[ShogoKawahara]

引き受けます。

[ShogoKawahara]

r10781, r10782

で修正しました。

プロフィール項目に関しては、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 というサフィックスを付けるようにしています。

ご確認ください。

[kiwa]

「絞り込んだメンバーにメッセージ／Eメールを送る」がプロフィール項目の絞り込みに対応していません。

[ShogoKawahara]

r10793(2.12), r10794(trunk)

にて、「絞り込んだメンバーにメッセージ/Ｅメールを送る」でプロフィール項目の絞り込みに対応するように修正しました。

ご確認ください。

[kiwa]

動作に影響はなさそうなのですが質問させてください。メンバーリスト(admin_page_list_c_member)にて

1. プロフィール「hoge」を選択項目「moge」絞り込み
2. プロフィール「hoge」で絞り込まれている状態で「moge」を「▼選択」に戻し、絞り込みを解除する

という動作を行うと、URLに

&profile[hoge]=moge&profile[hoge]=

と、プロフィール「hoge」の絞り込みが2つ入りますが、これは問題ないでしょうか？2つ以上にはなりません。

[ebihara]

現時点の修正は問題ないと思います。

comment:23 について修正するようでしたら、その修正分だけチェックします。とりあえず確認中のままにしておきます。

[kiwa]

Replying to kiwa: 2.12.7で既に発生しているので、別チケット作成しました。

• #3650 メンバーリストにて、URLに同じ絞り込み項目が2つ入る

[ShogoKawahara]

comment:23について

同じパラメタがあったとき後のほうのパラメタが優先されるので挙動的には問題ありませんが、コンボボックスによってリクエストするときに、現在の絞り込み設定値に、あたらしいパラメータを追加してリクエストするために起きているようです。

これを修正するには、パラメータを追加する際に、現在の設定値に同じ項目があったら削除するような処理が必要になりそうです。

あまり重要度も高くない問題であり、小さな修正できかない気がしますので、この修正を見送りたいです。

[kiwa]

追加の修正が入らないということなので、テスト待ちにします。

[kiwa]

テスト中に絞り込み・並び替え関係で不具合を発見しましたが、2.12.7の時点で既に発生していたものなので、別チケットにしました。

• #3646: メンバーリストにて、プロフィール項目で並び替えを行うとプロフィール項目が空欄のメンバーが表示されない
• #3647: メンバーリストにて、フォームタイプが「複数選択(チェックボックス)」のプロフィール項目で並べ替えを行うと同じメンバーが複数表示される

今回の修正については問題なかったので、このチケットは閉じます。