Beefy Boxes and Bandwidth Generously Provided by pair Networks
P is for Practical
 
PerlMonks  

Re^7: XML::XSLT problem

by gellyfish (Monsignor)
on Nov 17, 2006 at 21:21 UTC ( #584802=note: print w/ replies, xml ) Need Help??


in reply to Re^6: XML::XSLT problem
in thread XML::XSLT problem

XML::XSLT should just ignore the fo stuff, though that isn't being tested.

But yes you have uncovered a bug in a code path which wasn't being tested. Basically the problem is with an outer <xsl:variable /> declaration where there is a literal value (like you have there) rather than supplied with a value attribute. It needs to create a document fragment to store the results of evaluating the literal value and createDocumentFragment() needs to be called as a method of the XML::DOM::Document object, but as the top level variables are evaluated on the first parse of the XSL document there isn't an XML::DOM::Document yet :-(.

The bottom line is that whilst your example above could quite happily be expressed as:

<xsl:variable name="TextFontSize" value="10" />
I can't promise a quick fix that does the right thing as surprisingly it's going to require quite a large change, You might want to check out the version in CVS but that cheats with the document fragment and elicits a warning from XML::DOM and also doesn't respect any XML in the variable value but YMMV. Caveat Emptor etc etc.

But I have a test for this now so it will get fixed eventually

Thanks for the bug report, sorry I can't help you straight away.

/J\


Comment on Re^7: XML::XSLT problem
Select or Download Code

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://584802]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others drinking their drinks and smoking their pipes about the Monastery: (8)
As of 2014-08-20 16:05 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    The best computer themed movie is:











    Results (118 votes), past polls