You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@sling.apache.org by ro...@apache.org on 2017/03/09 09:22:52 UTC

svn commit: r1786117 - /sling/trunk/bundles/commons/log/src/main/java/org/apache/sling/commons/log/logback/internal/LogConfigManager.java

Author: rombert
Date: Thu Mar  9 09:22:52 2017
New Revision: 1786117

URL: http://svn.apache.org/viewvc?rev=1786117&view=rev
Log:
SLING-6624 - LogConfigManager reports incorrect config pid when
duplicate categories are defined 

* corrected error message

Modified:
    sling/trunk/bundles/commons/log/src/main/java/org/apache/sling/commons/log/logback/internal/LogConfigManager.java

Modified: sling/trunk/bundles/commons/log/src/main/java/org/apache/sling/commons/log/logback/internal/LogConfigManager.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/commons/log/src/main/java/org/apache/sling/commons/log/logback/internal/LogConfigManager.java?rev=1786117&r1=1786116&r2=1786117&view=diff
==============================================================================
--- sling/trunk/bundles/commons/log/src/main/java/org/apache/sling/commons/log/logback/internal/LogConfigManager.java (original)
+++ sling/trunk/bundles/commons/log/src/main/java/org/apache/sling/commons/log/logback/internal/LogConfigManager.java Thu Mar  9 09:22:52 2017
@@ -531,7 +531,7 @@ public class LogConfigManager implements
                 final LogConfig cfg = configByCategory.get(cat);
                 if (cfg != null && !pid.equals(cfg.getConfigPid())) {
                     throw new ConfigurationException(LogConfigManager.LOG_LOGGERS, "Category " + cat
-                        + " already defined by configuration " + pid);
+                        + " already defined by configuration " + cfg.getConfigPid());
                 }
             }