A couple of goodies

[David Carter]

On Wed, 25 Jul 2007, Bron Gondwana wrote:

> See also the attached, our latest patch - it includes a bugfix which stops
> you being able to randomly move some user's home directory to a non-home
> directory elsewhere.  Something that you could do only because of the bug
> in mboxname_isusermailbox.

I think that the problem is specific to virtual domain setups.

> It also makes mboxname_isusermailbox a fair bit easier to read - though 
> it does a config item lookup each use - might be worth making that 
> static or even a bit more global.  Hmm.  What do you think?

config_getstring() is actually just an array lookup: the hard work is done 
at compile time.

mboxlist_delayed_delete_isenabled() and mboxlist_in_delete_hierachy() have 
some caching, but this is really just habit on my part.

-- 
David Carter                             Email: David.Carter at ucs.cam.ac.uk
University Computing Service,            Phone: (01223) 334502
New Museums Site, Pembroke Street,       Fax:   (01223) 334679
Cambridge UK. CB2 3QH.