commit 743875048abd2577f8cb1558e2e618efeef38b74
author Jeremy Bauer <jrbauer@apache.org> Mon May 13 01:43:45 2013 +0000
committer Jeremy Bauer <jrbauer@apache.org> Mon May 13 01:43:45 2013 +0000
tree 85f2f55324017352de770ce2db3885ab83ffb01a
parent 521fecd2d9b91c27e9f90d97e5f5479d17239eb8

Refrain from logging if verifying log factory plugin.

git-svn-id: https://svn.apache.org/repos/asf/openjpa/branches/1.1.x@1481682 13f79535-47bb-0310-9956-ffa450edef68

openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/PluginValue.java

diff --git a/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/PluginValue.java b/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/PluginValue.java
index 4b22020..197aa4e 100644
--- a/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/PluginValue.java
+++ b/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/PluginValue.java
@@ -20,6 +20,7 @@
 
 import org.apache.commons.lang.StringUtils;
 import org.apache.openjpa.lib.log.Log;
+import org.apache.openjpa.lib.log.LogFactory;
 import org.apache.openjpa.lib.util.Localizer;
 import org.apache.openjpa.lib.util.ParseException;
 
@@ -105,7 +106,8 @@
         
         // ensure plugin value is compatible with plugin type
         if (obj != null && !type.isAssignableFrom(obj.getClass())) {
-            Log log = (conf == null) ? null : conf.getConfigurationLog();
+            Log log = (conf == null || type.equals(LogFactory.class)) 
+                      ? null : conf.getConfigurationLog();
             String msg = getIncompatiblePluginMessage(obj, type);
             if (log != null && log.isErrorEnabled()) {
             	log.error(msg);