- Nuiton-utils-commits - mailman.cloud.codelutin.com

r2205 - trunk
by maven-release＠users.nuiton.org Sept. 9, 2011

Sept. 9, 2011

Author: maven-release Date: 2011-09-09 17:50:50 +0200 (Fri, 09 Sep 2011) New Revision: 2205 Url: http://nuiton.org/repositories/revision/nuiton-utils/2205 Log: Update mavenpom4redmineAndCentral to 3.0.3. Modified: trunk/pom.xml Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2011-09-09 10:38:17 UTC (rev 2204) +++ trunk/pom.xml 2011-09-09 15:50:50 UTC (rev 2205) @@ -22,9 +22,7 @@ <http://www.gnu.org/licenses/lgpl-3.0.html>. #L% --> -<project xmlns="http://maven.apache.org/POM/4.0.0" - xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" - xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd"> +<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd"> <modelVersion>4.0.0</modelVersion> @@ -35,7 +33,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>mavenpom4redmineAndCentral</artifactId> - <version>3.0.2</version> + <version>3.0.3</version> </parent> <artifactId>nuiton-utils-parent</artifactId>

1 0

r2204 - in trunk/nuiton-utils/src: main/java/org/nuiton/util test/java/org/nuiton/util
by athimel＠users.nuiton.org Sept. 9, 2011

Sept. 9, 2011

Author: athimel Date: 2011-09-09 12:38:17 +0200 (Fri, 09 Sep 2011) New Revision: 2204 Url: http://nuiton.org/repositories/revision/nuiton-utils/2204 Log: #1734 Add getOptionAsLong on ApplicationConfig Modified: trunk/nuiton-utils/src/main/java/org/nuiton/util/ApplicationConfig.java trunk/nuiton-utils/src/test/java/org/nuiton/util/ApplicationConfigTest.java Modified: trunk/nuiton-utils/src/main/java/org/nuiton/util/ApplicationConfig.java =================================================================== --- trunk/nuiton-utils/src/main/java/org/nuiton/util/ApplicationConfig.java 2011-09-09 05:09:07 UTC (rev 2203) +++ trunk/nuiton-utils/src/main/java/org/nuiton/util/ApplicationConfig.java 2011-09-09 10:38:17 UTC (rev 2204) @@ -1258,6 +1258,21 @@ return result; } + /** + * Get option value as {@code long}. + * + * @param key the option's key + * @return value as {@code long} + */ + public long getOptionAsLong(String key) { + Long result = getOption(Long.class, key); + if (result == null) { + // primitive value can not be null + result = 0L; + } + return result; + } + /** * Get option value as {@code float}. * Modified: trunk/nuiton-utils/src/test/java/org/nuiton/util/ApplicationConfigTest.java =================================================================== --- trunk/nuiton-utils/src/test/java/org/nuiton/util/ApplicationConfigTest.java 2011-09-09 05:09:07 UTC (rev 2203) +++ trunk/nuiton-utils/src/test/java/org/nuiton/util/ApplicationConfigTest.java 2011-09-09 10:38:17 UTC (rev 2204) @@ -426,6 +426,7 @@ Assert.assertNotNull(instance.getOptionAsBoolean("dfsdfgqsgqfg")); Assert.assertNotNull(instance.getOptionAsDouble("dfsdfgqsgqfg")); Assert.assertNotNull(instance.getOptionAsInt("dfsdfgqsgqfg")); + Assert.assertNotNull(instance.getOptionAsLong("dfsdfgqsgqfg")); // list option can not be null Assert.assertNotNull(instance.getOptionAsList("dfsdfgqsgqfg")); @@ -508,4 +509,21 @@ Assert.assertEquals(expected, actual); } + + @Test + public void testGetOptionAsLong() { + ApplicationConfig instance = new ApplicationConfig(); + + // test long is not null + long actual = instance.getOptionAsLong("toto"); + Assert.assertNotNull(actual); + Assert.assertEquals(0l, actual); + + // not null version + long expected = System.currentTimeMillis(); + instance.setOption("toto", "" + expected); + actual = instance.getOptionAsLong("toto"); + Assert.assertEquals(expected, actual); + + } }

1 0

r2203 - trunk/nuiton-utils/src/main/java/org/nuiton/util/beans
by tchemit＠users.nuiton.org Sept. 9, 2011

Sept. 9, 2011

