Ticket #519 (closed バグ指摘: 修正済)
SCUtils::sfInsertProductClass内のSQL文の見直し
| Reported by: | AMUAMU | Owned by: | somebody |
|---|---|---|---|
| Priority: | 中 | Milestone: | EC-CUBE2.5.0alpha |
| Component: | 管理画面 | Version: | コミュニティ (eccube-comu) |
| Keywords: | Cc: | seasoft | |
| 修正済み: |
Description
psyark様より報告がありました。
http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=4308&forum=9&post_id=19160
はじめまして。SCUtils::sfInsertProductClassに以下のようなコードがあるのですが
// すでに規格登録があるかどうかをチェックする。
$where = "product_id = ? AND classcategory_id1 <> 0 AND classcategory_id1 <> 0";
$count = $objQuery->count("dtb_products_class", $where, array($product_id));
classcategory_id1 <> 0が繰り返されているのですが、classcategory_id2の間違いではないかと思い書き込みました。
Change History
comment:1 Changed 13 years ago by AMUAMU
- Status changed from new to closed
- Version changed from 2.4.0 to 2.4.1
- Resolution set to 修正済
comment:2 Changed 13 years ago by Seasoft
- Status changed from closed to reopened
- Cc seasoft added
- Priority changed from 低 to 中
- Version changed from 2.4.1 to コミュニティ (eccube-comu)
- Milestone EC-CUBE2.x.x deleted
- Resolution 修正済 deleted
本改訂の適用により、規格1のみを設定している商品の情報(価格や在庫)が異常になると、 私どものお客様より報告がありました。 当方では、クリーンインストールでのテストは行なっていませんが、取り急ぎ報告まで。 (追記) ぱっと見ですが、「<>0 AND <>0」はまずいような・・・ OR 比較する必要があるような。 (追記2) むしろ、EC-CUBE のデータ的には、classcategory_id1 のみの比較で十分で、 classcategory_id2 を評価する必要性は薄いのかもしれません。 (商品規格2のみ登録するのは不可能な気がするので。 CSV登録やDB直接編集などが関わってくると、分かりませんが。)
詳細はフォーラムにて・・・
http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=5079&forum=11
comment:5 Changed 13 years ago by Seasoft
- Status changed from reopened to closed
- Resolution set to 修正済
Note: See
TracTickets for help on using
tickets.
r18427 にて修正しました。 自分の確認した範囲では恐らく影響は出ないと思います。