28627 2015-05-12 15:33:19 +0000 [QT3TS] Serialization-html-4 2015-06-24 08:14:37 +0000 1 1 1 Unclassified XPath / XQuery / XSLT XQuery 3 & XPath 3 Test Suite Working drafts PC Windows NT RESOLVED FIXED P2 normal --- 1 tim oneil christian.gruen public-qt-comments oldest_to_newest 120265 0 tim 2015-05-12 15:33:19 +0000 The element <isindex /> appears to be missing from the expected results. <test><![CDATA[ declare boundary-space strip; declare namespace output = "http://www.w3.org/2010/xslt-xquery-serialization"; declare option output:method "html"; declare option output:indent "no"; declare option output:version "5.0"; <html xmlns="http://www.w3.org/1999/xhtml"><body><area/><base/><br/><col/><embed /><frame/><hr/><img/><input/><isindex/><link/><meta/><param/></body></html> ]]></test> <result> <all-of> <serialization-matches><![CDATA[<html xmlns="http://www.w3.org/1999/xhtml"><body><area><base><br><col><embed><hr><img><input><link><meta><param></body></html>]]></serialization-matches> <serialization-matches flags="i"><![CDATA[<!DOCTYPE .*]]></serialization-matches> </all-of> </result> 120319 1 oneil 2015-05-15 13:38:36 +0000 According to the change log Christian Gruen removed the isindex element in light of the details in section 7.1 of the Serialisation spec. I must say I agree with this change and if you are happy I will mark this bug as resolved-Invalid. 120344 2 tim 2015-05-18 15:58:03 +0000 Sorry - I don't follow. Note that the query contains <isindex/> but that the expected result does not. Which bit of 7.1 says that <isindex> should be discarded when serializing? I can see test saying that it is implementation defined whether it should be treated as a void/empty element. 120345 3 oneil 2015-05-18 16:05:04 +0000 Yes the spec does state 'implementation defined'. I think we should have two alternative expected results. One with <isindex/> and the other without. 120360 4 tim 2015-05-19 14:29:27 +0000 There are two possible results, but neither of them is the one given. The test should accept either <serialization-matches><![CDATA[<html xmlns="http://www.w3.org/1999/xhtml"><body><area><base><br><col><embed><hr><img><input><isindex><link><meta><param></body></html>]]></serialization-matches> or <serialization-matches><![CDATA[<html xmlns="http://www.w3.org/1999/xhtml"><body><area><base><br><col><embed><hr><img><input><isindex></isindex><link><meta><param></body></html>]]></serialization-matches> The current expected results completely misses <isindex>. 121438 5 oneil 2015-06-23 12:52:52 +0000 At the telcon #611 the working group agreed to incorporate Tim's suggestion in the test case. See comment #4. 121440 6 tim 2015-06-23 14:24:21 +0000 This hasn't been fixed correctly. In CVS I see: <test-case name="Serialization-html-4" covers-30="html-output"> <description>Test that elements in the XHTML namespace whose local names match those of HTML 5.0 elements with empty content model are serialized using only a start tag</description> <created by="Henry Zongaro" on="2012-10-26"/> <modified by="O'Neil Delpratt" on="2013-05-30" change="added dependency inline with bug issue #21868"/> <modified by="Christian Gruen" on="2015-04-28" change="isindex and frame removed (implementation-defined, see 7.1)"/> <modified by="Christian Gruen" on="2015-05-20" change="isindex and frame removed (fix)"/> <modified by="O'Neil Delpratt, Saxonica" on="2015-06-23" change="Added isindex element bug issue #28627"/> <dependency type="feature" value="serialization" satisfied="true"/> <dependency type="spec" value="XQ30+"/> <test><![CDATA[ declare boundary-space strip; declare namespace output = "http://www.w3.org/2010/xslt-xquery-serialization"; declare option output:method "html"; declare option output:indent "no"; declare option output:version "5.0"; <html xmlns="http://www.w3.org/1999/xhtml"><body><area/><base/><br/><col/><embed /><hr/><img/><input/><link/><meta/><param/></body></html> ]]></test> <result> <all-of> <any-of> <serialization-matches><![CDATA[<html xmlns="http://www.w3.org/1999/xhtml"><body><area><base><br><col><embed><hr><img><input><isindex><link><meta><param></body></html>]]></serialization-matches> <serialization-matches><![CDATA[<html xmlns="http://www.w3.org/1999/xhtml"><body><area><base><br><col><embed><hr><img><input><isindex></isindex><link><meta><param></body></html>]]></serialization-matches> </any-of> <serialization-matches flags="i"><![CDATA[<!DOCTYPE .*]]></serialization-matches> </all-of> </result> </test-case> The expected results now expect "isindex", but this no longer appears in the query. 121445 7 christian.gruen 2015-06-23 17:22:42 +0000 One day after Tim's last comment (on May 20), I had already fixed the test. After consultation with O'Neil and Tim, I have just reverted the test case to this version. 121456 8 tim 2015-06-24 08:14:37 +0000 Confirmed fixed. Thanks.