<?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>5721</bug_id>
          
          <creation_ts>2008-05-28 22:30:24 +0000</creation_ts>
          <short_desc>Statement about definitions in definition docs vs. instance docs would help</short_desc>
          <delta_ts>2008-08-14 20:14:05 +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>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>editorial</keywords>
          <priority>P2</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Julia McCarthy">julia</reporter>
          <assigned_to name="Virginia Smith">virginia.smith</assigned_to>
          <cc>james.lynn</cc>
    
    <cc>johnarwe</cc>
    
    <cc>kumarp</cc>
    
    <cc>pratul.dublish</cc>
    
    <cc>virginia.smith</cc>
          
          <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>20291</commentid>
    <comment_count>0</comment_count>
    <who name="Julia McCarthy">julia</who>
    <bug_when>2008-05-28 22:30:24 +0000</bug_when>
    <thetext>I spent a lot of time trying to understand things like
1) Why it wasn&apos;t ridiculous to say sml:ref=&quot;true&quot; followed by sml:nilref=&quot;true&quot;;
2) Why it made sense to define acyclic, targetType, etc. on a type that did not have an sml:ref attribute.

After much puzzling (and questions to John) I believe the reason is because it can be useful to define some things in the model definition documents while leaving some decisions to the instance document author. For example:
1) The definition document might say that a particular type is an sml reference but the instance document author may decide to not to define it as a reference;
2) The definition document author might want to add constraints while leaving the decision about whether to use an sml reference up to the instance document author.

I believe it would be helpful to readers if there were a non-normatve note (or two) that explained this.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>20384</commentid>
    <comment_count>1</comment_count>
    <who name="Kumar Pandit">kumarp</who>
    <bug_when>2008-06-05 18:37:29 +0000</bug_when>
    <thetext>resolution (6/5/08 conf call):  mark needsAgreement
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>20596</commentid>
    <comment_count>2</comment_count>
    <who name="Julia McCarthy">julia</who>
    <bug_when>2008-06-19 13:07:09 +0000</bug_when>
    <thetext>Proposal 1 (addresses part 1 of this issue re: sml:nilref attribute):
Add this non-normative statement to the last paragraph of section 4.1.2 between the current first and second (last) sentences: For example, sml:nilref may be useful in the case where the complex type defines sml:ref=&quot;true&quot; and an instance document author creating an element with that type does not choose to include a reference.

Proposal 2 (addresses part 2 of this issue re: constraint attributes):

Add one or more non-normative sentences at the end of section 5.1.3 that make these points:
	sml:ref can be defined either on a ComplexType or in the element declaration.
	sml:acyclic can be defined only on a ComplexType
	the sml:targetXXX attributes can be defined only on element declarations
	this is the reason that an element that is not an sml reference could be defined with one of the constraints on sml references
	this is valuable because the decision about whether to include a constraint and the decision about whether to make the element an sml reference can be made independently - some choices made by the schema author, other choices made by the instance document author


</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>20608</commentid>
    <comment_count>3</comment_count>
    <who name="Virginia Smith">virginia.smith</who>
    <bug_when>2008-06-19 19:26:22 +0000</bug_when>
    <thetext>Resolution 6/19:

Fix per proposal 1 in Comment #2.
Fix per proposal 2 in Comment #2, bullets 4 and 5 only.

No needsReview necessary.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>20793</commentid>
    <comment_count>4</comment_count>
    <who name="James Lynn">james.lynn</who>
    <bug_when>2008-06-24 13:50:04 +0000</bug_when>
    <thetext>Fixed as per Comment #3
Marked as needsReview as the changes in Proposal 1 might read better is the non-normative note is either mover to the end of the paragraph or the entire paragraph is incorporated into the note. Alternatively, the last sentence could be reworded.

Proposal 1:
http://dev.w3.org/cvsweb/~checkout~/2007/xml/sml/build/sml.html?content-type=text/html;%20charset=utf-8#Null_Reference

Proposal 2:
http://dev.w3.org/cvsweb/~checkout~/2007/xml/sml/build/sml.html?content-type=text/html;%20charset=utf-8#constraints_summary</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>20827</commentid>
    <comment_count>5</comment_count>
    <who name="John Arwe">johnarwe</who>
    <bug_when>2008-06-25 09:14:52 +0000</bug_when>
    <thetext>f2f consensus review of comment 4

proposal 1: wg ignored the fact that no diff was given, grudgingly, however believes the text should be changed to:

It is a consequence of the preceding that this specification assigns no meaning to the sml:nilref attribute when it is used on an element that is not an SML reference.  Model validators MAY choose to warn their invokers should they detect this condition in a document.

Note:

sml:nilref may be useful in the case where the schema author defines a complex type specifying sml:ref with a fixed value of &quot;true&quot;, but the instance author wants to signal the absence of a target.  

proposal 2: wg ignored the fact that no diff was given, grudgingly, however it prefers the following

The constraints described above can be useful even on element declarations whose instances are not necessarily SML references, because the decision about whether to include a constraint and the decision about whether to make the element an SML reference can be made independently - some choices made by the schema author, other choices made by the instance document author.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>21035</commentid>
    <comment_count>6</comment_count>
    <who name="James Lynn">james.lynn</who>
    <bug_when>2008-07-03 04:01:02 +0000</bug_when>
    <thetext>Fixed as per changes in Comment #5</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>21045</commentid>
    <comment_count>7</comment_count>
    <who name="John Arwe">johnarwe</who>
    <bug_when>2008-07-03 15:51:44 +0000</bug_when>
    <thetext>The following sentence is now repeated before and after the Note in SML 4.1.2, one of them (probably the second copy) should be removed. 

Model validators MAY choose to warn their invokers should they detect this condition in a document.

</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>21565</commentid>
    <comment_count>8</comment_count>
    <who name="Virginia Smith">virginia.smith</who>
    <bug_when>2008-08-14 20:14:05 +0000</bug_when>
    <thetext>The fix requested in comment #7 was made as part of another bug fix since I noticed the repetition when editing that section.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>