Ticket #2162 (closed 改善提案: 修正済)

Opened 11 years ago

Last modified 11 years ago

pageクラスからdtb_best_productsテーブルを直接指定している箇所をなくす

Reported by: pineray Owned by: pineray
Priority: Milestone: EC-CUBE2.13.0
Component: フロント Version:
Keywords: Cc:
修正済み: yes

Description

pageクラスからはデータベースを直接触れないようにする。
SC_Helper_Recommendクラスを作成し、dtb_best_productsに関する処理を集約する。

Change History

comment:1 Changed 11 years ago by pineray

  • Owner changed from somebody to pineray
  • 修正済み set
  • Status changed from new to assigned

r22582 にて対応しました。

comment:2 follow-up: ↓ 3 Changed 11 years ago by hiro151

  • Owner changed from pineray to hiro151
  • 修正済み unset
  • Status changed from assigned to new

テーブルが、dtb_reccommendとdtb_best_productsがあるため、dtb_best_productsを操作するHelperは命名を変更するほうがいいと思います。

comment:3 in reply to: ↑ 2 ; follow-up: ↓ 4 Changed 11 years ago by hiro151

hiro151 への返信

テーブルが、dtb_reccommendとdtb_best_productsがあるため、dtb_best_productsを操作するHelperは命名を変更するほうがいいと思います。

r22625にて、対応しました。

comment:4 in reply to: ↑ 3 Changed 11 years ago by hiro151

hiro151 への返信

hiro151 への返信

テーブルが、dtb_reccommendとdtb_best_productsがあるため、dtb_best_productsを操作するHelperは命名を変更するほうがいいと思います。

r22625 にて、対応しました。

comment:5 Changed 11 years ago by hiro151

r22711 削除処理のバグを修正メッソド内で呼んでいるsfDeleteRankRecordメソッドの引数が足りない

comment:6 Changed 11 years ago by kim

  • Owner changed from hiro151 to pineray

comment:7 Changed 11 years ago by pineray

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

comment:8 Changed 11 years ago by h_yoshimoto

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

comment:9 Changed 11 years ago by m_uehara

  • 修正済み unset

非公開の商品がフロントでおすすめ商品として表示できるようになってしまっています。
ご確認のほどよろしくお願いいたします。

comment:10 Changed 11 years ago by pineray

  • 修正済み set

表示判定の条件式に誤りがあったので、
r23117 r23118 にて対応しました。

comment:11 Changed 11 years ago by h_yoshimoto

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

ご対応ありがとうございました。
こちらクローズさせていただきます。

Note: See TracTickets for help on using tickets.