You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by ro...@apache.org on 2013/10/22 16:31:59 UTC

svn commit: r1534647 - /sling/trunk/tooling/support/install/src/main/java/org/apache/sling/tooling/support/install/impl/InstallServlet.java

Author: rombert
Date: Tue Oct 22 14:31:58 2013
New Revision: 1534647

URL: http://svn.apache.org/r1534647
Log:
SLING-3019 - Provide a mechanism to install a bundle based on a
directory

Refresh bundles when both installing and updating, to make sure wirings
are correct.

Modified:
    sling/trunk/tooling/support/install/src/main/java/org/apache/sling/tooling/support/install/impl/InstallServlet.java

Modified: sling/trunk/tooling/support/install/src/main/java/org/apache/sling/tooling/support/install/impl/InstallServlet.java
URL: http://svn.apache.org/viewvc/sling/trunk/tooling/support/install/src/main/java/org/apache/sling/tooling/support/install/impl/InstallServlet.java?rev=1534647&r1=1534646&r2=1534647&view=diff
==============================================================================
--- sling/trunk/tooling/support/install/src/main/java/org/apache/sling/tooling/support/install/impl/InstallServlet.java (original)
+++ sling/trunk/tooling/support/install/src/main/java/org/apache/sling/tooling/support/install/impl/InstallServlet.java Tue Oct 22 14:31:58 2013
@@ -237,13 +237,15 @@ public class InstallServlet extends Http
         if (bundle != null) {
             // update
             bundle.update(in);
-
-            packageAdmin.refreshPackages(new Bundle[] { bundle });
         } else {
             // install
             final Bundle b = bundleContext.installBundle(location, in);
             b.start();
         }
+
+        // take into account added/removed packages for updated bundles and newly satisfied optional package imports
+        // for new installed bundles
+        packageAdmin.refreshPackages(new Bundle[] { bundle });
     }
 
     private Bundle getBundle(final String symbolicName) {