You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by dk...@apache.org on 2009/04/01 20:17:25 UTC

svn commit: r760992 - in /cxf/branches/2.1.x-fixes: ./ tools/validator/src/main/java/org/apache/cxf/tools/validator/internal/WSDL11Validator.java

Author: dkulp
Date: Wed Apr  1 18:17:24 2009
New Revision: 760992

URL: http://svn.apache.org/viewvc?rev=760992&view=rev
Log:
Merged revisions 760171 via svnmerge from 
https://svn.apache.org/repos/asf/cxf/trunk

........
  r760171 | dkulp | 2009-03-30 17:02:52 -0400 (Mon, 30 Mar 2009) | 2 lines
  
  When validating a wsdl, collect all the errors and spit them all out, not just the first.
........

Modified:
    cxf/branches/2.1.x-fixes/   (props changed)
    cxf/branches/2.1.x-fixes/tools/validator/src/main/java/org/apache/cxf/tools/validator/internal/WSDL11Validator.java

Propchange: cxf/branches/2.1.x-fixes/
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Wed Apr  1 18:17:24 2009
@@ -1 +1 @@
-/cxf/trunk:743446,753380,753397,753421,754585,755365,757499,757859,757899,757935,757951,758195,758303,758308,758378,758690,758910,759890,759961,759963-759964,759966,760029,760073,760150,760178,760456,760468
+/cxf/trunk:743446,753380,753397,753421,754585,755365,757499,757859,757899,757935,757951,758195,758303,758308,758378,758690,758910,759890,759961,759963-759964,759966,760029,760073,760150,760171,760178,760456,760468

Propchange: cxf/branches/2.1.x-fixes/
------------------------------------------------------------------------------
Binary property 'svnmerge-integrated' - no diff available.

Modified: cxf/branches/2.1.x-fixes/tools/validator/src/main/java/org/apache/cxf/tools/validator/internal/WSDL11Validator.java
URL: http://svn.apache.org/viewvc/cxf/branches/2.1.x-fixes/tools/validator/src/main/java/org/apache/cxf/tools/validator/internal/WSDL11Validator.java?rev=760992&r1=760991&r2=760992&view=diff
==============================================================================
--- cxf/branches/2.1.x-fixes/tools/validator/src/main/java/org/apache/cxf/tools/validator/internal/WSDL11Validator.java (original)
+++ cxf/branches/2.1.x-fixes/tools/validator/src/main/java/org/apache/cxf/tools/validator/internal/WSDL11Validator.java Wed Apr  1 18:17:24 2009
@@ -130,12 +130,16 @@
             validators.add(new MIMEBindingValidator(this.def));
         }
 
+        boolean notValid = false;
         for (AbstractValidator validator : validators) {
             if (!validator.isValid()) {
+                notValid = true;
                 addErrorMessage(validator.getErrorMessage());                
-                throw new ToolException(this.getErrorMessage());
             }
         }
+        if (notValid) {
+            throw new ToolException(this.getErrorMessage());            
+        }
 
         // By default just use WsdlRefValidator
         if (!env.optionSet(ToolConstants.CFG_VALIDATE_WSDL)) {