Re: Auth bug
| From: | Pierre-Alain Joye | Date: | Mon, 12 Aug 2002 15:42:43 +0000 |
| Subject: | Re: Auth bug | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-8324@lists.php.net to get a copy of this message | ||
On Mon, 12 Aug 2002 17:18:04 +0200
LIMBOURG Arnaud <arnaud.limbourg@cegetel.fr> wrote:
> Hello,
>
> There is a small bug in PEAR::Auth.
>
> When you pass a dsn as an array (performance reason), Auth crashes.
>
> A fix is tho change line 97 of DB.php of the last release to
>
> if (is_string($dsn) || is_array($dsn)) {
>
> instead of
> if (is_string($dsn)) {
Well, the current Auth package does not contain this line :
97 if (is_string($dsn)) {
98 $this->db = DB::Connect($dsn);
But, cannot be consider as a bug, afaik, DB::Connect accepts dsn as array. In this case, the options
given to the Auth constructor can be :
$options['dsn'] = array(
.... dsn array stuffs....
);
I remember a discussion with Martin months ago about the choosed parsing method in Auth, so I m not
pretty sure with that (:)), but seems to be normal and need to add the array test to the current
cvs.
hth
pa