Author: tchemit Date: 2011-09-09 07:09:07 +0200 (Fri, 09 Sep 2011) New Revision: 2203 Url: http://nuiton.org/repositories/revision/nuiton-utils/2203 Log: Evolution #1728: add diff method in Binder to compare the values of two bean : add type in PropertyDiff + reformat code Modified: trunk/nuiton-utils/src/main/java/org/nuiton/util/beans/Binder.java Modified: trunk/nuiton-utils/src/main/java/org/nuiton/util/beans/Binder.java =================================================================== --- trunk/nuiton-utils/src/main/java/org/nuiton/util/beans/Binder.java 2011-09-09 05:08:34 UTC (rev 2202) +++ trunk/nuiton-utils/src/main/java/org/nuiton/util/beans/Binder.java 2011-09-09 05:09:07 UTC (rev 2203) @@ -140,16 +140,16 @@ * Obtain from the given object all properties registered in the binder * model. * - * @param source the bean to read - * @param propertyNames subset of properties to load + * @param source the bean to read + * @param propertyNames subset of properties to load * @param includeNullValues get all the properties and - * values for the given bean. If false, you'll get only the values + * values for the given bean. If false, you'll get only the values * @return the map of properties obtained indexed by their property name, * or an empty map is the given {@code from} is {@code null}. * @since 2.3 */ public Map<String, Object> obtainProperties(I source, - boolean includeNullValues, String... propertyNames) { + boolean includeNullValues, String... propertyNames) { if (source == null) { // special limit case return Collections.emptyMap(); @@ -188,7 +188,7 @@ } } - boolean include = read != null || read == null && includeNullValues; + boolean include = read != null || includeNullValues; if (include) { result.put(sourceProperty, read); } @@ -329,7 +329,7 @@ } protected Object readProperty(String sourceProperty, Object source, - Method readMethod) { + Method readMethod) { try { Object read = null; if (source != null) { @@ -368,6 +368,8 @@ public class PropertyDiff { + protected Class<?> propertyType; + protected String sourceProperty; protected Object sourceValue; @@ -376,13 +378,19 @@ protected Object targetValue; - public PropertyDiff() {} + public PropertyDiff() { + } - public PropertyDiff(String sourceProperty, Object sourceValue, String targetProperty, Object targetValue) { + public PropertyDiff(String sourceProperty, + Object sourceValue, + String targetProperty, + Object targetValue, + Class<?> propertyType) { this.sourceProperty = sourceProperty; this.sourceValue = sourceValue; this.targetProperty = targetProperty; this.targetValue = targetValue; + this.propertyType= propertyType; } public String getSourceProperty() { @@ -416,10 +424,20 @@ public void setTargetValue(Object targetValue) { this.targetValue = targetValue; } + + public Class<?> getPropertyType() { + return propertyType; + } + + public void setPropertyType(Class<?> propertyType) { + this.propertyType = propertyType; + } } - protected List<PropertyDiff> diff(I source, O target, boolean excludeProperties, - String... propertyNames) { + protected List<PropertyDiff> diff(I source, + O target, + boolean excludeProperties, + String... propertyNames) { if (source == null) { throw new NullPointerException("parameter 'source' can no be null"); } @@ -437,16 +455,24 @@ Method sourceReadMethod = model.getSourceReadMethod(sourceProperty); - Object sourceRead = readProperty(sourceProperty, source, sourceReadMethod); + Object sourceRead = readProperty(sourceProperty, source, + sourceReadMethod); String targetProperty = model.getTargetProperty(sourceProperty); Method targetReadMethod = model.getTargetReadMethod(targetProperty); - Object targetRead = readProperty(targetProperty, target, targetReadMethod); + Object targetRead = readProperty(targetProperty, target, + targetReadMethod); if (ObjectUtils.notEqual(sourceRead, targetRead)) { - PropertyDiff propertyDiff = new PropertyDiff(sourceProperty, sourceRead, targetProperty, targetRead); + PropertyDiff propertyDiff = new PropertyDiff( + sourceProperty, + sourceRead, + targetProperty, + targetRead, + sourceReadMethod.getReturnType() + ); result.add(propertyDiff); } } @@ -454,34 +480,39 @@ return result; } - /** Compare two beans property by property according to the model. - * + /** + * Compare two beans property by property according to the model. + * * List contains one element per property with different values (according * to the result of an equals() call) * * @param source a bean of type I * @param target a bean of type O * @return a list with all the properties which values differ in source - * and target. Properties with equal values are not included. + * and target. Properties with equal values are not included. * @since 2.3 */ public List<PropertyDiff> diff(I source, O target) { return diff(source, target, false); } - /** Compare two beans property by property according to the model. - * + /** + * Compare two beans property by property according to the model. + * * List contains one element per property with different values (according * to the result of an equals() call) * * @param source a bean of type I * @param target a bean of type O + * @param propertyNames property names to exclude from the diff * @return a list with all the properties which values differ in source - * and target. Properties with equal values and excluded properties will - * not be contained in the result + * and target. Properties with equal values and excluded properties + * will not be contained in the result * @since 2.3 */ - public List<PropertyDiff> diffExcluding(I source, O target, String... propertyNames) { + public List<PropertyDiff> diffExcluding(I source, + O target, + String... propertyNames) { return diff(source, target, true, propertyNames); }

1 0

r2202 - trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field
by tchemit＠users.nuiton.org Sept. 9, 2011

Sept. 9, 2011

Author: tchemit Date: 2011-09-09 07:08:34 +0200 (Fri, 09 Sep 2011) New Revision: 2202 Url: http://nuiton.org/repositories/revision/nuiton-utils/2202 Log: add svn properties Modified: trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/SkipableRequiredFieldValidator.java trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/SkipableRequiredStringFieldValidator.java Property changes on: trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/SkipableRequiredFieldValidator.java ___________________________________________________________________ Added: svn:keywords + Author Date Id Revision HeadURL Added: svn:eol-style + native Property changes on: trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/SkipableRequiredStringFieldValidator.java ___________________________________________________________________ Added: svn:keywords + Author Date Id Revision HeadURL Added: svn:eol-style + native

1 0

r2201 - in trunk/nuiton-utils/src: main/java/org/nuiton/util/beans test/java/org/nuiton/util/beans
by bleny＠users.nuiton.org Sept. 1, 2011

Sept. 1, 2011

