Backport(バックポート) #1795
Did not consider the waiting for approval of community participation(マイホームのコミュニティ参加数が承認待ちを考慮していない)
開始日:
2010-10-19
期日:
進捗率:
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 がほぼ14年前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
更新履歴 a28df4e74ea3a60afbcfe3b3341efbcbcf4f70e7 で適用されました。
#2
Shinichi Urabe が13年以上前に更新
3.4.9のタグを切る前のコミットとなります。(タグを作成する前に確認するべきでした。。)
すでに3.4.9に含まれてしまっているのですが、テスト、コードチェックを済ませ、3.4.10 では動作に問題ないという方針で保証する形にします。
#3
Yuma Sakata が13年以上前に更新
テストOKです。
#4
Shinichi Urabe が13年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
false === $cache にする必要があります。コーディング規約違反です 修正をお願いします
345 if ($cache === false)
false をいれて型が変わるのが、不自然なので、null を代入してください
#5
Masato Nagasawa が13年以上前に更新
- ステータス を Rejected(差し戻し) から Pending Review(レビュー待ち) に変更
更新履歴 d18753e79a41293e38388e738022e1c0a052c996 で適用されました。
#6
Shinichi Urabe が13年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
確認しました
#7
Shinichi Urabe が13年以上前に更新
- ステータス を Pending Testing(テスト待ち) から Fixed(完了) に変更
- 進捗率 を 70 から 100 に変更
再テスト完了しました。OKです