[dev] Re: [cvs] commit: horde/lib/Auth cyrsql.php

[marcus@riboflavin.net]

I originally was going to do this, but it seemed more obvious/easier
to understand the use of this function the way it is.  Why?  This is
cyrus-specific.  The way it is now matches the setting in imapd.conf.
 I like the idea of matching parameters between the two config files.

I'm open to changing it, but I'd like a more compelling argument.

Quoting Mike Cochrane <mike@graftonhall.co.nz>:

> 
> > ----- Message from marcus@riboflavin.net ---------
> >
> > marcus      2002/09/08 18:40:20 PDT
> >
> >   Modified files:
> >     lib/Auth             cyrsql.php
> >   Log:
> >   * tab->spaces
> >   * Delete used 'cyradmin' instead of params['cyradmin'] so
> removing
> >     accounts didn't work.
> >   * Add new variable $_separator and a paramater 'unixhier'.  If
> >     unixhier is true, then we should use '/' instead of '.' for
> the
> >     hierarchy separator.
> why not a new parameter 'separator' and default it to '.' if it
> doesn't exist in
> the contstructor?
> 
> just my 2c worth :-)
> 
> - Mike :-)
> 
> --
> Horde developers mailing list
> Frequently Asked Questions: http://horde.org/faq/
> To unsubscribe, mail: dev-unsubscribe@lists.horde.org