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 md...@apache.org on 2012/09/13 13:53:14 UTC

svn commit: r1384288 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/type/NodeTypeImpl.java

Author: mduerig
Date: Thu Sep 13 11:53:14 2012
New Revision: 1384288

URL: http://svn.apache.org/viewvc?rev=1384288&view=rev
Log:
OAK-66: JCR Node Type Management
FIXME

Modified:
    jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/type/NodeTypeImpl.java

Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/type/NodeTypeImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/type/NodeTypeImpl.java?rev=1384288&r1=1384287&r2=1384288&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/type/NodeTypeImpl.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/type/NodeTypeImpl.java Thu Sep 13 11:53:14 2012
@@ -484,6 +484,7 @@ class NodeTypeImpl implements NodeType {
 
     private static boolean isProtected(ItemDefinition definition) {
         // TODO need a better way for setting protected items internally
+        // FIXME admins should not be able to set protected items
         Subject subject = Subject.getSubject(AccessController.getContext());
         return (subject == null || !subject.getPrincipals().contains(AdminPrincipal.INSTANCE)) && definition.isProtected();
     }