[Home]

Summary:ASTERISK-00214: New Pending agents are created with last config values read from the agent.conf
Reporter:tclark (tclark)Labels:
Date Opened:2003-09-03 18:51:31Date Closed:2004-09-25 02:14:13
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_queue
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:
Description:A side effect is that if there are a mix of agents that use ackcall=yes or ackcall=always or ackcall=no, and if the last
ackcall value is set to ackcall=no, then any agent that requires a '#' to acknowledge the call will hear 2 beeps
if they belong to an ':' agent group and the call
is bridged using a Pending Place holder agent.
OR
if the last
ackcall value is set to ackcall=always, then any agent that does not require a '#' to acknowledge the call will NOT
hear any beep if they belong to an ':' agent group and the call
is bridged using a Pending Place holder agent


****** ADDITIONAL INFORMATION ******

Steps to reproduce
agent.conf
group=1
ackcall=no
agent => 1,,Agent 1
group=2
ackcall=always
agent => 2,,Agent 2
queues.conf

[q]
strategy = ringall
timeout = 30
retry = 1
maxlen = 0
member => Agent/:1
member => Agent/:2

1) exten => *1,1,AgentLogin(1)
2) put 2 calls in the q
3) the first call will bridge as a non-pending agent
4) press * to hangup the call

The next call will be bridged w/o a beep

if the last ackcall is a ackcall=no,
then the ackcall=always agent will hear 2 beeps


Comments:By: tclark (tclark) 2003-09-03 19:35:58

This patch appears to resolve this

--- chan_agent.c        2003-09-03 17:24:58.000000000 -0700
+++ chan_agent.new      2003-09-03 17:24:21.000000000 -0700
@@ -779,6 +779,7 @@
                       chan = agent_new(newlyavailable, AST_STATE_DOWN);
                       parent = p->owner;
                       p->abouttograb = 1;
+                       p->ackcall = newlyavailable->ackcall;
                       ast_mutex_unlock(&p->lock);
                       break;
               }

By: Mark Spencer (markster) 2003-09-08 14:41:46

Fixed in CVS