You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@ace.apache.org by ja...@apache.org on 2014/03/10 22:51:39 UTC

svn commit: r1576099 - /ace/trunk/org.apache.ace.client.repository/src/org/apache/ace/client/repository/stateful/impl/StatefulTargetRepositoryImpl.java

Author: jawi
Date: Mon Mar 10 21:51:39 2014
New Revision: 1576099

URL: http://svn.apache.org/r1576099
Log:
ACE-467 - ensure that unregistered targets do not cause exceptions in the check for reachability.


Modified:
    ace/trunk/org.apache.ace.client.repository/src/org/apache/ace/client/repository/stateful/impl/StatefulTargetRepositoryImpl.java

Modified: ace/trunk/org.apache.ace.client.repository/src/org/apache/ace/client/repository/stateful/impl/StatefulTargetRepositoryImpl.java
URL: http://svn.apache.org/viewvc/ace/trunk/org.apache.ace.client.repository/src/org/apache/ace/client/repository/stateful/impl/StatefulTargetRepositoryImpl.java?rev=1576099&r1=1576098&r2=1576099&view=diff
==============================================================================
--- ace/trunk/org.apache.ace.client.repository/src/org/apache/ace/client/repository/stateful/impl/StatefulTargetRepositoryImpl.java (original)
+++ ace/trunk/org.apache.ace.client.repository/src/org/apache/ace/client/repository/stateful/impl/StatefulTargetRepositoryImpl.java Mon Mar 10 21:51:39 2014
@@ -842,6 +842,11 @@ public class StatefulTargetRepositoryImp
      * @return <code>true</code> if the given entity is reachable from the given target, <code>false</code> otherwise.
      */
     private boolean isReachableFrom(StatefulTargetObjectImpl target, RepositoryObject entity) {
+        // ACE-467 ensure we only take registered targets into consideration...
+        if (!target.isRegistered()) {
+            return false;
+        }
+
         if (entity instanceof DistributionObject) {
             return target.isAssociated(entity, DistributionObject.class);
         }