[asterisk-dev] [Code Review] Add ConnectedLineNum and ConnectedLineName AMI headers to several events.

Russell Bryant reviewboard at asterisk.org
Wed May 25 10:39:01 CDT 2011


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

Ship it!


- Russell


On 2011-05-23 15:29:58, rmudgett wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1227/
> -----------------------------------------------------------
> 
> (Updated 2011-05-23 15:29:58)
> 
> 
> Review request for Asterisk Developers and Russell Bryant.
> 
> 
> Summary
> -------
> 
> The ConnectedLineNum and ConnectedLineName headers were added to many AMI
> events/responses if the CallerIDNum/CallerIDName headers were also present.
> 
> The addition of connected line support changes the behavior of the channel
> caller ID somewhat.  The channel caller ID value no longer time shares with
> the connected line ID on outgoing call legs.  The timing of some AMI
> events/responses output the connected line ID as caller ID.  These party ID's
> are now separate.
> 
> 
> This addresses bug 18252.
>     https://issues.asterisk.org/view.php?id=18252
> 
> 
> Diffs
> -----
> 
>   /branches/1.8/CHANGES 320708 
>   /branches/1.8/UPGRADE.txt 320708 
>   /branches/1.8/apps/app_dial.c 320708 
>   /branches/1.8/apps/app_fax.c 320708 
>   /branches/1.8/apps/app_meetme.c 320708 
>   /branches/1.8/apps/app_queue.c 320708 
>   /branches/1.8/main/channel.c 320708 
>   /branches/1.8/main/features.c 320708 
>   /branches/1.8/main/manager.c 320708 
> 
> Diff: https://reviewboard.asterisk.org/r/1227/diff
> 
> 
> Testing
> -------
> 
> Compile testing and code inspection at this point.
> 
> 
> Thanks,
> 
> rmudgett
> 
>

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


More information about the asterisk-dev mailing list