Author: bleny Date: 2011-09-01 15:30:35 +0200 (Thu, 01 Sep 2011) New Revision: 2201 Url: http://nuiton.org/repositories/revision/nuiton-utils/2201 Log: #1729 add flag argument in Binder#obtainProperties to get all values even if null Modified: trunk/nuiton-utils/src/main/java/org/nuiton/util/beans/Binder.java trunk/nuiton-utils/src/test/java/org/nuiton/util/beans/BinderTest.java Modified: trunk/nuiton-utils/src/main/java/org/nuiton/util/beans/Binder.java =================================================================== --- trunk/nuiton-utils/src/main/java/org/nuiton/util/beans/Binder.java 2011-09-01 12:23:57 UTC (rev 2200) +++ trunk/nuiton-utils/src/main/java/org/nuiton/util/beans/Binder.java 2011-09-01 13:30:35 UTC (rev 2201) @@ -137,19 +137,19 @@ } /** - * Obtain from the given object all properties registred in the binder + * Obtain from the given object all properties registered in the binder * model. - * - * Note: If a property's value is null, it will not be injected in - * the result. * * @param source the bean to read * @param propertyNames subset of properties to load + * @param includeNullValues get all the properties and + * values for the given bean. If false, you'll get only the values * @return the map of properties obtained indexed by their property name, * or an empty map is the given {@code from} is {@code null}. + * @since 2.3 */ public Map<String, Object> obtainProperties(I source, - String... propertyNames) { + boolean includeNullValues, String... propertyNames) { if (source == null) { // special limit case return Collections.emptyMap(); @@ -174,22 +174,24 @@ // for primitive type case, force nullity read = null; } - if (read == null) { - continue; + if (read != null) { + if (model.containsBinderProperty(sourceProperty)) { + if (model.containsCollectionProperty(sourceProperty)) { + read = bindCollection(sourceProperty, read); + } else { + read = bindProperty(sourceProperty, read); + } + } else if (model.containsCollectionProperty(sourceProperty)) { + + // specific collection strategy is set, must use it + read = getCollectionValue(sourceProperty, read); + } } - if (model.containsBinderProperty(sourceProperty)) { - if (model.containsCollectionProperty(sourceProperty)) { - read = bindCollection(sourceProperty, read); - } else { - read = bindProperty(sourceProperty, read); - } - } else if (model.containsCollectionProperty(sourceProperty)) { - // specific collection strategy is set, must use it - read = getCollectionValue(sourceProperty, read); + boolean include = read != null || read == null && includeNullValues; + if (include) { + result.put(sourceProperty, read); } - - result.put(sourceProperty, read); } catch (IllegalAccessException e) { throw new RuntimeException(e); } catch (InvocationTargetException e) { @@ -202,6 +204,23 @@ } /** + * Obtain from the given object all properties registered in the binder + * model. + * + * Note: If a property's value is null, it will not be injected in + * the result. + * + * @param source the bean to read + * @param propertyNames subset of properties to load + * @return the map of properties obtained indexed by their property name, + * or an empty map is the given {@code from} is {@code null}. + */ + public Map<String, Object> obtainProperties(I source, + String... propertyNames) { + return obtainProperties(source, false, propertyNames); + } + + /** * Copy properties from a source bean to a destination one according to * the model of the binder. If {@code propertyNames} is defined, only * those properties will be copied. @@ -265,7 +284,6 @@ getAllPropertiesExclude(propertyNames) : getProperties(propertyNames); - for (String sourceProperty : propertyNames) { String targetProperty = model.getTargetProperty(sourceProperty); Modified: trunk/nuiton-utils/src/test/java/org/nuiton/util/beans/BinderTest.java =================================================================== --- trunk/nuiton-utils/src/test/java/org/nuiton/util/beans/BinderTest.java 2011-09-01 12:23:57 UTC (rev 2200) +++ trunk/nuiton-utils/src/test/java/org/nuiton/util/beans/BinderTest.java 2011-09-01 13:30:35 UTC (rev 2201) @@ -107,6 +107,9 @@ map = binderA.obtainProperties(a); Assert.assertEquals(0, map.size()); + map = binderA.obtainProperties(a, true); + Assert.assertEquals(1, map.size()); + map = binderB.obtainProperties(a); Assert.assertEquals(0, map.size());

1 0

r2200 - trunk/nuiton-utils/src/main/java/org/nuiton/util/beans
by bleny＠users.nuiton.org Sept. 1, 2011

Sept. 1, 2011

Author: bleny Date: 2011-09-01 14:23:57 +0200 (Thu, 01 Sep 2011) New Revision: 2200 Url: http://nuiton.org/repositories/revision/nuiton-utils/2200 Log: #1728: add diffExluding and @since tag Modified: trunk/nuiton-utils/src/main/java/org/nuiton/util/beans/Binder.java Modified: trunk/nuiton-utils/src/main/java/org/nuiton/util/beans/Binder.java =================================================================== --- trunk/nuiton-utils/src/main/java/org/nuiton/util/beans/Binder.java 2011-09-01 10:09:16 UTC (rev 2199) +++ trunk/nuiton-utils/src/main/java/org/nuiton/util/beans/Binder.java 2011-09-01 12:23:57 UTC (rev 2200) @@ -445,11 +445,28 @@ * @param target a bean of type O * @return a list with all the properties which values differ in source * and target. Properties with equal values are not included. + * @since 2.3 */ public List<PropertyDiff> diff(I source, O target) { return diff(source, target, false); } + /** Compare two beans property by property according to the model. + * + * List contains one element per property with different values (according + * to the result of an equals() call) + * + * @param source a bean of type I + * @param target a bean of type O + * @return a list with all the properties which values differ in source + * and target. Properties with equal values and excluded properties will + * not be contained in the result + * @since 2.3 + */ + public List<PropertyDiff> diffExcluding(I source, O target, String... propertyNames) { + return diff(source, target, true, propertyNames); + } + /** * Get the model of the binder. *

