[asterisk-dev] [Code Review] 2648: func_channel: set dtmf_features

jrose reviewboard at asterisk.org
Tue Jul 2 14:05:42 CDT 2013


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

(Updated July 2, 2013, 7:05 p.m.)


Review request for Asterisk Developers, Matt Jordan and rmudgett.


Changes
-------

Address Matt's notes


Bugs: ASTERISK-21876
    https://issues.asterisk.org/jira/browse/ASTERISK-21876


Repository: Asterisk


Description
-------

Allows setting dtmf features via a channel set function (namely CHANNEL(dtmf_features)


Diffs (updated)
-----

  /trunk/bridges/bridge_builtin_features.c 393426 
  /trunk/funcs/func_channel.c 393426 
  /trunk/include/asterisk/bridging_basic.h 393426 
  /trunk/main/bridging_basic.c 393426 

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


Testing
-------

Stuck two channels in the following extension:

exten => 10,1,NoOp
exten => 10,n,Answer()
exten => 10,n,Set(CHANNEL(dtmf_features)=wktx)
exten => 10,n,Wait(50)

Then bridged them together with the manager bridge command and checked to see if both channels could park, transfer, and attended transfer.

It's worth noting that the dial application overrides this datastore entirely, so these don't get applied to dial as-is. This might need to be addressed.


Thanks,

jrose

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


More information about the asterisk-dev mailing list