Merge pull request octobercms#2461 from gpasztor87/develop

JS fixes and typo

modules/system/assets/ui/js/chart.bar.js

@@ -116,7 +116,7 @@
             if (!data)
                 $this.data('oc.barChart', new BarChart(this, options))
         })
-      }
+    }
 
     $.fn.barChart.Constructor = BarChart
 

modules/system/assets/ui/js/chart.line.js

@@ -87,7 +87,7 @@
 
         this.chartOptions = $.extend({}, this.chartOptions, parsedOptions)
 
-        this.options       = options,
+        this.options       = options
         this.$el           = $(element)
         this.fullDataSet   = []
         this.resetZoomLink = $(options.resetZoomLink)

modules/system/assets/ui/js/chart.meter.js

@@ -54,7 +54,7 @@
             else
                 data.update(option)
         })
-      }
+    }
 
     $.fn.goalMeter.Constructor = GoalMeter
 

modules/system/assets/ui/js/chart.pie.js

@@ -11,7 +11,7 @@
  *
  * Dependences:
  * - Raphaël (raphael-min.js)
- * - October chart utilities (october.chartutils.js)
+ * - October chart utilities (chart.utils.js)
  */
 +function ($) { "use strict";
 
@@ -119,7 +119,7 @@
             if (!data) 
                 $this.data('oc.pieChart', new PieChart(this, options))
         })
-      }
+    }
 
     $.fn.pieChart.Constructor = PieChart
 

modules/system/assets/ui/js/chart.utils.js

@@ -3,6 +3,7 @@
  */
 
 +function ($) { "use strict";
+
     var ChartUtils = function() {}
 
     ChartUtils.prototype.defaultValueColor = '#b8b8b8';

modules/system/assets/ui/js/checkbox.balloon.js

@@ -41,7 +41,7 @@
 
             if (!data) $this.data('oc.balloon-selector', (data = new BalloonSelector(this, options)))
         })
-      }
+    }
 
     $.fn.balloonSelector.Constructor = BalloonSelector
 

modules/system/assets/ui/js/checkbox.js

@@ -3,14 +3,14 @@
  *
  */
 
-(function($){
+(function($) {
 
-    $(document).on('keydown', 'div.custom-checkbox', function(e){
+    $(document).on('keydown', 'div.custom-checkbox', function(e) {
         if (e.keyCode == 32)
             e.preventDefault()
     })
 
-    $(document).on('keyup', 'div.custom-checkbox', function(e){
+    $(document).on('keyup', 'div.custom-checkbox', function(e) {
         if (e.keyCode == 32) {
             var $cb = $('input', this)
 

modules/system/assets/ui/js/datepicker.js

@@ -12,6 +12,7 @@
  */
 
 +function ($) { "use strict";
+
     var Base = $.oc.foundation.base,
         BaseProto = Base.prototype
 

modules/system/assets/ui/js/drag.scroll.js

@@ -419,7 +419,7 @@
                 data[option].apply(data, methodArgs)
             }
         })
-      }
+    }
 
     $.fn.dragScroll.Constructor = DragScroll
 

modules/system/assets/ui/js/drag.sort.js

@@ -8,6 +8,7 @@
  */
 
  +function ($) { "use strict";
+
     var Base = $.oc.foundation.base,
         BaseProto = Base.prototype
 
@@ -204,7 +205,7 @@
             if (!data) $this.data('oc.sortable', (data = new Sortable(this, options)))
             if (typeof option == 'string') data[option].apply(data, args)
         })
-      }
+    }
 
     $.fn.sortable.Constructor = Sortable
 

modules/system/assets/ui/js/drag.value.js

@@ -131,7 +131,7 @@
         // IE
         if (document.selection) {
             el.focus()
-            sel = document.selection.createRange()
+            var sel = document.selection.createRange()
             sel.text = insertValue
             el.focus()
         }

modules/system/assets/ui/js/filter.js

@@ -20,7 +20,7 @@
 
     var FilterWidget = function (element, options) {
 
-        var $el = this.$el = $(element);
+        this.$el = $(element);
 
         this.options = options || {}
         this.scopeValues = {}
@@ -427,7 +427,7 @@
         })
 
         return result ? result : this
-      }
+    }
 
     $.fn.filterWidget.Constructor = FilterWidget
 

modules/system/assets/ui/js/flashmessage.js

@@ -27,7 +27,7 @@
 
         $(document.body).append($element)
 
-        setTimeout(function(){
+        setTimeout(function() {
             $element.addClass('in')
         }, 1)
 

modules/system/assets/ui/js/foundation.baseclass.js

@@ -51,8 +51,7 @@
         this.proxiedMethods = {}
     }
 
-    Base.prototype.dispose = function()
-    {
+    Base.prototype.dispose = function() {
         for (var key in this.proxiedMethods) {
             this.proxiedMethods[key] = null
         }

modules/system/assets/ui/js/foundation.controlutils.js

@@ -40,7 +40,7 @@
     $.oc.foundation.controlUtils = ControlUtils;
 
     $(document).on('ajaxBeforeReplace', function(ev){
-        // Automatically displose controls in an element
+        // Automatically dispose controls in an element
         // before the element contents is replaced.
         // The ajaxBeforeReplace event is triggered in 
         // framework.js

modules/system/assets/ui/js/input.hotkey.js

@@ -195,7 +195,7 @@
             if (!data) $this.data('oc.hotkey', (data = new HotKey(this, options)))
             if (typeof option == 'string') data[option].apply(data, args)
         })
-      }
+    }
 
     $.fn.hotKey.Constructor = HotKey
 
@@ -210,7 +210,7 @@
     // HOTKEY DATA-API
     // ==============
 
-    $(document).render(function(){
+    $(document).render(function() {
         $('[data-hotkey]').hotKey()
     })
 

modules/system/assets/ui/js/input.monitor.js

@@ -9,7 +9,7 @@
         BaseProto = Base.prototype
 
     var ChangeMonitor = function (element, options) {
-        var $el = this.$el = $(element);
+        this.$el = $(element);
 
         this.paused = false
         this.options = options || {}
@@ -135,7 +135,7 @@
 
             if (!data) $this.data('oc.changeMonitor', (data = new ChangeMonitor(this, options)))
         })
-      }
+    }
 
     $.fn.changeMonitor.Constructor = ChangeMonitor
 

modules/system/assets/ui/js/input.preset.js

@@ -305,7 +305,8 @@
     // INPUT CONVERTER DATA-API
     // ===============
 
-    $(document).render(function(){
+    $(document).render(function() {
         $('[data-input-preset]').inputPreset()
     })
+
 }(window.jQuery);

modules/system/assets/ui/js/input.trigger.js

@@ -157,7 +157,7 @@
 
             if (!data) $this.data('oc.triggerOn', (data = new TriggerOn(this, options)))
         })
