[asterisk-dev] [Code Review] Properly support the d() option in VoiceMail

Matt Jordan reviewboard at asterisk.org
Tue May 1 20:59:19 CDT 2012

This is an automatically generated e-mail. To reply, visit:

(Updated May 1, 2012, 8:59 p.m.)

Review request for Asterisk Developers.


Clarified the description

Summary (updated)

The VoiceMail d() option is documented to do the following:

    d([c]): Accept digits for a new extension in context <c>, if played
    during the greeting. Context defaults to the current context.

This option worked fine if the extension being redirected to existed in the current context of the caller; or, if the context <c> option were used, if the extension being redirected to happened to have an initial digit that also existed in the caller's current context.  If that digit did not happen to exist in the current caller's context, the extension would fail to match.

This means the following would fail:


exten => 1234,1,VoiceMail(1234 at default,d(othercontext))


exten => 5,1,NoOp(I will never be reached, because there is no extension that begins with a 5 in default)

This patch fixes it such that if the <c> option is used, the extensions are matched in that context as opposed to the caller's original context.

This addresses bug ASTERISK-18243.


  /branches/1.8/apps/app_voicemail.c 364705 
  /branches/1.8/main/pbx.c 364705 

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


Testing covered by the leave_voicemail_contexts test, in review 1893.



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20120502/e5bb2b99/attachment.htm>

More information about the asterisk-dev mailing list