10432 2010-08-25 15:39:39 +0000 alternative result for namespace-errors-q16 test 2010-09-28 16:35:35 +0000 1 1 1 Unclassified XQuery Update Facility Test Suite XQuery Update Facility Test Suite 1.0.1 PC Linux CLOSED FIXED P2 normal --- 1 markos_za andrew.eisenberg mike oldest_to_newest 37845 0 markos_za 2010-08-25 15:39:39 +0000 The reference result for the namespace-errors-q16 test adds a namespace declaration attribute (xmlns="") to un-declare the defaule element namespace binding. This attribute is added in the <PhD> node. However, it could also be added on the parent of that node. Specifically, the attached file is also a valid result. 37846 1 912 markos_za 2010-08-25 15:42:57 +0000 Created attachment 912 alternative result for test namespace-errors-q16 38265 2 mike 2010-08-30 22:59:11 +0000 Please note for reference that this is related to bug #9432 and bug #9312. 38268 3 mike 2010-08-30 23:15:51 +0000 Personal response: I think this bug is invalid - that is, the proposed alternative result is not a legitimate result. (However, it's difficult to be definitive until we see the changed wording introduced as a result of the resolution of bug #9432). propagate-namespace() adds a namespace binding (prefix, uri) to all child elements that do not already have a binding for that prefix, and then calls propagate namespace on that child element so it propagates down the tree. The effect of bug 9312/9432 is to extend the meaning of "do not already have a binding for that prefix" so that elements whose name is unprefixed are considered for this purpose to have a binding for the prefix "" even if it is a binding to no namespace. So in this case the namespace binding ("", "http://www.example.com/test") must be propagated to the univ:lab element, and must not be propagated to the PhD element. The proposed alternative result has xmlns="" on the univ:lab element, which means the namespace binding ("", "http://www.example.com/test") has not been propagated to this element, as the [yet to be written] spec requires. Michael Kay Saxonica 38338 4 mike 2010-08-31 16:10:32 +0000 It looks as if my comment #3 may have been based on an incorrect recollection of how we decided to resolve bug #9432. The decision on that bug is recorded in the Minutes of the Oxford F2F meeting: the decision recorded there was not to propagate the binding ("", "xyz"), whereas my recollection was that it should be propagated, but only so far as an element whose name is unprefixed. 38350 5 markos_za 2010-08-31 20:40:31 +0000 Given comment #4, what I reported here as a valid alternative result is actually the only valid result, and the current result in XQUTS is wrong. Right? 38352 6 mike 2010-08-31 20:51:16 +0000 Yes. I'd prefer to wait until I see final text for the bug #9432 resolution, but from today's discussion it's looking as if your proposed result for namespace-errors-q16 will be the only acceptable result. 39317 7 andrew.eisenberg 2010-09-24 22:05:52 +0000 The final text for the Bug #9432 has been entered. I have changed the expected result of this test case to the alternative that has been provided, as was suggested in comment #5. Markos, please close this bug report if you agree with this resolution. 912 2010-08-25 15:42:57 +0000 2010-08-25 15:42:57 +0000 alternative result for test namespace-errors-q16 namespace-errors-q16.res application/octet-stream 889 markos_za PG15Z3JhbnQgeG1sbnM6bmFyYT0iaHR0cDovL3d3dy5hbnIuZnIvbmFyYSIgeG1sbnM9Imh0dHA6 Ly93d3cuZXhhbXBsZS5jb20vdGVzdCI+IAogIDxuYXJhOmxhYiB4bWxucz0iIiBuYW1lPSJBZ3Jv UGx1cyI+IAogICAgIDxuYXJhOmFkZHJlc3M+U2FjbGF5LCBGcmFuY2U8L25hcmE6YWRkcmVzcz4g CiAgICAgPG5hcmE6cmVzZWFyY2hlciByaWQ9InIxIiBuYW1lPSJGcmVkIiBwb3NpdGlvbj0idGVj aG5pY2FsIHN0YWZmIi8+IAogICAgIDxuYXJhOnJlc2VhcmNoZXIgcmlkPSJyMiIgbmFtZT0iTGl6 IiBwb3NpdGlvbj0ibGFiIGFzc2lzdGFudCIvPiAKICAgICA8Y29vcDpQaEQgeG1sbnM6Y29vcD0i aHR0cDovL3d3dy5nb3V2LmZyL3VuaXYtaW5kdXN0cnktY29vcC8iIG5hbWU9Ik1hcmllIiBhZHZp c29yPSJyMSIvPiAKICAgICA8YWdybzpsYWIgeG1sbnM6YWdybz0iaHR0cDovL3d3dy5hZ3JvcGx1 cy5jb20iIG5hbWU9IkRhaXJ5IERlcHQiPiAKICAgICAgIDxhZ3JvOmVuZ2luZWVyIG5hbWU9Ik1h cmMiLz4gCiAgICAgPC9hZ3JvOmxhYj4gCiAgPC9uYXJhOmxhYj4gCiAgPHVuaXY6bGFiIHhtbG5z OnVuaXY9Imh0dHA6Ly93d3cuZWR1Y2F0aW9uLmdvdXYuZnIiIHhtbG5zPSIiIG5hbWU9IkZvb2Qg RW5naW5lZXJpbmcgRGVwdCwgT3JzYXkgVS4iPiAKICAgIDx1bml2OmFkZHJlc3M+T3JzYXksIEZy YW5jZTwvdW5pdjphZGRyZXNzPiAKICAgIDx1bml2OnJlc2VhcmNoZXIgcmlkPSJyMyIgbmFtZT0i SGVucnkiIHBvc2l0aW9uPSJhc3NvY2lhdGUgcHJvZmVzc29yIi8+IAogICAgPHVuaXY6UGhEIG5h bWU9IlJvYmVydCIgYWR2aXNvcj0icjMiLz4gCiAgICA8UGhEIG5hbWU9Ikp1bGlhIiBhZHZpc29y PSJyMSIvPiAKICA8L3VuaXY6bGFiPiAKPC9teWdyYW50Pg==