[dev] [PATCH] More IMP quota patches ...

Michael M Slusarz slusarz at bigworm.colorado.edu
Tue Sep 30 11:34:19 PDT 2003


Quoting Etienne Goyer <etienne.goyer at linuxquebec.com>:

| Hi list,
|
| I have a patch that fix three issue with IMP_Quota :
|
| 1. Typo in function documentation (kB -> bytes).

Committed.

| 2. Bug with an earlier patch of mine in templates/quota/quota.inc
| regarding error display.

Nope.  We already get the error message from the PEAR_Error object so there
should be no reason to have this gettext string here.

| 3. Wrong assumption regarding mailbox name in imp/lib/Quota/cyrus.php.
| The code was assuming a standard namespace (user<sep><mailbox>) and
| break on Cyrus server that use altnamespace.  I tested my patch with
| both standard namespace and altnamespace, and it worked for me.
| However, since my system have the imap_get_quotaroot function available,
| I did no tried the alternate path of using the imap_get_quota function
| instead so I don't know how it react in this circumstance.  To be
| checked, I suppose.

Committed.  I don't use cyrus so others will have to verify this works.

michael

______________________________________________
Michael Slusarz [slusarz at bigworm.colorado.edu]
The University of Colorado at Boulder


More information about the dev mailing list