Update of /var/cvs/src/org/mmbase/util/xml
In directory james.mmbase.org:/tmp/cvs-serv8531/src/org/mmbase/util/xml

Modified Files:
        BuilderReader.java 
Log Message:
MMB-1588 Cannot override state and readonly attributes of fields.

SYSTEM fields were default readonly


See also: http://cvs.mmbase.org/viewcvs/src/org/mmbase/util/xml
See also: http://www.mmbase.org/jira/browse/MMB-1588


Index: BuilderReader.java
===================================================================
RCS file: /var/cvs/src/org/mmbase/util/xml/BuilderReader.java,v
retrieving revision 1.93
retrieving revision 1.94
diff -u -b -r1.93 -r1.94
--- BuilderReader.java  1 Feb 2008 17:19:41 -0000       1.93
+++ BuilderReader.java  16 Feb 2008 22:07:39 -0000      1.94
@@ -38,7 +38,7 @@
  * @author Rico Jansen
  * @author Pierre van Rooden
  * @author Michiel Meeuwissen
- * @version $Id: BuilderReader.java,v 1.93 2008/02/01 17:19:41 michiel Exp $
+ * @version $Id: BuilderReader.java,v 1.94 2008/02/16 22:07:39 nklasens Exp $
  */
 public class BuilderReader extends DocumentReader {
 
@@ -784,7 +784,14 @@
         if (state != def.getState()) def.setState(state);
 
 
-        boolean readOnly = "true".equalsIgnoreCase(fieldReadOnly);
+        boolean readOnly = false;
+        if ("".equals(fieldReadOnly)) {
+            readOnly = state == Field.STATE_SYSTEM || state == 
Field.STATE_SYSTEM_VIRTUAL;
+        }
+        else {
+            readOnly = "true".equalsIgnoreCase(fieldReadOnly);
+        }
+
         if (def.isReadOnly() != readOnly) {
             def.setReadOnly(readOnly);
         }
_______________________________________________
Cvs mailing list
[email protected]
http://lists.mmbase.org/mailman/listinfo/cvs

Reply via email to