You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@avalon.apache.org by co...@apache.org on 2002/03/08 00:06:28 UTC
cvs commit: jakarta-avalon-excalibur/src/java/org/apache/avalon/excalibur/logger DefaultLogTargetFactoryManager.java
colus 02/03/07 15:06:28
Modified: src/java/org/apache/avalon/excalibur/logger
DefaultLogTargetFactoryManager.java
Log:
Fixed wrong sequence of life cycle calls.
Submitted By: Rajendra Ghorpade<RG...@onebridge.de>
Revision Changes Path
1.7 +6 -6 jakarta-avalon-excalibur/src/java/org/apache/avalon/excalibur/logger/DefaultLogTargetFactoryManager.java
Index: DefaultLogTargetFactoryManager.java
===================================================================
RCS file: /home/cvs/jakarta-avalon-excalibur/src/java/org/apache/avalon/excalibur/logger/DefaultLogTargetFactoryManager.java,v
retrieving revision 1.6
retrieving revision 1.7
diff -u -r1.6 -r1.7
--- DefaultLogTargetFactoryManager.java 9 Feb 2002 00:29:56 -0000 1.6
+++ DefaultLogTargetFactoryManager.java 7 Mar 2002 23:06:28 -0000 1.7
@@ -23,7 +23,7 @@
* from a configuration file.
*
* @author <a href="mailto:giacomo@apache.org">Giacomo Pati</a>
- * @version CVS $Revision: 1.6 $ $Date: 2002/02/09 00:29:56 $
+ * @version CVS $Revision: 1.7 $ $Date: 2002/03/07 23:06:28 $
* @since 4.0
*/
public class DefaultLogTargetFactoryManager
@@ -119,11 +119,6 @@
((LogEnabled) logTargetFactory).enableLogging( getLogger() );
}
- if( logTargetFactory instanceof Configurable )
- {
- ((Configurable) logTargetFactory).configure( confs[i] );
- }
-
if( logTargetFactory instanceof Contextualizable )
{
try
@@ -134,6 +129,11 @@
{
throw new ConfigurationException( "cannot contextualize LogTargetFactory " + factoryClass, ce );
}
+ }
+
+ if( logTargetFactory instanceof Configurable )
+ {
+ ((Configurable) logTargetFactory).configure( confs[i] );
}
if( logTargetFactory instanceof LogTargetFactoryManageable )
--
To unsubscribe, e-mail: <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>