Summary:	ASTERISK-00865: [patch] Add "Lookup" command for full channel ID to manager interface
Reporter:	iain (iain)	Labels:
Date Opened:	2004-01-17 15:57:27.000-0600	Date Closed:	2011-06-07 14:04:46
Priority:	Major	Regression?	No
Status:	Closed/Complete	Components:	Core/General
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) manager.patch ( 1) Manager.patch
Description:	When setting up call monitoring it's necessary to use the full phone ID to retrieve channel information.  This ID is the phone ID eg as specified in sip.conf augmented with a specific session code generated by *.  Given the phone ID this function retrieves the session code making it easier to write apps that implement call monitoring
Comments:	By: jrollyson (jrollyson) 2004-01-27 18:30:20.000-0600 How is this going to behave when the channel in question has multiple active calls? Also, has this been disclaimed? By: Brian West (bkw918) 2004-01-27 20:29:20.000-0600 each channel has a unique id with it tagged on.  so it shouldn't matter. By: iain (iain) 2004-01-28 14:15:06.000-0600 It has been disclaomed. By: iain (iain) 2004-02-22 04:50:15.000-0600 This isn't in cvs yet.  I'd like it included if poss 'cos I'd like to make a simple web dial and monitor app available for download and it needs this patch. By: James Golovich (jamesgolovich) 2004-02-22 15:40:35.000-0600 The use of the variable name 'Phone' doesn't seem like a good idea.  Why not just use 'Channel'.  Also there is lots of extra \n in the patch that don't belong and horrible alignment of code throughout As well as the terminology not being consistant.  If a 'Phone' variable is not specified the error is 'No phone specified'.  If the channel specified in the 'Phone' variable is not active the error is 'Extension is inactive' By: Olle Johansson (oej) 2004-03-21 17:01:00.000-0600 iain: Any new patches? By: iain (iain) 2004-03-22 04:14:37.000-0600 There will be - just had a few system problems recently that have taken up my time. By: iain (iain) 2004-03-29 03:54:27.000-0600 New patch should fix the variable name and alignment issues. Iain By: twisted (twisted) 2004-04-29 09:25:53 What is the status of this? iain? james? By: iain (iain) 2004-04-30 07:02:22 It's wiaitng to make it into cvs - I'd like it there so I can put some simple code I've developed for web dial and monitoring on the 'net.  It needs this extra command By: twisted (twisted) 2004-04-30 11:28:36 We are going to need a little more information before we consider for commit to CVS.  Usage? does it work with current CVS? By: James Golovich (jamesgolovich) 2004-04-30 12:21:24 I dont think this should go into cvs the way it is currently.  I think this would be better served by an action that can retrieve channel info based on arbitrary information (not exactly arbitrary because all the possible variable would have to be hardcoded in the app). So calling something like: Action: Lookup Callerid: 5551212 Would return any/all channels that have a callerid that contains the substring 5551212 Likewise it could be called with: Action: Lookup Channel: 5551212 to see any channel with 5551212 in it By: Mark Spencer (markster) 2004-05-02 15:01:22 We already have a command that gets you all the channels.  I see no reason at this point not to leave that up to the calling program to parse through.