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

Opened 11 years ago

Last modified 10 years ago

dtb_order_detail に主キーが無い

Reported by: nanasess Owned by: nanasess
Priority: Milestone: EC-CUBE2.11.0
Component: その他 Version: 2.11.0 β
Keywords: Cc:
修正済み:

Description (last modified by nanasess) (diff)

order_id と product_class_id を主キーにした方が良いと思われる.

可能であれば, product_id も削除を検討する

シーケンス値の dtb_order_detail.order_detail_id を作成する

Change History

comment:1 follow-up: ↓ 3 Changed 11 years ago by nanasess

  • Description modified (diff)
  • Summary changed from dtb_other_deliv に主キーが無い to dtb_order_detail に主キーが無い

comment:2 Changed 11 years ago by AMUAMU

部分発送のような管理や商品明細の分解等のカスタマイズ性や自由度を考えると複合キーではなく、別途serialな主キーを入れた方が良いのでは無いでしょうか?
安易な複合キーは暗黙的なシステム拡張の制約になるので、個人的には避けて欲しいと思います。

また、コミュニティでも以前並び順が規格順であることの違和感が質問に上がっていましたので、顧客の購入順に固定し、顧客違和感を無くすという意味でも別主キー発行は有効かと思われます。
 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=6861&forum=11&post_id=35169

comment:3 in reply to: ↑ 1 Changed 11 years ago by Seasoft

nanasess への返信 私も過去のカスタマイズ経験から、商品規格をテーブルのキーにするのは避けて欲しいです。どうしてもキーにするならば、カートセッションの配列キーにもして、統一して欲しいです。

order_id 毎の rank のようなものがあると、良いのかなとも思います。

comment:4 Changed 10 years ago by nanasess

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

ありがとうございます. 別主キーの方向で実装しようと思います.

comment:5 Changed 10 years ago by nanasess

  • Status changed from assigned to closed
  • Resolution set to 修正済
  • Description modified (diff)

r20435 で対応しました

Note: See TracTickets for help on using tickets.