1 0

r2199 - trunk/nuiton-utils/src/main/java/org/nuiton/util/beans
by bleny＠users.nuiton.org Sept. 1, 2011

Sept. 1, 2011

Author: bleny Date: 2011-09-01 12:09:16 +0200 (Thu, 01 Sep 2011) New Revision: 2199 Url: http://nuiton.org/repositories/revision/nuiton-utils/2199 Log: #1728: a bit of refactoring on diff() Modified: trunk/nuiton-utils/src/main/java/org/nuiton/util/beans/Binder.java Modified: trunk/nuiton-utils/src/main/java/org/nuiton/util/beans/Binder.java =================================================================== --- trunk/nuiton-utils/src/main/java/org/nuiton/util/beans/Binder.java 2011-09-01 09:53:58 UTC (rev 2198) +++ trunk/nuiton-utils/src/main/java/org/nuiton/util/beans/Binder.java 2011-09-01 10:09:16 UTC (rev 2199) @@ -310,10 +310,10 @@ } } - protected Object readSourceProperty(String sourceProperty, I source) { + protected Object readProperty(String sourceProperty, Object source, + Method readMethod) { try { Object read = null; - Method readMethod = model.getSourceReadMethod(sourceProperty); if (source != null) { // obtain value from source read = readMethod.invoke(source); @@ -348,44 +348,6 @@ } } - protected Object readTargetProperty(String targetProperty, O target) { - try { - Object read = null; - Method readMethod = model.getTargetReadMethod(targetProperty); - if (target != null) { - // obtain value from source - read = readMethod.invoke(target); - } - // obtain acceptable null value (for primitive types, use - // default values). - if (read == null) { - read = ObjectUtil.getNullValue(readMethod.getReturnType()); - } - if (log.isDebugEnabled()) { - log.debug("property " + targetProperty + ", type : " + - readMethod.getReturnType() + ", value = " + read); - } - - if (model.containsBinderProperty(targetProperty)) { - if (model.containsCollectionProperty(targetProperty)) { - read = bindCollection(targetProperty, read); - } else { - read = bindProperty(targetProperty, read); - } - } else if (model.containsCollectionProperty(targetProperty)) { - - // specific collection strategy is set, must use it - read = getCollectionValue(targetProperty, read); - } - - return read; - } catch (Exception e) { - throw new RuntimeException( - "could not read property " + targetProperty + - " on target " + target); - } - } - public class PropertyDiff { protected String sourceProperty; @@ -440,6 +402,9 @@ protected List<PropertyDiff> diff(I source, O target, boolean excludeProperties, String... propertyNames) { + if (source == null) { + throw new NullPointerException("parameter 'source' can no be null"); + } if (target == null) { throw new NullPointerException("parameter 'target' can no be null"); } @@ -452,12 +417,16 @@ for (String sourceProperty : propertyNames) { - Object sourceRead = readSourceProperty(sourceProperty, source); + Method sourceReadMethod = model.getSourceReadMethod(sourceProperty); + Object sourceRead = readProperty(sourceProperty, source, sourceReadMethod); + String targetProperty = model.getTargetProperty(sourceProperty); - Object targetRead = readTargetProperty(targetProperty, target); + Method targetReadMethod = model.getTargetReadMethod(targetProperty); + Object targetRead = readProperty(targetProperty, target, targetReadMethod); + if (ObjectUtils.notEqual(sourceRead, targetRead)) { PropertyDiff propertyDiff = new PropertyDiff(sourceProperty, sourceRead, targetProperty, targetRead); result.add(propertyDiff);

1 0

r2198 - in trunk/nuiton-utils/src: main/java/org/nuiton/util/beans test/java/org/nuiton/util/beans
by bleny＠users.nuiton.org Sept. 1, 2011

Sept. 1, 2011

