Check for global scope violations for Dojo 1.3 Release

[dylan]

Check for global scope violations for the 1.3 release.

[dylan]

(In [16889]) refs #8798, declare a variable that was scoped globally in Dijit's scroll.js, \!strict

[dylan]

NodeList?.js, line 481:

return d.query(queryOrListOrNode).place(item[0], position); // dojo.NodeList

item is out of scope.

[dylan]

(In [16890]) refs #8798, bundleMap, not bundle, \!strict

[dylan]

(In [16891]) refs #8798, add fixme note to TablePlugin? for the editor, and fix another global scope issue, \!strict

[dylan]

(In [16966]) refs #8798, missing var for local variable, \!strict

[dylan]

(In [16967]) refs #8798, missing var for local variable, \!strict

[dylan]

(In [16968]) refs #8798, unintended syntax in variable declaration block, \!strict

[dylan]

In layout/dnd/PlottedDnd.js, the following are unexplained global vars. They either need to be fixed or documented:

IEonselectstart autoScrollActive

[dylan]

Ok, once the two comments are addressed in this bug, this one is done.

[dante]

the NodeList? one does seem to exist any longer (assuming Eugene's NodeList? patch took care of it?). will look into plotteddnd.js now ...

[dante]

(In [16992]) refs #8798 - global refs (half) in PlottedDnd?.js - autoScrollActive -> this.autoScrollActive ... A little concerned about the IOonselectstart fix there. It stores a ref to the orig handler and restores it on dnd-stop/cancel. Could just wrap it in cl oseure, but wondering if IE will leak here by doing so. !strict