You are viewing a plain text version of this content. The canonical link for it is here.
Posted to wsrp4j-dev@portals.apache.org by es...@apache.org on 2007/08/01 05:19:37 UTC

svn commit: r561630 - /portals/wsrp4j/trunk/commons/src/java/org/apache/wsrp4j/commons/util/ParameterChecker.java

Author: esm
Date: Tue Jul 31 20:19:37 2007
New Revision: 561630

URL: http://svn.apache.org/viewvc?view=rev&rev=561630
Log:
Removing ClientData data structure as a required element of the MarkupParams data structure.  

Reading WSRP 6.1.9, I don't see where ClientData is a required element. (I checked the errata as well).

Modified:
    portals/wsrp4j/trunk/commons/src/java/org/apache/wsrp4j/commons/util/ParameterChecker.java

Modified: portals/wsrp4j/trunk/commons/src/java/org/apache/wsrp4j/commons/util/ParameterChecker.java
URL: http://svn.apache.org/viewvc/portals/wsrp4j/trunk/commons/src/java/org/apache/wsrp4j/commons/util/ParameterChecker.java?view=diff&rev=561630&r1=561629&r2=561630
==============================================================================
--- portals/wsrp4j/trunk/commons/src/java/org/apache/wsrp4j/commons/util/ParameterChecker.java (original)
+++ portals/wsrp4j/trunk/commons/src/java/org/apache/wsrp4j/commons/util/ParameterChecker.java Tue Jul 31 20:19:37 2007
@@ -1180,7 +1180,7 @@
     /**
      * Check the MarkupParams. The following attributes are mandatory:
      * 
-     * 1) ClientData 2) Locale 3) MimeType 4) Mode 5) WindowState
+     * 1) Locale 2) MimeType 3) Mode 4) WindowState
      * 
      * @param markupParams
      *            MarkupParams
@@ -1200,11 +1200,7 @@
         // check only, if object not null, otherwise ignore. Object is defined
         // as nillable
         if (markupParams != null) {
-            // check ClientData, is mandatory
-            if (markupParams.getClientData() == null) {
-                throwMissingParametersFault("ClientData in " +
-                        "MarkupParams is null");
-            }
+
             if (markupParams.getLocales() == null) {
                 throwMissingParametersFault("Locales in " +
                         "MarkupParams is null");