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

[Tilghman Lesher]

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



/trunk/main/pbx.c
<https://reviewboard.asterisk.org/r/3011/#comment19443>

    I think you're better off using ast_true() for this.


- Tilghman Lesher


On Nov. 12, 2013, 11:44 p.m., Jonathan Rose wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/3011/
> -----------------------------------------------------------
> 
> (Updated Nov. 12, 2013, 11:44 p.m.)
> 
> 
> Review request for Asterisk Developers, Mark Michelson and rmudgett.
> 
> 
> 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/20131113/8d69b938/attachment.html>