From 9249dbedc1a778cf02203a1bcb14880c9bdb0b4e Mon Sep 17 00:00:00 2001 From: Benjamin Paassen Date: Wed, 4 Feb 2015 22:40:40 +0100 Subject: allowed actual field descriptor references. --- src/core/model/Domain.cpp | 26 +++++++++++++------------- src/core/model/Domain.hpp | 27 ++++++++++++++++++++++++--- src/plugins/xml/XmlParser.cpp | 2 +- test/plugins/xml/XmlParserTest.cpp | 23 ++++++++++++++++------- 4 files changed, 54 insertions(+), 24 deletions(-) diff --git a/src/core/model/Domain.cpp b/src/core/model/Domain.cpp index 9368005..f8c0779 100644 --- a/src/core/model/Domain.cpp +++ b/src/core/model/Domain.cpp @@ -155,19 +155,6 @@ bool Descriptor::doValidate(Logger &logger) const } else { valid = valid & validateName(logger); } - // check if all FieldDescriptors have this Descriptor as parent. - for (Handle fd : fieldDescriptors) { - if (fd->getParent() != this) { - logger.error(std::string("Descriptor \"") + getName() + - "\" has " - "field \"" + - fd->getName() + - "\" as child but the field does not " - "have the Descriptor as parent.", - *this); - valid = false; - } - } // check the FieldDescriptors themselves. return valid & continueValidationCheckDuplicates(fieldDescriptors, logger); } @@ -248,7 +235,20 @@ bool Descriptor::continuePath(Handle target, return found; } + void Descriptor::addFieldDescriptor(Handle fd) +{ + // only add it if we need to. + if (fieldDescriptors.find(fd) == fieldDescriptors.end()) { + invalidate(); + fieldDescriptors.push_back(fd); + } + if (fd->getParent() == nullptr) { + fd->setParent(this); + } +} + +void Descriptor::moveFieldDescriptor(Handle fd) { // only add it if we need to. if (fieldDescriptors.find(fd) == fieldDescriptors.end()) { diff --git a/src/core/model/Domain.hpp b/src/core/model/Domain.hpp index bef7919..dd0af4c 100644 --- a/src/core/model/Domain.hpp +++ b/src/core/model/Domain.hpp @@ -507,6 +507,27 @@ public: return fieldDescriptors; } + /** + * Adds the given FieldDescriptor to this Descriptor. This also sets the + * parent of the given FieldDescriptor if it is not set yet. + * + * @param fd is a FieldDescriptor. + */ + void addFieldDescriptor(Handle fd); + + /** + * Adds the given FieldDescriptors to this Descriptor. This also sets the + * parent of each given FieldDescriptor if it is not set yet. + * + * @param fds are FieldDescriptors. + */ + void addFieldDescriptors(const std::vector> &fds) + { + for (Handle fd : fds) { + addFieldDescriptor(fd); + } + } + /** * Adds the given FieldDescriptor to this Descriptor. This also sets the * parent of the given FieldDescriptor if it is not set to this Descriptor @@ -514,7 +535,7 @@ public: * * @param fd is a FieldDescriptor. */ - void addFieldDescriptor(Handle fd); + void moveFieldDescriptor(Handle fd); /** * Adds the given FieldDescriptors to this Descriptor. This also sets the @@ -523,10 +544,10 @@ public: * * @param fds are FieldDescriptors. */ - void addFieldDescriptors(const std::vector> &fds) + void moveFieldDescriptors(const std::vector> &fds) { for (Handle fd : fds) { - addFieldDescriptor(fd); + moveFieldDescriptor(fd); } } diff --git a/src/plugins/xml/XmlParser.cpp b/src/plugins/xml/XmlParser.cpp index c288f40..2d62c11 100644 --- a/src/plugins/xml/XmlParser.cpp +++ b/src/plugins/xml/XmlParser.cpp @@ -411,7 +411,7 @@ public: name, parent, logger(), [](Handle field, Handle parent, Logger &logger) { if (field != nullptr) { - parent.cast()->copyFieldDescriptor( + parent.cast()->addFieldDescriptor( field.cast()); } }); diff --git a/test/plugins/xml/XmlParserTest.cpp b/test/plugins/xml/XmlParserTest.cpp index 6619199..0512fd0 100644 --- a/test/plugins/xml/XmlParserTest.cpp +++ b/test/plugins/xml/XmlParserTest.cpp @@ -156,14 +156,25 @@ static void checkFieldDescriptor( } static void checkFieldDescriptor( - Handle desc, NodeVector children, + Handle desc, Handle parent, + NodeVector children, const std::string &name = DEFAULT_FIELD_NAME, FieldDescriptor::FieldType type = FieldDescriptor::FieldType::TREE, Handle primitiveType = nullptr, bool optional = false) { auto res = desc->resolve(RttiTypes::FieldDescriptor, name); ASSERT_EQ(1, res.size()); - checkFieldDescriptor(res[0].node, name, desc, children, type, primitiveType, + checkFieldDescriptor(res[0].node, name, parent, children, type, + primitiveType, optional); +} + +static void checkFieldDescriptor( + Handle desc, NodeVector children, + const std::string &name = DEFAULT_FIELD_NAME, + FieldDescriptor::FieldType type = FieldDescriptor::FieldType::TREE, + Handle primitiveType = nullptr, bool optional = false) +{ + checkFieldDescriptor(desc, desc, children, name, type, primitiveType, optional); } @@ -216,8 +227,8 @@ TEST(XmlParser, domainParsing) Rooted heading = checkStructuredClass("heading", "heading", headings_domain, single, nullptr, nullptr, true, false); - // which should allow text content - checkFieldDescriptor(heading, {text}); + // which should be a reference to the paragraph descriptor. + checkFieldDescriptor(heading, paragraph, {text}); // and each struct in the book domain (except for text) should have a // heading field now. checkFieldDescriptor(book, {heading}, "heading", @@ -261,9 +272,7 @@ TEST(XmlParser, domainParsing) // paragraph should have comment as child now as well. checkFieldDescriptor(paragraph, {text, comment}); // as should heading, because it references the paragraph default field. - // TODO: This does not work as of now, because in fact fields get copied, - // not referenced. Should we reference fields, though? - //checkFieldDescriptor(heading, {text, comment}); + checkFieldDescriptor(heading, paragraph, {text, comment}); } } -- cgit v1.2.3