<?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>4537</bug_id>
          
          <creation_ts>2007-05-04 23:35:45 +0000</creation_ts>
          <short_desc>[UPD] Section 3.1.11 upd:rename: Add well-formedness check for attribute duplicates</short_desc>
          <delta_ts>2007-06-15 21:04:16 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>XPath / XQuery / XSLT</product>
          <component>Update Facility</component>
          <version>Working drafts</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Windows XP</op_sys>
          <bug_status>CLOSED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>editorial</keywords>
          <priority>P2</priority>
          <bug_severity>minor</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Michael Rys">mrys</reporter>
          <assigned_to name="Andrew Eisenberg">andrew.eisenberg</assigned_to>
          
          
          <qa_contact name="Mailing list for public feedback on specs from XSL and XML Query WGs">public-qt-comments</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>15013</commentid>
    <comment_count>0</comment_count>
    <who name="Michael Rys">mrys</who>
    <bug_when>2007-05-04 23:35:45 +0000</bug_when>
    <thetext>Add well-formedness check for attribute duplicates. While upd:applyUpdates seems to check for it, other sections call it out.
Make spec consistent.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>15070</commentid>
    <comment_count>1</comment_count>
    <who name="Don Chamberlin">chamberl</who>
    <bug_when>2007-05-11 23:52:56 +0000</bug_when>
    <thetext>Michael,
I don&apos;t understand what you are requesting here. I don&apos;t see any sections (apart from upd:applyUpdates) that check for duplicate attribute names. Can you give me an example of a rule in some section that you think should be replicated in Section 3.1.11? (Note that upd:rename() can only rename a single attribute node, so there is no possibility of an internal name conflict within that primitive.)
--Don Chamberlin</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>15072</commentid>
    <comment_count>2</comment_count>
    <who name="Michael Rys">mrys</who>
    <bug_when>2007-05-11 23:57:37 +0000</bug_when>
    <thetext>A rename could change the name of an attribute to a name that conflicts with an already existing name.

The application of upd:applyUpdates() makes sure that this does not happen, but somebody who just reads the section on rename does not see that this check occurs.

It would be useful to call it out editorially with a note that such checks will occur when we talk about the operation, and not only burry it deep down in the semantics.
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>15444</commentid>
    <comment_count>3</comment_count>
    <who name="Don Chamberlin">chamberl</who>
    <bug_when>2007-06-15 21:04:05 +0000</bug_when>
    <thetext>This editorial suggestion has been accepted and implemented.
--Don Chamberlin (for the Query Working Group)</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>