<?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>5108</bug_id>
          
          <creation_ts>2007-09-30 19:16:58 +0000</creation_ts>
          <short_desc>sml 4.3.1 sml:acyclic</short_desc>
          <delta_ts>2007-12-04 20:21:26 +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>Core</component>
          <version>unspecified</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>editorial</keywords>
          <priority>P2</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>LC</target_milestone>
          <dependson>4793</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="John Arwe">johnarwe</reporter>
          <assigned_to name="Valentina Popescu">popescu</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>16912</commentid>
    <comment_count>0</comment_count>
    <who name="John Arwe">johnarwe</who>
    <bug_when>2007-09-30 19:16:58 +0000</bug_when>
    <thetext>Separate the concerns
from:
Model validators that conform to this specification MUST support the sml:acyclic attribute on any &lt;xs:complexType&gt; element in a schema document. 
to:
Model authors MAY specify the sml:acyclic attribute on any &lt;xs:complexType&gt; element in a schema document. 
Model validators that conform to this specification MUST support the sml:acyclic attribute.

from:
This is a boolean attribute and its value can be either true or false.
to: (re-use text from 4.1.1.1 SML Reference)

add: pointer to notation used for defining xml schema elements in 2.x. We are using its notation ([],{}) in 4.1.1.1 SML Reference, in sml:acyclic, in other places too.

4.3.1.1 Mapping from Schema
Since xs:anyType has no value for this, we need to state that its value is false.  i.e. set the default to provide a base case for the recursive definition.

headings are not consistent in the least between 4.1.x and 4.2.x.
4.3.1.1 Mapping from Schema seems to be the equivalent of 4.1.1.1 Ref Defs/SML Ref, 4.3.1.2+3 equiv? to 4.1.2.  There is probably a sensible cleavage, like syntactic definition and semantics, that apply to each one.  There should be a consistent structure so it doesn&apos;t get in the way of readers comprehension.

Not sure what the distinction is between 4.3.1.2 Rules and 4.3.1.3 Validation.  A validator would have to enforce the constraints in both sections, no?  They both seem like statement of semantics, suggest combining and calling them Semantics.

4.3.1.2 Rules
from: &quot;...but all derived types of an acylic reference type are     acyclic...&quot;
to  : &quot;...but all derived types of an acylic reference type must be acyclic...&quot;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>17565</commentid>
    <comment_count>1</comment_count>
    <who name="Virginia Smith">virginia.smith</who>
    <bug_when>2007-11-01 23:49:44 +0000</bug_when>
    <thetext>These changes depend on bug 4639 so I&apos;m removing the editorial keyword and adding the dependency.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>17603</commentid>
    <comment_count>2</comment_count>
    <who name="James Lynn">james.lynn</who>
    <bug_when>2007-11-06 02:03:14 +0000</bug_when>
    <thetext>Regarding the last change from &quot;derived types of an acylic reference type are acyclic...&quot; to &quot;derived types of an acylic reference type must be acyclic...&quot; I suspect there is a correct way to say this in terms of how Schema handles derived types, but appeal to our Schema experts for advice.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>17910</commentid>
    <comment_count>3</comment_count>
    <who name="John Arwe">johnarwe</who>
    <bug_when>2007-11-29 18:58:59 +0000</bug_when>
    <thetext>Most of the original comments are satisfied by the proposal for 4793 at 
http://lists.w3.org/Archives/Public/public-sml/2007Nov/0326.html

Separation of concerns: 4793 removes separate responsibilities for validators and authors, so covered.
true/false: 4793 covers
add: pointer to notation used  ... I do think this needs to be done ultimately; 4793 did not address this in copies of the proposal I saw
anyType remark: I forced 4793 to cover it.
headings are not consistent: editorial, assume 4793 covers it but did not check.  I trust you guys once the inconsistency is pointed out to fix it.
distinction between 4.3.1.2 and 4.3.1.3: covered by 4793
are -&gt; must be: covered by 4793

ok if we want to make this editorial once 4793 is accepted, assuming its fix still covers these points in substantively the same manner as the uri above.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>17958</commentid>
    <comment_count>4</comment_count>
    <who name="Valentina Popescu">popescu</who>
    <bug_when>2007-12-04 20:21:26 +0000</bug_when>
    <thetext>all fixes required by this defect have been applied under defect http://www.w3.org/Bugs/Public/show_bug.cgi?id=4793</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>