副管理者要請メッセージを送るところでコミュニティ管理者交代要請メッセージを送っている

[kiwa]

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

OpenPNE2.10.5最新版で確認
コミュニティ管理者がメンバー管理画面で「副管理者に指名」する処理の中で、対象メンバーに対し
「副管理者要請メッセージ」が送られるべき所が「コミュニティ管理者交代要請メッセージ」を送るようになっていた
/webapp/modules/pc/do/c_sub_admin_request_insert_c_commu_sub_admin_confirm.php
75行目
//list($msg_subject, $msg_body) = create_message_commu_admin_request($u, $body, $target_c_member_id, $target_c_commu_id);
list($msg_subject, $msg_body) = create_message_commu_sub_admin_request($u, $body, $target_c_member_id, $target_c_commu_id);
使うメッセージ送付関数の取り違え。

[nakasone]

2.10.xで再現確認しました。 2.8.x,trunkでは再現せず。

• 副管理者要請メッセージ送信
• 送信済みメッセージ確認
• 受信メッセージ確認

[ebihara]

やります。

[ebihara]

2.10.x のみで発生した現象ということで、どのようにしてバグとなるコードが混入したのか調査してみました。

このバグは、 OpenPNE2.10alpha2 においてアクション内に記述していたメッセージ本文の一部をwebapp/lib/util/message.php に移動したことにより混入しました（r3965）。

しかし、OpenPNE2.11.1 において、これと同様の修正をメッセージ全体にわたって適用した際にこのバグは取り除かれています（r4555）。

こういった経緯から、2.10系のみにバグが残るという状態に陥っていました。

修正方法としてはバグの原因となっている箇所のみ2.11系にあわせる、という形を取ります（提案していただいたコードと相違しないとは思いますが）。

[ebihara]

r6936 で修正しました。ご確認ください。

[kiwa]

確認しました。問題なさそうです。