[Usagi-developer 99] Re: サイドブロックの新着レビュー

Back to archive index

Kenji kenji****@club*****
2009年 7月 16日 (木) 08:09:47 JST


Kenji です。


バグとして修正しました。

http://usagi-project.org/redmine/issues/show/252
http://usagi-project.org/redmine/issues/show/253


// Kenji


On Thu, 18 Jun 2009 10:59:36 +0900
Kenji <kenji****@club*****> wrote:

> > 現在、サイドブロックの新着レビューには、すでにレビューがあるアイテム
> > にレビューが追加された場合は、表示されません。
> > 
> > これは、レビュー検索の結果と異なります。
> > 
> > サイドブロックの新着レビューについて、このような仕様にあえてする理由が
> > わかりませんので、これはバグではないでしょうか?
> > 
> > 原因は、サイドブロックの新着レビューで GROUP BY しているため、複数の
> > レビューがある場合は、データが捨てらてしまうためです。
> > 
> > ===================================================================
> > --- webapp_ext/modules/pc/page/h_home_right_side.php	(リビジョン 713)
> > +++ webapp_ext/modules/pc/page/h_home_right_side.php	(作業コピー)
> > @@ -160,8 +160,8 @@
> >                              . MYNETS_PREFIX_NAME . "c_review_comment AS crc ON (cr.c_review_id = crc.c_review_id) "
> >  //                . "INNER JOIN "
> >  //                            . MYNETS_PREFIX_NAME . "c_review_category AS crc2 ON (cr.c_review_category_id = crc2.c_review_category_id) "
> > -          . "GROUP BY "
> > -                . "crc.c_review_id "
> > +//          . "GROUP BY "
> > +//                . "crc.c_review_id "
> >            . "ORDER BY "
> >                  . "crc.r_datetime DESC";
> 
> これだと、同じ本が複数回表示されることがありますね。
> こうですね。
> 
> ===================================================================
> --- h_home_right_side.php	(リビジョン 713)
> +++ h_home_right_side.php	(作業コピー)
> @@ -163,7 +163,7 @@
>            . "GROUP BY "
>                  . "crc.c_review_id "
>            . "ORDER BY "
> -                . "crc.r_datetime DESC";
> +                . "MAX(crc.r_datetime) DESC";
>  
>      $result = db_get_all_limit($sql , 0, intval($size));
>      return $result;
> 
> 
> // Kenji
> 
> _______________________________________________
> Usagi-developer mailing list
> Usagi****@lists*****
> http://lists.sourceforge.jp/mailman/listinfo/usagi-developer




Usagi-developer メーリングリストの案内
Back to archive index