Bug(バグ) #65
[PATCH] エンドユーザ宛にメールが飛ぶ機能でユーザ側でメール受信設定を行えないものがある [仕様バグ]
100%
Description
概要¶
エンドユーザ宛にメールが飛ぶ機能でユーザ側でメール受信設定を行えないものがある。
- コミュニティ参加お知らせメール
- コミュニティ参加申請メール
- フレンドリンク承認完了メール
- フレンドの誕生日お知らせメール
備考¶
以前のチケットは http://trac.openpne.jp/ticket/3887 です
Related issues
Associated revisions
added Member::sendNotificationMail() to send mail following member's configurationa (refs #65)
fixed some conditional expression of the Member::sendNotificationMail() (refs #65)
moved function of Member::sendNotificationMail() to opMailSend::sendTemplateMailToMember() (refs #65)
fixed words for i18n (fixes #65)
Fixed for coding standard (fixes #65)
fixed to initialize i18n object (fixes #65)
History
#1
Updated by Kiwa Sakai about 12 years ago
- Tracker changed from Enhancement(機能追加・改善) to Bug(バグ)
- Subject changed from メール受信設定機能の追加 to エンドユーザ宛にメールが飛ぶ機能でユーザ側でメール受信設定を行えないものがある [仕様バグ]
- Priority changed from Low(低め) to Normal(通常)
- 3.6 で発生するか set to Yes
#2
Updated by Kiwa Sakai about 12 years ago
- File pc_frontend-member-config_mail.png View added
- File mobile_frontend-member-config_mail.png View added
- File pc_frontend-community-edit.png View added
- File mobile_frontend-community-edit.png View added
受信設定のUIイメージを添付します
#3
Updated by Kousuke Ebihara about 12 years ago
- Target version set to OpenPNE 3.7.0
#4
Updated by Kousuke Ebihara about 12 years ago
- Priority changed from Normal(通常) to High(高め)
#5
Updated by Shogo Kawahara about 12 years ago
- Assignee set to Shogo Kawahara
#6
Updated by Shogo Kawahara about 12 years ago
- Status changed from New(新規) to Accepted(着手)
#7
Updated by Shogo Kawahara over 11 years ago
- Assignee deleted (
Shogo Kawahara)
https://github.com/kawahara/OpenPNE3/tree/t65
で、修正案を作成していました。現状、テストして取り込む時間が取れないので
誰かに依頼します。
#8
Updated by Shogo Kawahara over 11 years ago
- Subject changed from エンドユーザ宛にメールが飛ぶ機能でユーザ側でメール受信設定を行えないものがある [仕様バグ] to [PATCH] エンドユーザ宛にメールが飛ぶ機能でユーザ側でメール受信設定を行えないものがある [仕様バグ]
#9
Updated by Shingo Yamada over 11 years ago
- Assignee set to Yuya Watanabe
#10
Updated by Yuya Watanabe over 11 years ago
説明の部分には四つのメール送信の設定が示されていますが、提示された修正案では「コミュニティ参加申請メール」についての項目が実装されていません。
現状の本体のソースを確認してみると、「コミュニティ参加申請メール」自体が実装されていない状態であると確認しました。
提示された修正案の取り込みを行いますが、この「コミュニティ参加申請メール」の機能を実装し、設定項目を追加する必要があると思います。
また、修正案に国際化対応がなされていないため、この部分についても修正を行う必要があると思います。
この点については取り込みの際に私の方で修正を行います。
#11
Updated by wa ta over 11 years ago
- Status changed from Accepted(着手) to Pending Review(レビュー待ち)
- % Done changed from 0 to 50
更新履歴 645e3a1ef15b4d49c499b08fd91f9ba9a86a489d で適用されました。
#12
Updated by wa ta over 11 years ago
更新履歴 ea65d18463fe4193f56fe1debf1a76b31502fc7d で適用されました。
#13
Updated by Naoya Tozuka over 11 years ago
- Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
#14
Updated by Yuya Watanabe over 11 years ago
Naoya Tozuka は書きました:
- #11 の修正( 645e3a1e )の MemberConfigMailForm.class.php ですが、configure() の中で $i18n が用いられる前に初期化されていません。
- 川原さんの修正の取り込み + #11, #12 の修正適用の段階では「コミュニティ参加申請メール」についての項目は未実装という認識でよろしいでしょうか。
以上ご確認よろしくお願いします。
1については修正を行いました。
2についてもその認識で大丈夫です。
現在の状態では「コミュニティ参加申請メール」設定の実装以前にメール送信部分が実装されていない状態だと思われます。
#15
Updated by Naoya Tozuka over 11 years ago
1. の修正箇所については問題ないことを確認しました。2. (「コミュニティ参加申請メール」についての項目の実装)については本チケットで(実装から動作確認まで)取り扱うべき問題と思われます。
#16
Updated by wa ta over 11 years ago
- Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)
更新履歴 b908613d2d3190f1149353537bdaf42ae933b427 で適用されました。
#17
Updated by Naoya Tozuka over 11 years ago
- Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
- 修正箇所 (b908613d2) については問題ないことを確認しました。
- 本チケットにて取り扱われるべきと思われる「コミュニティ参加申請メール」についての項目の実装が未だですので、一旦差し戻します。
#18
Updated by Shingo Yamada over 11 years ago
- Status changed from Rejected(差し戻し) to Accepted(着手)
#19
Updated by Shingo Yamada over 11 years ago
- Status changed from Accepted(着手) to Pending Review(レビュー待ち)
Naoya Tozuka は書きました:
- 本チケットにて取り扱われるべきと思われる「コミュニティ参加申請メール」についての項目の実装が未だですので、一旦差し戻します。
上記の件は #2260 にて進捗します。
本件は「テスト待ち」にしてよいと認識しています。
#20
Updated by Maki Takahashi over 11 years ago
#21
Updated by Rimpei Ogawa over 11 years ago
- Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
- % Done changed from 50 to 70
本チケットでの修正内容については既にレビュー済みのようですのでステータスを変更します。
#22
Updated by Shingo Yamada over 11 years ago
- 360対象 set to beta13
#23
Updated by Fumie Toyooka over 11 years ago
- Status changed from Pending Testing(テスト待ち) to Fixed(完了)
- % Done changed from 70 to 100
テストOKです。
#24
Updated by kaoru n over 7 years ago
- 3.8 で発生するか set to Unknown (未調査)