[Home]

Summary:ASTERISK-12658: [patch] Attended transfers do not call update_queue until after transfered call ends
Reporter:Bradley Watkins (marquis)Labels:
Date Opened:2008-08-28 17:15:19Date Closed:2008-10-14 17:30:19
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_queue
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) app_queue.c.transfer.patch
Description:The member struct's lastcall and calls elements are not updated until after the transferred call is completed.  This has the effect of breaking leastrecent and fewestcalls queueing strategies.  In particular, the agent that transferred the call will receive multiple calls back-to-back while other agents are free.
Comments:By: Bradley Watkins (marquis) 2008-10-14 15:51:00

I hate being a squeaky wheel, but this has been in the bug tracker for a month and a half now with no action.

Is there anything I can do to move this forward?
By: Mark Michelson (mmichelson) 2008-10-14 16:00:43

Actually, thanks for bringing this up. This had slipped from my view, and now your comment has helped to bump it back up. I'll take a further look at this today.
By: Mark Michelson (mmichelson) 2008-10-14 17:24:35

The patch seems fine by me. I'll get it committed. Sorry it took so long to get around to this.
By: Bradley Watkins (marquis) 2008-10-14 17:29:44

Thanks, and no problem.  I know everyone has plenty of things to keep them occupied. :)
By: Digium Subversion (svnbot) 2008-10-14 17:30:17

Repository: asterisk
Revision: 149200

U   branches/1.4/apps/app_queue.c

------------------------------------------------------------------------
r149200 | mmichelson | 2008-10-14 17:30:17 -0500 (Tue, 14 Oct 2008) | 12 lines

Update the queue with the correct number of calls and
whether the call was completed within the service level
when a transfer takes place. This way, we do not "break"
the leastrecent and fewestcalls strategies by not logging
a call until after the transferred call has ended.

(closes issue ASTERISK-12658)
Reported by: Marquis
Patches:
     app_queue.c.transfer.patch uploaded by Marquis (license 32)


------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=149200