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

Opened 11 years ago

Closed 11 years ago

#2171 closed defect (fixed)

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

Reported by: kiwa Owned by: ebihara
Priority: minor Milestone: OpenPNE2.10.6
Component: その他 Version: 2.10.x
Keywords: Cc:

Description

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);
使うメッセージ送付関数の取り違え。

Change History (9)

comment:1 Changed 11 years ago by kiwa

Priority: minormajor

comment:2 Changed 11 years ago by nakasone

Keywords: 再現待ち removed

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

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

comment:3 Changed 11 years ago by ebihara

Milestone: OpenPNE2.10.6
Version: 2.8.x & 2.10.x & 2.11.x2.10.x

comment:4 Changed 11 years ago by ebihara

Priority: majorminor

comment:5 Changed 11 years ago by ebihara

Owner: changed from nobody to ebihara
Status: newassigned

やります。

comment:6 Changed 11 years ago by ebihara

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

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

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

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

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

comment:7 Changed 11 years ago by ebihara

Keywords: 確認中 added

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

comment:8 Changed 11 years ago by ogawa

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

comment:9 Changed 11 years ago by kiwa

Keywords: テスト待ち removed
Resolution: fixed
Status: assignedclosed

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

Note: See TracTickets for help on using tickets.