You are viewing a plain text version of this content. The canonical link for it is here.
Posted to woden-dev@ws.apache.org by jk...@apache.org on 2006/01/18 16:09:34 UTC

svn commit: r370164 - /incubator/woden/java/src/org/apache/woden/internal/xml/XMLAttrImpl.java

Author: jkaputin
Date: Wed Jan 18 07:09:21 2006
New Revision: 370164

URL: http://svn.apache.org/viewcvs?rev=370164&view=rev
Log:
Corrected the use of fValid so that it is init to true
and set to false if the attribute cannot be converted 
into the appropriate type.

Modified:
    incubator/woden/java/src/org/apache/woden/internal/xml/XMLAttrImpl.java

Modified: incubator/woden/java/src/org/apache/woden/internal/xml/XMLAttrImpl.java
URL: http://svn.apache.org/viewcvs/incubator/woden/java/src/org/apache/woden/internal/xml/XMLAttrImpl.java?rev=370164&r1=370163&r2=370164&view=diff
==============================================================================
--- incubator/woden/java/src/org/apache/woden/internal/xml/XMLAttrImpl.java (original)
+++ incubator/woden/java/src/org/apache/woden/internal/xml/XMLAttrImpl.java Wed Jan 18 07:09:21 2006
@@ -60,7 +60,7 @@
     {
         fExternalForm = attrValue;
         fContent = convert(ownerEl, attrValue);
-        if(fContent != null) fValid = true;
+        if(fContent == null) fValid = false;
     }
     
     public Object getContent() {
@@ -90,10 +90,10 @@
     }
     
     /*
-     * Validity is initialized to false, but set to true during initialization
-     * if the attribute string format is valid (i.e. if it can be converted to
-     * an object of the appropriate type). This method may be used to change the
-     * validity later (e.g. if Validation determines an error).
+     * Validity is initialized to true, but set to false if the attribute's
+     * value is null or if it cannot be converted to an object of the 
+     * appropriate type). This method may be used to change the validity later
+     * (e.g. if Validation determines an error).
      * 
      * TODO confirm this method is needed, otherwise remove it.
      */



---------------------------------------------------------------------
To unsubscribe, e-mail: woden-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: woden-dev-help@ws.apache.org