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