10:47 p.m.
Author: jason.greene(a)jboss.com
Date: 2008-03-13 17:47:59 -0400 (Thu, 13 Mar 2008)
New Revision: 5428
Added:
pojo/branches/2.1/src/test/java/org/jboss/cache/pojo/test/DoubleRef.java
Modified:
pojo/branches/2.1/src/main/java/org/jboss/cache/pojo/impl/PojoInstance.java
pojo/branches/2.1/src/test/java/org/jboss/cache/pojo/LocalTest.java
Log:
Fix PCACHE-61
Modified: pojo/branches/2.1/src/main/java/org/jboss/cache/pojo/impl/PojoInstance.java
===================================================================
--- pojo/branches/2.1/src/main/java/org/jboss/cache/pojo/impl/PojoInstance.java 2008-03-13
16:51:00 UTC (rev 5427)
+++ pojo/branches/2.1/src/main/java/org/jboss/cache/pojo/impl/PojoInstance.java 2008-03-13
21:47:59 UTC (rev 5428)
@@ -104,10 +104,6 @@
referencedBy_ = new ArrayList();
}
- if (referencedBy_.contains(sourceFqn))
- throw new IllegalStateException("PojoReference.incrementRefCount(): source
fqn: " +
- sourceFqn + " is already present.");
-
if (util_ == null) util_ = new PojoUtil();
refCount_ = util_.incrementReferenceCount(sourceFqn, refCount_, referencedBy_);
// referencedBy_.add(sourceFqn);
Modified: pojo/branches/2.1/src/test/java/org/jboss/cache/pojo/LocalTest.java
===================================================================
--- pojo/branches/2.1/src/test/java/org/jboss/cache/pojo/LocalTest.java 2008-03-13
16:51:00 UTC (rev 5427)
+++ pojo/branches/2.1/src/test/java/org/jboss/cache/pojo/LocalTest.java 2008-03-13
21:47:59 UTC (rev 5428)
@@ -19,8 +19,10 @@
import org.jboss.cache.Fqn;
import org.jboss.cache.pojo.impl.PojoReference;
import org.jboss.cache.pojo.test.Address;
+import org.jboss.cache.pojo.test.DoubleRef;
import org.jboss.cache.pojo.test.Person;
import org.jboss.cache.pojo.test.Student;
+import org.testng.AssertJUnit;
import org.testng.annotations.AfterMethod;
import org.testng.annotations.BeforeMethod;
import org.testng.annotations.Test;
@@ -406,4 +408,15 @@
PojoReference ref = (PojoReference) cache_.getCache().get(fqn,
PojoReference.KEY);
assertTrue(cache_.exists(ref.getFqn()));
}
+
+ public void testDoubleRef() throws Exception
+ {
+ DoubleRef ref = new DoubleRef();
+ cache_.attach("/test", ref);
+
+ AssertJUnit.assertSame(ref.getOne(), ref.getTwo());
+
+ ref.setOne(null);
+ ref.setTwo(null);
+ }
}
\ No newline at end of file
Added: pojo/branches/2.1/src/test/java/org/jboss/cache/pojo/test/DoubleRef.java
===================================================================
--- pojo/branches/2.1/src/test/java/org/jboss/cache/pojo/test/DoubleRef.java
(rev 0)
+++ pojo/branches/2.1/src/test/java/org/jboss/cache/pojo/test/DoubleRef.java 2008-03-13
21:47:59 UTC (rev 5428)
@@ -0,0 +1,56 @@
+/*
+* JBoss, Home of Professional Open Source
+* Copyright 2005, JBoss Inc., and individual contributors as indicated
+* by the @authors tag. See the copyright.txt in the distribution for a
+* full listing of individual contributors.
+*
+* This is free software; you can redistribute it and/or modify it
+* under the terms of the GNU Lesser General Public License as
+* published by the Free Software Foundation; either version 2.1 of
+* the License, or (at your option) any later version.
+*
+* This software is distributed in the hope that it will be useful,
+* but WITHOUT ANY WARRANTY; without even the implied warranty of
+* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+* Lesser General Public License for more details.
+*
+* You should have received a copy of the GNU Lesser General Public
+* License along with this software; if not, write to the Free
+* Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
+* 02110-1301 USA, or see the FSF site: http://www.fsf.org.
+*/
+package org.jboss.cache.pojo.test;
+
+import org.jboss.cache.pojo.annotation.Replicable;
+
+/**
+ * Contains two fields referring to the same object reference
+ *
+ * @author Jason T. Greene
+ */
+@Replicable
+public class DoubleRef
+{
+ private Student one = new Student();
+ private Student two = one;
+
+ public Student getOne()
+ {
+ return one;
+ }
+
+ public void setOne(Student one)
+ {
+ this.one = one;
+ }
+
+ public Student getTwo()
+ {
+ return two;
+ }
+
+ public void setTwo(Student two)
+ {
+ this.two = two;
+ }
+}