[asterisk-dev] [Code Review] 3011: Say suite of applications: Add the ability to listen for DTMF and jump to a new extension (like Background)

[Jonathan Rose]

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

(Updated Nov. 14, 2013, 2:32 p.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers, Mark Michelson and rmudgett.


Changes
-------

Committed in revision 402819


Repository: Asterisk


Description
-------

Say applications didn't currently have a way to wait for DTMF like the background application does. One exception to this rule existed, that being the SayUnixTime application with 'jump' option enabled. Since some of these applications could not be changed to accept new arguments without breaking existing dialplan, a new channel variable 'SAY_DTMF_INTERRUPT' will now be checked for by these applications, and if it is true on the channel that is running the application, it will listen for DTMF similar to Background.


Diffs
-----

  /trunk/main/pbx.c 402698 
  /trunk/apps/app_sayunixtime.c 402698 
  /trunk/CHANGES 402698 

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


Testing
-------

Tested entering extensions in each type of Say application modified with and without setting the SAY_DTMF_INTERRUPT variable to 'true'.  Also tested SayCounted applications (which all already act like Background unconditionally and won't be changed).


Thanks,

Jonathan Rose

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20131114/bc920cda/attachment.html>