You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@aries.apache.org by tj...@apache.org on 2014/07/24 17:52:34 UTC

svn commit: r1613167 - /aries/trunk/subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/ResourceInstaller.java

Author: tjwatson
Date: Thu Jul 24 15:52:33 2014
New Revision: 1613167

URL: http://svn.apache.org/r1613167
Log:
ARIES-1231 - coordinator rollback action incorrectly added


Modified:
    aries/trunk/subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/ResourceInstaller.java

Modified: aries/trunk/subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/ResourceInstaller.java
URL: http://svn.apache.org/viewvc/aries/trunk/subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/ResourceInstaller.java?rev=1613167&r1=1613166&r2=1613167&view=diff
==============================================================================
--- aries/trunk/subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/ResourceInstaller.java (original)
+++ aries/trunk/subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/ResourceInstaller.java Thu Jul 24 15:52:33 2014
@@ -92,19 +92,19 @@ public abstract class ResourceInstaller 
 		// to track all constituents for persistence purposes, which includes
 		// resources that were provisioned to the subsystem as dependencies of
 		// other resources.
-		if (isReferencedSubsystem())
+		if (isReferencedSubsystem()) {
 			Activator.getInstance().getSubsystems().addReference(subsystem, resource);
-		coordination.addParticipant(new Participant() {
-			@Override
-			public void ended(Coordination arg0) throws Exception {
-				// Nothing
-			}
-
-			@Override
-			public void failed(Coordination arg0) throws Exception {
-				Activator.getInstance().getSubsystems().removeReference(subsystem, resource);
-			}
-		});
+			coordination.addParticipant(new Participant() {
+				@Override
+				public void ended(Coordination arg0) throws Exception {
+					// Nothing
+				}
+				@Override
+				public void failed(Coordination arg0) throws Exception {
+					Activator.getInstance().getSubsystems().removeReference(subsystem, resource);
+				}
+			});
+		}
 	}
 	
 	protected String getLocation() {