svn commit: r746298 - /servicemix/smx4/nmr/trunk/jbi/deployer/src/main/java/org/apache/servicemix/jbi/deployer/impl/AdminCommandsImpl.java

[ulhasbhole]

Author: ulhasbhole
Date: Fri Feb 20 17:15:33 2009
New Revision: 746298

URL: http://svn.apache.org/viewvc?rev=746298&view=rev
Log:
Fixed uninstall-component failure doesn't throw exception if 
installationService.unloadInstaller() call returns false which is the case in 
worng/non-existant SA

Modified:
    
servicemix/smx4/nmr/trunk/jbi/deployer/src/main/java/org/apache/servicemix/jbi/deployer/impl/AdminCommandsImpl.java

Modified: 
servicemix/smx4/nmr/trunk/jbi/deployer/src/main/java/org/apache/servicemix/jbi/deployer/impl/AdminCommandsImpl.java
URL: 
http://svn.apache.org/viewvc/servicemix/smx4/nmr/trunk/jbi/deployer/src/main/java/org/apache/servicemix/jbi/deployer/impl/AdminCommandsImpl.java?rev=746298&r1=746297&r2=746298&view=diff
==============================================================================
--- 
servicemix/smx4/nmr/trunk/jbi/deployer/src/main/java/org/apache/servicemix/jbi/deployer/impl/AdminCommandsImpl.java
 (original)
+++ 
servicemix/smx4/nmr/trunk/jbi/deployer/src/main/java/org/apache/servicemix/jbi/deployer/impl/AdminCommandsImpl.java
 Fri Feb 20 17:15:33 2009
@@ -70,8 +70,10 @@
      */
     public String uninstallComponent(final String name) throws Exception {
         try {
-            installationService.unloadInstaller(name, true);
-            return 
ManagementSupport.createSuccessMessage("uninstallComponent", name);
+            if (installationService.unloadInstaller(name, true)) {
+               return 
ManagementSupport.createSuccessMessage("uninstallComponent", name);
+            }
+            throw new RuntimeException();
         } catch (Exception e) {
             throw ManagementSupport.failure("uninstallComponent", name, e);
         }