SlavePNEで初回認証後のプロフィール登録が完了しない

[ogawa]

■現象

SlavePNE有効時に初回認証後のプロフィール登録が完了せず、プロフィール入力画面が繰り返し表示されてしまう。

■原因

• c_password_query_id が登録できていなかった

■修正内容

• 「■関連情報」の修正パッチを適用
• #1544 の修正を適用

■関連情報

• 関連チケット：#1544
• 修正パッチ↓

  Index: webapp/modules/pc/do/h_regist_prof.php
  ===================================================================
  --- webapp/modules/pc/do/h_regist_prof.php
  +++ webapp/modules/pc/do/h_regist_prof.php
  @@ -85,7 +85,7 @@
           case "register":
               db_member_config_prof_new($u, $prof);
               db_member_update_c_member_profile($u, $c_member_profile_list);
  -            db_member_update_password_query($u, $prof['password_query_id'], $prof['password_query_answer']);
  +            db_member_update_password_query($u, $prof['c_password_query_id'], $prof['c_password_query_answer']);
               //管理画面で指定したコミュニティに強制参加
               $c_commu_id_list = db_commu_regist_join_list();
               foreach ($c_commu_id_list as $c_commu_id) {
  

[ebihara]

このコードが間違っているのは間違いないんですが、auth.incの

    if (($c_member['nickname'] === '')
     || ($c_member['birth_year'] === '')
     || ($c_member['birth_month'] === '')
     || ($c_member['birth_day'] === '')
     || ($c_member['c_password_query_id'] === '')
     || ($c_member['secure']['hashed_password_query_answer'] === '')

という間違った値チェックによって、再現しませんでした（このバグだと$c_memberc_password_query_id?が0に、$c_membersecure?hashed_password_query_answer?がd41d8cd98f00b204e9800998ecf8427e（空文字列のmd5ハッシュ値）になるため、チェックに引っかからない）。空文字列がmd5()に渡されることによって空文字列が返るPHPのバージョンがある、なんてこともなさそうです。

このチェックを正すのと、パッチの適用はやるべきだとは思いますが、プロフィール登録が完了しない件については別の原因かもしれません。引き続き調査してみます。

[ebihara]

おっと。上のコメントの括弧のなかが大変なことに。

このバグだと

$c_member['c_password_query_id']
}}
が0に、
{{{
$c_member['secure']['hashed_password_query_answer']
}}}
がd41d8cd98f00b204e9800998ecf8427e（空文字列のmd5ハッシュ値）になるため、チェックに引っかからない

ですね。

[ebihara]

再度訂正。

$c_member['c_password_query_id']

が0に、

$c_member['secure']['hashed_password_query_answer']

がd41d8cd98f00b204e9800998ecf8427e（空文字列のmd5ハッシュ値）になるため、チェックに引っかからない

[ebihara]

r4632で修正しました。

パッチで修正されている内容も原因のひとつではあるのですが、一番大きな要因はファンクションキャッシュがクリアできていなかったことでした。

ということで、SlavePNEの登録に関わる他のコードも見てみたのですが、

• プロフィール項目入力済かどうかを判定する際は、 db_member_check_param_inputed() を共通して使用している
• db_member_check_param_inputed()以外に、SlavePNE 独自で使用している関数は見あたらない
• 既存の関数でファンクションキャッシュを用いている箇所でも、適切にキャッシュを削除しており、確認をした限りでは問題なく動作している

ということで、今回修正した箇所以外には特に問題は見あたりませんでした。

なお、プロフィール項目登録だけでなく、メールアドレス登録も完了せずにループしてしまうという現象がありましたが、同じ db_member_check_param_inputed() を使用しているため、r4632 で改善されたようです。

ご確認ください。

[ebihara]

あ、必須プロフィール項目が入力済かどうかのチェックについては、#1544で

[ebihara]

trunkにマージを忘れていました。以下のリビジョンでtrunkのコードでも対応しました。

r4635

[ogawa]

メールアドレス登録後のキャッシュクリアと、ktaiモジュールの確認をお願いします。

[ebihara]

確認します

[ebihara]

以下のリビジョンでメールアドレス登録後のファンクションキャッシュクリアに対応しました。

• trunk - r4636:4637
• 2.10.x - r4638

ktaiモジュールについても、PC版と同様に db_member_check_param_inputed() でプロフィール登録が完了しているかどうかチェックしているため、PC版で適切にキャッシュがクリアされてさえいれば問題ないように見えます。

[ebihara]

ktaiモジュールについても、PC版と同様に db_member_check_param_inputed() でプロフィール登録が完了しているかどうかチェックしているため、PC版で適切にキャッシュがクリアされてさえいれば問題ないように見えます。

のコメントは間違いでした。

ただし、プロフィールの登録処理においても、PC版と同一の関数を使用しており、db_member_update_password_query() で適切にキャッシュのクリアをおこなっているため、問題ないと思います。

[ebihara]

以下のリビジョンで、携帯メールアドレス登録後にもファンクションキャッシュをクリアするようにしました。

• trunk - r4641
• 2.10.x - r4640

[ogawa]

2.8系も修正しましょう。

[ebihara]

r4643で2.8系へも修正しました。

ご確認ください。

[kiwa]

動作テストが完了していないのでテスト待ちに変更します