[
https://issues.apache.org/jira/browse/LOG4J2-231?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Ralph Goers resolved LOG4J2-231.
--------------------------------
Resolution: Fixed
Fix Version/s: 2.0-beta6
Assignee: Ralph Goers
Fix committed in revision 1479236. Created LoggingTest in the log4j-1.2 api
package to test this. Please verify and close.
> Logger.getParent
> -----------------
>
> Key: LOG4J2-231
> URL: https://issues.apache.org/jira/browse/LOG4J2-231
> Project: Log4j 2
> Issue Type: Question
> Components: Core, log4j 1.2 emulation
> Affects Versions: 2.0-beta5
> Reporter: Remko Popma
> Assignee: Ralph Goers
> Fix For: 2.0-beta6
>
>
> Not 100% sure, but this code looked a bit strange:
> (o.a.l.l.core.Logger line 65)
> public Logger getParent() {
> final LoggerConfig lc = config.loggerConfig.getParent();
> if (lc == null) {
> return null;
> }
> if (context.hasLogger(lc.getName())) {
> return context.getLogger(getName(), getMessageFactory()); //
> <-------
> }
> return new Logger(context, getName(), this.getMessageFactory()); //
> <-------
> }
> the last two return statements use the name of *this* logger instead of the
> parent name lc.getName().
> Is that correct?
> (this method is not used in core internally but is used in the logtj12-api
> module, by Category#getParent)
> I'll try to write a JUnit test for this later, still need to figure out how.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscr...@logging.apache.org
For additional commands, e-mail: log4j-dev-h...@logging.apache.org
