Author: rfeng
Date: Thu Aug 23 09:12:45 2007
New Revision: 569057

URL: http://svn.apache.org/viewvc?rev=569057&view=rev
Log:
Improve the text for the exception from getSingleService()

Modified:
    
incubator/tuscany/java/sca/modules/core/src/main/java/org/apache/tuscany/sca/core/context/ComponentContextHelper.java

Modified: 
incubator/tuscany/java/sca/modules/core/src/main/java/org/apache/tuscany/sca/core/context/ComponentContextHelper.java
URL: 
http://svn.apache.org/viewvc/incubator/tuscany/java/sca/modules/core/src/main/java/org/apache/tuscany/sca/core/context/ComponentContextHelper.java?rev=569057&r1=569056&r2=569057&view=diff
==============================================================================
--- 
incubator/tuscany/java/sca/modules/core/src/main/java/org/apache/tuscany/sca/core/context/ComponentContextHelper.java
 (original)
+++ 
incubator/tuscany/java/sca/modules/core/src/main/java/org/apache/tuscany/sca/core/context/ComponentContextHelper.java
 Thu Aug 23 09:12:45 2007
@@ -73,8 +73,8 @@
      * @param processors The extension point for stax artifact processors
      */
     public ComponentContextHelper(AssemblyFactory assemblyFactory,
-                           JavaInterfaceFactory javaInterfaceFactory,
-                           StAXArtifactProcessorExtensionPoint processors) {
+                                  JavaInterfaceFactory javaInterfaceFactory,
+                                  StAXArtifactProcessorExtensionPoint 
processors) {
         this.assemblyFactory = assemblyFactory;
         this.javaInterfaceFactory = javaInterfaceFactory;
         this.staxProcessors = processors;
@@ -242,7 +242,7 @@
         }
         return null;
     }
-    
+
     public static CompositeActivator getCurrentCompositeActivator() {
         RuntimeComponent component = getCurrentComponent();
         if (component != null) {
@@ -251,7 +251,7 @@
         }
         return null;
     }
-    
+
     public static ComponentContextHelper getCurrentComponentContextHelper() {
         CompositeActivator activator = getCurrentCompositeActivator();
         if (activator != null) {
@@ -276,8 +276,12 @@
                 regularServices.add(service);
             }
         }
+        if (regularServices.size() == 0) {
+            throw new ServiceRuntimeException("No service is declared on 
component " + component.getURI());
+        }
         if (regularServices.size() != 1) {
-            throw new ServiceRuntimeException("The component doesn't have 
exactly one service");
+            throw new ServiceRuntimeException("More than one service is 
declared on component " + component.getURI()
+                + ". Service name is required to get the service.");
         }
         targetService = regularServices.get(0);
         return targetService;



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to