Add invalid object to ValidationException

Why don't you make available what you've got so far in form of a patch, and I'll have a look ?

Can I actually assume that this discussion makes sense in the context of 'multivalued' element definitions, i..e where there are collections of some type used in the generated classes ?

I don't have a patch yet, but I will work on one this weekend.

The problem is that we have a root object, that has many child objects, which contain child objects, etc. Some are multi-valued elements, others are single. So when you call the validate method and the ValidationException is thrown, you could traverse the validation exceptions' invalid objects to lead you to the exact object with the problem.

Dave, I do have an early patch ready to be posted here for review that improves the location processing upon creation of a ValidationException. I will make this available in a few hours through this issue.

Initial patch for preview. Please note that this improves the location processing, but does not deal with collections in the way that it e.g. emits locations similar to /test/first[2] to highlight that the second <first> sub-element of /test has failed validation.

Final patch for review.

For the XML schema shown below ....

<?xml version="1.0" encoding="UTF-8" ?>
<xs:schema xmlns:xs="http://www.w3.org/2001/XMLSchema">

  <xs:element name="test">
    <xs:complexType>
      <xs:sequence>
        <xs:element name="first" type="extraShort" maxOccurs="unbounded"/>
        <xs:element name="second" type="extraShort" />
      </xs:sequence>
      <xs:attribute name="third" type="extraShort" use="optional"/>
    </xs:complexType>
  </xs:element>
  
  <xs:simpleType name="extraShort">
    <xs:restriction base="xs:short">
        <xs:maxInclusive value="10" />
    </xs:restriction>
  </xs:simpleType>

</xs:schema>

the following XPATH fragments will be generated, depending on where the error is located:

/test/second
/test/@third
/test/first[3]

In other words, I have added some new logic to deal with the position within a collection.

Patch committed as is. Please feel free to re-open this issue should any issues arise, or create a follow-up issue and link it to this very instance.

Thanks Werner. I appreciate the quick turn around. X-Paths look good. I'll let you know if I have any problems related to it.

thanks

dave