Project

General

Profile

Bug（バグ） #65

[PATCH] エンドユーザ宛にメールが飛ぶ機能でユーザ側でメール受信設定を行えないものがある [仕様バグ]

Added by Kousuke Ebihara over 12 years ago. Updated over 6 years ago.

Status:
Fixed（完了）
Priority:
High（高め）
Assignee:
Yuya Watanabe
Target version:
OpenPNE 3.7.0
Start date:
2009-04-27
Due date:
% Done:

100%

3.6 で発生するか:
Yes
3.8 で発生するか:
Unknown (未調査)

Description

概要

エンドユーザ宛にメールが飛ぶ機能でユーザ側でメール受信設定を行えないものがある。

  • コミュニティ参加お知らせメール
  • コミュニティ参加申請メール
  • フレンドリンク承認完了メール
  • フレンドの誕生日お知らせメール

備考

以前のチケットは http://trac.openpne.jp/ticket/3887 です

pc_frontend-member-config_mail.png View - 「フレンドリンク承認完了メール」「フレンドの誕生日お知らせメール」受信設定のためのUIイメージ（PC） (19.3 KB) Kiwa Sakai, 2010-11-10 21:14

mobile_frontend-member-config_mail.png View - 「フレンドリンク承認完了メール」「フレンドの誕生日お知らせメール」受信設定のためのUIイメージ（携帯） (11.1 KB) Kiwa Sakai, 2010-11-10 21:14

pc_frontend-community-edit.png View - 「コミュニティ参加お知らせメール」「コミュニティ参加申請メール」受信設定のためのUIイメージ（PC） (32.2 KB) Kiwa Sakai, 2010-11-10 21:14

mobile_frontend-community-edit.png View - 「コミュニティ参加お知らせメール」「コミュニティ参加申請メール」受信設定のためのUIイメージ（携帯） (58.8 KB) Kiwa Sakai, 2010-11-10 21:14

Related issues

Related to OpenPNE 3 - Enhancement（機能追加・改善） #1509: The issue that notification email doesn't send to both mobile and PC. (通知メールがモバイルとPCの両方に送信されていない問題) New（新規） 2010-08-18
Related to OpenPNE 3 - Backport（バックポート） #2182: [PATCH] エンドユーザ宛にメールが飛ぶ機能でユーザ側でメール受信設定を行えないものがある [仕様バグ] Fixed（完了） 2011-06-10
Related to OpenPNE 3 - Enhancement（機能追加・改善） #2260: コミュニティ参加申請通知メールの実装 New（新規） 2011-07-05
Related to OpenPNE 3 - Bug（バグ） #2694: 3.4 系で、管理画面からフレンドリンク承認完了メールを有効に設定しても、メンバー側のメール設定にフレンドリンク承認完了メールの項目が表示されない Won't fix（対応せず） 2011-12-15
Related to OpenPNE 3 - Bug（バグ） #2512: 管理画面からデイリーニュースを無効に設定すると、メンバー側で「メール設定」のページが開けなくなる Fixed（完了） 2011-10-18

Associated revisions

Revision 68fc7758 (diff)
Added by Shogo Kawahara over 11 years ago

added Member::sendNotificationMail() to send mail following member's configurationa (refs #65)

Revision 7affd7aa (diff)
Added by Shogo Kawahara over 11 years ago

