Ticket #441 (closed バグ指摘: 修正済)

Opened 13 years ago

Last modified 11 years ago

お気に入り一覧で重複表示

Reported by: Seasoft Owned by: somebody
Priority: Milestone: EC-CUBE2.4.0
Component: フロント Version: 2.4.0RC-1
Keywords: Cc:
修正済み:

Description

お気に入りに登録した商品が、複数のカテゴリに割り当てされていると、割り当てられたカテゴリー分、お気に入り一覧に重複して表示される。

Linux-user 様の報告。  http://xoops.ec-cube.net/modules/newb/viewtopic.php?topic_id=3739&forum=1

コミュニティでは r17955 にて改修。
ramrun 様がテストを実施してくださり、詳細なテスト報告をいただきました。 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?viewmode=thread&topic_id=3739&forum=1&post_id=15247#forumpost15247

Attachments

LC_Page_Mypage_Favorite.php Download (11.3 KB) - added by kajiwara 13 years ago.
4/20 19:30時点での最新ファイル(LC_Page_Mypage_Favorite.php)を添付いたします。

Change History

comment:1 Changed 13 years ago by kajiwara

本件、ご報告、並びに改修、テスト作業を実施していただき、ありがとうございました。
>seasoft様、ramrun 様、及び開発コミュニティにご参加のみなさま

以下、r17980、r17981にて不具合を改修いたしました。

①r17980 にて商品が重複表示される不具合を改修いたしました。

コミュニティ版や開発コミュニティを参考にさせていただき、結果ビューを使用せず、必要最小限の情報を取得するSQLを記述する形で対応させていただきました。
(参照コミュニティ: http://xoops.ec-cube.net/modules/newbb/viewtopic.php?viewmode=thread&topic_id=3739&forum=1&post_id=15149#forumpost15149

以下の点より、ビューを使用しない対応としています。

○お気に入り機能に必要な情報(テーブル)がそれほど多くないため、そのままSQLをソースに記載しても問題ないレベルと判断。
○現状の一覧表示用のビューでは不必要なテーブルを参照するため、使用しない。
○その他ビューは近しいビューも存在するが、本来の使用目的ではない使用方法となる点、改修が必要になる点より、一旦使用しない。

②r17981 にてナビ部分のボタン画像反転処理の不具合を改修いたしました。
(参照コミュニティ: http://xoops.ec-cube.net/modules/newbb/viewtopic.php?viewmode=thread&topic_id=3691&forum=4&post_id=14928#forumpost14928

comment:2 Changed 13 years ago by kajiwara

  • Status changed from new to closed
  • Resolution set to 修正済

一旦解決にさせていただきます。

comment:3 Changed 13 years ago by kajiwara

  • Status changed from closed to reopened
  • Resolution 修正済 deleted

comment:4 Changed 13 years ago by kajiwara

バグ報告、誠にありがとうございます。>Linux-user様

r17983 にてページ送りの処理が正常に動作していない不具合を修正いたしました。

また、そもそもお気に入り商品が「最大20件表示」という部分の処理がどこにも含まれていないよう思いますので、こちらの文言をテンプレート上から削除いたしました。

Changed 13 years ago by kajiwara

4/20 19:30時点での最新ファイル(LC_Page_Mypage_Favorite.php)を添付いたします。

comment:5 Changed 13 years ago by kajiwara

  • Status changed from reopened to closed
  • Resolution set to 修正済

再度解決にさせていただきます。

comment:6 Changed 11 years ago by Seasoft

コミュニティ・関連 r18080

Note: See TracTickets for help on using tickets.