Backport（バックポート） #919
完了
PC・携帯登録時に携帯個体識別番号の登録を必須で登録すると携帯メールアドレスが登録されない
Naoto Inoue さんが14年以上前に追加.
14年以上前に更新.
説明
バグ？と思われる物を見つけたので報告
バージョン：3.4.2
現象：
1）招待制ではなく、新規登録できる様に設定
2）管理画面のSNS設定、携帯関連設定、メンバー登録時の携帯個体識別番号取得設定をPC・携帯登録時に携帯個体識別番号の登録を必須にするに設定
3）ログイン画面から新規登録。メールを飛ばし、招待状のURLから登録画面を開く
4）フォームに必要事項を入力し登録
5）エラー発生
Catchable fatal error: Argument 1 passed to MemberConfigMobileAddressForm::__construct() must be an instance of Member, boolean given, called in /home/xxx/openpne/apps/pc_frontend/modules/member/actions/actions.class.php on line 153 and defined in /home/xxx/openpne/lib/form/MemberConfigForm/MemberConfigMobileAddressForm.class.php on line 22
Catchable fatal error: Argument 1 passed to MemberConfigForm::__construct() must be an instance of Member, boolean given, called in /home/xxx/openpne/lib/form/MemberConfigForm/MemberConfigMobileAddressForm.class.php on line 24 and defined in /home/xxx/openpne/lib/form/doctrine/MemberConfigForm.class.php on line 26
Fatal error: Call to a member function getIsActive() on a non-object in /home/xxx/openpne/lib/form/doctrine/MemberConfigForm.class.php on line 35
opAuthMailAddressPluginのバグかコアのバグか判断出来なかったのでこちらのチケットを書きました
関連するチケット
2 (0件未完了 — 2件完了)
- ステータス を New（新規） から Pending Fixing（修正待ち） に変更
- 優先度 を Normal（通常） から Urgent（急いで） に変更
OpenPNE3.5.2 にて上の手順による再現が行えました。
修正内容
- apps/pc_frontend/modules/member/actions/actions.class.php
public function executeRegisterMobileToRegisterEnd(sfWebRequest $request)
{
opActivateBehavior::disable();
$this->form = new registerMobileForm($this->getUser()->getMember());
opActivateBehavior::enable();
if ($request->isMethod(sfWebRequest::POST))
{
$this->form->bind($request->getParameter('member_config'));
if ($this->form->isValid())
{
$this->form->save();
$this->redirect('member/registerMobileToRegisterEndFinish');
}
}
return sfView::SUCCESS;
}
public function executeRegisterMobileToRegisterEndFinish(sfWebRequest $request)
{
}
- 153行目($this->form = new ... の行)の前後に opActivateBehavior::disable() と opActivateBehavior::enable() を記述
- redirect先を member/registerMobileToRegisterEndFinish に変更
- executeRegisterMobileToRegisterEndFinish() を追加
- apps/pc_frontend/modules/member/templates/registerMobileToRegisterEndFinishSuccess.php
<?php op_include_box('finish', '携帯登録をおこなうことでメンバー登録が完了します。<br />入力された携帯メールアドレス宛てにメールを送信いたしましたので、メール内に記載されたURLから携帯登録をおこなってください。', array('title' => '携帯登録メール送信完了')); ?>
- ステータス を Pending Fixing（修正待ち） から Pending Review（レビュー待ち） に変更
- 担当者 を Minoru Takai にセット
- 対象バージョン を OpenPNE 3.4.4 にセット
- 進捗率 を 0 から 50 に変更
- トラッカー を Bug（バグ） から Backport（バックポート） に変更
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
- ステータス を Rejected（差し戻し） から Accepted（着手） に変更
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- ステータス を Pending Testing（テスト待ち） から Rejected（差し戻し） に変更
#568 （携帯への新規登録メール内のURLが間違っている)と同様の問題が起こっており、このチケットの動作が確認できません。
#568 の変更が3.4.xに取り込まれていない可能性がありますのでご確認をお願いします。
- ステータス を Rejected（差し戻し） から Pending Testing（テスト待ち） に変更
Imamura Mutsumi は書きました:
#568 （携帯への新規登録メール内のURLが間違っている)と同様の問題が起こっており、このチケットの動作が確認できません。
#568 の変更が3.4.xに取り込まれていない可能性がありますのでご確認をお願いします。
↑の#568 件とこのチケットの件について混乱していました。
このチケットはPC側の設定変更から携帯を登録した場合ですので、#568 の現象とは別件になります。
ただ、#568 と同じ現象は3.4.xでも再現しますのでバックポートチケットを作成し今後対応します。
Imamura Mutsumi は書きました:
Imamura Mutsumi は書きました:
#568 （携帯への新規登録メール内のURLが間違っている)と同様の問題が起こっており、このチケットの動作が確認できません。
#568 の変更が3.4.xに取り込まれていない可能性がありますのでご確認をお願いします。
↑の#568 件とこのチケットの件について混乱していました。
このチケットはPC側の設定変更から携帯を登録した場合ですので、#568 の現象とは別件になります。
ただ、#568 と同じ現象は3.4.xでも再現しますのでバックポートチケットを作成し今後対応します。
こちらについては次のバージョンで #1067 で対応しましょう
- ステータス を Pending Testing（テスト待ち） から Fixed（完了） に変更
- 進捗率 を 50 から 100 に変更
確認しました。PC・携帯ともに登録完了しログインできることは確認できました。
- #1069 PC・携帯登録時に携帯個体識別番号の登録を必須で登録すると携帯メールアドレスが登録されない
- #1071 PC・携帯登録時に携帯個体識別番号の登録を必須で新規登録すると登録中にdevモードだとWarningが発生する
上記2点の問題がありますが、別チケットで対応する方針にします。
- 題名 を PC・携帯登録時に携帯個体識別番号の登録を必須で登録 から PC・携帯登録時に携帯個体識別番号の登録を必須で登録すると携帯メールアドレスが登録されない に変更
他の形式にエクスポート: Atom
PDF
