You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jackrabbit.apache.org by ju...@apache.org on 2009/04/06 14:40:45 UTC
svn commit: r762314 -
/jackrabbit/commons/jcr-rmi/trunk/jcr-rmi/src/main/java/org/apache/jackrabbit/rmi/value/DateValue.java
Author: jukka
Date: Mon Apr 6 12:40:45 2009
New Revision: 762314
URL: http://svn.apache.org/viewvc?rev=762314&view=rev
Log:
JCRRMI-17: Multiple TCK test failures
Should throw ValueFormatException for invalid date strings.
Modified:
jackrabbit/commons/jcr-rmi/trunk/jcr-rmi/src/main/java/org/apache/jackrabbit/rmi/value/DateValue.java
Modified: jackrabbit/commons/jcr-rmi/trunk/jcr-rmi/src/main/java/org/apache/jackrabbit/rmi/value/DateValue.java
URL: http://svn.apache.org/viewvc/jackrabbit/commons/jcr-rmi/trunk/jcr-rmi/src/main/java/org/apache/jackrabbit/rmi/value/DateValue.java?rev=762314&r1=762313&r2=762314&view=diff
==============================================================================
--- jackrabbit/commons/jcr-rmi/trunk/jcr-rmi/src/main/java/org/apache/jackrabbit/rmi/value/DateValue.java (original)
+++ jackrabbit/commons/jcr-rmi/trunk/jcr-rmi/src/main/java/org/apache/jackrabbit/rmi/value/DateValue.java Mon Apr 6 12:40:45 2009
@@ -65,6 +65,9 @@
*/
protected DateValue(String value) throws ValueFormatException {
this(ISO8601.parse(value));
+ if (this.value == null) {
+ throw new ValueFormatException("Invalid date value: " + value);
+ }
}
/**