[Home]

Summary:ASTERISK-15657: [patch] Voicemail minsecs is not able to be overridden per-mailbox
Reporter:Mark Murawski (kobaz)Labels:
Date Opened:2010-02-18 11:51:52.000-0600Date Closed:2010-03-24 13:13:32
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_voicemail
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) voicemail.patch
( 1) voicemail2.patch
Description:When configuring voicemail users you have the following

[genera]
minsecs=3

[internal]
4200 => 1000,foo,foo@bar,pager@bar,attach=no|serveremail=myaddy@digium.com|tz=central|maxmsg=10|minsecs=5


Where the last column is per-mailbox overrides of the defauls.

minsecs is completely left out in the override processing.

Here's a patch.

This also affects trunk/1.6.1/1.6.2  (1.4 as well?)

Comments:By: Mark Murawski (kobaz) 2010-02-18 12:10:50.000-0600

Same with nextaftercmd... new patch
By: Digium Subversion (svnbot) 2010-03-24 13:13:31

Repository: asterisk
Revision: 254321

U   trunk/CHANGES
U   trunk/apps/app_voicemail.c
U   trunk/configs/voicemail.conf.sample

------------------------------------------------------------------------
r254321 | jpeeler | 2010-03-24 13:13:30 -0500 (Wed, 24 Mar 2010) | 12 lines

Allow configuration of minsecs and nextaftercmd per mailbox.

Previously only configurable globally. A unit test has also been written to
provide protection against parse failures for supported mailbox options.

(closes issue ASTERISK-15657)
Reported by: kobaz
Patches:
     voicemail2.patch uploaded by kobaz (license 834)

Review: https://reviewboard.asterisk.org/r/555/

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

http://svn.digium.com/view/asterisk?view=rev&revision=254321