[gatein-commits] gatein SVN: r5160 - components/wsrp/trunk/consumer/src/main/java/org/gatein/wsrp/consumer/registry.

[do-not-reply at jboss.org]

Author: chris.laprun at jboss.com
Date: 2010-11-18 09:43:00 -0500 (Thu, 18 Nov 2010)
New Revision: 5160

Modified:
   components/wsrp/trunk/consumer/src/main/java/org/gatein/wsrp/consumer/registry/ActivatingNullInvokerHandler.java
Log:
- GTNWSRP-165: Synchronize customer activation to avoid double registrations with federating invoker when multiple portlets try to access to that specific producer. 


Modified: components/wsrp/trunk/consumer/src/main/java/org/gatein/wsrp/consumer/registry/ActivatingNullInvokerHandler.java
===================================================================
--- components/wsrp/trunk/consumer/src/main/java/org/gatein/wsrp/consumer/registry/ActivatingNullInvokerHandler.java	2010-11-18 14:36:34 UTC (rev 5159)
+++ components/wsrp/trunk/consumer/src/main/java/org/gatein/wsrp/consumer/registry/ActivatingNullInvokerHandler.java	2010-11-18 14:43:00 UTC (rev 5160)
@@ -62,9 +62,12 @@
       else
       {
          // activate the consumer which should register it with this FederatingPortletInvoker
-         consumerRegistry.activateConsumerWith(invokerId);
+         synchronized (this)
+         {
+            consumerRegistry.activateConsumerWith(invokerId);
 
-         federated = callingInvoker.getFederatedInvoker(invokerId);
+            federated = callingInvoker.getFederatedInvoker(invokerId);
+         }
       }
 
       //