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

Opened 10 years ago

Last modified 8 years ago

SQL標準に準拠する

Reported by: nanasess Owned by: nanasess
Priority: Milestone: EC-CUBE2.12.0
Component: その他 Version: 2.4.4
Keywords: Cc:
修正済み: yes

Description (last modified by nanasess) (diff)

特定の RDBMS へ依存しないように, できるだけ SQL標準に準拠した関数を使用する また, カラム名に予約語を使用しないよう修正する

  • now() -> CURRENT_TIMESTAMP
  • rule -> rule_max

Change History

comment:1 Changed 10 years ago by nanasess

r18788 で, CREATE TABLE については CURRENT_TIMESTAMP に変更しました

comment:2 Changed 10 years ago by nanasess

  • Milestone changed from EC-CUBE2.5.0beta to EC-CUBE2.5.1(仮)

comment:3 Changed 10 years ago by kotani

  • 修正済み unset
  • Milestone changed from EC-CUBE2.11.1 to EC-CUBE2.12.0(仮)

comment:4 follow-up: ↓ 5 Changed 10 years ago by shutta

自分が携わった部分で恐縮ですが、売上集計の年代別集計にて、RDBMS依存の関数を使用しちゃってます。
r20242, r20243 あたりの部分が該当します。
これを書き直すのは骨が折れそうなので、とりあえずメモとして残しておきます。

comment:5 in reply to: ↑ 4 ; follow-up: ↓ 6 Changed 10 years ago by nanasess

shutta への返信

自分が携わった部分で恐縮ですが、売上集計の年代別集計にて、RDBMS依存の関数を使用しちゃってます。
r20242, r20243 あたりの部分が該当します。
これを書き直すのは骨が折れそうなので、とりあえずメモとして残しておきます。

DB_TYPE を使用しているのは, ちょっとまずいですね... 見落していてすみません.

DBFactory クラスに移動させないといけませんので, #801 を差し戻しておきます.

comment:6 in reply to: ↑ 5 Changed 10 years ago by shutta

DB_TYPEにより分岐する部分は、SC_DB_DBFactory で吸収するようになっているんですね。
とりあえず、 r20242:20243#801 に関する修正は、 r20792 にて対応してみました。

comment:7 Changed 9 years ago by shutta

r21185,r21186 で、CURRENT_TIMESTAMP を使用するように now() を置換。

comment:8 Changed 8 years ago by nanasess

  • Owner changed from somebody to nanasess
  • Status changed from new to assigned
  • Summary changed from SQL標準関数を使用する to SQL標準に準拠する

comment:9 Changed 8 years ago by nanasess

  • Description modified (diff)

comment:10 Changed 8 years ago by nanasess

  • 修正済み set

rule については r21810 で対応しました

comment:11 Changed 8 years ago by kajiwara

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

nanasessさん、shuttaさん

本件、ご対応ありがとうございます。

チケットに記載のある2点に関してはこちらで対応済と思いますので、クローズさせていただきます。

Note: See TracTickets for help on using tickets.