[asterisk-dev] [Code Review] New AMI Command LocalOptimizeAway

Mark Michelson mmichelson at digium.com
Wed Jun 23 12:56:38 CDT 2010


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

Ship it!


Looks good by me. Good job!

- Mark


On 2010-06-23 12:17:32, tim_ringenbach wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/732/
> -----------------------------------------------------------
> 
> (Updated 2010-06-23 12:17:32)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> Adds a new manager command LocalOptimizeAway. It clears the flag on a local channel that says not to optimize it away.
> 
> General use case: You need the behavior of "/n" for a while, but then something else happens (typically a transfer) and you no longer desire that behavior.
> 
> Specific Use case: Using local channels as queue members without stateinterface, using /n to keep the local channel up so the agent doesn't get another call, then on transfer issuing this manager command so the local channel goes away and the agent can get another call.
> 
> 
> Diffs
> -----
> 
>   http://svn.asterisk.org/svn/asterisk/trunk/channels/chan_local.c 272083 
> 
> Diff: https://reviewboard.asterisk.org/r/732/diff
> 
> 
> Testing
> -------
> 
> We have the 1.4 version of the patch in production at several sites. The trunk version actually submitted here has only minimal testing: I logged into the manager once and issued the command and observed that it worked as expected.
> 
> 
> Thanks,
> 
> tim_ringenbach
> 
>




More information about the asterisk-dev mailing list