[asterisk-dev] [Code Review] Make the FollowMe application optionally update the connected line information when the accepting endpoint is bridged.

[Mark Michelson]

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



/branches/1.8/apps/app_followme.c
<https://reviewboard.asterisk.org/r/1656/#comment9572>

    Did you make these changes because there is the possibility that winner and caller are the same channel?



/branches/1.8/apps/app_followme.c
<https://reviewboard.asterisk.org/r/1656/#comment9573>

    I think this change makes the message less clear. The reason is that saying "SIP/1000 placed on hold" can sanely be interpreted to mean Asterisk is placing SIP/1000 on hold. What the message is trying to convey is that SIP/1000 has placed Asterisk on hold. So either add back "Call on" or change it to something like "%s has placed us on hold".



/branches/1.8/apps/app_followme.c
<https://reviewboard.asterisk.org/r/1656/#comment9574>

    The way this is coded doesn't match the description of the 'I' option. The documentation says that any connected line change will be blocked. This only blocks connected line changes received from outbound channels. So either update the documentation to say what it actually does or change the way this behaves to match what the documentation says it does. For what it's worth, app_dial in Asterisk 1.8.X blocks connected line changes on the incoming channel as well. I'd suggest making this do the same.


- Mark


On Jan. 6, 2012, 4:52 p.m., rmudgett wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1656/
> -----------------------------------------------------------
> 
> (Updated Jan. 6, 2012, 4:52 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> Like Dial and Queue, FollowMe needs to deal with AST_CONTROL_CONNECTED_LINE information so when the parties are initially bridged, the connected line information will be correct.
> 
> Added the 'I' option just like the app_dial and app_queue 'I' option.
> 
> 
> This addresses bug ASTERISK-18969.
>     https://issues.asterisk.org/jira/browse/ASTERISK-18969
> 
> 
> Diffs
> -----
> 
>   /branches/1.8/CHANGES 349955 
>   /branches/1.8/apps/app_dial.c 349955 
>   /branches/1.8/apps/app_followme.c 349955 
> 
> Diff: https://reviewboard.asterisk.org/r/1656/diff
> 
> 
> Testing
> -------
> 
> Tested with and without the new FollowMe 'I' option.  Connected line is updated or blocked in a similar manner to app_dial and app_queue.
> 
> 
> Thanks,
> 
> rmudgett
> 
>

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