<?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>4172</bug_id>
          
          <creation_ts>2007-01-09 18:35:23 +0000</creation_ts>
          <short_desc>[UPD] Attribute name clash</short_desc>
          <delta_ts>2007-02-15 12:00:55 +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>Linux</op_sys>
          <bug_status>CLOSED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="John Snelson">john.snelson</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>13389</commentid>
    <comment_count>0</comment_count>
    <who name="John Snelson">john.snelson</who>
    <bug_when>2007-01-09 18:35:23 +0000</bug_when>
    <thetext>The XQuery Update specification does not deal with cases when the names of inserted attributes clash. They could clash with each other, or with attributes that already exist. I assume that this should be an error scenario.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>13428</commentid>
    <comment_count>1</comment_count>
    <who name="Andrew Eisenberg">andrew.eisenberg</who>
    <bug_when>2007-01-10 22:39:24 +0000</bug_when>
    <thetext>Answering as an individual, I believe that we have covered this with a very generic statement in section 3.2.2 upd:applyUpdates, Semantics, bullet 5:

&quot;If the resulting XDM instance violates any constraint specified in [XQuery/XPath Data Model (XDM)], a dynamic error is raised [err:XUDY0021].&quot;

</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>13441</commentid>
    <comment_count>2</comment_count>
    <who name="John Snelson">john.snelson</who>
    <bug_when>2007-01-11 11:06:47 +0000</bug_when>
    <thetext>I can see how that applies, and missed that on my readings through the spec. However, it might be more user friendly to expand on this statement in places where it applies - such as this case.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>13443</commentid>
    <comment_count>3</comment_count>
    <who name="Frans Englich">frans.englich</who>
    <bug_when>2007-01-11 12:36:14 +0000</bug_when>
    <thetext>I think the suggestion in #2 is sensible. The problem with having a broad statement, although if it correctly covers the conditions, is that it&apos;s then up to the reader to figure out where it applies. I believe having more targeted descriptions of error conditions improves the interoperability in these areas, as well as making test-suite creation easier.

Perhaps a sensible editorial change is to remove the broad statement(so we don&apos;t get overlaps, which is messy even if it&apos;s with identical codes) and add a sentence in each specific case it applies.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>13976</commentid>
    <comment_count>4</comment_count>
    <who name="Don Chamberlin">chamberl</who>
    <bug_when>2007-02-13 18:38:53 +0000</bug_when>
    <thetext>John,
Thanks for your comment, which was discussed by the working group on 13 Feb 2007. The consensus of the working group is that the error of attribute name clashes cannot be detected until applyUpdates time, because the clashing attribute names might have been created by different kinds of expressions (for example, a rename and an insert). So we think that the error is identified in the right place, Rule 5 of upd:applyUpdates. However, we agree that it would be helpful to identify attribute name clashes as a specific example of the rule. Therefore we will add a Note to the next version of the document identifying attribute name clashes as an example of a data model constraint violation, with error code XUDY0021.

I hope this change addresses your concern, and I am marking this bug as &quot;Fixed&quot;. You may change its status to &quot;Closed&quot; or reopen it if you have additional comments or concerns.

Regards,
Don Chamberlin (for the Query Working Group)</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>