Summary: | ASTERISK-10306: [patch] maxsecs option never set per mailbox due deprecated maxmessage, it's working globally only | ||
Reporter: | Caio Begotti (caio1982) | Labels: | |
Date Opened: | 2007-09-14 23:22:19 | Date Closed: | 2007-09-16 10:13:16 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | Applications/app_voicemail |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) voicemail_maxsecs_per_mailbox.diff | |
Description: | Maybe nobody noticed it before because it's not a much special option for most people, but it is to me :-) maxsecs is only working globally in trunk, never per mailbox. It's because it used to be named as maxmessage but the deprecated code was marked only for the general context not for all mailboxes' options, thus an if() ignored it until now or you was forced to keep using the old option name. The attached change fixes that and let people call the AMI command VoicemailUsersList happily again. | ||
Comments: | By: Digium Subversion (svnbot) 2007-09-16 10:13:16 Repository: asterisk Revision: 82496 ------------------------------------------------------------------------ r82496 | tilghman | 2007-09-16 10:13:15 -0500 (Sun, 16 Sep 2007) | 2 lines Option maxmessage should be maxsecs per-folder, too (closes issue ASTERISK-10306) ------------------------------------------------------------------------ |