[asterisk-bugs] [Asterisk 0019199]: [patch] MaxMsg Quota Only Enforced on INBOX Messages for Filesystem Based Voice Mail Boxes

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=19199 
====================================================================== 
Reported By:                Jamuel
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   19199
Category:                   Applications/app_voicemail
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:           1.6.2.18 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2011-04-28 19:13 CDT
Last Modified:              2011-04-28 19:23 CDT
====================================================================== 
Summary:                    [patch] MaxMsg Quota Only Enforced on INBOX Messages
for Filesystem Based Voice Mail Boxes
Description: 
When a caller attempts to leave a message for a VM user app_voicemail does
not count messages in other folders in the user's mailbox (e.g. Old,
Urgent, Friends, Family, CustX) in consideration of maxmsg limit.  

The maxmsg setting which is supposed limit total messages per folder does
not work as anticipated as a user could individually (or in aggregate) have
more messages than maxmsg allows just as long as the number contained in
INBOX is less than maxmsg.  It appears that the file-based count_messages
function only ever checks INBOX.

NB: the maxmsg setting has been interpreted to be a setting to control the
total number messages for each user (mailbox) folder and not each
sub-folder that may optionally be contained within the user's mailbox
folder.
====================================================================== 

---------------------------------------------------------------------- 
 (0134261) Jamuel (reporter) - 2011-04-28 19:23
 https://issues.asterisk.org/view.php?id=19199#c134261 
---------------------------------------------------------------------- 
Patch (unified diff) created that modifies the file system based
count_messages() function so that the path sent to it is recursively
scanned for *.txt files.  The total number of files is returned.

Instead of sending $VM_SPOOL_DIR/$VM_CONTEXT/$MAILBOX/INBOX as the path
for  count_messages() during leave_voicemail()
$VM_SPOOL_DIR/$VM_CONTEXT/$MAILBOX/INBOX is sent. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2011-04-28 19:23 Jamuel         Note Added: 0134261                          
======================================================================