Summary:	ASTERISK-09711: CDR channel column doesn't change to Agent/agent_id when using AgentMonitorOutgoing(c) with Originate script
Reporter:	KR Torio (krtorio)	Labels:
Date Opened:	2007-06-19 22:42:14	Date Closed:	2007-07-09 10:01:37
Priority:	Minor	Regression?	No
Status:	Closed/Complete	Components:	Core/General
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:
Description:	We have a click-to-dial script which uses asterisk manager outgoing command to start calls. Our calls pass through AgentMonitorOutgoing(c) application. This changes the channel column to the cdr row to Agent/agent_id. updatecdr=yes in agents.conf Works in version 1.2.x also 1.4.2 . Produced the problem in 1.4.4 and 1.4.5 Here's the dialplan and manager script to reproduce the problem: ======== Dialplan ======== ; Note: requires agent to login via AgentCallbackLogin exten => _NXXNXXXXXX,1,AgentMonitorOutgoing(c) exten => _NXXNXXXXXX,2,Dial(SIP/provider/${EXTEN}) ======================= Asterisk manager script ======================= Action: Originate Channel: Agent/<insert agent id here> Exten: <insert number to dial here> Context: <insert outgoing context here> Priority: 1
Comments:	By: KR Torio (krtorio) 2007-06-20 01:15:02 Also tried manual dialing in version 1.4.5 with AgentMonitorOutgoing(c), channel column still won't change to Agent/agent_id . ======== Dialplan ======== ; Note: requires agent to login via AgentCallbackLogin exten => _NXXNXXXXXX,1,AgentMonitorOutgoing(c) exten => _NXXNXXXXXX,2,Dial(SIP/provider/${EXTEN}) Found out this bug is similar to http://bugs.digium.com/view.php?id=9573 . It seems that all asterisk code that tries to alter the value of the channel column  to Agent/agent_id no longer works. Could this be caused by making some of the CDR variables (including channel) read-only? By: Joshua C. Colp (jcolp) 2007-06-25 09:53:51 I have located the issue, but due to the changed CDR system really have no idea how to change it so I've emailed murf (who is the one who changed the CDR system) telling him what the problem is and seeing what we can change. The actual issue is that the changed CDR system does some merging of CDRs to give an end result, and in some scenarios this yields the altered CDR with an Agent/agent_id channel name discarded and not used. By: Steve Murphy (murf) 2007-07-05 18:08:47 see patch on 9573; apply it, give it a spin, see if it helps, please! By: Digium Subversion (svnbot) 2007-07-08 21:56:36 Repository: asterisk Revision: 73980 ------------------------------------------------------------------------ r73980 | file | 2007-07-08 21:56:35 -0500 (Sun, 08 Jul 2007) | 2 lines Give Agent channel names priority when doing CDR merging. (issue ASTERISK-9711 reported by krtorio) ------------------------------------------------------------------------ By: Digium Subversion (svnbot) 2007-07-08 21:57:54 Repository: asterisk Revision: 73983 ------------------------------------------------------------------------ r73983 | file | 2007-07-08 21:57:53 -0500 (Sun, 08 Jul 2007) | 10 lines Merged revisions 73980 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r73980 | file | 2007-07-09 00:13:19 -0300 (Mon, 09 Jul 2007) | 2 lines Give Agent channel names priority when doing CDR merging. (issue ASTERISK-9711 reported by krtorio) ........ ------------------------------------------------------------------------ By: Digium Subversion (svnbot) 2007-07-09 10:01:37 Repository: asterisk Revision: 74071 ------------------------------------------------------------------------ r74071 | russell | 2007-07-09 10:01:35 -0500 (Mon, 09 Jul 2007) | 92 lines Merged revisions 73964,73983,73994,74024-74025,74044,74046,74048 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r73964 | file | 2007-07-08 21:29:00 -0500 (Sun, 08 Jul 2007) | 2 lines Use linkedlist macros for parking. ................ r73983 | file | 2007-07-08 22:14:34 -0500 (Sun, 08 Jul 2007) | 10 lines Merged revisions 73980 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r73980 | file | 2007-07-09 00:13:19 -0300 (Mon, 09 Jul 2007) | 2 lines Give Agent channel names priority when doing CDR merging. (issue ASTERISK-9711 reported by krtorio) ........ ................ r73994 | tilghman | 2007-07-08 23:09:16 -0500 (Sun, 08 Jul 2007) | 10 lines Merged revisions 73985 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r73985 | tilghman | 2007-07-08 23:03:20 -0500 (Sun, 08 Jul 2007) | 2 lines Doxygen formatting fixes; fixes errors while 'make progdocs'.  (Closes issue ASTERISK-9794) ........ ................ r74024 | oej | 2007-07-09 03:27:37 -0500 (Mon, 09 Jul 2007) | 7 lines Implementation of a feature that will disable "missed calls" counters on SIP phones. If the call is answered by another phone, other phones won't display the call as "missed". You can also add an option to the dial command so that you can have a "followme" scenario and not count the calls as "missed" when you cancel the call. Thanks to Ramon and Frank for feedback on this feature. ................ r74025 | oej | 2007-07-09 03:30:04 -0500 (Mon, 09 Jul 2007) | 2 lines Update with new features ................ r74044 | file | 2007-07-09 09:38:19 -0500 (Mon, 09 Jul 2007) | 10 lines Merged revisions 74043 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r74043 | file | 2007-07-09 11:34:33 -0300 (Mon, 09 Jul 2007) | 2 lines Use AC_CHECK_HEADER to check for ptlib/openh323 to allow for cross compiling. (issue ASTERISK-9392 reported by zandbelt) ........ ................ r74046 | file | 2007-07-09 09:50:04 -0500 (Mon, 09 Jul 2007) | 10 lines Merged revisions 74045 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r74045 | file | 2007-07-09 11:49:05 -0300 (Mon, 09 Jul 2007) | 2 lines Few minor thread synchronization tweaks. (issue ASTERISK-9777 reported by gzero) ........ ................ r74048 | mmichelson | 2007-07-09 09:58:42 -0500 (Mon, 09 Jul 2007) | 12 lines Merged revisions 74047 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r74047 | mmichelson | 2007-07-09 09:57:41 -0500 (Mon, 09 Jul 2007) | 4 lines Fixed a logic error in leave_voicemail. Pass the mailbox instead of the context to inbox_count when the context is "default." (closes issue ASTERISK-9821, reported by yannj, repaired by me) ........ ................ ------------------------------------------------------------------------