Project

General

Profile

Backport(バックポート) #1794

Did not consider the waiting for approval of community participation(マイホームのコミュニティ参加数が承認待ちを考慮していない)

Added by Masato Nagasawa over 8 years ago. Updated about 8 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2010-10-19
Due date:
% Done:

100%


Description

Overview (現象)

承認が必要なコミュニティに参加申請している場合に、
リストに表示されていないにも関わらずカウントされてしまう。

Causes (原因)

Doctrineのリレーション情報から member_id が一致する CommunityMember のレコードを取得しており、
community_member.is_pre を考慮していないことが原因です。
また、 レコードの取得後にカウント処理を行っているため、速度面でも問題があります。

$member->CommunityMember->count()

Way to fix (修正内容)

Member クラスに is_pre を考慮して Doctrine_Query::count() を行うメソッドを追加して置換する。


Related issues

Related to OpenPNE 3 - Bug(バグ) #1707: Did not consider the waiting for approval of community participation(マイホームのコミュニティ参加数が承認待ちを考慮していない) Fixed(完了) 2010-10-19

Associated revisions

Revision f3676e9c (diff)
Added by Masato Nagasawa over 8 years ago

fixed the pre-member included to counted of the join-communities (fixes #1794, BP from #1707)

Revision dc3a2e24 (diff)
Added by Masato Nagasawa about 8 years ago

fixed the pre-member included to counted of the join-communities (fixes #1794, BP from #1707)

History

#1 Updated by Masato Nagasawa over 8 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 f3676e9c9ae0f5390d6ebceea6f88849f9ecac38 で適用されました。

#2 Updated by Kousuke Ebihara over 8 years ago

  • Target version changed from OpenPNE 3.6beta7 to OpenPNE 3.6beta8

#3 Updated by Kousuke Ebihara over 8 years ago

  • Target version changed from OpenPNE 3.6beta8 to OpenPNE3.6beta9

#4 Updated by Kousuke Ebihara over 8 years ago

  • Target version changed from OpenPNE3.6beta9 to OpenPNE3.6beta11

#5 Updated by Masato Nagasawa about 8 years ago

  • Target version changed from OpenPNE3.6beta11 to OpenPNE 3.6.0

#6 Updated by Masato Nagasawa about 8 years ago

  • Target version changed from OpenPNE 3.6.0 to OpenPNE3.6beta11

#7 Updated by Kousuke Ebihara about 8 years ago

  • Target version changed from OpenPNE3.6beta11 to OpenPNE3.6beta12

親チケットが「テスト待ち」「完了」でないものをすべて beta12 以降での対応とします。

#8 Updated by Kousuke Ebihara about 8 years ago

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

#9 Updated by Masato Nagasawa about 8 years ago

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

更新履歴 dc3a2e24fe6ab056a8466a9809d204906367a7f2 で適用されました。

#10 Updated by Kousuke Ebihara about 8 years ago

http://redmine.openpne.jp/issues/1794#note-9 のリリースブランチへの取り込みでステータスが「レビュー待ち」に巻き戻ってしまいましたので、もとに戻します。

#11 Updated by Kousuke Ebihara about 8 years ago

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

#12 Updated by Minoru Takai about 8 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

動作テスト

親チケットと同様に stable-3.6.x ブランチで動作テストを行いました。

apps/pc_frontend/modules/community/templates/_joinListBox.php が呼ばれるページ(マイホーム)で、修正前のコードでは自身が参加しているコミュニティ数(コミュニティリスト下部のリンク「全てを見る(10)」の括弧内の部分)が承認待ちのものも含まれており、修正後にはこれが排除されていることを確認しました。

修正内容を見ても、副作用などはなく、適切な修正であると判断できます。

完了とします。

Also available in: Atom PDF