[gatein-commits] gatein SVN: r1671 - components/wsrp/trunk/wsrp-producer-war/src/main/java/org/gatein/wsrp/endpoints.

[do-not-reply at jboss.org]

Author: chris.laprun at jboss.com
Date: 2010-02-14 10:58:04 -0500 (Sun, 14 Feb 2010)
New Revision: 1671

Modified:
   components/wsrp/trunk/wsrp-producer-war/src/main/java/org/gatein/wsrp/endpoints/RegistrationEndpoint.java
Log:
- Check for null after modifyRegistration as we might not have a resulting RegistrationState.

Modified: components/wsrp/trunk/wsrp-producer-war/src/main/java/org/gatein/wsrp/endpoints/RegistrationEndpoint.java
===================================================================
--- components/wsrp/trunk/wsrp-producer-war/src/main/java/org/gatein/wsrp/endpoints/RegistrationEndpoint.java	2010-02-14 15:55:04 UTC (rev 1670)
+++ components/wsrp/trunk/wsrp-producer-war/src/main/java/org/gatein/wsrp/endpoints/RegistrationEndpoint.java	2010-02-14 15:58:04 UTC (rev 1671)
@@ -1,6 +1,6 @@
 /*
  * JBoss, a division of Red Hat
- * Copyright 2009, Red Hat Middleware, LLC, and individual
+ * Copyright 2010, Red Hat Middleware, LLC, and individual
  * contributors as indicated by the @authors tag. See the
  * copyright.txt in the distribution for a full listing of
  * individual contributors.
@@ -100,8 +100,12 @@
 
       RegistrationState result = producer.modifyRegistration(modifyRegistration);
 
-      registrationState.value = result.getRegistrationState();
-      extensions.value = result.getExtensions();
+      // it is possible (if not likely) that result of modifyRegistration be null
+      if (result != null)
+      {
+         registrationState.value = result.getRegistrationState();
+         extensions.value = result.getExtensions();
+      }
    }
 
    public List<Extension> deregister(