kkpowar / gmaps-utility-library-dev

Automatically exported from code.google.com/p/gmaps-utility-library-dev
0 stars 0 forks source link

dragzoom.js uses global variables unnecessarily #26

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
If your calling program uses global variables "center", "rect",
"polyBounds" or "zoomLevel", then using dragzoom.js changes the content of
those variables.

If your page contains HTML elements that use those names as "id" values,
then MSIE will crash in a manner that's rather difficult to debug.

http://groups.google.com/group/Google-Maps-API/browse_thread/thread/1c9627af52ae
126b/0e331d032aa88d74?lnk=raot#0e331d032aa88d74

All this could be avoided if dragzoom.js user "var center", etc.

Original issue reported on code.google.com by goo...@econym.demon.co.uk on 30 Mar 2008 at 7:20

GoogleCodeExporter commented 8 years ago
Patch attached against r583

Original comment by robert.c...@gmail.com on 2 Apr 2008 at 8:10

Attachments:

GoogleCodeExporter commented 8 years ago
Fixed in r588.

Original comment by lem...@gmail.com on 16 Apr 2008 at 3:36

GoogleCodeExporter commented 8 years ago

Original comment by lem...@gmail.com on 26 May 2008 at 4:40