Summary:	ASTERISK-12076: [patch] warning about non-existent dialplan context is garbled, sometimes spurious
Reporter:	David Chappell (chappell)	Labels:
Date Opened:	2008-05-23 15:51:25	Date Closed:	2008-06-04 08:28:05
Priority:	Minor	Regression?	No
Status:	Closed/Complete	Components:	PBX/pbx_config
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) dialplan_reload_2.diff ( 1) dialplan_reload.diff
Description:	Every time the dialplan is loaded, the following warning is printed: WARNING[7633]: pbx.c:6322 ast_context_verify_includes: Context '_______________' tries includes nonexistent context 'featuremap' Note the phrase "tries includes".  Also note that in this case the warning is spurious since featuremap is a pseudo context defined in features.conf. Patch fixes wording and suppresses warning for featuremap. However, suppressing the warning creates a new problem.  On some servers  "dialplan reload" takes up to a minute to run after the prompt has returned.  Currently the spurious warning, since it occurs at the very end of the process, servers as an indicate that the command has finished its work.  Therefore this patch fixes "dialplan reload" so that it prints "Dialplan reloaded." on the console when it finishes.
Comments:	By: Joshua C. Colp (jcolp) 2008-05-27 07:48:17 There is no pseudo context featuremap that is created in the dialplan unless you include that file in your extensions.conf. It just ends up getting parsed by res_features and saved in some internal data structures. By: David Chappell (chappell) 2008-05-27 11:58:48 It has been reported that one must put: include => featuremap at some appropriate place in the dialplan before the codes defined in features.conf will work.  I cannot find this statement in any formal documentation, but it is mentioned on several forums: http://www.voipinfo.org/wiki/index.php?page=PBX+CallTransferhttp://www.voip-info.org/wiki/index.php?page=Asterisk+config+features.conf http://www.voip-info.org/wiki/index.php?page=Asterisk+config+features.conf In my bug report I called featuremap a "pseudo context" because I have been informed that it must be included as if it were a context even though it does not actually exist. The note posted by administrator suggests that one must also do: #include features.conf in extensions.conf.  Is that what was meant? If so, I will be happy to create a new patch which removes the suppression of the message about the non-existence of featuremap and fix the documentation instead. By: Joshua C. Colp (jcolp) 2008-05-27 12:01:23 That documentation is incorrect, and the example config file for features.conf does not mention needing to include it at all. By: David Chappell (chappell) 2008-06-03 10:57:17 Interesting.  Earlier my tests seemed to confirm the report that there is a featuremap pseudo context which must be included.  However it now appears that the differences were actually an issue of whether the disconnect sequence was detected or not. To be doubly sure, I grepped the entire source for "featuremap".  I found nothing to indicate that features.conf creates anything at all called "featuremap". So, I have corrected my patch removing the code to suppress the warning about a missing featuremap context.  The patch now corrects the wording of the message and adds a console message indicating that "dialplan reload" has finished. By: Digium Subversion (svnbot) 2008-06-04 08:24:46 Repository: asterisk Revision: 120282 U   branches/1.4/main/pbx.c U   branches/1.4/pbx/pbx_config.c ------------------------------------------------------------------------ r120282 | file | 2008-06-04 08:24:39 -0500 (Wed, 04 Jun 2008) | 6 lines Fix a log message and add a message for when the dialplan is done reloading. (closes issue ASTERISK-12076) Reported by: chappell Patches:      dialplan_reload_2.diff uploaded by chappell (license 8) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=120282 By: Digium Subversion (svnbot) 2008-06-04 08:27:29 Repository: asterisk Revision: 120283 _U  trunk/ U   trunk/pbx/pbx_config.c ------------------------------------------------------------------------ r120283 | file | 2008-06-04 08:27:28 -0500 (Wed, 04 Jun 2008) | 14 lines Merged revisions 120282 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r120282 | file | 2008-06-04 10:31:09 -0300 (Wed, 04 Jun 2008) | 6 lines Fix a log message and add a message for when the dialplan is done reloading. (closes issue ASTERISK-12076) Reported by: chappell Patches:      dialplan_reload_2.diff uploaded by chappell (license 8) ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=120283 By: Digium Subversion (svnbot) 2008-06-04 08:28:05 Repository: asterisk Revision: 120284 _U  branches/1.6.0/ U   branches/1.6.0/pbx/pbx_config.c ------------------------------------------------------------------------ r120284 | file | 2008-06-04 08:28:05 -0500 (Wed, 04 Jun 2008) | 22 lines Merged revisions 120283 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r120283 | file | 2008-06-04 10:33:59 -0300 (Wed, 04 Jun 2008) | 14 lines Merged revisions 120282 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r120282 | file | 2008-06-04 10:31:09 -0300 (Wed, 04 Jun 2008) | 6 lines Fix a log message and add a message for when the dialplan is done reloading. (closes issue ASTERISK-12076) Reported by: chappell Patches:      dialplan_reload_2.diff uploaded by chappell (license 8) ........ ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=120284