You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jackrabbit.apache.org by an...@apache.org on 2009/06/10 15:29:28 UTC
svn commit: r783356 -
/jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/Util.java
Author: angela
Date: Wed Jun 10 13:29:27 2009
New Revision: 783356
URL: http://svn.apache.org/viewvc?rev=783356&view=rev
Log:
JCR-1104: JSR 283 support
- resolve TODO: determine length of an internalvalue
Modified:
jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/Util.java
Modified: jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/Util.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/Util.java?rev=783356&r1=783355&r2=783356&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/Util.java (original)
+++ jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/Util.java Wed Jun 10 13:29:27 2009
@@ -344,15 +344,16 @@
* cannot be determined.
*/
public static long getLength(InternalValue value) {
- // TODO: support new JSR 283 property types
- if (value.getType() == PropertyType.BINARY) {
- return value.getBLOBFileValue().getLength();
- } else
if (value.getType() == PropertyType.NAME
|| value.getType() == PropertyType.PATH) {
return -1;
} else {
- return value.toString().length();
+ try {
+ return value.getLength();
+ } catch (RepositoryException e) {
+ log.warn("Unable to determine length of value.", e.getMessage());
+ return -1;
+ }
}
}
}