[asterisk-bugs] [Asterisk 0014071]: [patch] Improve debugging output for specific context inclusion failure case in pbx_config

[Asterisk Bug Tracker]

The following issue has been ASSIGNED. 
====================================================================== 
http://bugs.digium.com/view.php?id=14071 
====================================================================== 
Reported By:                kshumard
Assigned To:                file
====================================================================== 
Project:                    Asterisk
Issue ID:                   14071
Category:                   PBX/pbx_config
Reproducibility:            always
Severity:                   text
Priority:                   normal
Status:                     assigned
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 163761 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2008-12-12 16:04 CST
Last Modified:              2008-12-16 10:50 CST
====================================================================== 
Summary:                    [patch] Improve debugging output for specific
context inclusion failure case in pbx_config
Description: 
The dialplan below results in a generic, "WARNING[18233]: pbx_config.c:1560
pbx_load_config: Unable to include context 'local' in context
'longdistance'" message, output by ast_context_add_include2(). This patch
copies the "switch (errno)" block out of handle_cli_dialplan_add_include()
that follows the call to ast_context_add_include(). This makes the warning
much more descriptive, i.e. "WARNING[20865]: pbx_config.c:1570
pbx_load_config: Context 'local' already included in 'longdistance'
context".

[longdistance]
include => local

[local]
exten => s,1,Noop

[longdistance]
exten => s,1,Noop
include => local

While the example seems somewhat contrived, a student in an Asterisk
Advanced training class encountered it. Every improvement to documentation
helps, right? : )
====================================================================== 

---------------------------------------------------------------------- 
 (0096453) svnbot (reporter) - 2008-12-16 10:50
 http://bugs.digium.com/view.php?id=14071#c96453 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 164733

U   trunk/pbx/pbx_config.c

------------------------------------------------------------------------
r164733 | file | 2008-12-16 10:50:46 -0600 (Tue, 16 Dec 2008) | 6 lines

Be more detailed about why the include did not get included.
(closes issue http://bugs.digium.com/view.php?id=14071)
Reported by: kshumard
Patches:
      pbx_config.patch.improvederroroutput.txt uploaded by kshumard
(license 92)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=164733 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-12-16 10:50 svnbot         Checkin                                      
2008-12-16 10:50 svnbot         Note Added: 0096453                          
2008-12-16 10:50 svnbot         Status                   new => assigned     
2008-12-16 10:50 svnbot         Assigned To               => file            
======================================================================