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

Posted to commits@sling.apache.org by bd...@apache.org on 2013/09/04 12:14:15 UTC

svn commit: r1519966 - /sling/trunk/bundles/scripting/core/src/main/java/org/apache/sling/scripting/core/impl/DefaultSlingScript.java

Author: bdelacretaz
Date: Wed Sep  4 10:14:15 2013
New Revision: 1519966

URL: http://svn.apache.org/r1519966
Log:
SLING-3037 - avoid $ in logger names

Modified:
    sling/trunk/bundles/scripting/core/src/main/java/org/apache/sling/scripting/core/impl/DefaultSlingScript.java

Modified: sling/trunk/bundles/scripting/core/src/main/java/org/apache/sling/scripting/core/impl/DefaultSlingScript.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/scripting/core/src/main/java/org/apache/sling/scripting/core/impl/DefaultSlingScript.java?rev=1519966&r1=1519965&r2=1519966&view=diff
==============================================================================
--- sling/trunk/bundles/scripting/core/src/main/java/org/apache/sling/scripting/core/impl/DefaultSlingScript.java (original)
+++ sling/trunk/bundles/scripting/core/src/main/java/org/apache/sling/scripting/core/impl/DefaultSlingScript.java Wed Sep  4 10:14:15 2013
@@ -707,7 +707,8 @@ class DefaultSlingScript implements Slin
     private String getLoggerName() {
         String name = scriptName;
         name = name.substring(1);       // cut-off leading slash
-        name = name.replace('.', '$');  // extension separator as part of name
+        name = name.replace('.', '_');  // extension separator as part of name - used to be $ but 
+                                        // logback considers that as a separator (SLING-3037)
         name = name.replace('/', '.');  // hierarchy defined by dot
         return name;
     }