[asterisk-commits] gtjoseph: branch 1.8 r421327 - /branches/1.8/funcs/func_config.c

[SVN commits to the Asterisk project]

Author: gtjoseph
Date: Mon Aug 18 15:14:32 2014
New Revision: 421327

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=421327
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/

Modified:
    branches/1.8/funcs/func_config.c

Modified: branches/1.8/funcs/func_config.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.8/funcs/func_config.c?view=diff&rev=421327&r1=421326&r2=421327
==============================================================================
--- branches/1.8/funcs/func_config.c (original)
+++ branches/1.8/funcs/func_config.c Mon Aug 18 15:14:32 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;