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

Matt Jordan reviewboard at asterisk.org
Tue May 1 16:31:01 CDT 2012


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

Review request for Asterisk Developers.


Summary
-------

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 redirect to existed in the current context of the caller, or, if the context <c> option were used, if the extension being dialed happened to have an initial digit that matched in the caller's current context.

This means the following would fail:

[default]

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

[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.
    https://issues.asterisk.org/jira/browse/ASTERISK-18243


Diffs
-----

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

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


Testing
-------

Testing covered by the leave_voicemail_contexts test, in review 1893.


Thanks,

Matt

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


More information about the asterisk-dev mailing list