Ticket #519 (closed バグ指摘: 修正済)

Opened 11 years ago

Last modified 10 years ago

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 11 years ago by AMUAMU

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

r18427 にて修正しました。 自分の確認した範囲では恐らく影響は出ないと思います。

comment:2 Changed 11 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:3 Changed 11 years ago by Seasoft

r18435 にて修正してみました。

comment:4 Changed 11 years ago by Seasoft

  • Milestone set to EC-CUBE2.5.0

comment:5 Changed 11 years ago by Seasoft

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

comment:6 Changed 10 years ago by nanasess

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

正式版へ未反映ですので差し戻します

comment:7 Changed 10 years ago by nanasess

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

version-2_5-dev ブランチで対応済みになりますので close します

Note: See TracTickets for help on using tickets.