Summary:	ASTERISK-11591: res_agi.c can still tell you to use DeadAGI for hung up channels
Reporter:	Frank Waller (explidous)	Labels:
Date Opened:	2008-03-06 12:37:33.000-0600	Date Closed:	2008-03-06 14:36:00.000-0600
Priority:	Minor	Regression?	No
Status:	Closed/Complete	Components:	Resources/res_agi
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) res_agi_12161.patch
Description:	When I went to see what changes were made to with regards to issue ASTERISK-11491 I noticed the following error message: ast_log(LOG_ERROR, "If you want to run AGI on hungup channels you should use DeadAGI!\n"); in the eagi_exec function at line 2947. This could cause some confusion. ****** ADDITIONAL INFORMATION ****** This is actually effecting 1.6.0-beta5, but that is not a selectable Asterisk Version yet in the bug tracker.
Comments:	By: Donny Kavanagh (donnyk) 2008-03-06 12:46:02.000-0600 Sorry for the oversight, attached is a patch to clarify the error message. By: Russell Bryant (russell) 2008-03-06 13:02:42.000-0600 assigned back to juggie, as he can now commit the patch himself :) By: Digium Subversion (svnbot) 2008-03-06 13:28:06.000-0600 Repository: asterisk Revision: 106399 U   trunk/res/res_agi.c ------------------------------------------------------------------------ r106399 | juggie | 2008-03-06 13:28:05 -0600 (Thu, 06 Mar 2008) | 9 lines trivial fix for an agi error when attempting to use EAGI on a dead/hungup channel, we now print an error that makes sense given our removal of deadagi as an actual application. (closes issue ASTERISK-11591) Reported by: explidous Patches:      res_agi_12161.patch uploaded by juggie (license 24) Tested by: juggie ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=106399 By: Digium Subversion (svnbot) 2008-03-06 14:36:00.000-0600 Repository: asterisk Revision: 106433 _U  branches/1.6.0/ U   branches/1.6.0/res/res_agi.c ------------------------------------------------------------------------ r106433 | juggie | 2008-03-06 14:35:57 -0600 (Thu, 06 Mar 2008) | 17 lines Merged revisions 106399 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ........ r106399 | juggie | 2008-03-06 14:31:50 -0500 (Thu, 06 Mar 2008) | 9 lines trivial fix for an agi error when attempting to use EAGI on a dead/hungup channel, we now print an error that makes sense given our removal of deadagi as an actual application. (closes issue ASTERISK-11591) Reported by: explidous Patches:      res_agi_12161.patch uploaded by juggie (license 24) Tested by: juggie ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=106433