You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by ar...@apache.org on 2009/01/23 19:58:10 UTC
svn commit: r737145 -
/myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/FrameBorderLayoutRenderer.java
Author: arobinson74
Date: Fri Jan 23 10:58:10 2009
New Revision: 737145
URL: http://svn.apache.org/viewvc?rev=737145&view=rev
Log:
TRINIDAD-1378 Do not call .toString() on a possibly null value
Modified:
myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/FrameBorderLayoutRenderer.java
Modified: myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/FrameBorderLayoutRenderer.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/FrameBorderLayoutRenderer.java?rev=737145&r1=737144&r2=737145&view=diff
==============================================================================
--- myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/FrameBorderLayoutRenderer.java (original)
+++ myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/FrameBorderLayoutRenderer.java Fri Jan 23 10:58:10 2009
@@ -284,14 +284,15 @@
String attr,
StringBuffer result)
{
- String val = frame.getAttributes().get(attr).toString();
+ Object val = frame.getAttributes().get(attr);
- if ((val==null))
+ if (val == null)
{
if (_LOG.isWarning())
- _LOG.warning("FRAME_MISSING_ATTRIBUTE", new Object[]{frame.getId(), attr});
- val="0";
+ _LOG.warning("FRAME_MISSING_ATTRIBUTE", new Object[] { frame.getId(), attr });
+ val = "0";
}
+
result.append(val);
return result;
}