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

Opened 8 years ago

Last modified 8 years ago

SC_Queryのupdateメソッドの使用を推奨

Reported by: shutta Owned by: shutta
Priority: Milestone: EC-CUBE2.12.3
Component: その他 Version: 2.12.1
Keywords: Cc:
修正済み: yes

Description

地のSQL文でUPDATEを実行している部分を、SC_Queryのupdateメソッドを使用するように改善したい。

updateメソッドを使用することを規約にするとかまでは考えておらず、個人的に、DBクラス側で制御しやすいようにUPDATEに関しては統一されていれば良いかな、と思った次第です。

気に入らなければ却下してもらって結構なので。

Change History

comment:1 Changed 8 years ago by shutta

  • Owner changed from somebody to shutta
  • Status changed from new to assigned

comment:2 Changed 8 years ago by shutta

  • 修正済み set

r22012 にてコミット。

ただ、LC_Page_Admin_Products_ProductRankクラスのlfRenumberメソッド内のUPDATE処理に関しては、構文が少し複雑なのでupdateメソッドに描き直さずにそのままにしています。
書き換えること自体は可能ですが、可読性が落ちるかなぁと思い敢えて書き換えていません。

やはり、全部書き換えるのは無理があるかもしれません。
あくまで、(個人的な)推奨ということで。。。

comment:3 Changed 8 years ago by kim

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

Shutta様 ありがとうございます。 まずは、可能なところだけでOKです。 こちらでクローズいたします。

Note: See TracTickets for help on using tickets.