[ https://issues.apache.org/jira/browse/XMLBEANS-347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12540981 ]
Jacob Danner commented on XMLBEANS-347: --------------------------------------- the method is used internally in three places, and one such usage has a method that explicitly does not allow exceptions (compareTo) * (2) it is allowed to return 2 for "incomparable"; * it should not throw an exception. org.apache.xmlbeans.impl.values [XmlObjectBase[compare_to(XmlObject)[(2318, 13) |if| |(|equal_to|(|xmlobj|)|)], valueEqualsImpl(XmlObject)[(2137, 41) |return| |(|underlying|(|xmlobj|)|)|.|equal_to|(|this|)|;, (2139, 16) |return| |equal_to|(|xmlobj|)|;]]] XmlObjectBase[compare_to(XmlObject)[(2318, 13) |if| |(|equal_to|(|xmlobj|)|)], valueEqualsImpl(XmlObject)[(2137, 41) |return| |(|underlying|(|xmlobj|)|)|.|equal_to|(|this|)|;, (2139, 16) |return| |equal_to|(|xmlobj|)|;]] compare_to(XmlObject)[(2318, 13) |if| |(|equal_to|(|xmlobj|)|)] (2318, 13) |if| |(|equal_to|(|xmlobj|)|) valueEqualsImpl(XmlObject)[(2137, 41) |return| |(|underlying|(|xmlobj|)|)|.|equal_to|(|this|)|;, (2139, 16) |return| |equal_to|(|xmlobj|)|;] (2137, 41) |return| |(|underlying|(|xmlobj|)|)|.|equal_to|(|this|)|; (2139, 16) |return| |equal_to|(|xmlobj|)|; > XmlComplexContentImpl.equal_to() fails to throw exception when unable to > return correct value > --------------------------------------------------------------------------------------------- > > Key: XMLBEANS-347 > URL: https://issues.apache.org/jira/browse/XMLBEANS-347 > Project: XMLBeans > Issue Type: Bug > Affects Versions: Version 2.3 > Environment: n/a > Reporter: Charles Butterfield > Assignee: Jacob Danner > Priority: Critical > > I was recently horrified to discover that valueEquals() fails silently for > complex types. For complex types the comparison is delegated to > XmlComplexContentImpl.equal_to() which simply returns "true" with the comment > "BUGBUG: by-value structure comparison undone". > Clearly the method should throw an exception to alert users to the issue, as > do various neighboring methods. > I first ran into this problem (today) while using an old xbean.jar (built in > early 2004). I then checked the 2.3.0 source and the bug still exists. So > its been there unchanged since version 1.0.4 at least. > Hopefully adding the exception is a no-brainer. Perhaps a working > implementation could follow. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]