プロジェクト

全般

プロフィール

Bug(バグ) #2512

完了

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

pnetan   さんが約13年前に追加. 約9年前に更新.

ステータス:
Fixed(完了)
優先度:
Urgent(急いで)
担当者:
対象バージョン:
開始日:
2011-10-18
期日:
進捗率:

100%

予定工数:
3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Unknown (未調査)

説明

概要

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

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

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

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

原因

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

修正方針

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

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

報告元

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

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

ファイル

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

関連するチケット 5 (0件未完了5件完了)

関連している OpenPNE 3 - Backport(バックポート) #2606: 管理画面からデイリーニュースを無効に設定すると、メンバー側で「メール設定」のページが開けなくなるFixed(完了)Minoru Takai2011-10-18

操作
関連している OpenPNE 3 - Bug(バグ) #2668: #2512 に対応した 3.6 系以上では、管理画面からデイリーニュースを無効に設定しても、メンバー側のメール設定にデイリーニュースの項目が表示されているWon't fix(対応せず)Yuya Watanabe2011-12-07

操作
関連している OpenPNE 3 - Bug(バグ) #990: 管理画面でデイリー・ニュースを「送信しない」にしてもユーザ側にデイリー・ニュースの受信設定が表示されるFixed(完了)tu nguyen ngoc2010-04-27

操作
関連している OpenPNE 3 - Bug(バグ) #65: [PATCH] エンドユーザ宛にメールが飛ぶ機能でユーザ側でメール受信設定を行えないものがある [仕様バグ]Fixed(完了)Yuya Watanabe2009-04-27

操作
関連している OpenPNE 3 - Backport(バックポート) #3116: 管理画面からデイリーニュースを無効に設定すると、メンバー側で「メール設定」のページが開けなくなるFixed(完了)開 石切山2011-10-18

操作

Mutsumi Imamura さんが約13年前に更新

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

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

Kousuke Ebihara さんが約13年前に更新

  • 優先度Normal(通常) から Urgent(急いで) に変更

Minoru Takai さんが約13年前に更新

  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者Minoru Takai にセット
  • 対象バージョンOpenPNE 3.7.0 にセット

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

そして着手します。

Minoru Takai さんが約13年前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

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

Minoru Takai さんが約13年前に更新

  • ステータスPending Review(レビュー待ち) から 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版と併せて修正お願いします。

Minoru Takai さんが約13年前に更新

調査

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

メンバー側では

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

Minoru Takai さんが約13年前に更新

  • 題名openpne3.6 「携帯メールアドレス向け」の「デイリー・ニュース」を管理画面より「通知しない」に設定すると、携帯ホームから「メール設定」のリンクが消えてしまう から 管理画面からデイリーニュースを無効に設定すると、メンバー側で「メール設定」のページが開けなくなる に変更
  • 説明 を更新 (差分)
  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

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

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

Kousuke Ebihara さんが約13年前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

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

Yuma Sakata さんがほぼ13年前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更
  • 3.6 で発生するかUnknown (未調査) にセット
  • 3.4 で発生するかUnknown (未調査) にセット

テストOKです。

kaoru n さんが約9年前に更新

  • 3.8 で発生するかUnknown (未調査) にセット

他の形式にエクスポート: Atom PDF