Re: PDO driver in CVS

Lukas Kahwe Smith wrote:
> Arnaud Limbourg wrote:
> 
>> Hi All,
>>
>> I added a PDO driver in CVS today. If you want to try it grab cvs (it is
>> not bundled) and go to the docs/examples/example5 directory, there is a
>> conf_pdo.php you need to rename to conf.php. Modify the parameters
>> according to your system settings and you're good to go.
>>
>> It is not quite there yet but it does work :)
>>
>> Feedback is appreciated !
> 
> 
> Cool .. I will take a look.

Looks good to me (I havent tested it).
Unfortunately PDO insists on this rather clumsy dsn/username/password 
handling, so I guess sticking it inside the options is the only sensible 
way. However since you may not need a password or even user in some 
cases but you may need attributes I think we should check each item 
separately if its exists in the options array before using it.

regards,
Lukas