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:55:03 UTC

svn commit: r737144 - /myfaces/trinidad/trunk_1.2.x/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/FrameBorderLayoutRenderer.java

Author: arobinson74
Date: Fri Jan 23 10:55:02 2009
New Revision: 737144

URL: http://svn.apache.org/viewvc?rev=737144&view=rev
Log:
TRINIDAD-1378
Do not call .toString() on a possibly null value

Modified:
    myfaces/trinidad/trunk_1.2.x/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/FrameBorderLayoutRenderer.java

Modified: myfaces/trinidad/trunk_1.2.x/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/FrameBorderLayoutRenderer.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/trunk_1.2.x/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/FrameBorderLayoutRenderer.java?rev=737144&r1=737143&r2=737144&view=diff
==============================================================================
--- myfaces/trinidad/trunk_1.2.x/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/FrameBorderLayoutRenderer.java (original)
+++ myfaces/trinidad/trunk_1.2.x/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/FrameBorderLayoutRenderer.java Fri Jan 23 10:55:02 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;
   }