[gatein-commits] gatein SVN: r5183 - 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-19 12:29:02 -0500 (Fri, 19 Nov 2010)
New Revision: 5183

Modified:
   components/wsrp/trunk/consumer/src/main/java/org/gatein/wsrp/consumer/registry/AbstractConsumerRegistry.java
Log:
- GTNWSRP-166: The FederatingPortletInvoker wasn't properly updated when a producer's id was changed. Not sure how we never realized that before, kudos to Michal for exposing the issue! :)

Modified: components/wsrp/trunk/consumer/src/main/java/org/gatein/wsrp/consumer/registry/AbstractConsumerRegistry.java
===================================================================
--- components/wsrp/trunk/consumer/src/main/java/org/gatein/wsrp/consumer/registry/AbstractConsumerRegistry.java	2010-11-19 14:26:10 UTC (rev 5182)
+++ components/wsrp/trunk/consumer/src/main/java/org/gatein/wsrp/consumer/registry/AbstractConsumerRegistry.java	2010-11-19 17:29:02 UTC (rev 5183)
@@ -252,7 +252,15 @@
       if (oldId != null)
       {
          remove(getConsumer(oldId));
-         createConsumerFrom(producerInfo);
+         WSRPConsumer consumer = createConsumerFrom(producerInfo);
+
+         // update the federating portlet invoker:
+         FederatedPortletInvoker invoker = federatingPortletInvoker.getFederatedInvoker(oldId);
+         if (invoker != null)
+         {
+            federatingPortletInvoker.unregisterInvoker(oldId);
+            federatingPortletInvoker.registerInvoker(producerInfo.getId(), consumer);
+         }
       }
    }