You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@juddi.apache.org by ks...@apache.org on 2010/03/18 18:43:38 UTC

svn commit: r924918 - /webservices/juddi/trunk/juddi-core/src/main/java/org/apache/juddi/subscription/SubscriptionNotifier.java

Author: kstam
Date: Thu Mar 18 17:43:38 2010
New Revision: 924918

URL: http://svn.apache.org/viewvc?rev=924918&view=rev
Log:
JUDDI-375 fixing NPE

Modified:
    webservices/juddi/trunk/juddi-core/src/main/java/org/apache/juddi/subscription/SubscriptionNotifier.java

Modified: webservices/juddi/trunk/juddi-core/src/main/java/org/apache/juddi/subscription/SubscriptionNotifier.java
URL: http://svn.apache.org/viewvc/webservices/juddi/trunk/juddi-core/src/main/java/org/apache/juddi/subscription/SubscriptionNotifier.java?rev=924918&r1=924917&r2=924918&view=diff
==============================================================================
--- webservices/juddi/trunk/juddi-core/src/main/java/org/apache/juddi/subscription/SubscriptionNotifier.java (original)
+++ webservices/juddi/trunk/juddi-core/src/main/java/org/apache/juddi/subscription/SubscriptionNotifier.java Thu Mar 18 17:43:38 2010
@@ -84,13 +84,15 @@ public class SubscriptionNotifier extend
 				if (subscription.getExpiresAfter()==null || subscription.getExpiresAfter().getTime() > startTime) {
 					try {
 						GetSubscriptionResults getSubscriptionResults = buildGetSubscriptionResults(subscription, new Date(scheduledExecutionTime()));
-						getSubscriptionResults.setSubscriptionKey(subscription.getSubscriptionKey());
-						UddiEntityPublisher publisher = new UddiEntityPublisher();
-						publisher.setAuthorizedName(subscription.getAuthorizedName());
-						SubscriptionResultsList resultList = subscriptionImpl.getSubscriptionResults(getSubscriptionResults, publisher);
-						if (resultListContainsChanges(resultList)) {
-							log.info("We have a change and need to notify..");
-							notify(getSubscriptionResults,resultList);
+						if (getSubscriptionResults!=null) {
+							getSubscriptionResults.setSubscriptionKey(subscription.getSubscriptionKey());
+							UddiEntityPublisher publisher = new UddiEntityPublisher();
+							publisher.setAuthorizedName(subscription.getAuthorizedName());
+							SubscriptionResultsList resultList = subscriptionImpl.getSubscriptionResults(getSubscriptionResults, publisher);
+							if (resultListContainsChanges(resultList)) {
+								log.info("We have a change and need to notify..");
+								notify(getSubscriptionResults,resultList);
+							}
 						}
 					} catch (Exception e) {
 						log.error("Could not obtain subscriptionResult for subscriptionKey " 



---------------------------------------------------------------------
To unsubscribe, e-mail: juddi-cvs-unsubscribe@ws.apache.org
For additional commands, e-mail: juddi-cvs-help@ws.apache.org