Author: bpoussin Date: 2014-07-30 15:50:45 +0200 (Wed, 30 Jul 2014) New Revision: 1587 Url: http://forge.nuiton.org/projects/wikitty/repository/revisions/1587 Log: - no add null object in collection - "" for date become null date (permit reset to default in web form) Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyUtil.java trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/FieldType.java Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyUtil.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyUtil.java 2014-02-14 13:52:55 UTC (rev 1586) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyUtil.java 2014-07-30 13:50:45 UTC (rev 1587) @@ -696,7 +696,9 @@ */ static public Date toDate(Object value) { Date result = null; - if (value != null) { + // poussin 20140730 pour null ou chaine vide on laisse a null + // c'est util pour les formulaires web qui sont soumis avec des champs sans valeurs + if (value != null && !"".equals(value)) { if (value instanceof Date) { result = (Date) value; } else if (value instanceof Calendar) { Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/FieldType.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/FieldType.java 2014-02-14 13:52:55 UTC (rev 1586) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/FieldType.java 2014-07-30 13:50:45 UTC (rev 1587) @@ -187,12 +187,18 @@ // copy all value in new collections if (value instanceof Collection) { for ( Object o : (Collection<?>) value ) { - col.add( getContainedValidObject(o) ); + Object tmp = getContainedValidObject(o); + if (tmp != null) { + col.add( tmp ); + } } } else { // is Array for ( Object o : (Object[]) value ) { - col.add( getContainedValidObject(o) ); + Object tmp = getContainedValidObject(o); + if (tmp != null) { + col.add( tmp ); + } } } result = col;