From portal-commits at lists.jboss.org Fri Nov 30 09:00:36 2007
Content-Type: multipart/mixed; boundary="===============9008264025192754801=="
MIME-Version: 1.0
From: portal-commits at lists.jboss.org
To: portal-commits at lists.jboss.org
Subject: [portal-commits] JBoss Portal SVN: r9218 - in
 branches/JBoss_Portal_Branch_2_6:
 core/src/main/org/jboss/portal/core/impl/portlet/state and 1 other
 directories.
Date: Fri, 30 Nov 2007 08:56:36 -0500
Message-ID: <E1Iy6M8-00087y-A5@committer01.frg.pub.inap.atl.jboss.com>

--===============9008264025192754801==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable

Author: julien(a)jboss.com
Date: 2007-11-30 08:56:36 -0500 (Fri, 30 Nov 2007)
New Revision: 9218

Modified:
   branches/JBoss_Portal_Branch_2_6/core/build.xml
   branches/JBoss_Portal_Branch_2_6/core/src/main/org/jboss/portal/core/imp=
l/portlet/state/PersistentRegistration.java
   branches/JBoss_Portal_Branch_2_6/registration/src/main/org/jboss/portal/=
test/registration/AbstractRegistrationPersistenceManagerTestCase.java
Log:
JBPORTAL-1826 : bug fix and test case for "Registration data is not properl=
y persisted on updates"

Modified: branches/JBoss_Portal_Branch_2_6/core/build.xml
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
--- branches/JBoss_Portal_Branch_2_6/core/build.xml	2007-11-30 13:47:54 UTC=
 (rev 9217)
+++ branches/JBoss_Portal_Branch_2_6/core/build.xml	2007-11-30 13:56:36 UTC=
 (rev 9218)
@@ -613,7 +613,7 @@
                <parameter name=3D"CacheNaturalId" value=3D"true"/>
                <parameter name=3D"Config" value=3D"persistent-jboss-beans.=
xml"/>
             </zest>
-         	=

+
             <!--
             <zest todir=3D"${test.reports}" name=3D"org.jboss.portal.test.=
core.model.instance.InstanceContainerTestCase"
                   outfile=3D"TEST-PersistedLocally-ClonedOnCreate-Instance=
ContainerTestCase">

Modified: branches/JBoss_Portal_Branch_2_6/core/src/main/org/jboss/portal/c=
ore/impl/portlet/state/PersistentRegistration.java
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
--- branches/JBoss_Portal_Branch_2_6/core/src/main/org/jboss/portal/core/im=
pl/portlet/state/PersistentRegistration.java	2007-11-30 13:47:54 UTC (rev 9=
217)
+++ branches/JBoss_Portal_Branch_2_6/core/src/main/org/jboss/portal/core/im=
pl/portlet/state/PersistentRegistration.java	2007-11-30 13:56:36 UTC (rev 9=
218)
@@ -23,6 +23,7 @@
 package org.jboss.portal.core.impl.portlet.state;
 =

 import org.jboss.portal.common.NotYetImplemented;
+import org.jboss.portal.common.util.MapAccessor;
 import org.jboss.portal.common.util.TypedMap;
 import org.jboss.portal.registration.Consumer;
 import org.jboss.portal.registration.Registration;
@@ -30,10 +31,10 @@
 =

 import javax.xml.namespace.QName;
 import java.util.Collections;
+import java.util.HashMap;
 import java.util.HashSet;
 import java.util.Map;
 import java.util.Set;
-import java.util.HashMap;
 =

 /**
  * @author <a href=3D"mailto:julien(a)jboss.org">Julien Viet</a>
@@ -175,7 +176,7 @@
 =

    public Map getProperties()
    {
-      return Collections.unmodifiableMap(persistentProperties);
+      return Collections.unmodifiableMap(properties);
    }
 =

    public void setPropertyValueFor(QName propertyName, Object value)
@@ -289,13 +290,21 @@
       }
    };
 =

+   private final MapAccessor accessor =3D new MapAccessor()
+   {
+      public Map getMap(boolean b)
+      {
+         return persistentProperties;
+      }
+   };
+
    /** Implement registration properties semantics, mostly validation and =
equality. */
    public class Properties extends TypedMap
    {
 =

       public Properties()
       {
-         super(persistentProperties, KEY_CONVERTER, VALUE_CONVERTER);
+         super(accessor, KEY_CONVERTER, VALUE_CONVERTER);
       }
 =

       public void setProperty(String propertyName, Object value)

Modified: branches/JBoss_Portal_Branch_2_6/registration/src/main/org/jboss/=
portal/test/registration/AbstractRegistrationPersistenceManagerTestCase.java
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
--- branches/JBoss_Portal_Branch_2_6/registration/src/main/org/jboss/portal=
/test/registration/AbstractRegistrationPersistenceManagerTestCase.java	2007=
-11-30 13:47:54 UTC (rev 9217)
+++ branches/JBoss_Portal_Branch_2_6/registration/src/main/org/jboss/portal=
/test/registration/AbstractRegistrationPersistenceManagerTestCase.java	2007=
-11-30 13:56:36 UTC (rev 9218)
@@ -29,6 +29,7 @@
 import org.jboss.portal.registration.NoSuchRegistrationException;
 import org.jboss.portal.registration.Registration;
 import org.jboss.portal.registration.RegistrationPersistenceManager;
+import org.jboss.portal.common.util.MapBuilder;
 =

 import javax.xml.namespace.QName;
 import java.util.ArrayList;
@@ -397,4 +398,40 @@
       assertEquals(null, getManager().getRegistration(regId));
       stopInteraction();
    }
+
+   public void testBulkUpdateRegistrationProperties() throws Exception
+   {
+      startInteraction();
+      ConsumerGroup group =3D getManager().createConsumerGroup("Foo");
+      Consumer consumer =3D getManager().createConsumer("Bar", "Bar");
+      group.addConsumer(consumer);
+      getManager().addRegistrationFor("Bar", registrationProperties);
+      stopInteraction();
+
+      //
+      startInteraction();
+      consumer =3D getManager().getConsumerById("Bar");
+      Registration reg =3D (Registration)consumer.getRegistrations().itera=
tor().next();
+      registrationProperties.remove(new QName("prop1"));
+      reg.updateProperties(registrationProperties);
+      assertEquals(Collections.singletonMap(new QName("prop2"), "value2"),=
 reg.getProperties());
+      stopInteraction();
+
+      //
+      startInteraction();
+      consumer =3D getManager().getConsumerById("Bar");
+      reg =3D (Registration)consumer.getRegistrations().iterator().next();
+      assertEquals(Collections.singletonMap(new QName("prop2"), "value2"),=
 reg.getProperties());
+      registrationProperties.put(new QName("prop3"), "value3");
+      reg.updateProperties(registrationProperties);
+      assertEquals(MapBuilder.hashMap().put(new QName("prop2"), "value2").=
put(new QName("prop3"), "value3").get(), reg.getProperties());
+      stopInteraction();
+
+      //
+      startInteraction();
+      consumer =3D getManager().getConsumerById("Bar");
+      reg =3D (Registration)consumer.getRegistrations().iterator().next();
+      assertEquals(MapBuilder.hashMap().put(new QName("prop2"), "value2").=
put(new QName("prop3"), "value3").get(), reg.getProperties());
+      stopInteraction();
+   }
 }


--===============9008264025192754801==--