[Home]

Summary:ASTERISK-09802: [patch] manager_event tweak for the AgentCalled event
Reporter:Paul W (kwakwaversal)Labels:
Date Opened:2007-07-04 07:53:19Date Closed:2007-07-09 11:19:01
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_queue
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) app_queue_manager_2.patch
( 1) app_queue_manager.patch
Description:Firstly this is my 1st proper submission and I've read everything I could to make sure its ok.  Apologies if not.

This is basically a tweak to allow capturing the Queue from the manager interface for the AgentCalled queue event.  

Every other event which is triggered using the manager_event() function contains the "Queue" AMI header but for some reason AgentCalled does not.  I've supplied a patch to rectify this and added an extra header which lets the manager know which Agent channel is being called. This might not actually be necessary for some people but I used it so I could match it against the AgentConnect event.  I stole the "DestinationChannel" AMI header for this because I didn't want to create a new one.

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

There's another possible bug I've found with app_queue which is why I added the "DestinationChannel" header but I'll provide more information when I'm sure.
Comments:By: Russell Bryant (russell) 2007-07-06 11:20:06

Since your license agreement was not approved, I'm suspending this issue.  Feel free to reopen this issue if you are able to resolve the issues with your license with our legal team.  Thanks!

By: Paul W (kwakwaversal) 2007-07-09 08:32:53

I'd like to re-open this issue, my license agreement has been accepted now.

By: Digium Subversion (svnbot) 2007-07-09 11:19:00

Repository: asterisk
Revision: 74084

------------------------------------------------------------------------
r74084 | russell | 2007-07-09 11:18:59 -0500 (Mon, 09 Jul 2007) | 4 lines

Add Queue and DestinationChannel headers to the AgentCalled manager event
to be more like the rest of the events in this module.
(closes issue ASTERISK-9802, patch by kwakwaversal)

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