278467 - String object (not literal string) as value to setParameter causes error.

Status: RESOLVED FIXED

(Core :: XSLT, defect)

Description

[William J. Edney]

It seems that using setParameter() with a String object as opposed to a literal
string will cause the XSLT processor to give up with an error.

In other words, this is bad:

myProcessor = new XSLTProcessor();
myProcessor.setParameter(null,'foo',new String('bar'));

Note that the following works:

myProcessor.setParameter(null,'foo','bar');

Testcase is coming up.

Cheers,

- Bill

Comment 1

[William J. Edney]

Attachment: Test case showing that using String 'objects' as opposed to literals doesn't work.

Run the test case. It works because the parameter is a string literal.

Edit the JS in the transformIt() function in the head of the document and
uncomment the line that assigns a String object to the parameter value.

The transformation no longer works and gives an error in the JavaScript
console.

Comment 2

[Peter Van der Beken [:peterv]]

So I could fix this in Transformiix, but I wonder if it wouldn't be better to
have XPConnect handle this itself. XPCVariant::InitializeData
(http://lxr.mozilla.org/seamonkey/source/js/src/xpconnect/src/xpcvariant.cpp#240)
special-cases JS primitive types and JS arrays (and nsIJSID, nsIJSIID and
nsIJSCID). Anything else just gets wrapped and stored as an nsISupports in the
variant. Would it make sense to have some of the builtin JS classes converted
into a specific nsVariant type (VTYPE_DOUBLE for Number, VTYPE_BOOL for Boolean,
...)?

Comment 3

[Peter Van der Beken [:peterv]]

Attachment: Better testcase

Comment 4

[Peter Van der Beken [:peterv]]

I'll do this as part of bug 248025. I'll convert all JS objects to string
parameters.

Comment 5

[Peter Van der Beken [:peterv]]

Comment on attachment 171306 [details]
Test case showing that using String 'objects' as opposed to literals doesn't work.

This doesn't crash in any way.

Comment 6

[Peter Van der Beken [:peterv]]

Fixed by patch in bug 248025, all JS objects are converted to string now.