-      }
+    }
 
     $.fn.triggerOn.Constructor = TriggerOn
 

modules/system/assets/ui/js/list.sortable.js

@@ -41,6 +41,7 @@
  */
 
 +function ($) { "use strict";
+
     var Base = $.oc.foundation.base,
         BaseProto = Base.prototype,
         listSortableIdCounter = 0,
@@ -94,6 +95,8 @@
     }
 
     ListSortable.prototype.unregisterListHandlers = function(list) {
+        var $list = $(list)
+
         $list.off('dragstart', '> li', this.proxy(this.onDragStart))
         $list.off('dragover', '> li', this.proxy(this.onDragOver))
         $list.off('dragenter', '> li', this.proxy(this.onDragEnter))
@@ -138,7 +141,7 @@
     ListSortable.prototype.elementBelongsToManagedList = function(element) {
         for (var i=this.lists.length-1; i >= 0; i--) {
             var list = this.lists[i],
-                children = [].slice.call(list.children) // Converts HTMLCollection to array
+                children = [].slice.call(list.children); // Converts HTMLCollection to array
 
             if (children.indexOf(element) !== -1) {
                 return true
@@ -442,7 +445,7 @@
                 }
             }
         })
-      }
+    }
 
     $.fn.listSortable.Constructor = ListSortable
 

modules/system/assets/ui/js/loader.base.js

@@ -19,7 +19,7 @@
 
     var LoadIndicator = function (element, options) {
 
-        var $el = this.$el = $(element)
+        this.$el = $(element)
 
         this.options = options || {}
         this.tally = 0
@@ -93,7 +93,7 @@
                 }
             }
         })
-      }
+    }
 
     $.fn.loadIndicator.Constructor = LoadIndicator
 

modules/system/assets/ui/js/loader.cursor.js

@@ -39,9 +39,7 @@
         if (this.counter > 1)
             return
 
-        var self = this,
-            $window = $(window);
-
+        var self = this;
 
         if (event !== undefined && event.clientY !== undefined) {
             self.indicator.css({
@@ -73,7 +71,7 @@
         }
     }
 
-    $(document).ready(function(){
+    $(document).ready(function() {
         $.oc.cursorLoadIndicator = new CursorLoadIndicator();
     })
 

modules/system/assets/ui/js/popover.js

@@ -386,7 +386,7 @@
                 data[option].apply(data, methodArgs)
             }
         })
-      }
+    }
 
     $.fn.ocPopover.Constructor = Popover