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 mr...@apache.org on 2013/02/11 10:33:15 UTC

svn commit: r1444694 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/observation/ChangeProcessor.java

Author: mreutegg
Date: Mon Feb 11 09:33:15 2013
New Revision: 1444694

URL: http://svn.apache.org/r1444694
Log:
OAK-618: ChangeProcessor must ignore hidden properties

Modified:
    jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/observation/ChangeProcessor.java

Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/observation/ChangeProcessor.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/observation/ChangeProcessor.java?rev=1444694&r1=1444693&r2=1444694&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/observation/ChangeProcessor.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/observation/ChangeProcessor.java Mon Feb 11 09:33:15 2013
@@ -159,6 +159,9 @@ class ChangeProcessor implements Runnabl
 
         @Override
         public void propertyAdded(PropertyState after) {
+            if (NodeStateUtils.isHidden(after.getName())) {
+                return;
+            }
             if (!stopping && filterRef.get().include(Event.PROPERTY_ADDED, jcrPath(), associatedParentNode)) {
                 Event event = generatePropertyEvent(Event.PROPERTY_ADDED, path, after);
                 events.add(Iterators.singletonIterator(event));
@@ -167,6 +170,9 @@ class ChangeProcessor implements Runnabl
 
         @Override
         public void propertyChanged(PropertyState before, PropertyState after) {
+            if (NodeStateUtils.isHidden(before.getName())) {
+                return;
+            }
             if (!stopping && filterRef.get().include(Event.PROPERTY_CHANGED, jcrPath(), associatedParentNode)) {
                 Event event = generatePropertyEvent(Event.PROPERTY_CHANGED, path, after);
                 events.add(Iterators.singletonIterator(event));
@@ -175,6 +181,9 @@ class ChangeProcessor implements Runnabl
 
         @Override
         public void propertyDeleted(PropertyState before) {
+            if (NodeStateUtils.isHidden(before.getName())) {
+                return;
+            }
             if (!stopping && filterRef.get().include(Event.PROPERTY_REMOVED, jcrPath(), associatedParentNode)) {
                 Event event = generatePropertyEvent(Event.PROPERTY_REMOVED, path, before);
                 events.add(Iterators.singletonIterator(event));