[asterisk-commits] gtjoseph: trunk r421341 - in /trunk: ./ funcs/func_config.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Mon Aug 18 15:21:02 CDT 2014


Author: gtjoseph
Date: Mon Aug 18 15:20:59 2014
New Revision: 421341

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=421341
Log:
func_config: Change 'Not Found' message from ERROR to DEBUG

When you call the CONFIG dialplan function with the name of a variable that
doesn't exist in the target context you get an ERROR.  This does nothing but
clutter up the logs with messages that may be perfectly acceptable.  Just
because a variable wasn't in the context doesn't mean it's an error.  Maybei
t's optional or just needs to be defaulted or ignored.

This patch changes the log level from ERROR to DEBUG.  If a dialplan developer
wants to debug their dialplan they still canby setting the console debug level 
as needed.

Tested by: George Joseph
Review: https://reviewboard.asterisk.org/r/3919/
........

Merged revisions 421327 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........

Merged revisions 421328 from http://svn.asterisk.org/svn/asterisk/branches/11
........

Merged revisions 421329 from http://svn.asterisk.org/svn/asterisk/branches/12
........

Merged revisions 421337 from http://svn.asterisk.org/svn/asterisk/branches/13

Modified:
    trunk/   (props changed)
    trunk/funcs/func_config.c

Propchange: trunk/
------------------------------------------------------------------------------
Binary property 'branch-13-merged' - no diff available.

Modified: trunk/funcs/func_config.c
URL: http://svnview.digium.com/svn/asterisk/trunk/funcs/func_config.c?view=diff&rev=421341&r1=421340&r2=421341
==============================================================================
--- trunk/funcs/func_config.c (original)
+++ trunk/funcs/func_config.c Mon Aug 18 15:20:59 2014
@@ -165,7 +165,7 @@
 	}
 
 	if (!(val = ast_variable_retrieve(cfg, args.category, args.variable))) {
-		ast_log(LOG_ERROR, "'%s' not found in [%s] of '%s'\n", args.variable, 
+		ast_debug(1, "'%s' not found in [%s] of '%s'\n", args.variable,
 			args.category, args.filename);
 		AST_RWLIST_UNLOCK(&configs);
 		return -1;




More information about the asterisk-commits mailing list