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

Opened 9 years ago

Closed 8 years ago

#3210 closed enhancement (fixed)

管理画面のPostgreSQL 8.3対応

Reported by: tietew Owned by: tietew
Priority: minor Milestone: OpenPNE2.13.6
Component: pne-postgresql Version: 2.12.x & 2.14.x
Keywords: Cc:

Description (last modified by kiwa)

■概要

管理画面内でデータ取り出し時にr_datetimeで文字の切り出し(substr)を行っているが、PostgreSQL8からは対応されない。この部分を修正して、PostgreSQL8以上のバージョンでも管理画面の機能が動作するようにしてほしい。

■仕様

#3242で既にsubstrで使うr_datetimeをtextにキャストすることで対応していたが、読みやすさを優先してsubstrをto_charに変更。PostgreSQL8.3に対応した。

■関連情報

元のDescription

PostgreSQL 8.3で動かなかった場所のパッチです。 今のところ、これ以外で動いていなさそうな場所は見つかっていません。 パッチは OpenPNE 2.12.6 ベースで作成しました。

PostgreSQL 8.2のリリースから二年、8.3からも半年以上経ちますので、そろっと対応を進めて欲しいです。

Attachments (1)

db_admin.pgsql-analysis-sqlfix.patch (7.8 KB) - added by tietew 9 years ago.

Download all attachments as: .zip

Change History (14)

Changed 9 years ago by tietew

comment:1 Changed 9 years ago by kiwa

Milestone: OpenPNE2.13.6

ありがとうございます!是非取り込みたいのでMilestoneを開発版の次のバージョンにしておきました。

comment:2 Changed 8 years ago by kiwa

Owner: changed from nobody to kiwa
Status: newassigned
Summary: webapp/modules/admin/lib/db_admin.php patch for PostgreSQL 8.3+管理画面のPostgreSQL 8.3対応

取り込みます

comment:3 Changed 8 years ago by kiwa

Component: 指定しないpne-postgresql
Description: modified (diff)
Keywords: 確認中 added
Version: 2.12.x2.12.x & 2.13.x

以下のリビジョンで対応しました。

既にr9726, r10177, r10178 でPostgreSQL8.3に対応していましたが、tietewさんの提供してくださったコードのほうが読みやすかったので独断でこちらを取り込ませていただきました。

2.12と2.13ではコードが変わっていたので、パッチは使用せずにdb_admin.php内のr_datetime::textになっているsubstrを総置換した形になっています。

異論などありましたらコメントお願いします。

comment:4 Changed 8 years ago by kiwa

r10842, r10843, r10844 にて、取り込み漏れの部分を修正しました

comment:5 Changed 8 years ago by ShogoKawahara

Keywords: テスト待ち added; 確認中 removed

comment:6 Changed 8 years ago by ShogoKawahara

Keywords: 確認中 added; テスト待ち removed

ミスしました。

comment:7 Changed 8 years ago by ShogoKawahara

クエリーの文字列連結は下のように = と . を揃えるのが望ましいです。

$sql = "なんやら"
     . " かんやら";

ただし、今回の更新以外でも共通化していない部分が多く見受けられるので、対応は別でも構わないと思います。

comment:8 in reply to:  7 Changed 8 years ago by kiwa

Replying to kawahara:

今回の修正外の話ですし、この際他の場所も統一したいので別チケットで対応しましょう。

  • #3671: クエリーの文字列連結方法の統一

comment:9 Changed 8 years ago by ShogoKawahara

Keywords: テスト待ち added; 確認中 removed

comment:10 Changed 8 years ago by kiwa

Keywords: テスト待ち removed
Resolution: fixed
Status: assignedclosed

PostgreSQL7.4の環境がないため、今までの対応バージョンのDBで確認はできませんでしたが、PostgreSQL8.3環境での動作は問題ありませんでした。

comment:11 Changed 8 years ago by kiwa

Resolution: fixed
Status: closedreopened

owner変更のためreopenします。

comment:12 Changed 8 years ago by kiwa

Owner: changed from kiwa to tietew
Status: reopenednew

comment:13 Changed 8 years ago by kiwa

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.