ここの情報は古いです。ご理解頂いた上でお取り扱いください。

Opened 11 years ago

Closed 11 years ago

Last modified 10 years ago

#3833 closed enhancement (fixed)

メッセージお知らせメールでメッセージ本文に対してリンクするようにする

Reported by: ShogoKawahara Owned by: fukamachi
Priority: minor Milestone: OpenPNE2.13.8
Component: 指定しない Version:
Keywords: Cc:

Description (last modified by shingo)

■概要

メッセージお知らせメールでメッセージ本文に対してリンクするようにする

■仕様

#2583で、PC版メッセージお知らせメールでも件名・本文を確認できるようにしたが、字数を省略するならメッセージへのリンクを張るほうが適切だと言える。

このチケットでは、お知らせメールからメッセージの件名・本文を削除し、代わりにメッセージへのリンクを追加する。

■関連情報

Change History (12)

comment:1 Changed 11 years ago by fukamachi

Description: modified (diff)
Owner: changed from nobody to fukamachi
Status: newassigned

やります

comment:2 Changed 11 years ago by fukamachi

Keywords: 確認中 added

r11378, r11379で対応しました。

それに伴い、webapp/lib/util/mail_send.php の do_common_send_message_mail_send() が取る引数を変更しました。

見る限りではこの関数を使用していたコードは、このチケットの修正箇所のみでしたので、特に問題はないかと思います。

comment:3 Changed 11 years ago by imamura623

Keywords: 差し戻し added; 確認中 removed

動作確認しました。 ログイン状態では問題なく動作しましたが、以下の確認お願いします。

  • 未ログインの状態ではメッセージへのリンクをクリックするとログインを要求され、ログイン後マイホームにリダイレクトされるためメッセージにたどり着けない。

よろしく御願い致します。

comment:4 Changed 11 years ago by fukamachi

変更内容は、メールにメッセージURLをべたうちしただけなので、直接ブラウザにURLを入力してテストしました。

結果、ポータルクリエイターを使ったSNSの場合のみログイン後にマイホームにリダイレクトされるようです。使用していないSNSでは症状が再現されませんでした。

原因は、未ログイン時にポータルクリエイター使用SNSの指定ページを開こうとした際、forwardではなくログイン画面にredirectさせているため、遷移URLが保持されないためと思われます。

これはポータルクリエイターを使っている場合、メッセージに限らず発生する可能性があります。

comment:5 in reply to:  3 ; Changed 11 years ago by kiwa

Replying to imamura623:

  • 未ログインの状態ではメッセージへのリンクをクリックするとログインを要求され、ログイン後マイホームにリダイレクトされるためメッセージにたどり着けない。

恐らくSNSポータルクリエイターが原因と思われますので、このチケットには関係ありません。


このチケットでは、お知らせメールからメッセージの件名・本文を削除し、代わりにメッセージへのリンクを追加する。

これでは元の機能追加を削除してしまうことになります。

メッセージお知らせメールに一部件名・本文を記載するのは「ログインしなくてもメッセージの内容を件名や本文の一部だけでも確認したい」という要望が元となります。この要望に応えられる形で対応していただきたいです。

comment:6 in reply to:  5 Changed 11 years ago by fukamachi

Keywords: 確認中 added; 差し戻し removed

Replying to kiwa:

r11495で対応しました。

comment:7 Changed 11 years ago by nagasawa

Keywords: 差し戻し added; 確認中 removed

PC版では$subject、$bodyを渡していないのでかならず空になります。

webapp/lib/util/mail_send.php
 529     $params = array(<
 530         "c_member_to"   => db_member_c_member4c_member_id($c_member_id_to),<
 531         "c_member_from" => db_member_c_member4c_member_id($c_member_id_from),<
 532         "c_message_id" => $c_message_id,<
 533     );

comment:8 in reply to:  7 Changed 11 years ago by fukamachi

Keywords: 確認中 added; 差し戻し removed

Replying to nagasawa:

r11510で常に空のメッセージが表示されてしまう問題を修正しました。
ご指摘の通り、メール送信用の関数で、subjectとbodyを受け渡していなかったことが原因でした。

comment:9 Changed 11 years ago by nagasawa

Keywords: テスト待ち added; 確認中 removed

チェックしました。

comment:10 Changed 11 years ago by imamura623

Keywords: 差し戻し added; テスト待ち removed

動作確認しました。 通常はあり得ないと思いますが以下ご確認ください

  • メッセージ送信者がログイン状態で、受信者へのお知らせメールに記載されているメッセージリンクをクリックすると、送信者の送信済みのメッセージに遷移し、送信済みのメッセージに返信ボタンが表示されてしまう。(その返信ボタンを押すとエラーメッセージが表示される)

その他は問題無さそうです。

comment:11 Changed 11 years ago by imamura623

Keywords: 差し戻し removed
Resolution: fixed
Status: assignedclosed

メッセージ送信者がログイン状態で、受信者へのお知らせメールに記載されているメッセージリンクをクリックすると、送信者の送信済みのメッセージに遷移し、送信済みのメッセージに返信ボタンが表示されてしまう。(その返信ボタンを押すとエラーメッセージが表示される)

上記は致命的なバグでは無いと判断し、別チケットで対応することにします。 http://trac.openpne.jp/ticket/3892

また、本チケットは修正完了とし閉じます。

comment:12 Changed 10 years ago by shingo

Description: modified (diff)
Note: See TracTickets for help on using tickets.