RE: [PEAR-DEV] Auth, [set|get]AuthData(), suggestions and MDB container
| From: | Lukas Smith | Date: | Wed, 05 Mar 2003 16:16:42 +0000 |
| Subject: | RE: [PEAR-DEV] Auth, [set|get]AuthData(), suggestions and MDB container | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-14087@lists.php.net to get a copy of this message | ||
> -----Original Message-----
> From: Martin Jansen [mailto:mj@php.net]
> Sent: Wednesday, March 05, 2003 5:09 PM
> To: Lorenzo Alberton
> Cc: pear-dev@lists.php.net
> Subject: Re: [PEAR-DEV] Auth, [set|get]AuthData(), suggestions and MDB
> container
>
> On Sun Mar 02, 2003 at 09:3154PM +0100, Lorenzo Alberton wrote:
> > Wouldn't it a good idea just to add a new method for adding
> > (and not only replacing) a new user value?
> >
> > BTW, for those interested, I attached a MDB container for Auth.
>
> I've added the container to CVS!
> BTW: From what I can see, the best idea would be to make it a subclass
> of Auth_Container_DB, don't you think?
Dunno if that makes sense.
However one thing that is not used in the container ATM is the type
conversion which might be a good idea to add at some point. That woul
get rid of the hardcoded quotes etc.
Btw: you don't need the freeResult on line 227 because fetchRow always
frees the result set.
Regards,
Lukas