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

Opened 10 years ago

Closed 10 years ago

#3906 closed enhancement (fixed)

コミュニティを管理しているメンバーがSNSを退会したときの対処がされていない

Reported by: ShogoKawahara Owned by: nagasawa
Priority: minor Milestone: OpenPNE3.0.5
Component: core Version: 3.0.x & 3.1.x
Keywords: 3系未実装項目 OpenPNE3.1.1 Cc:

Description

コミュニティを管理しているメンバーが、SNSを退会したときの対処がされていない。

Change History (13)

comment:1 Changed 10 years ago by ShogoKawahara

Type: defectenhancement

comment:2 Changed 10 years ago by nagasawa

Owner: changed from nobody to nagasawa
Status: newassigned

やります

comment:3 Changed 10 years ago by nagasawa

Keywords: 確認中 added

r11799 で追加しました。

comment:4 Changed 10 years ago by nagasawa

r11800 で以下の内容を修正しました。

  • 不要なコメントアウト
  • ハードタブ

comment:5 Changed 10 years ago by nagasawa

何度もすいません、コメントアウトの修正が入っていませんでしたので
r11801 で修正しました。

comment:6 Changed 10 years ago by ShogoKawahara

3.0.xでは対応されていますが、3.1.xでは対応されていないようです。

comment:7 Changed 10 years ago by ShogoKawahara

Milestone: OpenPNE3.0.5

comment:8 Changed 10 years ago by ShogoKawahara

Keywords: 確認中 removed

comment:9 Changed 10 years ago by ShogoKawahara

Keywords: 3系未実装項目 OpenPNE3.1.1 added; 3系未実装項目 3.0.5&3.1.0 removed
Version: 3.0.x3.0.x & 3.1.x

comment:10 Changed 10 years ago by nagasawa

Keywords: 確認中 added

r11992 で3.1.xに追加しました。

comment:11 Changed 10 years ago by ShogoKawahara

Keywords: 差し戻し added; 確認中 removed

./lib/model/doctrine/CommunityMemberTable.class.php

123   public function getCommunityMemberCount($communityId)
124   {
125     $communityMember = $this->createQuery()
126       ->select('COUNT(*)')
127       ->where('community_id = ?', $communityId)
128       ->addWhere('position = ?', '')
129       ->fetchArray();
130
131     return $communityMember[0]['COUNT'];
132   }

3.1.xでのこの実装は正しくないように思えます。

public function getCommunityMembers($communityId)
{
  return $this->createQuery()
    ->where('community_id = ?', $communityId)
    ->addWhere('position = ?', '')
    ->execute();
}

のようにして、 カウント時に、

Doctrine::getTable('CommunityMember')
  ->getCommunityMembers($id)
  ->count();

とした方が望ましく思います。

comment:12 Changed 10 years ago by nagasawa

Keywords: 確認中 added; 差し戻し removed

r12330 で修正しました。

comment:13 Changed 10 years ago by ebihara

Keywords: 3系未実装項目 OpenPNE3.1.1 added; 3系未実装項目 OpenPNE3.1.1 確認中 removed
Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.