Author: bleny Date: 2011-09-01 11:53:58 +0200 (Thu, 01 Sep 2011) New Revision: 2198 Url: http://nuiton.org/repositories/revision/nuiton-utils/2198 Log: #1728: fisrt try by computing a diff given two bean. test included Modified: trunk/nuiton-utils/src/main/java/org/nuiton/util/beans/Binder.java trunk/nuiton-utils/src/test/java/org/nuiton/util/beans/BinderTest.java Modified: trunk/nuiton-utils/src/main/java/org/nuiton/util/beans/Binder.java =================================================================== --- trunk/nuiton-utils/src/main/java/org/nuiton/util/beans/Binder.java 2011-08-30 10:48:22 UTC (rev 2197) +++ trunk/nuiton-utils/src/main/java/org/nuiton/util/beans/Binder.java 2011-09-01 09:53:58 UTC (rev 2198) @@ -25,6 +25,7 @@ package org.nuiton.util.beans; +import org.apache.commons.lang.ObjectUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.nuiton.util.ObjectUtil; @@ -38,6 +39,7 @@ import java.util.Collection; import java.util.Collections; import java.util.HashSet; +import java.util.LinkedList; import java.util.List; import java.util.Map; import java.util.Set; @@ -308,6 +310,177 @@ } } + protected Object readSourceProperty(String sourceProperty, I source) { + try { + Object read = null; + Method readMethod = model.getSourceReadMethod(sourceProperty); + if (source != null) { + // obtain value from source + read = readMethod.invoke(source); + } + // obtain acceptable null value (for primitive types, use + // default values). + if (read == null) { + read = ObjectUtil.getNullValue(readMethod.getReturnType()); + } + if (log.isDebugEnabled()) { + log.debug("property " + sourceProperty + ", type : " + + readMethod.getReturnType() + ", value = " + read); + } + + if (model.containsBinderProperty(sourceProperty)) { + if (model.containsCollectionProperty(sourceProperty)) { + read = bindCollection(sourceProperty, read); + } else { + read = bindProperty(sourceProperty, read); + } + } else if (model.containsCollectionProperty(sourceProperty)) { + + // specific collection strategy is set, must use it + read = getCollectionValue(sourceProperty, read); + } + + return read; + } catch (Exception e) { + throw new RuntimeException( + "could not read property " + sourceProperty + + " on source " + source); + } + } + + protected Object readTargetProperty(String targetProperty, O target) { + try { + Object read = null; + Method readMethod = model.getTargetReadMethod(targetProperty); + if (target != null) { + // obtain value from source + read = readMethod.invoke(target); + } + // obtain acceptable null value (for primitive types, use + // default values). + if (read == null) { + read = ObjectUtil.getNullValue(readMethod.getReturnType()); + } + if (log.isDebugEnabled()) { + log.debug("property " + targetProperty + ", type : " + + readMethod.getReturnType() + ", value = " + read); + } + + if (model.containsBinderProperty(targetProperty)) { + if (model.containsCollectionProperty(targetProperty)) { + read = bindCollection(targetProperty, read); + } else { + read = bindProperty(targetProperty, read); + } + } else if (model.containsCollectionProperty(targetProperty)) { + + // specific collection strategy is set, must use it + read = getCollectionValue(targetProperty, read); + } + + return read; + } catch (Exception e) { + throw new RuntimeException( + "could not read property " + targetProperty + + " on target " + target); + } + } + + public class PropertyDiff { + + protected String sourceProperty; + + protected Object sourceValue; + + protected String targetProperty; + + protected Object targetValue; + + public PropertyDiff() {} + + public PropertyDiff(String sourceProperty, Object sourceValue, String targetProperty, Object targetValue) { + this.sourceProperty = sourceProperty; + this.sourceValue = sourceValue; + this.targetProperty = targetProperty; + this.targetValue = targetValue; + } + + public String getSourceProperty() { + return sourceProperty; + } + + public void setSourceProperty(String sourceProperty) { + this.sourceProperty = sourceProperty; + } + + public Object getSourceValue() { + return sourceValue; + } + + public void setSourceValue(Object sourceValue) { + this.sourceValue = sourceValue; + } + + public String getTargetProperty() { + return targetProperty; + } + + public void setTargetProperty(String targetProperty) { + this.targetProperty = targetProperty; + } + + public Object getTargetValue() { + return targetValue; + } + + public void setTargetValue(Object targetValue) { + this.targetValue = targetValue; + } + } + + protected List<PropertyDiff> diff(I source, O target, boolean excludeProperties, + String... propertyNames) { + if (target == null) { + throw new NullPointerException("parameter 'target' can no be null"); + } + + propertyNames = excludeProperties ? + getAllPropertiesExclude(propertyNames) : + getProperties(propertyNames); + + List<PropertyDiff> result = new LinkedList<PropertyDiff>(); + + for (String sourceProperty : propertyNames) { + + Object sourceRead = readSourceProperty(sourceProperty, source); + + String targetProperty = model.getTargetProperty(sourceProperty); + + Object targetRead = readTargetProperty(targetProperty, target); + + if (ObjectUtils.notEqual(sourceRead, targetRead)) { + PropertyDiff propertyDiff = new PropertyDiff(sourceProperty, sourceRead, targetProperty, targetRead); + result.add(propertyDiff); + } + } + + return result; + } + + /** Compare two beans property by property according to the model. + * + * List contains one element per property with different values (according + * to the result of an equals() call) + * + * @param source a bean of type I + * @param target a bean of type O + * @return a list with all the properties which values differ in source + * and target. Properties with equal values are not included. + */ + public List<PropertyDiff> diff(I source, O target) { + return diff(source, target, false); + } + /** * Get the model of the binder. * Modified: trunk/nuiton-utils/src/test/java/org/nuiton/util/beans/BinderTest.java =================================================================== --- trunk/nuiton-utils/src/test/java/org/nuiton/util/beans/BinderTest.java 2011-08-30 10:48:22 UTC (rev 2197) +++ trunk/nuiton-utils/src/test/java/org/nuiton/util/beans/BinderTest.java 2011-09-01 09:53:58 UTC (rev 2198) @@ -31,6 +31,7 @@ import org.junit.BeforeClass; import org.junit.Test; +import java.util.List; import java.util.Map; public class BinderTest { @@ -101,7 +102,7 @@ } @Test - public void testObtainProperties() throws Exception { + public void testObtainProperties() { Map<String, Object> map; map = binderA.obtainProperties(a); Assert.assertEquals(0, map.size()); @@ -142,7 +143,7 @@ } @Test - public void testCopy() throws Exception { + public void testCopy() { a.setA(VALUE_A); a.setB(VALUE_B); @@ -172,7 +173,7 @@ } @Test - public void testCopyIncluding() throws Exception { + public void testCopyIncluding() { a.setA(VALUE_A); a.setB(VALUE_B); @@ -189,7 +190,7 @@ } @Test - public void testCopyExcluding() throws Exception { + public void testCopyExcluding() { a.setA(VALUE_A); a.setB(VALUE_B); @@ -224,4 +225,26 @@ Assert.assertEquals(VALUE_E, b.getE2()); } + + @Test + public void testDiff() { + a.setA(VALUE_A); + b.setA(VALUE_A); + a.setB(VALUE_A); + b.setB(VALUE_B); + + List<Binder<BeanA, BeanB>.PropertyDiff> diff = binderB.diff(a, b); + + // only one property has changed + Assert.assertEquals(1, diff.size()); + + // names of the properties used for comparison + Binder.PropertyDiff propertyDiff = diff.get(0); + Assert.assertEquals(BeanA.PROPERTY_B, propertyDiff.getSourceProperty()); + Assert.assertEquals(BeanB.PROPERTY_B, propertyDiff.getTargetValue()); + + // old and new values + Assert.assertEquals(VALUE_A, propertyDiff.getSourceValue()); + Assert.assertEquals(VALUE_B, propertyDiff.getTargetValue()); + } }

