Author: geirm
Date: Fri Feb 11 07:24:10 2005
New Revision: 153411

URL: http://svn.apache.org/viewcvs?view=rev&rev=153411
Log:
The spec/javadoc is baffling, but I think that this is acceptable

Modified:
    
webservices/scout/trunk/modules/scout/src/java/org/apache/ws/scout/registry/BulkResponseImpl.java

Modified: 
webservices/scout/trunk/modules/scout/src/java/org/apache/ws/scout/registry/BulkResponseImpl.java
URL: 
http://svn.apache.org/viewcvs/webservices/scout/trunk/modules/scout/src/java/org/apache/ws/scout/registry/BulkResponseImpl.java?view=diff&r1=153410&r2=153411
==============================================================================
--- 
webservices/scout/trunk/modules/scout/src/java/org/apache/ws/scout/registry/BulkResponseImpl.java
 (original)
+++ 
webservices/scout/trunk/modules/scout/src/java/org/apache/ws/scout/registry/BulkResponseImpl.java
 Fri Feb 11 07:24:10 2005
@@ -55,9 +55,22 @@
         return this.collection;
     }
 
+    /**
+     * The javadoc is unclear.  it says for getExceptions() :
+     *   "Get the Collection of RegistryException instances in case of partial
+     *     commit. Caller thread will block here if result is not yet 
available.
+     *     Return null if result is available and there is no 
RegistryException(s)."
+     * Yet the return javadoc says :
+     *   "Collection of RegistryException instances. The Collection may be 
empty but not null."
+     *
+     * So my interpretation is return null if result avail, and empty 
collection otherwise
+     *
+     * @return
+     * @throws JAXRException
+     */
     public Collection getExceptions() throws JAXRException
     {
-        return this.exceptions;
+        return (this.exceptions.size() == 0 ? null : exceptions);
     }
 
     public boolean isPartialResponse() throws JAXRException



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

Reply via email to