[dojo.data] missing "var" in ItemFileWriteStore on line 278

[skinner]

on line 278, I think this

typeMap = this._datatypeMap[type];

should instead be this

var typeMap = this._datatypeMap[type];

(An aside: I wonder if it would be possible to write a simple unit test for this sort thing -- a unit test that pulls in *all* the dojo modules and then just checks to make sure that the only global variable that got added was "dojo"?)

[Adam Peller]

well, wouldn't you have to exercise most of the code, too? Perhapst his would be a good addition to DOH. I'd suggest a separate ticket.

[Jared Jurkiewicz]

You need other global variables now and then. For example, the FlickrStore? requires a global function for handling the data returned. So a test requiring only dojo being defined isn't so good.

[Adam Peller]

For examples, perhaps, but for the Dojo library, we're pretty strict about global variables. It's a worthwhile test to run. You can always be selective about the results, or perhaps if you think a DOH test needs to include a global variable, we could have an API to declare it so it doesn't get flagged as an error.

[Jared Jurkiewicz]

(In [9754]) Trivial fix in ItemFireWriteStore? fixes #3824