Re: Auth, [set|get]AuthData(), suggestions and MDB container
| From: | Martin Jansen | Date: | Wed, 05 Mar 2003 16:27:28 +0000 |
| Subject: | Re: Auth, [set|get]AuthData(), suggestions and MDB container | ||
| References: | 1 2 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-14088@lists.php.net to get a copy of this message | ||
On Wed Mar 05, 2003 at 05:1642PM +0100, Lukas Smith wrote:
> > 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.
It makes actually sense, because the MDB container uses two methods
which are identical to their DB counterparts.
> 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.
I don't see what you mean, but it would be cool if you could provide a
patch for that.
> Btw: you don't need the freeResult on line 227 because fetchRow always
> frees the result set.
Fixed in CVS.
--
- Martin Martin Jansen
http://martinjansen.com/