You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by an...@apache.org on 2012/04/26 14:50:51 UTC
svn commit: r1330819 -
/jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/value/ValueFactoryImpl.java
Author: angela
Date: Thu Apr 26 12:50:50 2012
New Revision: 1330819
URL: http://svn.apache.org/viewvc?rev=1330819&view=rev
Log:
OAK-16 : Proper ValueFactory implementation and Value handling (WIP)
- javadoc
Modified:
jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/value/ValueFactoryImpl.java
Modified: jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/value/ValueFactoryImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/value/ValueFactoryImpl.java?rev=1330819&r1=1330818&r2=1330819&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/value/ValueFactoryImpl.java (original)
+++ jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/value/ValueFactoryImpl.java Thu Apr 26 12:50:50 2012
@@ -38,7 +38,10 @@ import java.math.BigDecimal;
import java.util.Calendar;
/**
- * ValueFactoryImpl...
+ * Implementation of {@link ValueFactory} interface based on the
+ * {@link CoreValueFactory} exposed by the
+ * {@link org.apache.jackrabbit.oak.api.ContentSession#getCoreValueFactory()}
+ * being aware of namespaces remapped on the editing session.
*/
public class ValueFactoryImpl implements ValueFactory {
@@ -51,8 +54,11 @@ public class ValueFactoryImpl implements
private final NameMapper nameMapper;
/**
+ * Creates a new instance of {@code ValueFactory}.
*
- * @param factory
+ * @param factory The core value factory.
+ * @param nameMapper The name mapping used for converting JCR names/paths to
+ * the internal representation.
*/
public ValueFactoryImpl(CoreValueFactory factory, NameMapper nameMapper) {
this.factory = factory;