190631 - value-of creates a textnode even if stringvalue is an empty string

Status: VERIFIED FIXED

(Core :: XSLT, defect)

Description

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

We call resulthandler->characters in the code for xsl:value-of even if the
calculated stringvalue is an empty string. This has two effects:

1. In the standalone-outputhandlers the following stylesheet
   <out><xsl:value-of select="''"/></out>
   gives the output "<out></out>" rather then "<out/>". This makes us fail a lot
   of the string tests in xalan.

2. The following stylesheet
   <out><xsl:value-of select="''"/><xsl:attribute name="foo"/></out>
   will not create the attribute since we "close" the element when executing the
   xsl:value-of.
   The spec says on xsl:attribute: "The following are all errors: Adding an
   attribute to an element after children have been added to it..."
   And on xsl:value-of: "The string specifies the string-value of the created
   text node. If the string is empty, no text node will be created."
   So it's clear that we are doing the wrong thing.

There are two ways to solve this. Either we teach all the output-handlers to
ignore ::characters/::charactersNoOutputEscaping being called with an empty
string, or we make xsl:value-of not call the resulthandler if the string is empty.

I've done the latter since that's what the spec says.

Comment 1

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

Attachment: patch to fix

i also included a small fix to get rid of a warning in the HTML-output-handler
where we were peek()-ing an empty stack

Comment 2

[Axel Hecht]

Comment on attachment 112646 [details] [diff] [review]
patch to fix

if you believe it or not, r=[e-mail] ;-)

Comment 3

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

Comment on attachment 112646 [details] [diff] [review]
patch to fix

this is a very lowrisk patch since it hardly changes any logic. It's fully
contained to XSLT, no other pages will be affected.

Comment 4

[Asa Dotzler [:asa]]

Comment on attachment 112646 [details] [diff] [review]
patch to fix

a=asa (on behalf of drivers) for checkin to 1.3beta.

Comment 5

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

checked in. thanks for reviews

Comment 6

[Axel Hecht]

mass verifying