Project

General

Profile

Bug(バグ) #2512

管理画面からデイリーニュースを無効に設定すると、メンバー側で「メール設定」のページが開けなくなる

Added by pnetan   almost 8 years ago. Updated almost 4 years ago.

Status:
Fixed(完了)
Priority:
Urgent(急いで)
Assignee:
Target version:
Start date:
2011-10-18
Due date:
% Done:

100%

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

Description

概要

管理画面のメール通知送信設定では、他にもあるが少なくとも

  • (1) フレンド承認完了メール
  • (2) フレンドの誕生日お知らせメール
  • (3) デイリー・ニュース

の有効無効を設定することができる。

これらが有効なときはメンバー側での「メール設定」ページで、メンバーごとの設定ができるようになっているが、管理画面で (3) を無効にしてしまうと、メンバー側で「メール設定」ページが開けなくなってしまう。

原因

これはもともと「メール設定」ページに「デイリーニュース」しかなかった(それ以外を想定していなかった)時期に、管理画面でデイリーニュースを無効にしたのにメンバー側で設定ページが開けてしまうのは不適切だろうと #990 で対応したことに因るものである。

修正方針

このチケットでは「デイリーニュース」を無効にしてもメンバー側で「メール設定」を開けるようにすることを扱うが、本来 #990 で行うべき修正が打ち消されてしまう。

可能であればこれを併せてこのチケットで対応したかったが、修正方針が見えていないため、本来 #990 で行おうとしていた問題への対応については #2668 で行うことにした。

報告元

http://sns.openpne.jp/diary/26975 より転記

携帯メールアドレス向け」の「デイリー・ニュース」を管理画面より「通知しない」に設定すると、
携帯ホームから「メール設定」のリンクが消えてしまう為、
他の「フレンドの誕生日お知らせメール」や「フレンドリンク承認完了メール」の設定がユーザにより出来なくなってしまう。

mail_config_frontend.png View (70.3 KB) Minoru Takai, 2011-12-06 16:33

mail_config_backend.png View (84.9 KB) Minoru Takai, 2011-12-06 16:33


Related issues

Related to OpenPNE 3 - Backport(バックポート) #2606: 管理画面からデイリーニュースを無効に設定すると、メンバー側で「メール設定」のページが開けなくなる Fixed(完了) 2011-10-18
Related to OpenPNE 3 - Bug(バグ) #2668: #2512 に対応した 3.6 系以上では、管理画面からデイリーニュースを無効に設定しても、メンバー側のメール設定にデイリーニュースの項目が表示されている Won't fix(対応せず) 2011-12-07
Related to OpenPNE 3 - Bug(バグ) #990: 管理画面でデイリー・ニュースを「送信しない」にしてもユーザ側にデイリー・ニュースの受信設定が表示される Fixed(完了) 2010-04-27
Related to OpenPNE 3 - Bug(バグ) #65: [PATCH] エンドユーザ宛にメールが飛ぶ機能でユーザ側でメール受信設定を行えないものがある [仕様バグ] Fixed(完了) 2009-04-27
Related to OpenPNE 3 - Backport(バックポート) #3116: 管理画面からデイリーニュースを無効に設定すると、メンバー側で「メール設定」のページが開けなくなる Fixed(完了) 2011-10-18

Associated revisions

Revision 4109ae83 (diff)
Added by Minoru Takai almost 8 years ago

(fixes #2512) always display "Mail Setting" on frontend. revert #990

History

#1 Updated by Mutsumi Imamura almost 8 years ago

この問題が起こるに至った原因の時系列を調べてみると、 #990 のチケットの対応後に #65 の対応を行ったためと思われます。

#990 の問題を修正した時点では、メール設定画面(member/config?category=mail)にはデイリーニュースに関する設定のみだったため、メール設定ページへのリンクを表示させない修正で特に問題は有りませんでした。
しかし、 #990 の対応を考慮せずに、 #65 にてメール設定画面にフレンドリンク承認完了メールの設定などが追加されたため、本チケットの問題が起きてしまったと思われます。

#2 Updated by Kousuke Ebihara almost 8 years ago

  • Priority changed from Normal(通常) to Urgent(急いで)

#3 Updated by Minoru Takai almost 8 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 8 years ago

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

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

#5 Updated by Minoru Takai almost 8 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 8 years ago

調査

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

メンバー側では

  • (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 8 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 8 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 7 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 almost 4 years ago

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

Also available in: Atom PDF