[Proto-Scripty] Re: prototype-1.6.0.3 line1212 'if (params = Object.toQueryString(params))' is correct ?

2008-11-14 Thread josi
I've also seen this "bug", it's producing a warning in firefox-console if config is set to javascript.options.strict == true; Also jslint produces a "Expected a conditional expression and instead saw an assignment"-warning for this. this is not very nice style, in my opinion, but it works. On Nov

[Proto-Scripty] Re: prototype-1.6.0.3 line1212 'if (params = Object.toQueryString(params))' is correct ?

2008-11-13 Thread T.J. Crowder
It's not great style, but yes, I'm fairly certain it really does mean to be doing an assignment there and then testing the result of the assignment for falsity. E.g.: params = Object.toQueryString(params); if (params) { Why? Did it just look like a bug to you, or were you looking into