FuelPHP に Pull Request を送ったら merge されたはなし

2012/06/01

昨日の記事で、

Database_Connection::instance()->connect();
$pdo = Database_Connection::instance()->connection();

ポイントとしては、いきなり、

$pdo = Database_Connection::instance()->connection();

ってやってもダメなことでしょうか。instance() メソッドが呼ばれたタイミングではまだデータベースへの接続が確立されていないからです。ということで、取得する前に、connect() メソッドを呼んであげてます。

というふうに、FuelPHP で Database への接続オブジェクトを直接取得する方法を書いたんですが、どうせ接続を取得するメソッドなんだから、返す前に実際に接続できているかどうか確認してくれたらいいのに、と思ったので、これはちょうどいい機会だと思い、github で Pull Request を送ってみたら、なんと!速攻 merge されました!生まれて初めての github での Pull Request 成功です。ちょっと興奮。なんか嬉しい。生きててよかった!きゃー!

Pull Request #994: Make sure that the database is connected before return $_connection. by omoon · fuel/core

ということで、今日から FuelPHP で pdo オブジェクトなどのデータベース接続を直接取得したい場合は、いきなり

$pdo = Database_Connection::instance()->connection();

ってやっても大丈夫です。めでたし、めでたし。

<< 古い記事