[
https://issues.apache.org/jira/browse/WW-4344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13994457#comment-13994457
]
Johannes Geppert commented on WW-4344:
--------------------------------------
Use slf4j instead of own xwork logging facade is one of the point we should
discuss for the next Struts major release.
https://cwiki.apache.org/confluence/display/WW/Struts+Next
> Improve Logging layer implementation to avoid additional boilerplate with if
> (LOG.isDebugEnable)
> ------------------------------------------------------------------------------------------------
>
> Key: WW-4344
> URL: https://issues.apache.org/jira/browse/WW-4344
> Project: Struts 2
> Issue Type: Improvement
> Components: "New" API
> Affects Versions: 2.3.16.3
> Reporter: Lukasz Lenart
> Assignee: Lukasz Lenart
> Labels: logging
> Fix For: 2.3.18
>
>
> Instead using
> {code:java}
> if (LOG.isDebugEnable()) {
> LOG.debug("Action name is [#0]', actionName);
> }
> {code}
> can be reduced to
> {code:java}
> LOG.debug("Action name is [#0]', actionName);
> {code}
> with implementation in {{JdkLogger}} and any other implementation of
> {{Logger}} interface:
> {code:java}
> public void debug(String message, String... params) {
> if(isDebugEnable()) {
> // perform logging
> }
> }
> {code}
--
This message was sent by Atlassian JIRA
(v6.2#6252)
