[Home]

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:19Date Closed:2007-09-16 10:13:16
Priority:MinorRegression?No
Status:Closed/CompleteComponents: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)

------------------------------------------------------------------------