You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by cz...@apache.org on 2017/09/06 14:01:24 UTC

svn commit: r1807491 - in /sling/trunk/bundles/commons/classloader/src: main/java/org/apache/sling/commons/classloader/impl/Activator.java test/java/org/apache/sling/commons/classloader/impl/BundeChangesTest.java

Author: cziegeler
Date: Wed Sep  6 14:01:23 2017
New Revision: 1807491

URL: http://svn.apache.org/viewvc?rev=1807491&view=rev
Log:
SLING-7109 : Bundle not marked as used if a resource is loaded from a bundle

Modified:
    sling/trunk/bundles/commons/classloader/src/main/java/org/apache/sling/commons/classloader/impl/Activator.java
    sling/trunk/bundles/commons/classloader/src/test/java/org/apache/sling/commons/classloader/impl/BundeChangesTest.java

Modified: sling/trunk/bundles/commons/classloader/src/main/java/org/apache/sling/commons/classloader/impl/Activator.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/commons/classloader/src/main/java/org/apache/sling/commons/classloader/impl/Activator.java?rev=1807491&r1=1807490&r2=1807491&view=diff
==============================================================================
--- sling/trunk/bundles/commons/classloader/src/main/java/org/apache/sling/commons/classloader/impl/Activator.java (original)
+++ sling/trunk/bundles/commons/classloader/src/main/java/org/apache/sling/commons/classloader/impl/Activator.java Wed Sep  6 14:01:23 2017
@@ -122,9 +122,6 @@ public class Activator implements Synchr
                 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;
                 }

Modified: sling/trunk/bundles/commons/classloader/src/test/java/org/apache/sling/commons/classloader/impl/BundeChangesTest.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/commons/classloader/src/test/java/org/apache/sling/commons/classloader/impl/BundeChangesTest.java?rev=1807491&r1=1807490&r2=1807491&view=diff
==============================================================================
--- sling/trunk/bundles/commons/classloader/src/test/java/org/apache/sling/commons/classloader/impl/BundeChangesTest.java (original)
+++ sling/trunk/bundles/commons/classloader/src/test/java/org/apache/sling/commons/classloader/impl/BundeChangesTest.java Wed Sep  6 14:01:23 2017
@@ -127,7 +127,7 @@ public class BundeChangesTest {
         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());
     }
 }