fixed some conditional expression of the Member::sendNotificationMail() (refs #65)

Revision 177823a8 (diff)
Added by Shogo Kawahara over 11 years ago

moved function of Member::sendNotificationMail() to opMailSend::sendTemplateMailToMember() (refs #65)

Revision ea65d184 (diff)
Added by Yuya Watanabe almost 11 years ago

fixed words for i18n (fixes #65)

Revision 645e3a1e (diff)
Added by Yuya Watanabe almost 11 years ago

Fixed for coding standard (fixes #65)

Revision b908613d (diff)
Added by Yuya Watanabe almost 11 years ago

fixed to initialize i18n object (fixes #65)

History

#1 Updated by Kiwa Sakai over 11 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 over 11 years ago

受信設定のUIイメージを添付します

#3 Updated by Kousuke Ebihara over 11 years ago

  • Target version set to OpenPNE 3.7.0

#4 Updated by Kousuke Ebihara over 11 years ago

  • Priority changed from Normal（通常） to High（高め）

#5 Updated by Shogo Kawahara over 11 years ago

  • Assignee set to Shogo Kawahara

#6 Updated by Shogo Kawahara over 11 years ago

  • Status changed from New（新規） to Accepted（着手）

#7 Updated by Shogo Kawahara almost 11 years ago

  • Assignee deleted (Shogo Kawahara)

https://github.com/kawahara/OpenPNE3/tree/t65

で、修正案を作成していました。現状、テストして取り込む時間が取れないので
誰かに依頼します。

#8 Updated by Shogo Kawahara almost 11 years ago

  • Subject changed from エンドユーザ宛にメールが飛ぶ機能でユーザ側でメール受信設定を行えないものがある [仕様バグ] to [PATCH] エンドユーザ宛にメールが飛ぶ機能でユーザ側でメール受信設定を行えないものがある [仕様バグ]

#9 Updated by Shingo Yamada almost 11 years ago

  • Assignee set to Yuya Watanabe

#10 Updated by Yuya Watanabe almost 11 years ago

説明の部分には四つのメール送信の設定が示されていますが、提示された修正案では「コミュニティ参加申請メール」についての項目が実装されていません。

現状の本体のソースを確認してみると、「コミュニティ参加申請メール」自体が実装されていない状態であると確認しました。
提示された修正案の取り込みを行いますが、この「コミュニティ参加申請メール」の機能を実装し、設定項目を追加する必要があると思います。

また、修正案に国際化対応がなされていないため、この部分についても修正を行う必要があると思います。
この点については取り込みの際に私の方で修正を行います。

#11 Updated by wa ta almost 11 years ago

  • Status changed from Accepted（着手） to Pending Review（レビュー待ち）
  • % Done changed from 0 to 50

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

#12 Updated by wa ta almost 11 years ago

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

#13 Updated by Naoya Tozuka almost 11 years ago

  • Status changed from Pending Review（レビュー待ち） to Rejected（差し戻し）
  1. #11 の修正( 645e3a1e )の MemberConfigMailForm.class.php ですが、configure() の中で $i18n が用いられる前に初期化されていません。
  2. 川原さんの修正の取り込み + #11, #12 の修正適用の段階では「コミュニティ参加申請メール」についての項目は未実装という認識でよろしいでしょうか。

以上ご確認よろしくお願いします。

#14 Updated by Yuya Watanabe almost 11 years ago

Naoya Tozuka は書きました:

  1. #11 の修正( 645e3a1e )の MemberConfigMailForm.class.php ですが、configure() の中で $i18n が用いられる前に初期化されていません。
  2. 川原さんの修正の取り込み + #11, #12 の修正適用の段階では「コミュニティ参加申請メール」についての項目は未実装という認識でよろしいでしょうか。

以上ご確認よろしくお願いします。

1については修正を行いました。
2についてもその認識で大丈夫です。
現在の状態では「コミュニティ参加申請メール」設定の実装以前にメール送信部分が実装されていない状態だと思われます。

#15 Updated by Naoya Tozuka almost 11 years ago

1. の修正箇所については問題ないことを確認しました。
2. （「コミュニティ参加申請メール」についての項目の実装）については本チケットで（実装から動作確認まで）取り扱うべき問題と思われます。

#16 Updated by wa ta almost 11 years ago

  • Status changed from Rejected（差し戻し） to Pending Review（レビュー待ち）

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

#17 Updated by Naoya Tozuka almost 11 years ago

  • Status changed from Pending Review（レビュー待ち） to Rejected（差し戻し）
  1. 修正箇所 (b908613d2) については問題ないことを確認しました。
  2. 本チケットにて取り扱われるべきと思われる「コミュニティ参加申請メール」についての項目の実装が未だですので、一旦差し戻します。

#18 Updated by Shingo Yamada almost 11 years ago

  • Status changed from Rejected（差し戻し） to Accepted（着手）

#19 Updated by Shingo Yamada almost 11 years ago

  • Status changed from Accepted（着手） to Pending Review（レビュー待ち）

Naoya Tozuka は書きました:

  1. 本チケットにて取り扱われるべきと思われる「コミュニティ参加申請メール」についての項目の実装が未だですので、一旦差し戻します。

上記の件は #2260 にて進捗します。
本件は「テスト待ち」にしてよいと認識しています。

#21 Updated by Rimpei Ogawa almost 11 years ago

  • Status changed from Pending Review（レビュー待ち） to Pending Testing（テスト待ち）
  • % Done changed from 50 to 70

本チケットでの修正内容については既にレビュー済みのようですのでステータスを変更します。

#22 Updated by Shingo Yamada almost 11 years ago

  • 360対象 set to beta13

#23 Updated by Fumie Toyooka over 10 years ago

  • Status changed from Pending Testing（テスト待ち） to Fixed（完了）
  • % Done changed from 70 to 100

テストOKです。

#24 Updated by kaoru n over 6 years ago

  • 3.8 で発生するか set to Unknown (未調査)

Also available in: Atom PDF