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 2007/02/18 20:28:50 UTC

svn commit: r508949 - /myfaces/core/branches/jsf12/impl/src/main/java/org/apache/myfaces/application/ApplicationImpl.java

Author: mbr
Date: Sun Feb 18 11:28:49 2007
New Revision: 508949

URL: http://svn.apache.org/viewvc?view=rev&rev=508949
Log:
add null check to setVariableResolver 

Modified:
    myfaces/core/branches/jsf12/impl/src/main/java/org/apache/myfaces/application/ApplicationImpl.java

Modified: myfaces/core/branches/jsf12/impl/src/main/java/org/apache/myfaces/application/ApplicationImpl.java
URL: http://svn.apache.org/viewvc/myfaces/core/branches/jsf12/impl/src/main/java/org/apache/myfaces/application/ApplicationImpl.java?view=diff&rev=508949&r1=508948&r2=508949
==============================================================================
--- myfaces/core/branches/jsf12/impl/src/main/java/org/apache/myfaces/application/ApplicationImpl.java (original)
+++ myfaces/core/branches/jsf12/impl/src/main/java/org/apache/myfaces/application/ApplicationImpl.java Sun Feb 18 11:28:49 2007
@@ -349,6 +349,8 @@
      * @deprecated
      */
     public void setVariableResolver(VariableResolver variableResolver) {
+        checkNull(variableResolver, "variableResolver");
+        
         _resolverForFaces.addResolverFromLegacyVariableResolver(variableResolver);
         
         // TODO: fix FacesConfigurator so this won't happen