[asterisk-dev] [Code Review] 2712: Add a swap channel to BridgeEnter event and add a ParkedCallSwap event.

[svnbot]

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

(Updated Aug. 2, 2013, 9:13 a.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers.


Changes
-------

Committed in revision 396107


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


Repository: Asterisk


Description
-------

This does two things:

1) The stasis event for channels entering bridges now also includes the unique ID of a channel being swapped out of the bridge if relevant.
2) The stasis event for parking a call now has another event type for swapping out the parked channel.

Both of these are intended for clarity when reading the AMI event stream. The idea is that it becomes more clear that a channel entering a bridge or parking lot is doing so to take the place of a channel already there.


Diffs
-----

  /trunk/main/cel.c 395704 
  /trunk/main/bridge_channel.c 395704 
  /trunk/include/asterisk/stasis_bridges.h 395704 
  /trunk/include/asterisk/parking.h 395704 
  /trunk/main/stasis_bridges.c 395704 
  /trunk/res/parking/parking_bridge.c 395704 
  /trunk/res/parking/parking_manager.c 395704 
  /trunk/main/manager_bridges.c 395704 

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


Testing
-------

Performed a call where I did an attended transfer of a party to a parking lot. The BridgeEnter event when the transferee entered the parking lot indicated the unique ID of the transferer channel that initially was in the parking bridge. Then a ParkedCallSwap event occurred to show that the channel in the parking lot was replaced with a new channel.


Thanks,

Mark Michelson

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