[asterisk-dev] [Code Review] 2942: Give parking timeout comebacktoorigin channel DTMF features.

svnbot reviewboard at asterisk.org
Tue Oct 22 11:26:19 CDT 2013


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

(Updated Oct. 22, 2013, 11:26 a.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers.


Changes
-------

Committed in revision 401420


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


Repository: Asterisk


Description
-------

Parking timeouts did not set any DTMF features for the channel calling the parker back.

* Added code to set the parkedcalltransfers, parkedcallreparking, parkedcallhangup, and parkedcallrecording options appropriately for the channels when a parking timeout occurs.  The recall channel DTMF options are set using the BRIDGE_FEATURES channel variable to allow the other timeout options to have the DTMF features available.

* Updated the configuration option XML documentation to indicate that the parkedcalltransfers, parkedcallreparking, parkedcallhangup, and parkedcallrecording options also apply to parking timeouts.


Diffs
-----

  /branches/12/res/parking/parking_bridge.c 401272 
  /branches/12/res/parking/parking_bridge_features.c 401272 
  /branches/12/res/res_parking.c 401272 

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


Testing
-------

Tested with comebacktoorigin=yes and parkedcallreparking=caller configured in the parking lot.
Had A call B with the tTkK Dial option flags.
Made A park B.
When the parking timeout happens, A is called back and has the DTMF one-touch-parking feature enabled but does not have transfer ability.
Without the patch A has no DTMF features.


Thanks,

rmudgett

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


More information about the asterisk-dev mailing list