OpenPNE 3

このチケットに対象バージョンを指定します。

そして着手します。

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

バックポートチケットで差し戻されていましたが、元チケットで対応を扱います。

http://redmine.openpne.jp/issues/2606#note-3

PCメールアドレス向けメールのデイリー・ニュースを「通知しない」設定にした場合、SNS側のメール設定に表示されないか確認¶

• 試験手順
  1. 管理画面メール通知送信設定ページ(/pc_backend.php/mail)にアクセスする
  2. PCメールアドレス向けのディリー・ニュースを「通知しない」設定にする
  3. SNSログイン後、メール設定ページ(/member/config?category=mail)にアクセスする

• 試験結果
  メール設定にディリー・ニュース設定するラジオボタンが表示される

• 修正方針
  メール設定にディリー・ニュース設定するラジオボタンが表示されないように修正お願いします。
  フレンドリンク承認完了メール、フレンドの誕生日お知らせメールは同じ動作をした場合、メール設定にラジオボタンが表示されなかったので、挙動を合わせた方がいいと思います。

• 備考
  携帯版でも同じ現象が確認できましたので、PC版と併せて修正お願いします。

調査¶

• メンバー側の設定画面
  
• 管理画面での設定画面
  

メンバー側では

• (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 は補足的な指摘に過ぎないが、これを対応したほうがよいか判断できていない（というか対応したくてもどこを直せばいいか追えていない）。

note-5, note-6 の話は #2668 のチケットで扱います。

このチケットでは note-4 までの修正を評価してください。

note-5, note-6 の指摘事項は #2668 ということで、特に修正としては問題ないです。

テストOKです。