You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by ma...@apache.org on 2009/08/25 12:20:59 UTC

svn commit: r807541 - /myfaces/shared/trunk_3.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/RendererUtils.java

Author: matzew
Date: Tue Aug 25 10:20:59 2009
New Revision: 807541

URL: http://svn.apache.org/viewvc?rev=807541&view=rev
Log:
MYFACES-2339 - RenderKitUtils should give better error if someone uses an incorrect default-render-kit-id

throwing IllegalArgumentException if the underlying Factory/Factories can't return a proper RenderKit

Modified:
    myfaces/shared/trunk_3.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/RendererUtils.java

Modified: myfaces/shared/trunk_3.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/RendererUtils.java
URL: http://svn.apache.org/viewvc/myfaces/shared/trunk_3.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/RendererUtils.java?rev=807541&r1=807540&r2=807541&view=diff
==============================================================================
--- myfaces/shared/trunk_3.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/RendererUtils.java (original)
+++ myfaces/shared/trunk_3.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/RendererUtils.java Tue Aug 25 10:20:59 2009
@@ -1175,6 +1175,12 @@
                 renderKit = factory.getRenderKit(facesContext, renderKitId);
             }
         }
+        
+        if  (renderKit == null)
+        {
+            throw new IllegalArgumentException("Could not find a RenderKit for \"" + renderKitId + "\""); 
+        }
+        
         return renderKit.getResponseStateManager();
     }