asterisk users - Jul 2004 - Patch to test: Never say goodbye to an agent :-)

http://bugs.digium.com/bug_view_page.php?bug_id=0001693

This patch adds a lot of options for AgentLogin/AgentCallbackLogin
Please test and respond in the bug tracker!

/O


-------------------------------------------------------------------------------------
"This patch adds quite a few new features into __login_exec () of
channels/chan_agent.c for AgentLogin() and AgentCallbackLogin(). Only one
option ('s') existed previously.

Syntax/Documentation:

AgentLogin([AgentNo][|options]):
Asks the agent to login to the system. Always returns -1. While
logged in, the agent can receive calls and will hear a 'beep'
when a new call comes in. The agent can dump the call by pressing
the star key.
The option string may contain zero or more of the following characters:
'a-f' - Max Login Attemps (1-6)
's' -- silent login - do not announce the login ok segment


AgentCallbackLogin([AgentNo][|[options][exten]@context]):
Asks the agent to login to the system with callback.
The agent's callback extension is called (optionally with the specified
context.
Options: (defaults are 3 login attempts, play announcement, hangup, and return
-1)
a-f - Max Login Attemps (1-6)
g - play goodbye only on login failure
G - never play goodbye
h - hangup only on login failure (returns -1 on fail or 0 on success)
H - never hangug (returns 0)
l- hangup when agent logs off overriding option h/H (returns -1)
L - hangup when agent logs off overriding option h/H and g/G to play goodbye
(returns -1)
s - do not play the announcement
v - Set Local Variables AGENTNUMBER, AGENTPASS, AGENTEXTEN on successful login
V - Same as v - but set the variables globally"
-------------------------------------------------------------------------------------

Hi!
> "This patch adds quite a few new features into __login_exec () of
channels/chan_agent.c for AgentLogin() and AgentCallbackLogin(). Only one
> option ('s') existed previously.
> 
> AgentLogin([AgentNo][|options]):
[...]
> 
> AgentCallbackLogin([AgentNo][|[options][exten]@context]):
Just a word of warning: The patch's implementation now looks *very* 
different from the originally posted syntax that was posted here, so *do* 
read the discussion on the bugtracker.

Cheers, Philipp