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 2009/12/13 00:14:18 UTC

svn commit: r890001 - /incubator/wookie/trunk/src/org/apache/wookie/controller/PropertiesController.java

Author: scottbw
Date: Sat Dec 12 23:14:17 2009
New Revision: 890001

URL: http://svn.apache.org/viewvc?rev=890001&view=rev
Log:
Put guard to prevent null pointer exceptions when checking parameters

Modified:
    incubator/wookie/trunk/src/org/apache/wookie/controller/PropertiesController.java

Modified: incubator/wookie/trunk/src/org/apache/wookie/controller/PropertiesController.java
URL: http://svn.apache.org/viewvc/incubator/wookie/trunk/src/org/apache/wookie/controller/PropertiesController.java?rev=890001&r1=890000&r2=890001&view=diff
==============================================================================
--- incubator/wookie/trunk/src/org/apache/wookie/controller/PropertiesController.java (original)
+++ incubator/wookie/trunk/src/org/apache/wookie/controller/PropertiesController.java Sat Dec 12 23:14:17 2009
@@ -231,9 +231,9 @@
 	
 	private static boolean isPublic(HttpServletRequest request){
 		String is_public = request.getParameter("is_public"); //$NON-NLS-1$	
-		if (is_public != null && is_public.equals("true")|| is_public.equals("1")) return true;
+		if (is_public != null) if (is_public.equals("true")|| is_public.equals("1")) return true;
 		String requestId = request.getParameter("requestid"); //$NON-NLS-1$
-		if (requestId != null && requestId.equals("setpublicproperty")) return true;
+		if (requestId != null) if (requestId.equals("setpublicproperty")) return true;
 		return false;
 	}