This is an archived snapshot of W3C's public bugzilla bug tracker, decommissioned in April 2019. Please see the home page for more details.

Bug 28627 - [QT3TS] Serialization-html-4
Summary: [QT3TS] Serialization-html-4
Status: RESOLVED FIXED
Alias: None
Product: XPath / XQuery / XSLT
Classification: Unclassified
Component: XQuery 3 & XPath 3 Test Suite (show other bugs)
Version: Working drafts
Hardware: PC Windows NT
: P2 normal
Target Milestone: ---
Assignee: O'Neil Delpratt
QA Contact: Mailing list for public feedback on specs from XSL and XML Query WGs
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-05-12 15:33 UTC by Tim Mills
Modified: 2015-06-24 08:14 UTC (History)
1 user (show)

See Also:


Attachments

Description Tim Mills 2015-05-12 15:33:19 UTC
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>
Comment 1 O'Neil Delpratt 2015-05-15 13:38:36 UTC
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.
Comment 2 Tim Mills 2015-05-18 15:58:03 UTC
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.
Comment 3 O'Neil Delpratt 2015-05-18 16:05:04 UTC
Yes the spec does state 'implementation defined'. I think we should have two alternative expected results. One with <isindex/> and the other without.
Comment 4 Tim Mills 2015-05-19 14:29:27 UTC
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>.
Comment 5 O'Neil Delpratt 2015-06-23 12:52:52 UTC
At the telcon #611 the working group agreed to incorporate Tim's suggestion in the test case. See comment #4.
Comment 6 Tim Mills 2015-06-23 14:24:21 UTC
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.
Comment 7 Christian Gruen 2015-06-23 17:22:42 UTC
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.
Comment 8 Tim Mills 2015-06-24 08:14:37 UTC
Confirmed fixed.  Thanks.