Backport(バックポート) #919
完了PC・携帯登録時に携帯個体識別番号の登録を必須で登録すると携帯メールアドレスが登録されない
100%
説明
バグ?と思われる物を見つけたので報告
バージョン: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のバグかコアのバグか判断出来なかったのでこちらのチケットを書きました
Kiwa Sakai さんが14年以上前に更新
- ステータス を New(新規) から Pending Fixing(修正待ち) に変更
- 優先度 を Normal(通常) から Urgent(急いで) に変更
OpenPNE3.5.2 にて上の手順による再現が行えました。
Minoru Takai さんが14年以上前に更新
修正内容
- 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' => '携帯登録メール送信完了')); ?>
- このテンプレートを新規に作成
- apps/pc_frontend/modules/member/config/security.yml
registerMobileToRegisterEndFinish: is_secure: false
- security.yml の末に記述
Shinichi Urabe さんが14年以上前に更新
- ステータス を Pending Fixing(修正待ち) から Pending Review(レビュー待ち) に変更
- 担当者 を Minoru Takai にセット
- 対象バージョン を OpenPNE 3.4.4 にセット
- 進捗率 を 0 から 50 に変更
Minoru Takai さんが14年以上前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
#1059 で英語化も含めた修正を行いました。
Revision 6dc8ffbf2dbd80e025157db48c05525cccb9b84b で #919 に対する英語化の修正を行ったので確認をお願いします。
Mutsumi Imamura さんが14年以上前に更新
- ステータス を Rejected(差し戻し) から Pending Testing(テスト待ち) に変更
Shinichi Urabe さんが14年以上前に更新
Imamura Mutsumi は書きました:
Imamura Mutsumi は書きました:
#568 (携帯への新規登録メール内のURLが間違っている)と同様の問題が起こっており、このチケットの動作が確認できません。
#568 の変更が3.4.xに取り込まれていない可能性がありますのでご確認をお願いします。↑の#568 件とこのチケットの件について混乱していました。
このチケットはPC側の設定変更から携帯を登録した場合ですので、#568 の現象とは別件になります。
ただ、#568 と同じ現象は3.4.xでも再現しますのでバックポートチケットを作成し今後対応します。
こちらについては次のバージョンで #1067 で対応しましょう
Shogo Kawahara さんが14年以上前に更新
- 題名 を PC・携帯登録時に携帯個体識別番号の登録を必須で登録 から PC・携帯登録時に携帯個体識別番号の登録を必須で登録すると携帯メールアドレスが登録されない に変更