1 0

r2197 - in trunk/nuiton-validator/src: main/java/org/nuiton/validator/xwork2/field test/java/org/nuiton/validator/xwork2/field
by jcouteau＠users.nuiton.org Aug. 30, 2011

Aug. 30, 2011

Author: jcouteau Date: 2011-08-30 12:48:22 +0200 (Tue, 30 Aug 2011) New Revision: 2197 Url: http://nuiton.org/repositories/revision/nuiton-utils/2197 Log: Validator validate null and empty values (use required or requiredstring validators for those cases) Add a SkipableRequired and SkipableRequiredString validators Added: trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/SkipableRequiredFieldValidator.java trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/SkipableRequiredStringFieldValidator.java Modified: trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/EmailFieldValidator.java trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/FrenchCityNameFieldValidator.java trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/FrenchLastNameFieldValidator.java trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/FrenchPhoneNumberFieldValidator.java trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/FrenchPostCodeFieldValidator.java trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/FrenchSiretFieldValidator.java trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/VATIdentificationNumberFieldValidator.java trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/EmailFieldValidatorTest.java trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/FrenchCityNameFieldValidatorTest.java trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/FrenchLastNameFieldValidatorTest.java trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/FrenchPhoneNumberFieldValidatorTest.java trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/FrenchPostCodeFieldValidatorTest.java trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/FrenchSiretFieldValidatorTest.java trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/VATIdentificationNumberFieldValidatorTest.java Modified: trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/EmailFieldValidator.java =================================================================== --- trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/EmailFieldValidator.java 2011-08-29 14:19:18 UTC (rev 2196) +++ trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/EmailFieldValidator.java 2011-08-30 10:48:22 UTC (rev 2197) @@ -25,6 +25,7 @@ package org.nuiton.validator.xwork2.field; import com.opensymphony.xwork2.validator.ValidationException; +import org.apache.commons.lang.StringUtils; import org.nuiton.util.StringUtil; /** @@ -47,6 +48,10 @@ return; } if (value instanceof String) { + if (StringUtils.isEmpty((String)value)){ + // no value defined + return; + } if (!StringUtil.isEmail((String) value)) { addFieldError(fieldName, object); } Modified: trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/FrenchCityNameFieldValidator.java =================================================================== --- trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/FrenchCityNameFieldValidator.java 2011-08-29 14:19:18 UTC (rev 2196) +++ trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/FrenchCityNameFieldValidator.java 2011-08-30 10:48:22 UTC (rev 2197) @@ -56,6 +56,10 @@ return; } if (value instanceof String) { + if ("".equals(value)) { + // no value defined + return; + } Matcher m = p.matcher((String) value); if (!m.matches()) { addFieldError(fieldName, object); Modified: trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/FrenchLastNameFieldValidator.java =================================================================== --- trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/FrenchLastNameFieldValidator.java 2011-08-29 14:19:18 UTC (rev 2196) +++ trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/FrenchLastNameFieldValidator.java 2011-08-30 10:48:22 UTC (rev 2197) @@ -54,6 +54,10 @@ return; } if (value instanceof String) { + if ("".equals(value)) { + // no value defined + return; + } Matcher m = p.matcher((String) value); if (!m.matches()) { addFieldError(fieldName, object); Modified: trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/FrenchPhoneNumberFieldValidator.java =================================================================== --- trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/FrenchPhoneNumberFieldValidator.java 2011-08-29 14:19:18 UTC (rev 2196) +++ trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/FrenchPhoneNumberFieldValidator.java 2011-08-30 10:48:22 UTC (rev 2197) @@ -57,6 +57,10 @@ return; } if (value instanceof String) { + if ("".equals(value)) { + // no value defined + return; + } Matcher m = p.matcher((String) value); if (!m.matches()) { addFieldError(fieldName, object); Modified: trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/FrenchPostCodeFieldValidator.java =================================================================== --- trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/FrenchPostCodeFieldValidator.java 2011-08-29 14:19:18 UTC (rev 2196) +++ trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/FrenchPostCodeFieldValidator.java 2011-08-30 10:48:22 UTC (rev 2197) @@ -57,6 +57,10 @@ return; } if (value instanceof String) { + if ("".equals(value)){ + // no value defined + return; + } Matcher m = p.matcher((String) value); if (!m.matches()) { addFieldError(fieldName, object); Modified: trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/FrenchSiretFieldValidator.java =================================================================== --- trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/FrenchSiretFieldValidator.java 2011-08-29 14:19:18 UTC (rev 2196) +++ trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/FrenchSiretFieldValidator.java 2011-08-30 10:48:22 UTC (rev 2197) @@ -30,6 +30,7 @@ import java.util.Collection; import java.util.regex.Matcher; import java.util.regex.Pattern; +import org.apache.commons.lang.StringUtils; /** * Validator for French SIRET numbers @@ -80,6 +81,11 @@ siret = String.valueOf(value); } + if (StringUtils.isEmpty(siret)) { + // no value defined + return; + } + Matcher m = p.matcher(siret); if (!m.matches() || !luhnChecksum(siret)) { addFieldError(fieldName, object); Added: trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/SkipableRequiredFieldValidator.java =================================================================== --- trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/SkipableRequiredFieldValidator.java (rev 0) +++ trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/SkipableRequiredFieldValidator.java 2011-08-30 10:48:22 UTC (rev 2197) @@ -0,0 +1,20 @@ +package org.nuiton.validator.xwork2.field; + +import com.opensymphony.xwork2.validator.ValidationException; + +/** + * @author jcouteau <couteau(a)codelutin.com> + * @since 2.3 + */ +public class SkipableRequiredFieldValidator extends NuitonFieldValidatorSupport { + + @Override + protected void validateWhenNotSkip(Object object) throws ValidationException { + String fieldName = getFieldName(); + Object value = getFieldValue(fieldName, object); + + if (value == null) { + addFieldError(fieldName, object); + } + } +} Added: trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/SkipableRequiredStringFieldValidator.java =================================================================== --- trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/SkipableRequiredStringFieldValidator.java (rev 0) +++ trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/SkipableRequiredStringFieldValidator.java 2011-08-30 10:48:22 UTC (rev 2197) @@ -0,0 +1,20 @@ +package org.nuiton.validator.xwork2.field; + +import com.opensymphony.xwork2.validator.ValidationException; + +/** + * @author jcouteau <couteau(a)codelutin.com> + * @since 2.3 + */ +public class SkipableRequiredStringFieldValidator extends NuitonFieldValidatorSupport { + + @Override + protected void validateWhenNotSkip(Object object) throws ValidationException { + String fieldName = getFieldName(); + Object value = getFieldValue(fieldName, object); + + if (value == null || !(value instanceof String) || "".equals(value)) { + addFieldError(fieldName, object); + } + } +} Modified: trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/VATIdentificationNumberFieldValidator.java =================================================================== --- trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/VATIdentificationNumberFieldValidator.java 2011-08-29 14:19:18 UTC (rev 2196) +++ trunk/nuiton-validator/src/main/java/org/nuiton/validator/xwork2/field/VATIdentificationNumberFieldValidator.java 2011-08-30 10:48:22 UTC (rev 2197) @@ -28,6 +28,7 @@ import java.util.regex.Matcher; import java.util.regex.Pattern; +import org.apache.commons.lang.StringUtils; /** * Validator for EU VAT number. @@ -52,6 +53,10 @@ return; } if (value instanceof String) { + if (StringUtils.isEmpty((String) value)) { + // no value defined + return; + } Matcher m = p.matcher((String) value); if (!m.matches()) { addFieldError(fieldName, object); Modified: trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/EmailFieldValidatorTest.java =================================================================== --- trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/EmailFieldValidatorTest.java 2011-08-29 14:19:18 UTC (rev 2196) +++ trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/EmailFieldValidatorTest.java 2011-08-30 10:48:22 UTC (rev 2197) @@ -55,5 +55,15 @@ bean.setEmail("toto"); assertFieldInError(Contact.PROPERTY_EMAIL, "contact.email.format", true); + // use requiredstring for that case + bean.setEmail(""); + assertFieldInError(Contact.PROPERTY_EMAIL, "contact.email.format", + false); + + // use required for that case + bean.setEmail(null); + assertFieldInError(Contact.PROPERTY_EMAIL, "contact.email.format", + false); + } } Modified: trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/FrenchCityNameFieldValidatorTest.java =================================================================== --- trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/FrenchCityNameFieldValidatorTest.java 2011-08-29 14:19:18 UTC (rev 2196) +++ trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/FrenchCityNameFieldValidatorTest.java 2011-08-30 10:48:22 UTC (rev 2197) @@ -93,5 +93,15 @@ assertFieldInError(Contact.PROPERTY_CITY, "contact.city.format", true); + // Use requiredstring validator for this case + bean.setCity(""); + assertFieldInError(Contact.PROPERTY_CITY, "contact.city.format", + false); + + // Use required validator for this case + bean.setCity(null); + assertFieldInError(Contact.PROPERTY_CITY, "contact.city.format", + false); + } } Modified: trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/FrenchLastNameFieldValidatorTest.java =================================================================== --- trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/FrenchLastNameFieldValidatorTest.java 2011-08-29 14:19:18 UTC (rev 2196) +++ trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/FrenchLastNameFieldValidatorTest.java 2011-08-30 10:48:22 UTC (rev 2197) @@ -29,7 +29,6 @@ /** * @author jcouteau <couteau(a)codelutin.com> - * Date: 18/08/11 * @since 2.3 */ public class FrenchLastNameFieldValidatorTest extends AbstractFieldValidatorTest<Contact> { @@ -74,6 +73,16 @@ assertFieldInError(Contact.PROPERTY_LAST_NAME, "contact.name.format", true); + // Use requiredstring validator for that case + bean.setName(""); + assertFieldInError(Contact.PROPERTY_LAST_NAME, "contact.name.format", + false); + // Use required validator for that case + bean.setName(null); + assertFieldInError(Contact.PROPERTY_LAST_NAME, "contact.name.format", + false); + + } } Modified: trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/FrenchPhoneNumberFieldValidatorTest.java =================================================================== --- trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/FrenchPhoneNumberFieldValidatorTest.java 2011-08-29 14:19:18 UTC (rev 2196) +++ trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/FrenchPhoneNumberFieldValidatorTest.java 2011-08-30 10:48:22 UTC (rev 2197) @@ -29,7 +29,6 @@ /** * @author jcouteau <couteau(a)codelutin.com> - * Date: 18/08/11 * @since 2.3 */ public class FrenchPhoneNumberFieldValidatorTest extends AbstractFieldValidatorTest<Contact> { @@ -123,6 +122,14 @@ bean.setPhone("00-00-ab-00-00"); assertFieldInError(Contact.PROPERTY_PHONE, "contact.phoneNumber.format", true); + //Use required string validator for this case + bean.setPhone(""); + assertFieldInError(Contact.PROPERTY_PHONE, "contact.phoneNumber.format", + false); + //Use required validator for this case + bean.setPhone(null); + assertFieldInError(Contact.PROPERTY_PHONE, "contact.phoneNumber.format", + false); } } Modified: trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/FrenchPostCodeFieldValidatorTest.java =================================================================== --- trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/FrenchPostCodeFieldValidatorTest.java 2011-08-29 14:19:18 UTC (rev 2196) +++ trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/FrenchPostCodeFieldValidatorTest.java 2011-08-30 10:48:22 UTC (rev 2197) @@ -78,5 +78,17 @@ assertFieldInError(Contact.PROPERTY_POSTCODE, "contact.postCode.format", true); + // postCode can be empty (use a requiredstring validator for this check) + bean.setPostCode(""); + assertFieldInError(Contact.PROPERTY_POSTCODE, "contact.postCode.format", + false); + + // postCode can be null (use a requiredstring validator for this check) + bean.setPostCode(null); + assertFieldInError(Contact.PROPERTY_POSTCODE, "contact.postCode.format", + false); + + + } } Modified: trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/FrenchSiretFieldValidatorTest.java =================================================================== --- trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/FrenchSiretFieldValidatorTest.java 2011-08-29 14:19:18 UTC (rev 2196) +++ trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/FrenchSiretFieldValidatorTest.java 2011-08-30 10:48:22 UTC (rev 2197) @@ -29,7 +29,6 @@ /** * @author jcouteau <couteau(a)codelutin.com> - * Date: 19/08/11 * @since 2.3 */ public class FrenchSiretFieldValidatorTest extends AbstractFieldValidatorTest<Company> { @@ -85,5 +84,15 @@ bean.setSiret("4421bf1670300038"); assertFieldInError(Company.PROPERTY_SIRET, "company.siret.format", true); + + // Use requiredString for empty siret + bean.setSiret(""); + assertFieldInError(Company.PROPERTY_SIRET, "company.siret.format", + false); + + // Use required for null siret + bean.setSiret(null); + assertFieldInError(Company.PROPERTY_SIRET, "company.siret.format", + false); } } Modified: trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/VATIdentificationNumberFieldValidatorTest.java =================================================================== --- trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/VATIdentificationNumberFieldValidatorTest.java 2011-08-29 14:19:18 UTC (rev 2196) +++ trunk/nuiton-validator/src/test/java/org/nuiton/validator/xwork2/field/VATIdentificationNumberFieldValidatorTest.java 2011-08-30 10:48:22 UTC (rev 2197) @@ -62,5 +62,15 @@ bean.setVat("FR 57 442116703"); assertFieldInError(Company.PROPERTY_VAT_NUMBER, "company.vat.format", true); + + // Use required string validator for that + bean.setVat(""); + assertFieldInError(Company.PROPERTY_VAT_NUMBER, "company.vat.format", + false); + + // Use required validator for that + bean.setVat(null); + assertFieldInError(Company.PROPERTY_VAT_NUMBER, "company.vat.format", + false); } }

