PEAR::DB から PEAR::MDB2 への変更

[nanasess]

現在, DB の抽象化ライブラリとして PEAR::DB を使用しているが, 以下の問題がある.

• PEAR::DB は PHP ライセンスであり, GPL と矛盾する
• もうメンテナンスされてない

上記の理由により PEAR::MDB2 若しくは ADOdb へ変更する

MDB2

 http://pear.php.net/package/MDB2/

ADOdb

 http://adodb.sourceforge.net/

導入要件

• GPL と矛盾しないライセンスで配布されていること
• Sequence 取得の抽象化メソッドがあること
  • Sequence に関して PostgreSQL 及び MySQL を同一のコードベースで記述可能なこと
• データベースのメタデータを扱うメソッドがあること
• 活発にメンテナンスされていること

利点

• PEAR::MDB2
  • 2.5.x では, PDO 用のドライバも有り
  • エラーハンドリングを, 他の PEAR パッケージと統一可能
• ADOdb
  • 速いというウワサ
  • キャッシュ機構あり
  • adodb-errorpear.inc.php を include すれば, PEAR::Error も使用可能
  • adodb-errorhandler.inc.php で, 任意のエラー定数で trigger_error を使用可能

欠点

• PEAR::MDB2
  • PHP の include_path に, 他の PEAR クラスが存在すると, 関数の多重定義エラーになる
  • EC-CUBE のエラーハンドリングが貧弱なため, DBエラーが発生すると無限ループする
• ADOdb
  • DB::last_query に相当する機能が無い(EC-CUBE 本体では未使用)
  • adodb-errorpear.inc.php を使用した場合, 出力するエラー文字列が膨大になり, memory_limit を越える場合がある

その他

• DBエラーが発生した場合, PHP エラーではなく EC-CUBE のエラー画面を表示するのが望ましい

参考

•  http://www.phpied.com/db-2-mdb2/

[nanasess]

現状は, どちらも導入要件を満たしていそうなので, サンプルコードを書いてみることにします.

[nanasess]

PEAR::DB から PEAR::MDB2 へ移行のための参考サイト追加.

現状は, PEAR::DB への依存が激しすぎるので MDB2 の方が妥当かも...

[nanasess]

説明を更新.

PEAR::MDB2 の欠点の影響が強すぎ, 簡単に動いてくれないので, ADOdb を使用する方向で検討.

[nanasess]

r18390 で ADOdb を使用するよう改修した SC_DbConn のパッチをコミット

[nanasess]

エラー表示に関する説明を追記

PEAR::DB 版 SC_DbConn は下記のような実装になっている

• DB 接続エラーの場合
  • SC_DbConn::getAll() 関数の場合のみ 0 を返し, 「DB接続に失敗しました」という文言を表示する. その他の関数は SC_DbConn::send_err_mail() 関数を呼び exit する
• SQL エラーの場合
  • SC_DbConn::send_err_mail() 関数を呼び exit する

PEAR::DB 版 SC_DbConn の実装の問題点は, 接続エラーの場合は文言が表示されるものの, それ以外の場合は真っ白な画面になってしまう.

また, エラー画面を表示しようとしても, 表示するまでに DB 接続を行うため, 多重でエラーが発生する.

本来であれば, DB 関連のエラーはエラーコードを付与して EC-CUBE のエラー画面を表示するのが望ましい.

[miningbrownie]

MDB2ってオリジナルBSDライセンスだったような、、、 オリジナルBSDだとBSD宣伝条項が引っかかるような気がします。

[nanasess]

miningbrownie への返信

MDB2ってオリジナルBSDライセンスだったような、、、オリジナルBSDだとBSD宣伝条項が引っかかるような気がします。

実際に MDB2.php を覗いてみました.

 http://www.google.co.jp/codesearch/p?hl=ja#yNAEdmvlCkE/package/MDB2-2.2.2.tgz|dSu2EDsdFxU/MDB2-2.2.2/MDB2.php&q=MDB2%20PEAR%20lang:php%20pear.php.net

オリジナルBSDライセンスで問題となる, 下記の条項は見当たらないので大丈夫かと思います.

All advertising materials mentioning features or use of this software must display the following acknowledgement:

[nanasess]

エラーハンドリングの問題が解決できそうなので, MDB2 へ置き替えるコードも書いてみることにします

[nanasess]

r18754:18755 で, MDB2 に対応するためのパッチとライブラリを追加しました

[nanasess]

version-2_5-dev r18763 に MDB2 対応のコードをコミットしました.

大幅な修正を伴っていますので, 不具合などありましたらフィードバックお願いします!

[nanasess]

r18766 で ADOdb のライブラリを削除しました

[nanasess]

概ね動作していますので, milestone:EC-CUBE2.5.0beta へ変更します