Project

General

Profile

Backport(バックポート) #3682

コミュニティ作成不可カテゴリのコミュニティでコミュニティ副管理者が設定変更をおこなうとコミュニティカテゴリが空になる

Added by 誠二 天重 almost 5 years ago. Updated almost 5 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2012-06-08
Due date:
% Done:

100%


Description

現象

コミュニティカテゴリ設定で「メンバーのコミュニティ作成を許可」のチェックが入っていないコミュニティカテゴリが設定されているコミュニティについて、
コミュニティ副管理者がコミュニティ設定変更をおこなうと、コミュニティカテゴリが空になる。

再現手順

1. コミュニティカテゴリ設定 ( /pc_backend.php/community/categoryList ) にて『カテゴリA』の「メンバーのコミュニティ作成を許可」のチェックを外し、編集ボタンを押下して変更を完了させる
2. member_id:1のメンバーがコミュニティカテゴリに『カテゴリA』を選択した コミュニティa を作成する
3. メンバーB が コミュニティa に参加する
4. member_id:1 のメンバーが コミュニティa のコミュニティメンバー管理から メンバーB を副管理者に指名する
5. メンバーB が コミュニティa の副管理者依頼を承認する
6. メンバーB が コミュニティa のコミュニティ設定画面を開く → コミュニティカテゴリが未選択状態になっている
7. コミュニティ設定画面で設定を確定する → コミュニティカテゴリが空になる

確認バージョン

OpenPNE 3.6.3

原因

source:lib/form/doctrine/CommunityForm.class.php にて、コミュニティに「メンバーのコミュニティ作成を許可」されていないカテゴリが設定された状態で member_id=1 でない管理者(副管理者)が編集画面を開くといった状況が想定されていなかったことが原因。

修正内容

  • ログイン中のメンバーのIDが 1 であるか否かに関わらず、編集前の時点で設定されているカテゴリは「コミュニティカテゴリ」の選択肢に含めるようにする
  • ログイン中のメンバーのIDが 1 であるか否かに関わらず、編集前の時点で設定されているカテゴリは「メンバーのコミュニティ作成を許可」されていない場合でも設定できるようにする

Related issues

Copied from OpenPNE 3 - Bug(バグ) #3072: コミュニティ作成不可カテゴリのコミュニティでコミュニティ副管理者が設定変更をおこなうとコミュニティカテゴリが空になる Pending Testing(テスト待ち) 2012-06-08

Associated revisions

Revision d1e153ba (diff)
Added by Youichi Kimura almost 5 years ago

allow sub-admin not to change the category (refs #3682, BP from #3072)

Revision a7c060cb (diff)
Added by Youichi Kimura almost 5 years ago

fix a regression caused by the CommunityForm changes (refs #3682, BP from #3072)

History

#1 Updated by 誠二 天重 almost 5 years ago

  • Copied from Bug(バグ) #3072: コミュニティ作成不可カテゴリのコミュニティでコミュニティ副管理者が設定変更をおこなうとコミュニティカテゴリが空になる added

#2 Updated by 誠二 天重 almost 5 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

https://github.com/openpne/OpenPNE3/pull/180
にてプルリクエストしました。

#3 Updated by Shinichi Urabe almost 5 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#4 Updated by isao sano almost 5 years ago

  • Status changed from Pending Testing(テスト待ち) to Pending Merge(マージ待ち)
  • % Done changed from 70 to 80

単体試験完了しました。問題ありません。

#5 Updated by Shinichi Urabe almost 5 years ago

  • Status changed from Pending Merge(マージ待ち) to Fixed(完了)
  • % Done changed from 80 to 100

Also available in: Atom PDF