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

Opened 7 years ago

Last modified 6 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 7 years ago by pineray

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

r22582 にて対応しました。

comment:2 follow-up: ↓ 3 Changed 7 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 7 years ago by hiro151

hiro151 への返信

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

r22625にて、対応しました。

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

hiro151 への返信

hiro151 への返信

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

r22625 にて、対応しました。

comment:5 Changed 7 years ago by hiro151

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

comment:6 Changed 6 years ago by kim

  • Owner changed from hiro151 to pineray

comment:7 Changed 6 years ago by pineray

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

comment:8 Changed 6 years ago by h_yoshimoto

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

comment:9 Changed 6 years ago by m_uehara

  • 修正済み unset

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

comment:10 Changed 6 years ago by pineray

  • 修正済み set

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

comment:11 Changed 6 years ago by h_yoshimoto

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

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

Note: See TracTickets for help on using tickets.