ここの情報は古いです。ご理解頂いた上でお取り扱いください。

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#1537 closed defect (fixed)

postgresql用SQLにカラム名間違いがある

Reported by: imoto Owned by: ebihara
Priority: critical Milestone: OpenPNE2.10.2
Component: pne-postgresql Version: 2.10.x & 2.11.x
Keywords: OpenPNE2.11.2 Cc:

Description (last modified by ebihara)

■現象

  • この問題が直接的に影響を及ぼすのはPostgreSQL使用時のみ(MySQLではインサート時にプライマリキーを指定しておらず、常にNULL)
    • MySQLの場合は手動でプライマリキーを設定している場合(=プライマリキーが テーブル名+_id でない場合)に動作しない?(これについては別チケットを作成し、そこで対応する)

■原因

■修正内容

  • install用SQLについては、正しいカラム名に書き換え
  • すでに導入されている環境への対処として、setup/sql/postgres74/option/option-2.10.1to2.10.2-alter_c_pc_address_pre.sql を用意することにより対処(以下を実行)
        $ psql DBNAME -U username -f option/option-2.10.1to2.10.2-alter_c_pc_address_pre.sql
    

■関連情報

【バグ】OpenPNE2.10.1にてpostgresql用SQLに不備在り

テーブル: c_pc_address_pre

誤: c_pc_addess_pre_id
正: c_pc_address_pre_id

修正用SQL
ALTER TABLE c_pc_address_pre RENAME COLUMN c_pc_addess_pre_id TO c_pc_address_pre_id ;

2.10.0の時はPostgresql用SQLが付属されてこなかったから
適当に開発版のSQLを持ってきたけれども

2.10.1に付属されてくるSQLも同様の物のままで
カラム名に間違いが在った。

このままだと動作何て、しないのでカラム名修正して動作。

Change History (17)

comment:1 Changed 12 years ago by kiwa

Type: enhancementdefect

comment:2 Changed 12 years ago by kiwa

Component: pne-frameworkpne-postgresql
Keywords: 再現待ち removed

comment:3 Changed 12 years ago by kiwa

Priority: majorcritical

comment:4 Changed 12 years ago by kiwa

Milestone: OpenPNE2.10.2
Summary: 2.10.1でpostgresql用SQLにカラム名間違いがあるpostgresql用SQLにカラム名間違いがある

comment:5 Changed 12 years ago by kiwa

Description: modified (diff)

comment:6 Changed 12 years ago by ebihara

Owner: changed from nobody to ebihara
Status: newassigned

やります。

安定版の場合、update用SQLを追加することはできないので、以下のような形で対応しようと思います。

  • setup/sql/postgres74/install/install-2.10-create_tables.sql と setup/sql/postgres74/install/install-2.8-create_tables.sql は正しいものに修正
  • setup/sql/postgres74/option/ 以下に、報告元にある「修正用SQL」を実行するための .sql ファイルを置く

comment:7 Changed 12 years ago by ebihara

Description: modified (diff)
Keywords: 確認中 added
Owner: changed from ebihara to ogawa
Status: assignednew

以下のリビジョンで修正しました。ご確認ください。

前述の通り、install用SQLについてはカラム名を正しいものに書き換え、既存導入環境に対しての修正用SQL(setup/sql/postgres74/option/option-2.10.1to2.10.2-alter_c_pc_address_pre.sql)を用意することで対処しました。

Description の「■修正内容」を書き換えたので併せてご覧ください。

comment:8 Changed 12 years ago by ogawa

Keywords: 確認中 removed
Owner: changed from ogawa to ebihara

MySQL側も間違っていますね。

comment:9 Changed 12 years ago by ebihara

Description: modified (diff)
Status: newassigned

MySQLに対しても同様に修正します。

Descriptionの「■現象」の項に追記しました。

comment:10 Changed 12 years ago by ebihara

Keywords: 確認中 added

MySQLについては以下のリビジョンで修正しました。ご確認ください。

comment:11 Changed 12 years ago by ogawa

Keywords: 確認中 removed

MySQLについては動作上問題はなさそうなので、2.10では対応しないことにします。trunkのみupdate SQLで対応してください。

comment:12 Changed 12 years ago by ebihara

了解です。

  • 2.10.x で用意した MySQL用のoption SQL削除、install SQLの変更を取り消し
  • trunk の option SQL は削除し、update SQL にて対応

で行きます。

comment:13 Changed 12 years ago by ebihara

Keywords: 確認中 added

以下のリビジョンで対応しました。ご確認ください。

comment:14 Changed 12 years ago by ogawa

Keywords: 確認中 removed

他のテーブルと合わせる意味ではNOT NULLが必要ではないでしょうか。

comment:15 Changed 12 years ago by ebihara

Keywords: 確認中 added

r4759で NOT NULL を追加しました。

comment:16 Changed 12 years ago by ogawa

Keywords: 確認中 removed
Resolution: fixed
Status: assignedclosed

comment:17 Changed 12 years ago by kiwa

Keywords: OpenPNE2.11.2 added
Note: See TracTickets for help on using tickets.