Skip to content

Commit a5bf37c

Browse files
committed
Merge fix for #234 into 3.2.x branch
1 parent 7e68df0 commit a5bf37c

File tree

2 files changed

+19
-12
lines changed

2 files changed

+19
-12
lines changed

pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,7 @@
163163
<dependency>
164164
<groupId>org.apache.logging.log4j</groupId>
165165
<artifactId>log4j-core</artifactId>
166-
<version>2.0-rc1</version>
166+
<version>2.0.2</version>
167167
<optional>true</optional>
168168
</dependency>
169169
<dependency>

src/main/java/org/apache/ibatis/logging/log4j2/Log4j2AbstractLoggerImpl.java

Lines changed: 18 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -22,49 +22,56 @@
2222
import org.apache.logging.log4j.MarkerManager;
2323
import org.apache.logging.log4j.message.SimpleMessage;
2424
import org.apache.logging.log4j.spi.AbstractLogger;
25-
import org.apache.logging.log4j.spi.AbstractLoggerWrapper;
25+
import org.apache.logging.log4j.spi.ExtendedLoggerWrapper;
2626

2727
/**
2828
* @author Eduardo Macarron
2929
*/
3030
public class Log4j2AbstractLoggerImpl implements Log {
31-
31+
3232
private static Marker MARKER = MarkerManager.getMarker(LogFactory.MARKER);
33-
33+
3434
private static final String FQCN = Log4j2Impl.class.getName();
35-
36-
private AbstractLoggerWrapper log;
35+
36+
private ExtendedLoggerWrapper log;
3737

3838
public Log4j2AbstractLoggerImpl(AbstractLogger abstractLogger) {
39-
log = new AbstractLoggerWrapper(abstractLogger, abstractLogger.getName(), abstractLogger.getMessageFactory());
39+
log = new ExtendedLoggerWrapper(abstractLogger, abstractLogger.getName(), abstractLogger.getMessageFactory());
4040
}
4141

42+
@Override
4243
public boolean isDebugEnabled() {
4344
return log.isDebugEnabled();
4445
}
4546

47+
@Override
4648
public boolean isTraceEnabled() {
4749
return log.isTraceEnabled();
4850
}
4951

52+
@Override
5053
public void error(String s, Throwable e) {
51-
log.log(MARKER, FQCN, Level.ERROR, new SimpleMessage(s), e);
54+
log.logIfEnabled(FQCN, Level.ERROR, MARKER, new SimpleMessage(s), e);
5255
}
5356

57+
@Override
5458
public void error(String s) {
55-
log.log(MARKER, FQCN, Level.ERROR, new SimpleMessage(s), null);
59+
log.logIfEnabled(FQCN, Level.ERROR, MARKER, new SimpleMessage(s), null);
5660
}
5761

62+
@Override
5863
public void debug(String s) {
59-
log.log(MARKER, FQCN, Level.DEBUG, new SimpleMessage(s), null);
64+
log.logIfEnabled(FQCN, Level.DEBUG, MARKER, new SimpleMessage(s), null);
6065
}
6166

67+
@Override
6268
public void trace(String s) {
63-
log.log(MARKER, FQCN, Level.TRACE, new SimpleMessage(s), null);
69+
log.logIfEnabled(FQCN, Level.TRACE, MARKER, new SimpleMessage(s), null);
6470
}
6571

72+
@Override
6673
public void warn(String s) {
67-
log.log(MARKER, FQCN, Level.WARN, new SimpleMessage(s), null);
74+
log.logIfEnabled(FQCN, Level.WARN, MARKER, new SimpleMessage(s), null);
6875
}
6976

7077
}

0 commit comments

Comments
 (0)