You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jdo-commits@db.apache.org by an...@apache.org on 2007/10/20 11:55:01 UTC
svn commit: r586693 -
/db/jdo/trunk/tck2/src/java/org/apache/jdo/tck/pc/companyAnnotatedJPA/JPAAppPhoneNumber.java
Author: andyj
Date: Sat Oct 20 02:55:00 2007
New Revision: 586693
URL: http://svn.apache.org/viewvc?rev=586693&view=rev
Log:
JDO-539 Fix equals of Oid to use equals rather than == for comparing object equality
Modified:
db/jdo/trunk/tck2/src/java/org/apache/jdo/tck/pc/companyAnnotatedJPA/JPAAppPhoneNumber.java
Modified: db/jdo/trunk/tck2/src/java/org/apache/jdo/tck/pc/companyAnnotatedJPA/JPAAppPhoneNumber.java
URL: http://svn.apache.org/viewvc/db/jdo/trunk/tck2/src/java/org/apache/jdo/tck/pc/companyAnnotatedJPA/JPAAppPhoneNumber.java?rev=586693&r1=586692&r2=586693&view=diff
==============================================================================
--- db/jdo/trunk/tck2/src/java/org/apache/jdo/tck/pc/companyAnnotatedJPA/JPAAppPhoneNumber.java (original)
+++ db/jdo/trunk/tck2/src/java/org/apache/jdo/tck/pc/companyAnnotatedJPA/JPAAppPhoneNumber.java Sat Oct 20 02:55:00 2007
@@ -176,8 +176,8 @@
if( obj==null ||
!this.getClass().equals(obj.getClass()) ) return( false );
Oid o = (Oid) obj;
- if( this.person != o.person ) return( false );
- if( this.type != o.type ) return( false );
+ if( !this.person.equals(o.person) ) return( false );
+ if( !this.type.equals(o.type) ) return( false );
return( true );
}