Summary:	ASTERISK-04395: [patch] Optional exit context and extension matching for app_background
Reporter:	Michael Jerris (mikej)	Labels:
Date Opened:	2005-06-10 22:04:25	Date Closed:	2008-01-15 15:40:52.000-0600
Priority:	Major	Regression?	No
Status:	Closed/Complete	Components:	Core/General
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) 4511rev4.patch.txt ( 1) backgroundoptionaldigits.patch.txt
Description:	added option for a list of digits that app_background will break on.
Comments:	By: Michael Jerris (mikej) 2005-06-10 22:05:26 Totally untested.  Somone please give this a try and make sure it does not make anything really bad happen please. By: Michael Jerris (mikej) 2005-06-12 21:09:19 Okay, finally got this tested, it is working well as expected. By: Mark Spencer (markster) 2005-06-18 13:26:36 How about just making it an option which doesn't permit you to enter a digit which isn't part of a valid extension, wouldn't that be good enough? By: Michael Jerris (mikej) 2005-06-19 22:47:41 Attached as background.matchexten.txt is a quick hack at the idea of optionally matching to a valid single digit exten and only doing doing stoping the stream if a valid extension is hit.  Also, as a freebee I threw in specifying an exit context for background.  I think that allowing the extension match behavior for anything more than single digit extensions would be very counter intuitive for a user, so I did not add any code to address collecting a list of digits.  All code and formating review aside, are the ideas in this more to what you were thinking?  Also, as a seperate bug after this, I will go in and consolidate all of the waitstream (and probably some of the wait) functions into a single function, as they are largely duplicate code. By: Michael Jerris (mikej) 2005-06-20 22:22:53 Ok, spoke to kpfleming about this.  I will work on creating and testing some patches and will post when ready. By: Michael Jerris (mikej) 2005-06-28 23:12:53 updated 4511rev0.patch.txt.  Still need to test.  If anyone wants to give it a try, have a ball.  Why do we have seperate apps for background and playback when they are almost identical? By: Michael Jerris (mikej) 2005-06-29 23:11:24 The patch came out a little werid in file.c because the function above has the same end as the new functiton.  Fixed all the compiler errors.  Working on testing now. By: Michael Jerris (mikej) 2005-06-30 00:06:53 ok, tested and working.. somone code review me babay... By: Kevin P. Fleming (kpfleming) 2005-07-11 21:26:11 Committed to CVS HEAD, thanks! By: Digium Subversion (svnbot) 2008-01-15 15:40:52.000-0600 Repository: asterisk Revision: 6093 U   trunk/file.c U   trunk/include/asterisk/file.h U   trunk/pbx.c ------------------------------------------------------------------------ r6093 | kpfleming | 2008-01-15 15:40:52 -0600 (Tue, 15 Jan 2008) | 2 lines add 'exit context' and 'only stop on match' options to Background app (bug ASTERISK-4395) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=6093