diff options
author | Martin Smith <martin.smith@nokia.com> | 2012-05-18 10:18:57 +0200 |
---|---|---|
committer | Qt by Nokia <qt-info@nokia.com> | 2012-05-18 15:22:39 +0200 |
commit | f3a0422acb6d92be26e30c604b7092a8bfd52b7b (patch) | |
tree | fae13391d7a85f08765f979ec4b4e1cf0aa63ca9 /src/tools/qdoc/node.cpp | |
parent | 1c7421ad14f9321422cdfeede3902552a34ccf3b (diff) |
qdoc: Report multiple QML property docss
Documentation authors sometimes make the mistake of
documenting a QML property more than once. Here, we
refer to cases where a C++ class is documented in a
.cpp file as a QML type. In this context one QML
property might be documented in two qdoc comments,
because the author of the second comment does not
search the file for an existing qdoc comment for
the property before adding the second one. When DITA
XML is generated for this case, the QML type element
will contain two <qmlproperty> elements with identical
id attributes, which is invalid XML. id attributes
must be unique within an XML document.
qdoc now reports an error for this case, indicating
that the QMLN property has been documented multiple
times.
This problem can't occur when documenting QML in a
.qml file because in .qml files, each comment must
appear directly above the thing it applies to.
Change-Id: I3a22650a58371fbda2ac7a5429fc036f41750423
Reviewed-by: Martin Smith <martin.smith@nokia.com>
Diffstat (limited to 'src/tools/qdoc/node.cpp')
-rw-r--r-- | src/tools/qdoc/node.cpp | 41 |
1 files changed, 41 insertions, 0 deletions
diff --git a/src/tools/qdoc/node.cpp b/src/tools/qdoc/node.cpp index bb70200e7d..fc6dc4e35c 100644 --- a/src/tools/qdoc/node.cpp +++ b/src/tools/qdoc/node.cpp @@ -1594,6 +1594,25 @@ FakeNode* FakeNode::lookupQmlModuleNode(Tree* tree, const ArgLocPair& arg) } /*! + Returns true if this QML type or property group contains a + property named \a name. + */ +bool FakeNode::hasProperty(const QString& name) const +{ + foreach (Node* child, childNodes()) { + if (child->type() == Node::Fake && child->subType() == Node::QmlPropertyGroup) { + if (child->hasProperty(name)) + return true; + } + else if (child->type() == Node::QmlProperty) { + if (child->hasProperty(name)) + return true; + } + } + return false; +} + +/*! The constructor calls the FakeNode constructor with \a parent, \a name, and Node::Example. */ @@ -1614,6 +1633,7 @@ ExampleNode::ExampleNode(InnerNode* parent, const QString& name) EnumNode::EnumNode(InnerNode *parent, const QString& name) : LeafNode(Enum, parent, name), ft(0) { + // nothing. } /*! @@ -2362,6 +2382,10 @@ bool QmlPropertyNode::isWritable(Tree* tree) return true; } +/*! + Returns a pointer this QML property's corresponding C++ + property, if it has one. + */ PropertyNode* QmlPropertyNode::correspondingProperty(Tree *tree) { PropertyNode* pn; @@ -2405,6 +2429,23 @@ PropertyNode* QmlPropertyNode::correspondingProperty(Tree *tree) return 0; } +/*! + Returns true if this QML type or property group contains a + property named \a name. + */ +bool QmlPropertyNode::hasProperty(const QString& n) const +{ + if (name() == n) + return true; + foreach (Node* child, qmlPropNodes()) { + if (child->type() == Node::QmlProperty) { + if (child->name() == n) + return true; + } + } + return false; +} + /*! \class NameCollisionNode An instance of this node is inserted in the tree |