OpenPNE 3

原因¶

新規登録時に plugins/opAuthMailAddressPlugin/apps/mobile_mail_frontend/modules/opAuthMailAddress/actions/actions.class.php から InviteForm が呼ばれるが InviteForm::save() を実施された際に 143行目に以下のような処理が存在する。しかしながら #2044 で myUser クラスの親クラスを sfOpenPNESecurityUser　から sfBasicSecurityUser に変更されたことにより、getMemberId() は定義されていないため、この処理でエラーとなってしまう

• OpenPNE 3.6 用の opAuthMailAddressPlugin では InviteForm クラスは呼ばれず、構造自体異なるため、3.4.15-dev で発生するような問題はない
• OpenPNE 3.2 については myUser クラスの親クラスは fOpenPNESecurityUser であるので問題はでない

$this->member->setInviteMemberId($user->getMemberId());

修正方針¶

• InviteForm::save() 処理内、$user オブジェクトが sfOpenPNESecurityUser であれば、上記処理を実施するように分岐を追記する
  • 管理画面の AdminInviteForm クラスは InviteForm クラスを親クラスとして扱っているが、save() メソッドはオーバーライドされているので、Member::setInviteMemberId() が呼ばれることがないので上記修正で問題はでないと考えられる
  • Member::setInviteMemberId() に紹介者のIDをセットする場合は、いずれの場合も、sfUser::getMemberId() が呼べる処理の流れになっているため、上記修正で問題ないと考える

テスト範囲¶

1. 管理画面からの招待
2. PC,携帯からの招待（招待者としてひも付いているかも管理画面のメンバー一覧から確認）
3. 携帯からのメール投稿での新規登録
4. PCの新規登録

InviteForm が呼ばれる箇所の確認¶

$ ack InviteForm     

apps/pc_backend/modules/member/actions/actions.class.php
101:    $this->form = new AdminInviteForm(null, $options);
// 管理画面からの招待 AdminInviteForm::save() は親クラスの save() メソッドを呼んでいないので myUser::getMemberId() が呼ばれることがないが、親クラスの修正になるため念のために動作確認必要

apps/pc_backend/modules/member/lib/AdminInviteForm.class.php
18:class AdminInviteForm extends InviteForm
// AdminInviteForm の継承先が InviteForm

lib/action/sfOpenPNEMemberAction.class.php
348:    $this->form = new InviteForm(null, array('invited' => true));
// 動作確認必要(PC、携帯からのフレンド招待)

lib/form/doctrine/InviteForm.class.php
18:class InviteForm extends MemberConfigPcAddressForm
// クラス定義

plugins/opAuthMailAddressPlugin/apps/mobile_mail_frontend/modules/opAuthMailAddress/actions/actions.class.php
40:    $this->form = new InviteForm(null, array('authMode' => 'MailAddress'));
// 携帯から新規メール投稿（今回の問題の箇所）

plugins/opAuthMailAddressPlugin/apps/pc_frontend/modules/opAuthMailAddress/actions/actions.class.php
30:    $this->form = new InviteForm(null, array('authMode' => 'MailAddress'));
// PCから新規登録

Imamura Mutsumi は書きました:

携帯から新規登録を実施すると、登録用URLお知らせメールともに修正前と同様な「サーバーが混み合っています」というエラーメールが返ってきます。

その他の↓の動作は問題ありません。

1. 管理画面からの招待
2. PC,携帯からの招待（招待者としてひも付いているかも管理画面のメンバー一覧から確認）
4. PCの新規登録

userクラスから getCurrentAuthMode() が呼べないのに呼ばれている箇所があるため、
InviteForm::save() 内の parent::save() でメール送信されるが、そのあとの $this->member->setConfig('register_auth_mode', $this->getOption('authMode', $user->getCurrentAuthMode())); でエラーが発生している。option の authMode は plugins/opAuthMailAddressPlugin/apps/mobile_mail_frontend/modules/opAuthMailAddress/actions/actions.class.php の InviteForm を生成した際に 第二引数に設定されているので、空になることはないため、user オブジェクトが sfOpenPNESecurityUser でない場合、null をデフォルト値にセットするように修正した

新規登録の際に呼ばれることはないが、フレンドリンクを設定する if ($this->getOption('is_link')) 以下の処理でも $user オブジェクトが sfOpenPNESecurityUser でない場合は getMemberId() が呼べないので、念のために if ($this->getOption('is_link') && $isSfOpenPNESecurityUser) と修正した

エラーメールが来なくなることを確認しました。
再度、日記のメール投稿とメンバーから招待されて登録した場合の動作も確認し問題ないことを確認しました。
動作確認OKです。