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

Opened 10 years ago

Last modified 10 years ago

MYSQL 商品複製時にエラー

Reported by: Seasoft Owned by: Seasoft
Priority: Milestone: EC-CUBE2.11.1
Component: 管理画面 Version: 2.11.0
Keywords: Cc:
修正済み: yes

Description (last modified by Seasoft) (diff)

data\class\db\SC_DB_DBFactory.php (L1019) の SC_DB_DBFactory_MYSQL#sfGetColumnList が意図したとおりに値を返していない模様。

EC-CUBE	 2.11.1-dev
OS	 Windows NT SEVEN 6.1 build 7600 (Unknow Windows version Home Premium Edition) i586
DBサーバ	 MySQL 5.5.8
WEBサーバ	 Apache/2.2.17 (Win32) mod_ssl/2.2.17 OpenSSL/0.9.8o PHP/5.3.4 mod_perl/2.0.4 Perl/v5.10.1
PHP	 5.3.5
GD	 Loaded

r20781 で発現確認。

r20783 r20784

Change History

comment:1 Changed 10 years ago by Seasoft

  • Description modified (diff)

comment:2 Changed 10 years ago by Seasoft

  • Owner changed from somebody to Seasoft
  • 修正済み set
  • Summary changed from 商品複製時にエラー to MYSQL 商品複製時にエラー

comment:3 follow-up: ↓ 4 Changed 10 years ago by AMUAMU

SC_Query::listTableFields() を使うようにした方が良いですね。(なんで古い方を使ったままなんだろ・・・

comment:4 in reply to: ↑ 3 Changed 10 years ago by Seasoft

  • Status changed from new to assigned

@deprecated ですね。このあたり、リリースまでにきっちり削除したかったのですが、そこまで手が回らず無念です…

ざっと grep したところ、使用箇所はこの1個所ですね。

こういったケース、マイナーバージョンアップでメソッド削除も可能なものですかね? それとも、@deprecated マーク維持ですかね?

comment:5 Changed 10 years ago by Seasoft

  • Description modified (diff)

とりあえず、r20784 で SC_Query#listTableFields を使用するように改修しました。

comment:6 Changed 10 years ago by AMUAMU

リファクタは引き続き残った所をするということだったので、メソッドレベルは削除可能だと思いますが、ロックオンさんフォローコメント御願いします。

comment:7 follow-up: ↓ 8 Changed 10 years ago by kajiwara

Seasoftさん

返答が遅れましたが、リファクタリングをしていただくのはマイナーバージョンアップでも可能です。
ただし、仕様が変わってしまう等がある場合はご相談くださいませ。

comment:8 in reply to: ↑ 7 Changed 10 years ago by Seasoft

よく分からないので、r20850 で削除してみました。

問題があったら、差し戻してください。

comment:9 Changed 10 years ago by kajiwara

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

Seasoftさん

本件、ご報告及び修正作業いただき、誠にありがとうございました。

正常に動作していることを確認しておりますので、修正済みとさせていただきます。

Note: See TracTickets for help on using tickets.