管理画面のPostgreSQL 8.3対応

[tietew]

■概要

管理画面内でデータ取り出し時に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からも半年以上経ちますので、そろっと対応を進めて欲しいです。

[kiwa]

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

[kiwa]

取り込みます

[kiwa]

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

• trunk - r10783

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

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

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

[kiwa]

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

[ShogoKawahara]

ミスしました。

[ShogoKawahara]

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

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

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

[kiwa]

Replying to kawahara:

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

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

[kiwa]

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

[kiwa]

owner変更のためreopenします。