dojo.style() problem

[dwight]

I think I found a bug in Dojo v1.4.0 (20973), the dojo.style() function:

line 5895 of the uncompressed file is:

return (args == 1) ? s : _toStyleValue(n, style, s[style] ll n.style[style]); /* CSS2Properties

String

Number */

..and I believe it should be:

return (args == 1) ? s : (_toStyleValue(n, style, s[style]) ll n.style[style]); /* CSS2Properties

String

Number */

That is, it is missing some parentheses. It tries to use _toStyleValue() function regardless of returned 's' style object

note: pardon the l's for vertical bars...

[dwight]

Also, FireFox? 3.5.8 for Linux:

Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1.8) Gecko/20100214 Ubuntu/9.10 (karmic) Firefox/3.5.8 GTB6

FF complains about 's' object variable being null, although other browsers don't. I changed line 5888 from: var s = gcs(n);

to: var s = gcs(n) ll {};

...which stopped the error message.

I guess I should add that the original suggested change is my best guess of what was suppose to be the the way dojo.style() should work, but I could be missing something.

[James Burke]

Could you attach a test case that shows how the function is failing? That would help to make sure we are fixing the right thing, and make sure it does not break in the future.

[James Burke]

Please reopen with a test case/test dojo.style() call that does not work and is fixed by the suggested change.