Better solution for setting the context. The context is now known as soon as componen...

[utils] / system / general / src / main / java / org / wamblee / system / AbstractComponent.java

diff --git a/system/general/src/main/java/org/wamblee/system/AbstractComponent.java b/system/general/src/main/java/org/wamblee/system/AbstractComponent.java
index 3b4725835155e3cfc558fd370967e3fd4467ebd2..8b6109f233a951cf72edae3a6eb5765abc320e2a 100644 (file)
--- a/system/general/src/main/java/org/wamblee/system/AbstractComponent.java
+++ b/system/general/src/main/java/org/wamblee/system/AbstractComponent.java
@@ -70,6 +70,16 @@ public abstract class AbstractComponent implements Component {
                return _name;
        }
        
+       @Override
+       public void addContext(String aContext) {
+               if (_context == null ) { 
+                       _context = aContext; 
+               }
+               else { 
+                       _context = aContext + "." + _context;
+               }
+       }
+       
        @Override
        public String getQualifiedName() {
                if ( _context == null ) { 
@@ -89,16 +99,15 @@ public abstract class AbstractComponent implements Component {
        }
 
        @Override
-       public final void start(String aContext) {
-               LOG.info("Initializing '" + aContext + "." + _name + "'");
-               _context = aContext; 
-               doStart(aContext + "." + getName());
+       public final void start() {
+               LOG.info("Initializing '" + getQualifiedName() + "'");
+               doStart();
                _status = Status.RUNNING;
                if ( _running.size() != _provided.size()) { 
                        List<ProvidedInterface> remaining = 
                                new ArrayList<ProvidedInterface>(_provided);
                        remaining.removeAll(_running);
-                       throw new SystemAssemblyException(aContext + "." + getName() + ": not all services were started, missing " + remaining);
+                       throw new SystemAssemblyException(getQualifiedName() + ": not all services were started, missing " + remaining);
                }
        }
 
@@ -106,7 +115,7 @@ public abstract class AbstractComponent implements Component {
         * Must be implemented for initializing the subsystem. The implementation
         * must call {@link #addService(Service)} for each service that is started.
         */
-       protected abstract void doStart(String aContext);
+       protected abstract void doStart();
 
        /**
         * Implementations must call this method to indicate that a new service has
@@ -115,9 +124,9 @@ public abstract class AbstractComponent implements Component {
         * @param aService
         *            Service.
         */
-       protected final void addService(String aContext,
+       protected final void addService(
                        ProvidedInterface aDescriptor, Object aService) {
-               LOG.info(aContext + ": service '" + aService + "' started.");
+               LOG.info("Interface '" + getQualifiedName() + "." + aDescriptor.getName() + "' started.");
                _running.add(aDescriptor);
                aDescriptor.publish(aService);
        }