[asterisk-bugs] [Asterisk 0011415]: Better quota handling with IMAP storage

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=11415 
====================================================================== 
Reported By:                jaroth
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   11415
Category:                   Applications/app_voicemail
Reproducibility:            N/A
Severity:                   feature
Priority:                   normal
Status:                     new
Asterisk Version:            SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 90048 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             11-29-2007 11:05 CST
Last Modified:              11-29-2007 16:04 CST
====================================================================== 
Summary:                    Better quota handling with IMAP storage
Description: 
This patch has 3 components:

1 - Refactor quota checking to a check_quota() method
2 - Add additional quota check for number of messages in IMAP mailbox
(currently only IMAP quotas - disk usage - are checked with IMAP storage
enabled)
3 - Add quota check when user logs in to their own mailbox, to inform them
that their mailbox is over-quota (either in number of messages or IMAP
quota)
====================================================================== 

---------------------------------------------------------------------- 
 putnopvut - 11-29-07 16:04  
---------------------------------------------------------------------- 
This seems like a good (and much needed) patch.

The only thing I don't like is the hard-coded word "INBOX" since a user
may have a different folder specified (via the imapfolder voicemail.conf
option). The problem is, it's not enough just to modify this one patch,
since the mbox() function has "INBOX" hardcoded as well. 

I am going to add the project of removing the word "INBOX" from the IMAP
code path to my queue and I will report back here once it has been taken
care of. Once I have taken care of that, the only change needed for this
patch will be to change the literal "INBOX" to imapfolder instead. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
11-29-07 16:04  putnopvut      Note Added: 0074569                          
======================================================================