440974 - Unneeded <transformiix:root /> elements as a result of XSL transformation.

Status: RESOLVED FIXED

(Core :: XSLT, defect, P3)

Description

[Alexei Gorkov]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9) Gecko/2008052906 Firefox/3.0
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9) Gecko/2008052906 Firefox/3.0

Unneeded <transformiix:root /> elements appear after XSL transformation performed by XSLTProcessor.transformToDocument() method. It is not a wrapping <transformiix:root /> root element that is added to form valid XML document, but unneeded <transformiix:root /> elements wrapping child nodes of the resulting document.

Reproducible: Always

Steps to Reproduce:
1. XML document:

    <?xml version="1.0"?>
    <items>
        <item>
            <id>1</id>
        </item>
        <item>
            <id>2</id>
        </item>
        <item>
            <id>3</id>
        </item>           
    </items> 

2. XSL stylesheet:

<?xml version="1.0"?>
<xsl:stylesheet version="1.0" xmlns:xsl="http://www.w3.org/1999/XSL/Transform">
    <xsl:output method="html" />
    <xsl:template match="item">
        <tr id="{id}">
            <td>
                <xsl:value-of select="id" />
            </td>
        </tr>   
    </xsl:template>
</xsl:stylesheet>

3. Load these XML and XSL documents to Document instances.

4. Transform this XML document using given XSL stylesheet with the following code:

            var processor = new XSLTProcessor();
            processor.importStylesheet(xslDoc);           
            var doc = processor.transformToDocument(xmlDoc);             

Actual Results:  
<transformiix:result xmlns:transformiix="http://www.mozilla.org/TransforMiix">
    <transformiix:result>
        <TR id="1">
            <TD>1</TD>
        </TR>
        <TR id="2">
            <TD>2</TD>
        </TR>
    </transformiix:result>
    <TR id="3">
        <TD>3</TD>
    </TR>
</transformiix:result>


There is unexpected <transformiix:result /> element wrapping first two <tr /> elements in the resulting document.


Expected Results:  
 <transformiix:result xmlns:transformiix="http://www.mozilla.org/TransforMiix">
    <TR id="1">
        <TD>1</TD>
    </TR>
    <TR id="2">
        <TD>2</TD>
    </TR>
    <TR id="3">
        <TD>3</TD>
    </TR>
</transformiix:result>

With an increase of the number of items in the original XML document we get increase in nesting of <transformiix:result /> elements. For example with four <item /> elements I got:

<transformiix:result xmlns:transformiix="http://www.mozilla.org/TransforMiix">
    <transformiix:result>
        <transformiix:result>
            <TR id="1">
                <TD>1</TD>
            </TR>
            <TR id="2">
                <TD>2</TD>
            </TR>
        </transformiix:result>
        <TR id="3">
            <TD>3</TD>
        </TR>
    </transformiix:result>
    <TR id="4">
        <TD>4</TD>
    </TR>
</transformiix:result>

Comment 1

[Sergey Mosyakov]

Dear colleagues, 
I have the same problem in Windows XP SP 3 
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9) Gecko/2008052906 Firefox/3.0
Could anybody decide is it a feature of FF 3 or is it a bug?

Comment 2

[Aiko]

Attachment: Testcase, based on the code in comment 0

Attaching a testcase or at least providing a link increases your chances of someone looking at your bug report...

Comment 3

[Sergey Mosyakov]

It is very interesting how long bug can have status UNCONFIRMED. When I post any issue to Microsoft it works more faster. Can anybody decide is it the issue or not?

Comment 4

[Mark Omohundro]

Another Firefox workaround needed guys?  In order to get more-or-less
consistent output with the browsers I tested this example on, I needed XSLT
something like this:

<?xml version="1.0"?>
  <xsl:stylesheet version="1.0" xmlns:xsl="http://www.w3.org/1999/XSL/Transform">
    <xsl:output method="html" />
    <xsl:template match="items">
    <table>
    <xsl:for-each select="item">
      <tr><xsl:apply-templates/></tr>
    </xsl:for-each>
    </table>
    </xsl:template>
</xsl:stylesheet>

or this --

<?xml version="1.0"?>
  <xsl:stylesheet version="1.0" xmlns:xsl="http://www.w3.org/1999/XSL/Transform">
    <xsl:output method="html" />
    <xsl:template match="items">
    <tr id="{id}">
      <xsl:for-each select="item">
        <td align="left"><xsl:apply-templates/></td>
      </xsl:for-each>
    </tr>
    </xsl:template>
</xsl:stylesheet>

But is it necessary that everyone do a workaround?

Comment 5

[Peter Van der Beken [:peterv]]

Attachment: v1

Still need to automate testcase.

Comment 6

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Comment on attachment 337056 [details] [diff] [review]
v1

Yes, please do write tests as well.

Comment 7

[Peter Van der Beken [:peterv]]

Attachment: v1

Same as attachment 337056 [details] [diff] [review], but with mochitest. Carrying over r/sr=sicking.

Comment 8

[Phil Anderson]

Sorry for my ignorance about bugzilla, but where is this fixed?

I'm trying to work out why our XML files don't process properly in firefox 3.0.6 (but do in IE, & xsltproc).  I used the Firefox XSLT Results addin, which showed me that for some reason, there is a ridiculously large number of <transformiix:result> tags in our processed document.

Comment 9

[jwuertz]

So I'm working in Firefox 39.0a2 and this is still not fixed.
This should not be marked as resolved.

Comment 10

[Peter Van der Beken [:peterv]]

You'll have to file a new bug with a testcase. The bug as filed (see testcase in comment 0) was fixed.

Comment 11

[jwuertz]

The test case attached to this bug fails in 39.0a2, with a few changes. I have attached my new version of the test case. I don't see the point of opening a new bug for exactly the same problem.

Comment 12

[jwuertz]

Attachment: Updated version of the testcase so it runs in the latest version of firefox dev

Comment 13

[Peter Van der Beken [:peterv]]

(In reply to jwuertz from comment #11)
> The test case attached to this bug fails in 39.0a2, with a few changes.

It doesn't fail for me, it shows: <transformiix:result xmlns:transformiix="http://www.mozilla.org/TransforMiix"><tr xmlns="http://www.w3.org/1999/xhtml" id="1"><td>1</td></tr><tr xmlns="http://www.w3.org/1999/xhtml" id="2"><td>2</td></tr><tr xmlns="http://www.w3.org/1999/xhtml" id="3"><td>3</td></tr><tr xmlns="http://www.w3.org/1999/xhtml" id="4"><td>4</td></tr></transformiix:result>

> I don't see the point of
> opening a new bug for exactly the same problem.

The original problem was fixed in this bug, reopening is just going to add noise as any new problems probably have a different cause. If you are seeing multiple transformiix:result elements then please open a new bug.