[asterisk-dev] [Code Review] Allow configuration of minsecs and nextaftercmd per mailbox

[Jeff Peeler]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/555/
-----------------------------------------------------------

(Updated 2010-03-23 18:30:37.635115)


Review request for Asterisk Developers.


Changes
-------

Addressed Russell's comments.


Summary
-------

Make this work in voicemail.conf:

[default]
1234 => 4242,Example Mailbox,imapuser at localhost,,minsecs=5|nextaftercmd=yes


This addresses bug 16864.
    https://issues.asterisk.org/view.php?id=16864


Diffs (updated)
-----

  /trunk/apps/app_voicemail.c 254162 

Diff: https://reviewboard.asterisk.org/r/555/diff


Testing
-------

Tested both options to ensure they were respected and have also created a unit test that checks to see if the vm_user struct is populated correctly (I guess it'll protect against accidental deletions or typos).


Thanks,

Jeff