<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://www.w3.org/Bugs/Public/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4"
          urlbase="https://www.w3.org/Bugs/Public/"
          
          maintainer="sysbot+bugzilla@w3.org"
>

    <bug>
          <bug_id>5294</bug_id>
          
          <creation_ts>2007-11-28 17:38:04 +0000</creation_ts>
          <short_desc>Add an explanation on why the smlif:dataType has processContents=skip</short_desc>
          <delta_ts>2007-12-13 19:23:38 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>SML</product>
          <component>Interchange Format</component>
          <version>LC</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Windows XP</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>resolved</keywords>
          <priority>P2</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Valentina Popescu">popescu</reporter>
          <assigned_to name="Virginia Smith">virginia.smith</assigned_to>
          
          
          <qa_contact name="SML Working Group discussion list">public-sml</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>17878</commentid>
    <comment_count>0</comment_count>
    <who name="Valentina Popescu">popescu</who>
    <bug_when>2007-11-28 17:38:04 +0000</bug_when>
    <thetext>As part of the resolution for defect http://www.w3.org/Bugs/Public/show_bug.cgi?id=4775, the processContents 
for the smlif:dataType has been changed from lax to skip.

Since this change had some not so obvious reasons, I propose 
to have the spec explain this approach.

Proposal :

Add this comment to the sml-if.xsd schema, under the dataType type

  &lt;xs:complexType name=&quot;dataType&quot; mixed=&quot;false&quot;&gt;
    &lt;xs:annotation&gt;
      &lt;xs:documentation&gt;
The wildcard with processContents &quot;skip&quot; matches the root element of the   model document being packaged. The value of processContents is set to &quot;skip&quot; so that the contained element is not attempted for schema validation. As a result, validity of the packaged document will not affect validity of the IF document itself.
      &lt;/xs:documentation&gt;
    &lt;/xs:annotation&gt;    
    &lt;xs:sequence&gt;
      &lt;xs:any processContents=&quot;skip&quot; minOccurs=&quot;0&quot;/&gt;
    &lt;/xs:sequence&gt;
  &lt;/xs:complexType&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>17941</commentid>
    <comment_count>1</comment_count>
    <who name="Virginia Smith">virginia.smith</who>
    <bug_when>2007-12-03 21:46:29 +0000</bug_when>
    <thetext>Fix per proposal.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>17945</commentid>
    <comment_count>2</comment_count>
    <who name="Virginia Smith">virginia.smith</who>
    <bug_when>2007-12-03 22:20:11 +0000</bug_when>
    <thetext>The new schema reads as follows: 
--------
  &lt;xs:complexType name=&quot;dataType&quot; mixed=&quot;false&quot;&gt;
    &lt;xs:annotation&gt;
      &lt;xs:documentation&gt;
The wildcard with processContents &quot;skip&quot; matches the root element of the  
model document being packaged. The value of processContents is set to &quot;skip&quot; so
that the contained element is not processed for schema validation. As a result,
validity of the packaged document will not affect validity of the IF document
itself.
      &lt;/xs:documentation&gt;
    &lt;/xs:annotation&gt;    
    &lt;xs:sequence&gt;
      &lt;xs:any processContents=&quot;skip&quot; minOccurs=&quot;0&quot; namespace=&quot;##any&quot; maxOccurs=&quot;1&quot;/&gt;
    &lt;/xs:sequence&gt;
    &lt;xs:anyAttribute namespace=&quot;##other&quot; processContents=&quot;lax&quot;/&gt;
  &lt;/xs:complexType&gt;

--------

Note the phrase &quot;the contained element is not **processed** for schema validation&quot;. This is a change from the proposal.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>17962</commentid>
    <comment_count>3</comment_count>
    <who name="Kumar Pandit">kumarp</who>
    <bug_when>2007-12-05 08:18:39 +0000</bug_when>
    <thetext>I agree with the change in comment# 2.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>17965</commentid>
    <comment_count>4</comment_count>
    <who name="Sandy Gao">sandygao</who>
    <bug_when>2007-12-05 14:29:31 +0000</bug_when>
    <thetext>I also agree with changes in comment #2.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>17967</commentid>
    <comment_count>5</comment_count>
    <who name="Valentina Popescu">popescu</who>
    <bug_when>2007-12-05 14:36:56 +0000</bug_when>
    <thetext>+1 for change described in comment #2</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>18063</commentid>
    <comment_count>6</comment_count>
    <who name="Kirk Wilson">kirk.wilson</who>
    <bug_when>2007-12-13 16:26:38 +0000</bug_when>
    <thetext>+1 for change described in comment #2.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>