Opened 13 years ago
Closed 13 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 13 years ago by
Priority: | minor → major |
---|
comment:2 Changed 13 years ago by
Keywords: | 再現待ち removed |
---|
comment:3 Changed 13 years ago by
Milestone: | → OpenPNE2.10.6 |
---|---|
Version: | 2.8.x & 2.10.x & 2.11.x → 2.10.x |
comment:4 Changed 13 years ago by
Priority: | major → minor |
---|
comment:5 Changed 13 years ago by
Owner: | changed from nobody to ebihara |
---|---|
Status: | new → assigned |
やります。
comment:6 Changed 13 years ago by
2.10.x のみで発生した現象ということで、どのようにしてバグとなるコードが混入したのか調査してみました。
このバグは、 OpenPNE2.10alpha2 においてアクション内に記述していたメッセージ本文の一部をwebapp/lib/util/message.php に移動したことにより混入しました(r3965)。
しかし、OpenPNE2.11.1 において、これと同様の修正をメッセージ全体にわたって適用した際にこのバグは取り除かれています(r4555)。
こういった経緯から、2.10系のみにバグが残るという状態に陥っていました。
修正方法としてはバグの原因となっている箇所のみ2.11系にあわせる、という形を取ります(提案していただいたコードと相違しないとは思いますが)。
comment:8 Changed 13 years ago by
Keywords: | テスト待ち added; 確認中 removed |
---|
comment:9 Changed 13 years ago by
Keywords: | テスト待ち removed |
---|---|
Resolution: | → fixed |
Status: | assigned → closed |
確認しました。問題なさそうです。
Note: See
TracTickets for help on using
tickets.
2.10.xで再現確認しました。 2.8.x,trunkでは再現せず。