Bug(バグ) #2512
管理画面からデイリーニュースを無効に設定すると、メンバー側で「メール設定」のページが開けなくなる
100%
Description
概要¶
管理画面のメール通知送信設定では、他にもあるが少なくとも
- (1) フレンド承認完了メール
- (2) フレンドの誕生日お知らせメール
- (3) デイリー・ニュース
の有効無効を設定することができる。
これらが有効なときはメンバー側での「メール設定」ページで、メンバーごとの設定ができるようになっているが、管理画面で (3) を無効にしてしまうと、メンバー側で「メール設定」ページが開けなくなってしまう。
原因¶
これはもともと「メール設定」ページに「デイリーニュース」しかなかった(それ以外を想定していなかった)時期に、管理画面でデイリーニュースを無効にしたのにメンバー側で設定ページが開けてしまうのは不適切だろうと #990 で対応したことに因るものである。
修正方針¶
このチケットでは「デイリーニュース」を無効にしてもメンバー側で「メール設定」を開けるようにすることを扱うが、本来 #990 で行うべき修正が打ち消されてしまう。
可能であればこれを併せてこのチケットで対応したかったが、修正方針が見えていないため、本来 #990 で行おうとしていた問題への対応については #2668 で行うことにした。
報告元¶
http://sns.openpne.jp/diary/26975 より転記
携帯メールアドレス向け」の「デイリー・ニュース」を管理画面より「通知しない」に設定すると、 携帯ホームから「メール設定」のリンクが消えてしまう為、 他の「フレンドの誕生日お知らせメール」や「フレンドリンク承認完了メール」の設定がユーザにより出来なくなってしまう。
Related issues
Associated revisions
History
#1 Updated by Mutsumi Imamura almost 13 years ago
#2 Updated by Kousuke Ebihara almost 13 years ago
- Priority changed from Normal(通常) to Urgent(急いで)
#3 Updated by Minoru Takai almost 13 years ago
- Status changed from New(新規) to Accepted(着手)
- Assignee set to Minoru Takai
- Target version set to OpenPNE 3.7.0
このチケットに対象バージョンを指定します。
そして着手します。
#4 Updated by Minoru Takai almost 13 years ago
- Status changed from Accepted(着手) to Pending Review(レビュー待ち)
- % Done changed from 0 to 50
更新履歴 4109ae83a76d278df0cf1562db39bd701a7fc6d7 で適用されました。
#5 Updated by Minoru Takai almost 13 years ago
- Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
バックポートチケットで差し戻されていましたが、元チケットで対応を扱います。
http://redmine.openpne.jp/issues/2606#note-3
PCメールアドレス向けメールのデイリー・ニュースを「通知しない」設定にした場合、SNS側のメール設定に表示されないか確認¶
- 試験手順
1. 管理画面メール通知送信設定ページ(/pc_backend.php/mail)にアクセスする
2. PCメールアドレス向けのディリー・ニュースを「通知しない」設定にする
3. SNSログイン後、メール設定ページ(/member/config?category=mail)にアクセスする
- 試験結果
メール設定にディリー・ニュース設定するラジオボタンが表示される
- 修正方針
メール設定にディリー・ニュース設定するラジオボタンが表示されないように修正お願いします。
フレンドリンク承認完了メール、フレンドの誕生日お知らせメールは同じ動作をした場合、メール設定にラジオボタンが表示されなかったので、挙動を合わせた方がいいと思います。
- 備考
携帯版でも同じ現象が確認できましたので、PC版と併せて修正お願いします。
#6 Updated by Minoru Takai almost 13 years ago
- File mail_config_frontend.png View added
- File mail_config_backend.png View added
- Status changed from Rejected(差し戻し) to Accepted(着手)
- % Done changed from 50 to 0
調査¶
- メンバー側の設定画面
- 管理画面での設定画面
メンバー側では
- (1) フレンド承認完了メール
- (2) フレンドの誕生日お知らせメール
- (3) デイリー・ニュース
という設定項目があり、これに対応するように管理画面側でも、その項目の有効無効を設定できるようになっている。
現状では、管理画面側で (1), (2) を無効にするとメンバー側で (1), (2) の項目が非表示になるが、 (3) を無効にしても、メンバー側に (3) が表示されてしまっている、という問題で、これが note-5 の差し戻し内容である。
(3) を (1), (2) に合わせるには¶
(1), (2) は、メール配信の機能で lib/config/config/mail_template.yml に
130- friendLinkComplete: 131: caption: "Notification of Completion of Friend Link" 132- configurable: true 133- member_configurable: true 134- sample: 135- en: 136- - "{{ member.name }} accepted your {{ op_term.friend }} link request" 213- birthday: 214: caption: "Notification of Friend Birthday" 215- configurable: true 216- member_configurable: true 217- sample: 218- en: 219- - "There is {{ member.name }}'s {{ op_term.my_friend }} that its birthday is coming soon"
と記述されていて、ここでの configurable, member_configurable の設定で、メンバー側に設定変更のフォームが表示されるか否かを指定できている。これがあるため、管理画面側で無効にするとメンバー側では非表示になるようになっている。
デイリーニュースは member_configurable が記述されておらず次のようになっている。
248: dailyNews: 249- caption: "Daily News" 250- configurable: true 251- sample: 252- en: 253- - "Daily News"
しかし member_configurable を追加したところで note-5 を解消できなかった。というより、そもそもこれが記述されていなければ、 false を指定したのと同様に非表示になるはずである。
つまり、 (3) のデイリーニュースのメンバー側設定のフォームは、別のどこかで明示的に出力されているように思われるが、それが具体的にどこなのか追えていない。
修正方針¶
note-5 の指摘は尤もだが、これを仕様と考えることもできる。デイリーニュース設定に関しては、管理画面で無効にしてもメンバー側に表示されるということになる。
このチケットの問題は note-4 の修正で解消されていて、 note-5 は補足的な指摘に過ぎないが、これを対応したほうがよいか判断できていない(というか対応したくてもどこを直せばいいか追えていない)。
#7 Updated by Minoru Takai almost 13 years ago
- Subject changed from openpne3.6 「携帯メールアドレス向け」の「デイリー・ニュース」を管理画面より「通知しない」に設定すると、携帯ホームから「メール設定」のリンクが消えてしまう to 管理画面からデイリーニュースを無効に設定すると、メンバー側で「メール設定」のページが開けなくなる
- Description updated (diff)
- Status changed from Accepted(着手) to Pending Review(レビュー待ち)
- % Done changed from 0 to 50
note-5, note-6 の話は #2668 のチケットで扱います。
このチケットでは note-4 までの修正を評価してください。
#8 Updated by Kousuke Ebihara almost 13 years ago
- Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
- % Done changed from 50 to 70
note-5, note-6 の指摘事項は #2668 ということで、特に修正としては問題ないです。
#9 Updated by Yuma Sakata over 12 years ago
- Status changed from Pending Testing(テスト待ち) to Fixed(完了)
- % Done changed from 70 to 100
- 3.6 で発生するか set to Unknown (未調査)
- 3.4 で発生するか set to Unknown (未調査)
テストOKです。
#10 Updated by kaoru n about 9 years ago
- 3.8 で発生するか set to Unknown (未調査)