robregonm / YiiFirebird

Firebird Adapter for Yii 1.1.x
13 stars 8 forks source link

improve handling of default values #21

Closed mlorentz75 closed 11 years ago

mlorentz75 commented 11 years ago
robregonm commented 11 years ago

Checking the code of the pull request..., do this mean that this extension will be fully working for Firebird 2.5 and above? If this is correct, then we need to update the README.md in order to add this minimum version requirement.

mlorentz75 commented 11 years ago

The changes attempt to handle most context variables known in FB2.5 and that's currently the only release I'm testing against. I'm afraid I currently don't have time testing against older Firebird releases.

But I guess the patch shouldn't break things with former releases. Like, FB prior to 2.0 didn't have the optional precision in CURRENT_TIME (see: http://www.firebirdsql.org/refdocs/langrefupd20-current_time.html) so people running FB1.5 would just use 'CURRENT_TIME' without precision. The updated code would still work as it matches the beginning of the default value string.

That said, a note in README.md recommending to use the latest version of the Firebird PDO driver and probably DB too wouldn't hurt. I was quickly running into problems on CentOS 6 x86_64 when using the stock php-interbase RPM and thus switched development to FC18.

robregonm commented 11 years ago

It's ok, just wanted to make sure. I'll add the recommendation to the REAME.md file. Thanks.