[jboss-cvs] JBossAS SVN: r91364 - projects/logging-service-metadata/trunk/src/main/java/org/jboss/logging/metadata.

[jboss-cvs-commits at lists.jboss.org]

Author: david.lloyd at jboss.com
Date: 2009-07-16 19:16:57 -0400 (Thu, 16 Jul 2009)
New Revision: 91364

Modified:
   projects/logging-service-metadata/trunk/src/main/java/org/jboss/logging/metadata/LoggingMetaDataHelper.java
Log:
Use constructor parameters instead of properties when constructing a FileHandler so that JBossMC's propensity to set properties in random order doesn't cause the server log to be randomly truncated on redeploy

Modified: projects/logging-service-metadata/trunk/src/main/java/org/jboss/logging/metadata/LoggingMetaDataHelper.java
===================================================================
--- projects/logging-service-metadata/trunk/src/main/java/org/jboss/logging/metadata/LoggingMetaDataHelper.java	2009-07-16 22:49:57 UTC (rev 91363)
+++ projects/logging-service-metadata/trunk/src/main/java/org/jboss/logging/metadata/LoggingMetaDataHelper.java	2009-07-16 23:16:57 UTC (rev 91364)
@@ -191,8 +191,8 @@
             } else {
                 builder.setBean(FileHandler.class.getName());
             }
-            builder.addPropertyMetaData("append", Boolean.valueOf(fileHandlerMetaData.isAppend()));
-            builder.addPropertyMetaData("fileName", fileHandlerMetaData.getFileName());
+            builder.addConstructorParameter(String.class.getName(), fileHandlerMetaData.getFileName());
+            builder.addConstructorParameter(boolean.class.getName(), Boolean.valueOf(fileHandlerMetaData.isAppend()));
         } else if (handlerMetaData instanceof Log4jAppenderMetaData) {
             builder.setBean(Log4jAppenderHandler.class.getName());
         } else if (handlerMetaData instanceof NullHandlerMetaData) {