Project

General

Profile

Backport(バックポート) #1795

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

Added by Masato Nagasawa almost 9 years ago. Updated over 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 a28df4e7 (diff)
Added by Masato Nagasawa almost 9 years ago

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

Conflicts:

apps/pc_frontend/modules/community/templates/_joinListBox.php
lib/model/doctrine/Member.class.php

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

fixed initialize a cache variable(fixes #1795 BP from #1707)

History

#1 Updated by Masato Nagasawa almost 9 years ago

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

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

#2 Updated by Shinichi Urabe over 8 years ago

3.4.9のタグを切る前のコミットとなります。(タグを作成する前に確認するべきでした。。)
すでに3.4.9に含まれてしまっているのですが、テスト、コードチェックを済ませ、3.4.10 では動作に問題ないという方針で保証する形にします。

#3 Updated by Yuma Sakata over 8 years ago

テストOKです。

#4 Updated by Shinichi Urabe over 8 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

false === $cache にする必要があります。コーディング規約違反です 修正をお願いします

345         if ($cache === false)

false をいれて型が変わるのが、不自然なので、null を代入してください

#5 Updated by Masato Nagasawa over 8 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

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

#6 Updated by Shinichi Urabe over 8 years ago

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

確認しました

#7 Updated by Shinichi Urabe over 8 years ago

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

再テスト完了しました。OKです

Also available in: Atom PDF