BIZ：グループの管理者が SNS を退会するとグループを削除できなくなる

[kiwa]

■現象

BIZ：グループの管理者が SNS を退会するとグループを削除できなくなる。

■原因

グループの管理者が退会した場合でも、コミュニティと違って管理者交代などの救済措置が用意されていないため、グループの編集・削除がおこなえない。

■修正内容

グループの管理者のSNS退会時、 biz_group_member から当該メンバーの情報を削除し、グループ内のメンバーのうち最も早く参加したメンバー（biz_group_member_id が若いメンバー）を管理者に変更し、管理者交代通知メールを新しい管理者へ送信するようにした。

ただし、退会者しか所属していなかったグループは、メンバー退会時に削除される。

■関連情報

元のDescription

​http://sns.openpne.jp/?m=pc&a=page_c_topic_detail&target_c_commu_topic_id=3331 より転記

OpenPNE-2.10.3を利用しています。

BIZのグループのことで質問があります。

グループの管理者を強制退会させたため、グループ自体を削除できなくなりました。

どうすれば、グループを削除できるのでしょうか？
ご存知の方がいらしたら、ぜひ、教えてください。

[kiwa]

再現できました

■version

• 2.12beta
• 2.10.x
• 2.8.x

■手順

1. メンバーAがグループを作成する
2. 管理画面からメンバーAを強制退会させる
3. 1で作成したグループが管理者のいない状態で残る
4. 既存のグループメンバーへの管理者移行・管理画面からのグループ削除機能が存在しないため、グループが削除できない

[shingo]

修正します。

[shingo]

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

• 2.10.x - r8194 r8195 r8196
• 2.12.x - r8197
• trunk - r8198

[ebihara]

keywords は「確認中」としてください。

[ebihara]

• 削除対象のメンバーしか所属していないグループが存在した場合、そのグループが削除されることなく残ってしまいます。

• db_common_delete_c_member() 内： 文を複数行に渡って記述する場合、文字列結合演算子(.)を代入演算子(=)に揃えて記述することを推奨します。

  583         $sql = 'SELECT c_member_id FROM biz_group_member WHERE biz_group_id = ?'.
  584             ' ORDER BY biz_group_member_id';
  

  参考：​http://framework.zend.com/manual/ja/coding-standard.coding-style.html#coding-standard.coding-style.strings.string-concatenation

   文字列を "." 演算子で連結する際には、コードを読みやすくするために ひとつの文を複数行に分けることもできます。そのような場合は、 2 行目以降の行頭にスペースを入れ、各行の "." 演算子が最初の行の "=" 演算子と同じ位置にくるようにしなければなりません。
  
  $sql = "SELECT `id`, `name` FROM `people` "
       . "WHERE `name` = 'Susan' "
       . "ORDER BY `name` ASC ";
  

• db_common_delete_c_member() 内： do_common_send_mail_biz_group_admin_change() をコールする際、引数を intval() によって型変換していますが、これはあまり好ましくないのではないかという気がします。htmlspecialchars() 関数によるエスケープなどにも同じことが言えますが、このような処理は、変換後の値を用いる直前におこなうようにするのが望ましいです。 ※int型への型変換だとデメリットを感じにくいとは思いますが。。。

  588             do_common_send_mail_biz_group_admin_change(intval($new_admin_id), intval($biz_group['biz_group_id']));
  

  参考：​http://ml.php.gr.jp/pipermail/php-users/2008-February/033747.html

  > $val=htmlspecialchars($_POST['input'],ENT_QUOTES,'SJIS');
  
  あと、これも定石ですが、htmlspecialcharsはHTML出力の直前に
  かけるようにしてください。
  頭っからエスケープしたり、その値をDBに突っ込んだりすると、作った
  直後は良くてもそのうち面倒なことになりかねないので。
  

• do_common_send_mail_biz_group_admin_change() 内: 演算子の前後には半角スペースを入れてください。

  873     require_once OPENPNE_MODULES_BIZ_DIR.'/biz/lib/mysql_functions.php';
  

  参考：​http://framework.zend.com/manual/ja/coding-standard.coding-style.html#coding-standard.coding-style.strings.string-concatenation

  文字列の連結には "." 演算子を使用しなければなりません。コードを読みやすくするため、 "." 演算子の前後には常にスペースを入れなければなりません。 
  

• do_common_send_mail_biz_group_admin_change() 内: $biz_group_member_to と 代入演算子(=)の間に不要と思われるスペースが1個余分に存在します。

  875     $biz_group_member_to  = $biz_group_member = db_member_c_member4c_member_id($biz_group_id_to, true);
  

• do_common_send_mail_biz_group_admin_change() 内: 関数内で参照していない変数 $biz_group_member への代入があります。

  875     $biz_group_member_to  = $biz_group_member = db_member_c_member4c_member_id($biz_group_id_to, true);
  

• do_common_send_mail_biz_group_admin_change() 内: この $to_address の初期化は不要です。

  877     $to_address = ''; 
  

[shingo]

Replying to ebihara:

keywords は「確認中」としてください。

了解です。

[shingo]

Replying to shingo:「確認」は終わってました。訂正します。

[shingo]

再度修正します。

[shingo]

• 2.10.x - r8206
• 2.12.x - r8207
• trunk - r8208

[ebihara]

• 2.10.x への修正に存在する、コミュニティ管理者交代処理の不要な型変換の除去などが 2.12.x と trunk のコードに対しておこなわれていません
• 2.10.x への修正に存在する、不要な $to_address の初期化の除去などが 2.12.x と trunk のコードに対しておこなわれていません。また、 2.12.x と trunk には不要なスペースなどが散見されます

[shingo]

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

• 2.12.x - r8214
• trunk - r8215

[ebihara]

グループの画像が削除されない問題については #2996 で対応するとのことです。

このチケットの範囲の修正は特に問題ないと思います。

[kiwa]

確認しました。問題ありません。

[ebihara]

本現象は強制退会に限らないため Summary と Description を変更しました。

[ebihara]

「管理人」ではなく正しくは「管理者」なので、Summary と Description を修正しました。