1 0

r2196 - trunk/nuiton-profiling/src/site/apt
by bpoussin＠users.nuiton.org Aug. 29, 2011

Aug. 29, 2011

Author: bpoussin Date: 2011-08-29 16:19:18 +0200 (Mon, 29 Aug 2011) New Revision: 2196 Url: http://nuiton.org/repositories/revision/nuiton-utils/2196 Log: ajout de la documentation pour une utilisation avec tomcat Modified: trunk/nuiton-profiling/src/site/apt/index.apt Modified: trunk/nuiton-profiling/src/site/apt/index.apt =================================================================== --- trunk/nuiton-profiling/src/site/apt/index.apt 2011-08-26 16:01:12 UTC (rev 2195) +++ trunk/nuiton-profiling/src/site/apt/index.apt 2011-08-29 14:19:18 UTC (rev 2196) @@ -92,4 +92,15 @@ <%@page contentType="text/csv" pageEncoding="UTF-8"%> <%=NuitonTrace.getStatisticsCSVAndClear()%> -Il ne vous reste plus qu'a analyser votre application \ No newline at end of file +Il ne vous reste plus qu'a analyser votre application + +NuitonTrace pour une application Web lancée dans un Tomcat + + Procéder de la même façon que pour maven, le seul changement est le nom + de la variable d'environnement qui est + + export JAVA_OPTS="$JAVA_OPTS -javaagent:${maven_repo}/org/aspectj/aspectjweaver/1.6.11/aspectjweaver-1.6.11.jar" + + Si vous avez des problèmes, reconstruire le war avec un clean avant et supprimer + le répertoire pour votre application qui a peut être créé par tomcat dans le + répertoire webapps.

1 0