[jira] [Commented] (LANG-1082) Add option to disable the "objectsTriviallyEqual" test in DiffBuilder.

["Jonathan Baker (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/LANG-1082?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14286579#comment-14286579 ] 

Jonathan Baker commented on LANG-1082:
--------------------------------------

I'd be happy to throw some code out there, but I want to make sure I'm preserving the original spirit.

My gut tells me DiffBuilder should remain immutable.
Would adding a second constructor be the best fit?

public DiffBuilder( boolean testTriviallyEqual ) { ... }
public DiffBuilder() { this( true ); }

Thoughts?


> Add option to disable the "objectsTriviallyEqual" test in DiffBuilder.
> ----------------------------------------------------------------------
>
>                 Key: LANG-1082
>                 URL: https://issues.apache.org/jira/browse/LANG-1082
>             Project: Commons Lang
>          Issue Type: Improvement
>          Components: lang.builder.*
>    Affects Versions: 3.3.2
>            Reporter: Jonathan Baker
>            Priority: Trivial
>             Fix For: Patch Needed
>
>   Original Estimate: 0.25h
>  Remaining Estimate: 0.25h
>
> For example, a bean (e.g. a JPA record) has equals() overridden with a implementation that tests its "id" property.   If two beans have the same ID, they correspond to the same record in the database.  However, without tricking the objectsTriviallyEqual check in the DiffBuilder constructor, there is no way to build a DiffResult using "before" and "after" instances with changes to other properties (name, modification time, etc).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)