svn commit: r1617903 - in /jackrabbit/oak/trunk: oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/name/NameValidator.java oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/MiscTest.java

[tripod]

Author: tripod
Date: Thu Aug 14 09:35:03 2014
New Revision: 1617903

URL: http://svn.apache.org/r1617903
Log:
OAK-2032 Setting a MV Name property does not validate the namespaces

Modified:
    
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/name/NameValidator.java
    
jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/MiscTest.java

Modified: 
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/name/NameValidator.java
URL: 
http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/name/NameValidator.java?rev=1617903&r1=1617902&r2=1617903&view=diff
==============================================================================
--- 
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/name/NameValidator.java
 (original)
+++ 
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/name/NameValidator.java
 Thu Aug 14 09:35:03 2014
@@ -72,7 +72,7 @@ class NameValidator extends DefaultValid
 
     protected void checkValidValue(PropertyState property)
             throws CommitFailedException {
-        if (NAME.equals(property.getType())) {
+        if (NAME.equals(property.getType()) || 
NAMES.equals(property.getType())) {
             for (String value : property.getValue(NAMES)) {
                 checkValidValue(value);
             }

Modified: 
jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/MiscTest.java
URL: 
http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/MiscTest.java?rev=1617903&r1=1617902&r2=1617903&view=diff
==============================================================================
--- 
jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/MiscTest.java
 (original)
+++ 
jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/MiscTest.java
 Thu Aug 14 09:35:03 2014
@@ -46,7 +46,6 @@ public class MiscTest extends AbstractRe
         traverse(index);
     }
 
-    @Ignore("OAK-2032")
     @Test
     public void testMVNameProperty() throws Exception {
         Session session = getAdminSession();