[asterisk-dev] [Code Review] sig_pri and chan_misdn do not update connected line after call transfer

[rmudgett at digium.com]

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

Review request for Asterisk Developers and Russell Bryant.


Summary
-------

This patch corrects an oversight and is against the 1.8 branch.  The sig_pri and chan_misdn channel drivers did not update the connected line information after a call transfer.


Diffs
-----

  /branches/1.8/channels/chan_misdn.c 289425 
  /branches/1.8/channels/sig_pri.c 289425 

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


Testing
-------

Call transfers still work and the connected line information is updated.  Also the expected caller/callee connected line interception macros are run on each channel.

Basic transfer actions:
A calls B
A places B on hold
A calls C
A can optionally swap between B and C by flash hook.
A transfers B to C

Test cases:
A transfers B to C using hangup.
A transfers B to C using hangup while C ringing.
A transfers C to B using hangup.
A transfers B to C using ECTExecute message.
A transfers B to C using ECTExecute message while C ringing.
A transfers C to B using ECTExecute message.

Note chan_misdn does not support the ECTExecute method.

Also for chan_misdn, party B does not hear ringback when party C is ringing after the transfer.  I think this is an unrelated issue because the expected events happen that should start the ringback tones to party B.


Thanks,

rmudgett