You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by el...@apache.org on 2013/05/01 09:37:53 UTC

svn commit: r1477921 - /directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/DefaultDirectoryService.java

Author: elecharny
Date: Wed May  1 07:37:53 2013
New Revision: 1477921

URL: http://svn.apache.org/r1477921
Log:
Added some comments to clarify the code

Modified:
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/DefaultDirectoryService.java

Modified: directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/DefaultDirectoryService.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/DefaultDirectoryService.java?rev=1477921&r1=1477920&r2=1477921&view=diff
==============================================================================
--- directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/DefaultDirectoryService.java (original)
+++ directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/DefaultDirectoryService.java Wed May  1 07:37:53 2013
@@ -516,6 +516,8 @@ public class DefaultDirectoryService imp
             return;
         }
 
+        // We don't call getMethods() because it would get back the default methods
+        // from the BaseInterceptor, something we don't want.
         Method[] methods = interceptorClz.getDeclaredMethods();
 
         for ( Method method : methods )
@@ -549,6 +551,7 @@ public class DefaultDirectoryService imp
             }
         }
 
+        // Recurse on extended classes, as we have used getDeclaredMethods() instead of getmethods()
         gatherInterceptors( interceptor, interceptorClz.getSuperclass(), operation, selectedInterceptorList );
     }