You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by lu...@apache.org on 2010/12/01 19:25:49 UTC

svn commit: r1041115 - /myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/spi/impl/SpiUtils.java

Author: lu4242
Date: Wed Dec  1 18:25:48 2010
New Revision: 1041115

URL: http://svn.apache.org/viewvc?rev=1041115&view=rev
Log:
MYFACES-2986 Provide an interface to override how to find spi interfaces (add warning when just one SPI interface is expected)

Modified:
    myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/spi/impl/SpiUtils.java

Modified: myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/spi/impl/SpiUtils.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/spi/impl/SpiUtils.java?rev=1041115&r1=1041114&r2=1041115&view=diff
==============================================================================
--- myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/spi/impl/SpiUtils.java (original)
+++ myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/spi/impl/SpiUtils.java Wed Dec  1 18:25:48 2010
@@ -19,6 +19,7 @@
 package org.apache.myfaces.spi.impl;
 
 import java.util.List;
+import java.util.logging.Level;
 import java.util.logging.Logger;
 
 import javax.faces.context.ExternalContext;
@@ -41,6 +42,11 @@ public final class SpiUtils
         
         if (classList != null && !classList.isEmpty())
         {
+            if (classList.size() > 1)
+            {
+                getLogger().log(Level.WARNING, "More than one SPI interface for "+spiClass.getName()+
+                        " found :" + classList.toString() + ". Setting up the first one found.");
+            }
             return ClassUtils.newInstance(classList.get(0));
         }
         return ClassUtils.newInstance(defaultImpl);