OpenPNE 3

いくつか修正案を考えています。

1. 単純にキャッチして何もせず、成功時と同じ画面遷移にする
2. opMemberActionのエラーページに条件分岐でメールエラーを出す
3. メール専用にエラーページを作り、そこにリダイレクトする

とりあえず、現在の方針としては1を考えています。

taskかactionかの判別には、synfony 1.2では
http://blog.asial.co.jp/615
の方法がありますが、1.4では正常に動作しませんでした。
このため、アクションの名前を参照することで判別しています。

(未実装項目)
ログを残す部分

また、最初のopMemberAction固有のコミットはrevertしました。

更新履歴 59aaaa6dfbe97dd893347af76d44a1db00a4a0dd で適用されました。

更新履歴 17e886ec5d4ab06bcbe8942fae86ff4cab474318 で適用されました。

このチケットの状況を確認したのでコメントしておきます。

3.4.x 向けチケットでは http://redmine.openpne.jp/issues/1373#note-6 の通り、「メールサーバに接続できない場合には500エラーとならないようにする」という修正となっています。そして master 向けチケットでは http://redmine.openpne.jp/issues/1372#note-6 の通り、「メールサーバに接続できない場合には、500エラーとならないようにすることに加え、メール送信が失敗したことを示すためのエラーページに遷移させる」という修正となっています。

修正方針は良いと思いますので、メールサーバに接続できない状態でSNSからメールを送信した場合に、動作が不自然でないか、エラーページの表示は不自然でないかを確認できれば、この修正で問題を解消したと判断して良いと思います。

ちなみに簡単にコードを確認してみましたが、 apps/mobile_frontend/i18n/messages.ja.xml の修正 で「下さい>。」と句点の直前に不適切な文字が含まれていることを確認しました。

更新履歴 3586cc9458e4701d8d4bf3a6945a74a85a55bf8b で適用されました。

呼び出されている箇所

urabe@mac OpenPNE3$ ack opMailSend                                                                             11-08-09
apps/pc_backend/modules/member/lib/ReissuePasswordForm.class.php
36:    opMailSend::sendTemplateMail('reissuedPassword', $to, opConfig::get('admin_mail_address'), $params);

→アクション上で呼び出されているので、opExecutionFilterで対処

apps/pc_frontend/modules/member/lib/registerMobileForm.class.php
41:    $mail = new opMailSend();

→アクション上で呼び出されているので、opExecutionFilterで対処

lib/action/opAuthAction.class.php
45:      opMailSend::sendTemplateMailToMember('registerEnd', $member, $params);

→アクション上で呼び出されているので、opExecutionFilterで対処

lib/action/opCommunityAction.class.php
454:      opMailSend::sendTemplateMailToMember('joinCommunity', $community->getAdminMember(), $params, $options);

→アクション上で呼び出されているので、opExecutionFilterで対処

lib/action/opMemberAction.class.php
413:    $mail = new opMailSend();
420:    opMailSend::sendTemplateMailToMember('leave', $member, $param);

→アクション上で呼び出されているので、opExecutionFilterで対処

lib/form/doctrine/InviteForm.class.php
137:    opMailSend::sendTemplateMail('requestRegisterURL', $to, opConfig::get('admin_mail_address'), $params);

→アクション上で呼び出されているので、opExecutionFilterで対処

lib/form/MemberConfigForm/MemberConfigMobileAddressForm.class.php
58:    opMailSend::sendTemplateMail('changeMailAddress', $to, opConfig::get('admin_mail_address'), $params);

→継承している registerMobileFormはアクション上で呼び出されているので、opExecutionFilterで対処

lib/form/MemberConfigForm/MemberConfigPcAddressForm.class.php
58:    opMailSend::sendTemplateMail('changeMailAddress', $to, opConfig::get('admin_mail_address'), $params);

→継承している InviteFormはアクション上で呼び出されているので、opExecutionFilterで対処

lib/model/doctrine/MemberRelationshipTable.class.php
200:      opMailSend::sendTemplateMailToMember('friendLinkComplete', $toMember, $params);

→ MemberRelationshipTable::processFriendConfirm() はイベントで呼び出されるが、アクションないの処理となるので、opExecutionFilterで対処

lib/task/opBaseSendMailTask.class.php
33:    // opMailSend requires Zend

コメント

lib/task/openpneExecutemailactionTask.class.php
58:      opMailSend::execute($subject, $to, $from, $retval);

→ タスクでリターンメールを送信する際に呼び出される、例外をキャッチして別処理を追加する必要はないと考える

lib/task/openpneSendBirthDayMailTask.class.php
58:        opMailSend::sendTemplateMailToMember('birthday', $member, $params, array(), $context);

→ タスクで誕生日通知メールを送信する際に呼び出される、例外をキャッチして別処理を追加する必要はないと考える

lib/task/openpneSendDailyNewsTask.class.php
80:      opMailSend::sendTemplateMail('dailyNews', $address, opConfig::get('admin_mail_address'), $params, $context);

→ タスクでデイリーニュースを送信する際に呼び出される、例外をキャッチして別処理を追加する必要はないと考える

urabe@mac OpenPNE3$ ack sfOpenPNEMailSend                                                                      11-08-09
lib/util/sfOpenPNEMailSend.class.php
23:class sfOpenPNEMailSend extends opMailSend

plugins/opAuthMailAddressPlugin/lib/form/opAuthMailAddressPasswordRecoveryForm.class.php
102:    sfOpenPNEMailSend::sendTemplateMail('passwordRecovery', $this->member->getEMailAddress(), opConfig::get('admin_mail_address'), $params);

→アクション上で呼び出されているので、opExecutionFilterで対処

更新履歴 a568d894ecb91580b2d99d28f11963107a22ead1 で適用されました。

更新履歴 016e30d6bff01b3a4028f614b7a19d82c787755c で適用されました。

更新履歴 4a81cebbafc516e535c11588828e679fe2c02546 で適用されました。

更新履歴 22e2f57f5ebbcc1300ec5f53585cb22343802ba5 で適用されました。

いったん修正を戻し、クラス外から例外がわからないようにする修正は問題なので opMailSend::execute() では例外をキャッチしないように対処

• アクションは opExecutionFilter で例外をキャッチし、メール送信失敗のページにforwardされるように変更
• タスク（デイリーニュース、誕生日通知、メール投稿のリターンメール）については実行時にそのまま例外が表示できることが必要と判断し、例外をキャッチしていません

更新履歴 e099cc79c365f5d6ef96cf2e39e2002eab4d4961 で適用されました。