Changeset 2628 for TI01-discovery/trunk/ws-Discovery2

Timestamp:

20/06/07 16:43:10 (12 years ago)

Author:

mpritcha

Message:

Fix for ticket #775, now correctly sets status to false if all documents fail.

File:

• TI01-discovery/trunk/ws-Discovery2/src/ndg/services/discovery/PresentAgent.java (modified) (2 diffs)

TI01-discovery/trunk/ws-Discovery2/src/ndg/services/discovery/PresentAgent.java

@@ -344 +344 @@
                         }
 
-                        if ( returnDocs.isEmpty() )
+                        if ( returnDocs.isEmpty() || (returnDocs.size() ==  failedDocs.size()) )
                         {
                                 result.setStatus(false);
                                 result.setStatusMessage("Error retrieving all documents");
+
                         } else {
                                 result.setStatus(true);
@@ -353 +354 @@
                                 result.setDocuments( returnDocs );
                         }
-                        if ( !failedDocs.isEmpty() )
+                        if ( failedDocs.size() > returnDocs.size() )
                         {
                                 String failedDocMessage = " but some failed documents : ";