You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by mb...@apache.org on 2005/09/05 09:59:21 UTC
svn commit: r278688 -
/myfaces/impl/trunk/src/java/org/apache/myfaces/renderkit/html/HtmlRenderKitImpl.java
Author: mbr
Date: Mon Sep 5 00:59:17 2005
New Revision: 278688
URL: http://svn.apache.org/viewcvs?rev=278688&view=rev
Log:
fix for MYFACES-496 and MYFACES-497
Modified:
myfaces/impl/trunk/src/java/org/apache/myfaces/renderkit/html/HtmlRenderKitImpl.java
Modified: myfaces/impl/trunk/src/java/org/apache/myfaces/renderkit/html/HtmlRenderKitImpl.java
URL: http://svn.apache.org/viewcvs/myfaces/impl/trunk/src/java/org/apache/myfaces/renderkit/html/HtmlRenderKitImpl.java?rev=278688&r1=278687&r2=278688&view=diff
==============================================================================
--- myfaces/impl/trunk/src/java/org/apache/myfaces/renderkit/html/HtmlRenderKitImpl.java (original)
+++ myfaces/impl/trunk/src/java/org/apache/myfaces/renderkit/html/HtmlRenderKitImpl.java Mon Sep 5 00:59:17 2005
@@ -72,6 +72,14 @@
public Renderer getRenderer(String componentFamily, String rendererType)
{
+ if(componentFamily == null)
+ {
+ throw new NullPointerException("component family must not be null.");
+ }
+ if(rendererType == null)
+ {
+ throw new NullPointerException("renderer type must not be null.");
+ }
Renderer renderer = (Renderer) _renderers.get(key(componentFamily, rendererType));
if (renderer == null)
{
@@ -82,6 +90,14 @@
public void addRenderer(String componentFamily, String rendererType, Renderer renderer)
{
+ if(componentFamily == null)
+ {
+ throw new NullPointerException("component family must not be null.");
+ }
+ if(rendererType == null)
+ {
+ throw new NullPointerException("renderer type must not be null.");
+ }
_renderers.put(key(componentFamily, rendererType), renderer);
}