Merge pull request #6881 from epixa/uiletconst1

[internal] Replace var with const/let in the root of ui/public

src/ui/public/bound_to_config_obj.js

@@ -16,15 +16,15 @@ export default function BoundToConfigObjProvider($rootScope, config) {
    * @return {Object}
    */
   function BoundToConfigObj(input) {
-    var self = this;
+    const self = this;
 
     _.forOwn(input, function (val, prop) {
       if (!_.isString(val) || val.charAt(0) !== '=') {
         self[prop] = val;
         return;
       }
 
-      var configKey = val.substr(1);
+      const configKey = val.substr(1);
 
       update();
       $rootScope.$on('init:config', update);

src/ui/public/compile_recursive_directive.js

@@ -25,7 +25,7 @@ uiModules
       }
 
       // Break the recursion loop by removing the contents
-      var contents = element.contents().remove();
+      const contents = element.contents().remove();
       let compiledContents;
       return {
         pre: (link && link.pre) ? link.pre : null,

src/ui/public/elastic_textarea.js

@@ -1,7 +1,7 @@
 import _ from 'lodash';
 import uiModules from 'ui/modules';
-var NL_RE = /\n/g;
-var events = 'keydown keypress keyup change';
+const NL_RE = /\n/g;
+const events = 'keydown keypress keyup change';
 
 uiModules.get('kibana')
 .directive('elasticTextarea', function () {

src/ui/public/errors.js

@@ -1,12 +1,12 @@
 import _ from 'lodash';
 import angular from 'angular';
 
-var canStack = (function () {
-  var err = new Error();
+const canStack = (function () {
+  const err = new Error();
   return !!err.stack;
 }());
 
-var errors = {};
+const errors = {};
 
 // abstract error class
 function KbnError(msg, constructor) {
@@ -119,7 +119,7 @@ _.class(errors.MappingConflict).inherits(KbnError);
  * @param {String} field - the fields which contains the conflict
  */
 errors.RestrictedMapping = function RestrictedMapping(field, index) {
-  var msg = field + ' is a restricted field name';
+  let msg = field + ' is a restricted field name';
   if (index) msg += ', found it while attempting to fetch mapping for index pattern: ' + index;
 
   KbnError.call(this, msg, errors.RestrictedMapping);
@@ -165,7 +165,7 @@ _.class(errors.DuplicateField).inherits(KbnError);
 errors.SavedObjectNotFound = function SavedObjectNotFound(type, id) {
   this.savedObjectType = type;
   this.savedObjectId = id;
-  var idMsg = id ? ' (id: ' + id + ')' : '';
+  const idMsg = id ? ' (id: ' + id + ')' : '';
   KbnError.call(this,
     'Could not locate that ' + type + idMsg,
     errors.SavedObjectNotFound);

src/ui/public/events.js

@@ -3,7 +3,7 @@ import Notifier from 'ui/notify/notifier';
 import SimpleEmitter from 'ui/utils/simple_emitter';
 
 export default function EventsProvider(Private, Promise) {
-  var notify = new Notifier({ location: 'EventEmitter' });
+  const notify = new Notifier({ location: 'EventEmitter' });
 
   _.class(Events).inherits(SimpleEmitter);
   function Events() {
@@ -23,7 +23,7 @@ export default function EventsProvider(Private, Promise) {
       this._listeners[name] = [];
     }
 
-    var listener = {
+    const listener = {
       handler: handler
     };
     this._listeners[name].push(listener);
@@ -75,8 +75,8 @@ export default function EventsProvider(Private, Promise) {
    * @returns {Promise}
    */
   Events.prototype.emit = function (name) {
-    var self = this;
-    var args = _.rest(arguments);
+    const self = this;
+    const args = _.rest(arguments);
 
     if (!self._listeners[name]) {
       return self._emitChain;

src/ui/public/fixed_scroll.js

@@ -2,23 +2,23 @@ import $ from 'jquery';
 import _ from 'lodash';
 import uiModules from 'ui/modules';
 
-var SCROLLER_HEIGHT = 20;
+const SCROLLER_HEIGHT = 20;
 
 uiModules
 .get('kibana')
 .directive('fixedScroll', function ($timeout) {
   return {
     restrict: 'A',
     link: function ($scope, $el) {
-      var $window = $(window);
-      var $scroller = $('<div class="fixed-scroll-scroller">').height(SCROLLER_HEIGHT);
+      let $window = $(window);
+      let $scroller = $('<div class="fixed-scroll-scroller">').height(SCROLLER_HEIGHT);
 
 
       /**
        * Remove the listeners bound in listen()
        * @type {function}
        */
-      var unlisten = _.noop;
+      let unlisten = _.noop;
 
       /**
        * Listen for scroll events on the $scroller and the $el, sets unlisten()
@@ -74,15 +74,15 @@ uiModules
       function setup() {
         cleanUp();
 
-        var containerWidth = $el.width();
-        var contentWidth = $el.prop('scrollWidth');
-        var containerHorizOverflow = contentWidth - containerWidth;
+        const containerWidth = $el.width();
+        const contentWidth = $el.prop('scrollWidth');
+        const containerHorizOverflow = contentWidth - containerWidth;
 
-        var elTop = $el.offset().top - $window.scrollTop();
-        var elBottom = elTop + $el.height();
-        var windowVertOverflow = elBottom - $window.height();
+        const elTop = $el.offset().top - $window.scrollTop();
+        const elBottom = elTop + $el.height();
+        const windowVertOverflow = elBottom - $window.height();
 
-        var requireScroller = containerHorizOverflow > 0 && windowVertOverflow > 0;
+        const requireScroller = containerHorizOverflow > 0 && windowVertOverflow > 0;
         if (!requireScroller) return;
 
         // push the content away from the scroller

src/ui/public/modules.js

@@ -47,8 +47,8 @@ import _ from 'lodash';
  * "kibana" module's injector.
  *
  */
-var existingModules = {};
-var links = [];
+const existingModules = {};
+const links = [];
 
 /**
  * Take an angular module and extends the dependencies for that module to include all of the modules
@@ -79,7 +79,7 @@ function link(module) {
  * @return {AngularModule}
  */
 function get(moduleName, requires) {
-  var module = existingModules[moduleName];
+  let module = existingModules[moduleName];
 
   if (module === void 0) {
     // create the module
@@ -102,13 +102,13 @@ function get(moduleName, requires) {
 }
 
 function close(moduleName) {
-  var module = existingModules[moduleName];
+  const module = existingModules[moduleName];
 
   // already closed
   if (!module) return;
 
   // if the module is currently linked, unlink it
-  var i = links.indexOf(module);
+  const i = links.indexOf(module);
   if (i > -1) links.splice(i, 1);
 
   // remove from linked modules list of required modules