commit 6bb7afa18b37af69181a0188dd367fc30c3d0857
author Carsten Ziegeler <cziegeler@apache.org> Wed Sep 06 14:01:23 2017 +0000
committer Carsten Ziegeler <cziegeler@apache.org> Wed Sep 06 14:01:23 2017 +0000
tree 3c1702e19a0473adc93dcd4272fac88303df1379
parent 3343b988944960e5e54c6e1a29e9ad075ac0426a

SLING-7109 : Bundle not marked as used if a resource is loaded from a bundle

git-svn-id: https://svn.apache.org/repos/asf/sling/trunk@1807491 13f79535-47bb-0310-9956-ffa450edef68

src/main/java/org/apache/sling/commons/classloader/impl/Activator.java

diff --git a/src/main/java/org/apache/sling/commons/classloader/impl/Activator.java b/src/main/java/org/apache/sling/commons/classloader/impl/Activator.java
index 7e4f3bb..266845b 100644
--- a/src/main/java/org/apache/sling/commons/classloader/impl/Activator.java
+++ b/src/main/java/org/apache/sling/commons/classloader/impl/Activator.java
@@ -122,9 +122,6 @@
                 if ( this.service.hasUnresolvedPackages(event.getBundle()) ) {
                     reload = true;
                     logger.debug("Dynamic Class Loader is reloaded because the new bundle '{}' provides previously unresolved packages", event.getBundle());
-                } else if ( this.service.isBundleUsed(event.getBundle().getBundleId()) ) {
-                    reload = true;
-                    logger.debug("Dynamic Class Loader is reloaded because the bundle '{}' has been updated", event.getBundle());
                 } else {
                     reload = false;
                 }

src/test/java/org/apache/sling/commons/classloader/impl/BundeChangesTest.java

diff --git a/src/test/java/org/apache/sling/commons/classloader/impl/BundeChangesTest.java b/src/test/java/org/apache/sling/commons/classloader/impl/BundeChangesTest.java
index 656aa2a..18d616e 100644
--- a/src/test/java/org/apache/sling/commons/classloader/impl/BundeChangesTest.java
+++ b/src/test/java/org/apache/sling/commons/classloader/impl/BundeChangesTest.java
@@ -127,7 +127,7 @@
         assertFalse(registerCalled.get());
         assertFalse(unregisterCalled.get());
         listener.bundleChanged(new BundleEvent(BundleEvent.STARTED, bundle));
-        assertTrue(registerCalled.get());
-        assertTrue(unregisterCalled.get());
+        assertFalse(registerCalled.get());
+        assertFalse(unregisterCalled.get());
     }
 }