#3833 closed enhancement (fixed)
メッセージお知らせメールでメッセージ本文に対してリンクするようにする
Reported by: | ShogoKawahara | Owned by: | fukamachi |
---|---|---|---|
Priority: | minor | Milestone: | OpenPNE2.13.8 |
Component: | 指定しない | Version: | |
Keywords: | Cc: |
Description (last modified by )
Change History (12)
comment:1 Changed 11 years ago by
Description: | modified (diff) |
---|---|
Owner: | changed from nobody to fukamachi |
Status: | new → assigned |
comment:2 Changed 11 years ago by
Keywords: | 確認中 added |
---|
r11378, r11379で対応しました。
それに伴い、webapp/lib/util/mail_send.php の do_common_send_message_mail_send() が取る引数を変更しました。
見る限りではこの関数を使用していたコードは、このチケットの修正箇所のみでしたので、特に問題はないかと思います。
comment:3 follow-up: 5 Changed 11 years ago by
Keywords: | 差し戻し added; 確認中 removed |
---|
動作確認しました。 ログイン状態では問題なく動作しましたが、以下の確認お願いします。
- 未ログインの状態ではメッセージへのリンクをクリックするとログインを要求され、ログイン後マイホームにリダイレクトされるためメッセージにたどり着けない。
よろしく御願い致します。
comment:4 Changed 11 years ago by
変更内容は、メールにメッセージURLをべたうちしただけなので、直接ブラウザにURLを入力してテストしました。
結果、ポータルクリエイターを使ったSNSの場合のみログイン後にマイホームにリダイレクトされるようです。使用していないSNSでは症状が再現されませんでした。
原因は、未ログイン時にポータルクリエイター使用SNSの指定ページを開こうとした際、forwardではなくログイン画面にredirectさせているため、遷移URLが保持されないためと思われます。
これはポータルクリエイターを使っている場合、メッセージに限らず発生する可能性があります。
comment:5 follow-up: 6 Changed 11 years ago by
Replying to imamura623:
- 未ログインの状態ではメッセージへのリンクをクリックするとログインを要求され、ログイン後マイホームにリダイレクトされるためメッセージにたどり着けない。
恐らくSNSポータルクリエイターが原因と思われますので、このチケットには関係ありません。
このチケットでは、お知らせメールからメッセージの件名・本文を削除し、代わりにメッセージへのリンクを追加する。
これでは元の機能追加を削除してしまうことになります。
メッセージお知らせメールに一部件名・本文を記載するのは「ログインしなくてもメッセージの内容を件名や本文の一部だけでも確認したい」という要望が元となります。この要望に応えられる形で対応していただきたいです。
comment:6 Changed 11 years ago by
Keywords: | 確認中 added; 差し戻し removed |
---|
Replying to kiwa:
r11495で対応しました。
comment:7 follow-up: 8 Changed 11 years ago by
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 Changed 11 years ago by
Keywords: | 確認中 added; 差し戻し removed |
---|
Replying to nagasawa:
r11510で常に空のメッセージが表示されてしまう問題を修正しました。
ご指摘の通り、メール送信用の関数で、subjectとbodyを受け渡していなかったことが原因でした。
comment:10 Changed 11 years ago by
Keywords: | 差し戻し added; テスト待ち removed |
---|
動作確認しました。 通常はあり得ないと思いますが以下ご確認ください
- メッセージ送信者がログイン状態で、受信者へのお知らせメールに記載されているメッセージリンクをクリックすると、送信者の送信済みのメッセージに遷移し、送信済みのメッセージに返信ボタンが表示されてしまう。(その返信ボタンを押すとエラーメッセージが表示される)
その他は問題無さそうです。
comment:11 Changed 11 years ago by
Keywords: | 差し戻し removed |
---|---|
Resolution: | → fixed |
Status: | assigned → closed |
メッセージ送信者がログイン状態で、受信者へのお知らせメールに記載されているメッセージリンクをクリックすると、送信者の送信済みのメッセージに遷移し、送信済みのメッセージに返信ボタンが表示されてしまう。(その返信ボタンを押すとエラーメッセージが表示される)
上記は致命的なバグでは無いと判断し、別チケットで対応することにします。 http://trac.openpne.jp/ticket/3892
また、本チケットは修正完了とし閉じます。
comment:12 Changed 10 years ago by
Description: | modified (diff) |
---|
やります