[jbossseam-issues] [JBoss JIRA] Updated: (JBSEAM-1257) default equals() on javabean components is not reflexive

[Matt Drees (JIRA)]

     [ http://jira.jboss.com/jira/browse/JBSEAM-1257?page=all ]

Matt Drees updated JBSEAM-1257:
-------------------------------

    Attachment: reflexive-equals.txt

Here's a patch to JavaBeanIntereceptor.java that should do the trick, I think.  When intercepting an equals method, if the parameter is identical to the proxy, it unwraps the parameter.  So I suppose if you design your equals method to make calls on the parameter assuming interceptors will be run (for injection, etc), then this patch would hurt you.  But I think no one should be making those assumptions for an equals method.

> default equals() on javabean components is not reflexive
> --------------------------------------------------------
>
>                 Key: JBSEAM-1257
>                 URL: http://jira.jboss.com/jira/browse/JBSEAM-1257
>             Project: JBoss Seam
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: 1.2.1.GA
>            Reporter: Matt Drees
>         Assigned To: Gavin King
>         Attachments: reflexive-equals.txt
>
>
> If a javabean component doesn't override equals(), the javassist proxy generated for it has a nonreflexive equals method.  I.e., component.equals(component) returns false.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira