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 2010/02/23 22:02:19 UTC

svn commit: r915519 - /incubator/wookie/trunk/src/org/apache/wookie/helpers/WidgetHelper.java

Author: scottbw
Date: Tue Feb 23 21:02:19 2010
New Revision: 915519

URL: http://svn.apache.org/viewvc?rev=915519&view=rev
Log:
Put null guards in place when checking for ITS properties on widget metadata

Modified:
    incubator/wookie/trunk/src/org/apache/wookie/helpers/WidgetHelper.java

Modified: incubator/wookie/trunk/src/org/apache/wookie/helpers/WidgetHelper.java
URL: http://svn.apache.org/viewvc/incubator/wookie/trunk/src/org/apache/wookie/helpers/WidgetHelper.java?rev=915519&r1=915518&r2=915519&view=diff
==============================================================================
--- incubator/wookie/trunk/src/org/apache/wookie/helpers/WidgetHelper.java (original)
+++ incubator/wookie/trunk/src/org/apache/wookie/helpers/WidgetHelper.java Tue Feb 23 21:02:19 2010
@@ -97,7 +97,7 @@
 				+ "\" version=\"" + widget.getVersion() 
 				+ "\">\n";
 		out += "\t\t<title "; 
-		if (name.getDir()!=null) out+=" dir=\""+name.getDir()+"\"";
+		if (name != null && name.getDir()!=null) out+=" dir=\""+name.getDir()+"\"";
 		if (shortName != null) out +="short=\""+shortName + "\"";
 		out +=">";
 		if(longName != null) out += longName; 
@@ -106,7 +106,7 @@
 		// Do description
 		Description desc = (Description) LocalizationUtils.getLocalizedElement(Description.findByValue("widget", widget), locales);	
 		out += "\t\t<description";
-		if (desc.getDir()!=null) out+=" dir=\""+desc.getDir()+"\"";
+		if (desc!= null && desc.getDir()!=null) out+=" dir=\""+desc.getDir()+"\"";
 		out += ">";
 		if (desc != null) out += desc.getContent();
 		out += "</description>\n";