Minor nits
 
1.  2 errors in xupdate dtd (in working draft):
  a.  <!ELEMENT xupdate:insert-after ... > refers to the entity 'instuctions' (misspelled)
  b.  <!ENTITY % template ...> is missing a vertical bar '|' after #PCDATA
After these changes, the DTD becomes valid.
 
2.  In the working draft, 2nd para of introduction, change 'extensively' to 'extensive'
 
Keep up the good work.
 
Jeff
 
-------------------------------------- revised version of xupdate dtd ------------------------
 
 <!ENTITY % commands "
     xupdate:variable
   | xupdate:insert-before
   | xupdate:insert-after
   | xupdate:append
   | xupdate:update
   | xupdate:remove
   | xupdate:rename
 ">
 
 <!ENTITY % instructions "
     xupdate:element
   | xupdate:attribute
   | xupdate:text
   | xupdate:processing-instruction
   | xupdate:comment
 ">
 
 <!ENTITY % qname "NMTOKEN">
 
 <!ENTITY % template "
  (#PCDATA |
   %instructions;)*
 ">
 
 <!ELEMENT xupdate:modifications (%commands;)*>
 <!ATTLIST xupdate:modifications
    id          ID      #IMPLIED
    version     NMTOKEN #REQUIRED
    xmlns:xupdate CDATA   #FIXED "http://www.xmldb.org/xupdate"
 >
 
 <!ELEMENT xupdate:insert-before (%instructions;)*>
 <!ATTLIST xupdate:insert
    select      CDATA   #REQUIRED
 >
 
 <!ELEMENT xupdate:insert-after (%instructions;)*>
 <!ATTLIST xupdate:insert
    select      CDATA   #REQUIRED
 >
 
 <!ELEMENT xupdate:append (%instructions;)*>
 <!ATTLIST xupdate:insert
    select      CDATA   #REQUIRED
    child       CDATA   #IMPLIED
 >
 
 <!ELEMENT xupdate:element %template;>
 <!ATTLIST xupdate:element
    name        %qname; #REQUIRED
    namespace   CDATA   #IMPLIED
 >
 
 <!ELEMENT xupdate:attribute (#PCDATA)>
 <!ATTLIST xupdate:attribute
    name        %qname; #REQUIRED
    namespace   CDATA   #IMPLIED
 >
 
 <!ELEMENT xupdate:text (#PCDATA)>
 
 <!ELEMENT xupdate:processing-instruction (#PCDATA)>
 <!ATTLIST xupdate:processing-instruction
    name        NMTOKEN #REQUIRED
 >
 
 <!ELEMENT xupdate:update (#PCDATA)>
 <!ATTLIST xupdate:update
    select      CDATA   #REQUIRED
 >
 
 <!ELEMENT xupdate:remove EMPTY>
 <!ATTLIST xupdate:remove
    select      CDATA   #REQUIRED
 >
 
 <!ELEMENT xupdate:rename (#PCDATA)>
 <!ATTLIST xupdate:rename
    select      CDATA   #REQUIRED
 >
 
 <!ELEMENT xupdate:variable (#PCDATA)*>
 <!ATTLIST xupdate:variable
    name        NMTOKEN #REQUIRED
    select      CDATA   #IMPLIED
 >
 
 <!ELEMENT xupdate:value-of EMPTY>
 <!ATTLIST xupdate:value-of
    select      CDATA   #REQUIRED
 >
 
 <!ELEMENT xupdate:if %template;>
 <!ATTLIST xupdate:if
    test        CDATA   #REQUIRED
 >

Reply via email to