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

Opened 11 years ago

Closed 11 years ago

#2343 closed defect (fixed)

イベントでメッセージを送るとWarning

Reported by: imamura623 Owned by: ebihara
Priority: critical Milestone: OpenPNE2.12beta2
Component: その他 Version: 2.12.x & 2.14.x
Keywords: Cc:

Description

http://sns.openpne.jp/?m=pc&a=page_fh_diary&target_c_diary_id=16679 より転記

イベントでメッセージを送るとエラーが…

Warning: Missing argument 3 for do_common_send_message_mail_send_ktai() in 
/home/*****/webapp/lib/util/mail_send.php on line 719

Warning: Missing argument 4 for do_common_send_message_mail_send_ktai() in 
/home/*****/webapp/lib/util/mail_send.php on line 719

Warning: Cannot modify header information - headers already sent by (output 
started at /home/*****/webapp/lib/util/mail_send.php:719) in 
/home/*****/webapp/lib/util/util.php on line 34

ということで、調べてみると
webapp/lib/db/message.php の
do_common_send_message_mail_send_ktai($c_member_id_to, $c_member_id_from)
となっているのに、
webapp/lib/util/mail_send.phpは
function do_common_send_message_mail_send_ktai($c_member_id_to, $c_member_id_from, $subject, $body)
で受けてる。

それならということでwebapp/lib/db/message.phpの
do_common_send_message_mail_send_ktai($c_member_id_to, $c_member_id_from);
を
do_common_send_message_mail_send_ktai($c_member_id_to, $c_member_id_from,$subject, $body);
と、応急処置を施す。

Attachments (1)

message.patch (4.5 KB) - added by kiwa 11 years ago.
修正パッチをいただきました

Download all attachments as: .zip

Change History (13)

Changed 11 years ago by kiwa

Attachment: message.patch added

修正パッチをいただきました

comment:1 Changed 11 years ago by kiwa

Keywords: 再現待ち removed

あ、修正パッチは報告元のかたからです。

comment:2 Changed 11 years ago by kiwa

Milestone: OpenPNE2.12beta2

comment:3 Changed 11 years ago by ebihara

Priority: minormajor

comment:4 Changed 11 years ago by ebihara

Priority: majorcritical

comment:5 Changed 11 years ago by ebihara

Owner: changed from nobody to ebihara
Status: newassigned

やります。

comment:6 Changed 11 years ago by ebihara

Keywords: 確認中 added

以下のリビジョンで修正しました。ご確認ください。

comment:7 Changed 11 years ago by ogawa

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

webapp/modules/pc/do/f_message_send_insert_c_message.php に引数2つの呼び出しが残っているようです。互換性を考えて第3引数と第4引数にはデフォルト値を付けた方がよいのではないでしょうか。

comment:8 Changed 11 years ago by ebihara

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

以下のリビジョンで、 webapp/modules/pc/do/f_message_send_insert_c_message.php の関数呼び出しの際に第三引数と第四引数を指定するよう変更しました。

以下のリビジョンで、互換性維持のための対策をおこないました。

ご確認ください。

comment:9 Changed 11 years ago by ogawa

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

comment:10 Changed 11 years ago by ogawa

Milestone: OpenPNE2.12beta2

comment:11 Changed 11 years ago by ogawa

Version: 2.11.x2.12.x & 2.13.x

comment:12 Changed 11 years ago by kiwa

Keywords: テスト待ち removed
Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.