[asterisk-bugs] [Asterisk 0018547]: [patch] main/xmldoc.c error message is backwards -- fix that, and remove the sources of the error while we're at it

[Asterisk Bug Tracker]

The following issue has been UPDATED. 
====================================================================== 
https://issues.asterisk.org/view.php?id=18547 
====================================================================== 
Reported By:                jcovert
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   18547
Category:                   General
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:           1.8.1.1 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2010-12-28 19:41 CST
Last Modified:              2010-12-28 19:42 CST
====================================================================== 
Summary:                    [patch] main/xmldoc.c error message is backwards --
fix that, and remove the sources of the error while we're at it
Description: 
There have been a number of reports both in the bug tracker and in various
forums about five messages such as:

[May 12 08:56:54] DEBUG[14460]: xmldoc.c:1720 xmldoc_build_field: Cannot
find variable 'SIPPEER' in tree 'description'

This message should refer to "variable 'description' in tree 'SIPPEER'.

The current code in main/xmldoc.c is:

ast_log(LOG_DEBUG, "Cannot find variable '%s' in tree '%s'\n", name,
var);

and quite obviously should be:

ast_log(LOG_DEBUG, "Cannot find variable '%s' in tree '%s'\n", var,
name);

A patch is supplied to correct this.

HOWEVER, silly answers have appeared in the forums saying things like
"cannot read the xml" or "documentation is missing".  There are exactly
five occurrences of this error, and they are in cases where the
documentation is structured in a way that a "description" section is not
needed.

However, rather than indicating this (as is done appropriately in other
places) with

<description></description>

these five places (in three modules) have

<description />

which the xml parser does not support.

Patches are supplied to correct all of these.

Regards/john
====================================================================== 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-12-28 19:42 jcovert        Summary                  main/xmldoc.c error
message is backwards -- fix that, and remove the sources of the error while
we're at it => [patch] main/xmldoc.c error message is backwards -- fix that, and
remove the sources of the error while we're at it
======================================================================