Author: shawkins Date: 2009-03-25 08:42:28 -0400 (Wed, 25 Mar 2009) New Revision: 643 Modified: trunk/server/src/main/java/com/metamatrix/platform/config/spi/xml/XMLActionUpdateStrategy.java Log: TEIID-428 adding detail logging of property changes Modified: trunk/server/src/main/java/com/metamatrix/platform/config/spi/xml/XMLActionUpdateStrategy.java =================================================================== --- trunk/server/src/main/java/com/metamatrix/platform/config/spi/xml/XMLActionUpdateStrategy.java 2009-03-25 12:20:12 UTC (rev 642) +++ trunk/server/src/main/java/com/metamatrix/platform/config/spi/xml/XMLActionUpdateStrategy.java 2009-03-25 12:42:28 UTC (rev 643) @@ -89,7 +89,9 @@ import com.metamatrix.common.config.model.ConfigurationModelContainerImpl; import com.metamatrix.common.config.model.ConfigurationObjectEditorHelper; import com.metamatrix.common.config.model.PropertyValidations; +import com.metamatrix.common.log.LogManager; import com.metamatrix.common.namedobject.BaseID; +import com.metamatrix.common.util.LogCommonConstants; import com.metamatrix.common.util.crypto.CryptoException; import com.metamatrix.common.util.crypto.CryptoUtil; import com.metamatrix.core.util.Assertion; @@ -2037,8 +2039,6 @@ ComponentType type, ConfigurationModelContainerImpl config, String principal) throws InvalidPropertyValueException, ConfigurationException { -// System.out.println("STRATEGY: Process Property Changes 5 " + propName + " value: " + propValue ); - validateProperty.isPropertyValid(propName, propValue); if (propValue != null && propValue.trim().length() > 0 && @@ -2046,28 +2046,24 @@ ! CryptoUtil.isValueEncrypted(propValue)) { char[] pwd = null; try { -// System.out.println("STRATEGY: Process Property Changes 5a" ); - propValue = CryptoUtil.getCryptor().encrypt(propValue); } catch ( CryptoException e ) { throw new InvalidPropertyValueException(e, ConfigMessages.CONFIG_0108, ConfigPlugin.Util.getString(ConfigMessages.CONFIG_0108)); } } -// System.out.println("STRATEGY: Process Property Changes 5b " + propName + " value: " + propValue ); - if (operation == ConfigurationObjectEditor.ADD) { + LogManager.logDetail(LogCommonConstants.CTX_CONFIG, "adding", propName, "with value", propValue); //$NON-NLS-1$ //$NON-NLS-2$ ConfigurationObjectEditorHelper.addProperty(object, propName, propValue); } else if (operation == ConfigurationObjectEditor.SET) { + LogManager.logDetail(LogCommonConstants.CTX_CONFIG, "setting", propName, "to value", propValue); //$NON-NLS-1$ //$NON-NLS-2$ ConfigurationObjectEditorHelper.setProperty(object, propName, propValue); } else if (operation == ConfigurationObjectEditor.REMOVE) { + LogManager.logDetail(LogCommonConstants.CTX_CONFIG, "removing", propName); //$NON-NLS-1$ ConfigurationObjectEditorHelper.removeProperty(object, propName); - } -// System.out.println("STRATEGY: Process Property Changes 6" ); - } /**