Author: skitching Date: Sun Apr 16 04:36:37 2006 New Revision: 394461 URL: http://svn.apache.org/viewcvs?rev=394461&view=rev Log: Fix bug where priority of first commons-logging.properties file found was ignored. Also improve diagnostics output.
Modified: jakarta/commons/proper/logging/trunk/src/java/org/apache/commons/logging/LogFactory.java Modified: jakarta/commons/proper/logging/trunk/src/java/org/apache/commons/logging/LogFactory.java URL: http://svn.apache.org/viewcvs/jakarta/commons/proper/logging/trunk/src/java/org/apache/commons/logging/LogFactory.java?rev=394461&r1=394460&r2=394461&view=diff ============================================================================== --- jakarta/commons/proper/logging/trunk/src/java/org/apache/commons/logging/LogFactory.java (original) +++ jakarta/commons/proper/logging/trunk/src/java/org/apache/commons/logging/LogFactory.java Sun Apr 16 04:36:37 2006 @@ -1387,16 +1387,39 @@ if (newProps != null) { if (props == null) { props = newProps; + String priorityStr = props.getProperty(PRIORITY_KEY); + priority = 0.0; + if (priorityStr != null) { + priority = Double.parseDouble(priorityStr); + } + + if (isDiagnosticsEnabled()) { + logDiagnostic( + "[LOOKUP] First properties file found at '" + url + "'"); + } } else { String newPriorityStr = newProps.getProperty(PRIORITY_KEY); + double newPriority = 0.0; if (newPriorityStr != null) { - double newPriority = Double.valueOf(newPriorityStr).doubleValue(); - if (newPriority > priority) { - props = newProps; - priority = newPriority; + newPriority = Double.parseDouble(newPriorityStr); + } + + if (newPriority > priority) { + props = newProps; + priority = newPriority; + + if (isDiagnosticsEnabled()) { + logDiagnostic( + "[LOOKUP] New properties file found at '" + url + "'" + + " has higher priority than earlier file."); } + } else { + logDiagnostic( + "[LOOKUP] New properties file found at '" + url + "'" + + " has less priority than earlier file -- ignoring."); } } + } } } catch (SecurityException e) { --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]