polluting global window namespace less by defining all grid vars in o…

…ne global var window['wiceGrids']
leikind · Feb 4, 2014 · 7593b98 · 7593b98
1 parent bdb1f0f
commit 7593b98
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/vendor/assets/javascripts/wice_grid_init.js.coffee b/vendor/assets/javascripts/wice_grid_init.js.coffee
@@ -1,5 +1,7 @@
 $(document).on 'page:load ready', -> initWiceGrid()
 
+globalVarForAllGrids = 'wiceGrids'
+
 initWiceGrid = ->
 
   $(".wice-grid-container").each (index, wiceGridContainer) ->
@@ -26,7 +28,10 @@ initWiceGrid = ->
           templates   : filterDeclaration.declaration.templates
           ids         : filterDeclaration.declaration.ids
 
-    window[gridName] = gridProcessor
+    unless window[globalVarForAllGrids]
+      window[globalVarForAllGrids] = {}
+
+    window[globalVarForAllGrids][gridName] = gridProcessor
 
     setupDatepicker()
     setupSubmitReset wiceGridContainer, gridProcessor
@@ -241,8 +246,8 @@ setupBulkToggleForActionColumn = (wiceGridContainer) ->
 
 getGridProcessorForElement = (element) ->
   gridName = $(element).data('grid-name')
-  if gridName
-    window[gridName]
+  if gridName && window[globalVarForAllGrids]
+    window[globalVarForAllGrids][gridName]
   else
     null