Backport（バックポート） #1794
Did not consider the waiting for approval of community participation(マイホームのコミュニティ参加数が承認待ちを考慮していない)
100%
説明
Overview (現象)¶
承認が必要なコミュニティに参加申請している場合に、
リストに表示されていないにも関わらずカウントされてしまう。
Causes (原因)¶
Doctrineのリレーション情報から member_id が一致する CommunityMember のレコードを取得しており、
community_member.is_pre を考慮していないことが原因です。
また、 レコードの取得後にカウント処理を行っているため、速度面でも問題があります。
$member->CommunityMember->count()
Way to fix (修正内容)¶
Member クラスに is_pre を考慮して Doctrine_Query::count() を行うメソッドを追加して置換する。
関連するチケット
関係しているリビジョン
履歴
#1
Masato Nagasawa が13年以上前に更新
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
更新履歴 f3676e9c9ae0f5390d6ebceea6f88849f9ecac38 で適用されました。
#2
Kousuke Ebihara が13年以上前に更新
- 対象バージョン を OpenPNE 3.6beta7 から OpenPNE 3.6beta8 に変更
#3
Kousuke Ebihara が約13年前に更新
- 対象バージョン を OpenPNE 3.6beta8 から OpenPNE3.6beta9 に変更
#4
Kousuke Ebihara が約13年前に更新
- 対象バージョン を OpenPNE3.6beta9 から OpenPNE3.6beta11 に変更
#5
Masato Nagasawa がほぼ13年前に更新
- 対象バージョン を OpenPNE3.6beta11 から OpenPNE 3.6.0 に変更
#6
Masato Nagasawa がほぼ13年前に更新
- 対象バージョン を OpenPNE 3.6.0 から OpenPNE3.6beta11 に変更
#7
Kousuke Ebihara がほぼ13年前に更新
- 対象バージョン を OpenPNE3.6beta11 から OpenPNE3.6beta12 に変更
親チケットが「テスト待ち」「完了」でないものをすべて beta12 以降での対応とします。
#8
Kousuke Ebihara がほぼ13年前に更新
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
#9
Masato Nagasawa がほぼ13年前に更新
- ステータス を Pending Testing（テスト待ち） から Pending Review（レビュー待ち） に変更
- 進捗率 を 70 から 50 に変更
更新履歴 dc3a2e24fe6ab056a8466a9809d204906367a7f2 で適用されました。
#10
Kousuke Ebihara がほぼ13年前に更新
http://redmine.openpne.jp/issues/1794#note-9 のリリースブランチへの取り込みでステータスが「レビュー待ち」に巻き戻ってしまいましたので、もとに戻します。
#11
Kousuke Ebihara がほぼ13年前に更新
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
#12
Minoru Takai がほぼ13年前に更新
- ステータス を Pending Testing（テスト待ち） から Fixed（完了） に変更
- 進捗率 を 70 から 100 に変更
動作テスト¶
親チケットと同様に stable-3.6.x ブランチで動作テストを行いました。
apps/pc_frontend/modules/community/templates/_joinListBox.php が呼ばれるページ（マイホーム）で、修正前のコードでは自身が参加しているコミュニティ数（コミュニティリスト下部のリンク「全てを見る(10)」の括弧内の部分）が承認待ちのものも含まれており、修正後にはこれが排除されていることを確認しました。
修正内容を見ても、副作用などはなく、適切な修正であると判断できます。
完了とします。