func_config: Change 'Not Found' message from ERROR to DEBUG
authorGeorge Joseph <george.joseph@fairview5.com>
Mon, 18 Aug 2014 20:20:59 +0000 (20:20 +0000)
committerGeorge Joseph <george.joseph@fairview5.com>
Mon, 18 Aug 2014 20:20:59 +0000 (20:20 +0000)
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

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@421341 65c4cc65-6c06-0410-ace0-fbb531ad65f3

funcs/func_config.c

index d45ad2c..1123b0b 100644 (file)
@@ -165,7 +165,7 @@ static int config_function_read(struct ast_channel *chan, const char *cmd, char
        }
 
        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;