Inverted the logic of the stop() method.

When stopping, stop first all the dependent components and then mark itself as offline.
jpkrohling · Nov 14, 2013 · dccd4d9 · dccd4d9
1 parent 1e85650
commit dccd4d9
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/...r.component.core/src/main/java/org/exoplatform/services/jcr/impl/RepositoryContainer.java b/...r.component.core/src/main/java/org/exoplatform/services/jcr/impl/RepositoryContainer.java
@@ -594,8 +594,10 @@ public void start()
    @Override
    public synchronized void stop()
    {
-      RepositoryImpl repository = (RepositoryImpl)getComponentInstanceOfType(RepositoryImpl.class);
+      super.stop();
+      super.unregisterAllComponents();
 
+      RepositoryImpl repository = (RepositoryImpl)getComponentInstanceOfType(RepositoryImpl.class);
       try
       {
          repository.setState(ManageableRepository.OFFLINE);
@@ -604,9 +606,6 @@ public synchronized void stop()
       {
          log.error("Can not switch repository to OFFLINE", e);
       }
-
-      super.stop();
-      super.unregisterAllComponents();
    }
 
    /**