[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-17 15:00:10.822876)


Review request for Asterisk Developers.


Changes
-------

When removing the gotos I forgot to change all the else ifs, fixed.


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 253202 

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