Author: bpoussin Date: 2011-11-22 14:50:13 +0100 (Tue, 22 Nov 2011) New Revision: 1256 Url: http://nuiton.org/repositories/revision/wikitty/1256 Log: small bug in format method correction in toBigDecimal when argument is empty string Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyUtil.java trunk/wikitty-api/src/test/java/org/nuiton/wikitty/api/WikittyUtilTest.java Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyUtil.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyUtil.java 2011-11-20 14:58:36 UTC (rev 1255) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyUtil.java 2011-11-22 13:50:13 UTC (rev 1256) @@ -411,7 +411,7 @@ for (String field : fields) { values[i++] = w.getFqField(field); // i is incremented before replace, because String.format start at 1 not 0 - format = format.replaceAll("%"+Pattern.quote(field)+"\\$", "\\%"+i+"\\$"); + format = format.replaceAll("%"+Pattern.quote(field)+"(\\|.*?)?\\$", "\\%"+i+"\\$"); } // on remplace tous les champs non trouve @@ -460,7 +460,7 @@ */ static public BigDecimal toBigDecimal(Object value) { BigDecimal result = null; - if (value == null) { + if (value == null || "".equals(value)) { result = new BigDecimal(0); // default to 0 } else if (value instanceof BigDecimal) { result = (BigDecimal) value; Modified: trunk/wikitty-api/src/test/java/org/nuiton/wikitty/api/WikittyUtilTest.java =================================================================== --- trunk/wikitty-api/src/test/java/org/nuiton/wikitty/api/WikittyUtilTest.java 2011-11-20 14:58:36 UTC (rev 1255) +++ trunk/wikitty-api/src/test/java/org/nuiton/wikitty/api/WikittyUtilTest.java 2011-11-22 13:50:13 UTC (rev 1256) @@ -74,6 +74,12 @@ group.getWikitty()); Assert.assertEquals("Group Name = Mon Groupe", result); + // test un cas passant + result = WikittyUtil.format( + "Group Name = %"+WikittyGroupImpl.FQ_FIELD_WIKITTYGROUP_NAME+"|noname$s", + group.getWikitty()); + Assert.assertEquals("Group Name = Mon Groupe", result); + // test un cas passant avec deux fois le meme champs result = WikittyUtil.format( "Group Name = %"+WikittyGroupImpl.FQ_FIELD_WIKITTYGROUP_NAME+"$s(%"+WikittyGroupImpl.FQ_FIELD_WIKITTYGROUP_NAME+"$s)",