You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@wookie.apache.org by sc...@apache.org on 2014/02/09 20:33:00 UTC

svn commit: r1566367 - /wookie/trunk/wookie-services/wookie-spi/src/main/java/org/apache/wookie/services/impl/DefaultPreferencesService.java

Author: scottbw
Date: Sun Feb  9 19:33:00 2014
New Revision: 1566367

URL: http://svn.apache.org/r1566367
Log:
Ensure we don't get an NPE when getting the value of a preference

Modified:
    wookie/trunk/wookie-services/wookie-spi/src/main/java/org/apache/wookie/services/impl/DefaultPreferencesService.java

Modified: wookie/trunk/wookie-services/wookie-spi/src/main/java/org/apache/wookie/services/impl/DefaultPreferencesService.java
URL: http://svn.apache.org/viewvc/wookie/trunk/wookie-services/wookie-spi/src/main/java/org/apache/wookie/services/impl/DefaultPreferencesService.java?rev=1566367&r1=1566366&r2=1566367&view=diff
==============================================================================
--- wookie/trunk/wookie-services/wookie-spi/src/main/java/org/apache/wookie/services/impl/DefaultPreferencesService.java (original)
+++ wookie/trunk/wookie-services/wookie-spi/src/main/java/org/apache/wookie/services/impl/DefaultPreferencesService.java Sun Feb  9 19:33:00 2014
@@ -41,7 +41,12 @@ public class DefaultPreferencesService i
 			widgetpreferences = new HashMap<String, IPreference>();
 			this.preferences.put(token, widgetpreferences);
 		}
-		return widgetpreferences.get(name).getValue();
+		IPreference pref = widgetpreferences.get(name);
+		if (pref != null){
+			return widgetpreferences.get(name).getValue();
+		} else {
+			return null;
+		}
 	}
 
 	@Override