[asterisk-dev] [Code Review] VoicemailMain and VMauthenticate: Like VoiceMail allow escape to the 'a' extension when a single '*' is entered in Mailbox or Password

[Russell Bryant]

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


Please add the new feature to UPGRADE.txt.  It should probably go in the documentation of VoicemailMain(), as well.

- Russell


On 2010-02-05 03:31:43, Alec Davis wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/489/
> -----------------------------------------------------------
> 
> (Updated 2010-02-05 03:31:43)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> VoiceMail() already has the ability to escape to the single digit 'a' extension.
> 
> Where a site uses VoicemailMain(mailbox) the users have to be at their own extension to clear their voicemail, they have no way of escaping VoicemailMain to allow the escape to offer , there are other reasons to require escape also.
> Or alternately a 2nd extension is rung where VoicemailMain() is called without the mailbox parmeter, and asks for Mailbox and Password.
> 
> This patch, allows a site to include to 'a' priority in the VoicemailMain context, to allow an escape.
> 
> If the 'a' priority doesn't exist in the context VoicemailMain is called from then it acts as the old behaviour.
> 
> 
> Diffs
> -----
> 
>   trunk/apps/app_voicemail.c 244887 
> 
> Diff: https://reviewboard.asterisk.org/r/489/diff
> 
> 
> Testing
> -------
> 
> Tested both VoicemailMain and VMauthenticate with the 'a' extension present and not. All 4 senarios performed as expected.
> Using the following dialplan:
> 
> [voicemail-main]
> exten => s,1,Answer()
> 
> exten => s,n,VMAuthenticate()
> exten => s,n,VoicemailMain()
> exten => s,n,Verbose(0,After VoicemainMain)
> exten => s,n,Hangup
> 
> exten => a,1,Verbose(0,User entered *)
> exten => a,n,Playback(connecting)
> exten => a,n,Goto(s,1)
> 
> 
> Thanks,
> 
> Alec
> 
>