From 7b7d55e11abf8642622a4ad40bbbabc4480e2cf0 Mon Sep 17 00:00:00 2001
From: Dan Poltawski <dan@moodle.com>
Date: Wed, 29 Jun 2016 14:51:07 +0100
Subject: [PATCH] MDL-54944 js: split variable intialisations in multiple lines

To increase code clarity, we don't allow multiple intilisations to be
confused with each other.

Detected by http://eslint.org/docs/rules/one-var-declaration-per-line
---
 admin/tool/lp/amd/src/competencyactions.js           | 3 ++-
 admin/tool/lp/amd/src/competencypicker.js            | 3 ++-
 admin/tool/lp/amd/src/competencypicker_user_plans.js | 3 ++-
 lib/amd/src/form-course-selector.js                  | 3 ++-
 lib/amd/src/localstorage.js                          | 3 ++-
 mod/assign/amd/src/grading_navigation.js             | 6 ++++--
 mod/assign/amd/src/participant_selector.js           | 3 ++-
 7 files changed, 16 insertions(+), 8 deletions(-)

diff --git a/admin/tool/lp/amd/src/competencyactions.js b/admin/tool/lp/amd/src/competencyactions.js
index c0943c088cb..522e8272f2a 100644
--- a/admin/tool/lp/amd/src/competencyactions.js
+++ b/admin/tool/lp/amd/src/competencyactions.js
@@ -248,7 +248,8 @@ define(['jquery',
         $.when.apply(null, requests).done(function(competencies, framework) {
 
             // Expand the list of competencies into a tree.
-            var i, competenciestree = [];
+            var i;
+            var competenciestree = [];
             for (i = 0; i < competencies.length; i++) {
                 var onecompetency = competencies[i];
                 if (onecompetency.parentid == "0") {
diff --git a/admin/tool/lp/amd/src/competencypicker.js b/admin/tool/lp/amd/src/competencypicker.js
index 28c4ce3a28f..2ba49acbddd 100644
--- a/admin/tool/lp/amd/src/competencypicker.js
+++ b/admin/tool/lp/amd/src/competencypicker.js
@@ -248,7 +248,8 @@ define(['jquery',
             }
 
             // Expand the list of competencies into a tree.
-            var i, tree = [], comp;
+            var i, comp;
+            var tree = [];
             for (i = 0; i < competencies.length; i++) {
                 comp = competencies[i];
                 if (comp.parentid == "0") { // Loose check for now, because WS returns a string.
diff --git a/admin/tool/lp/amd/src/competencypicker_user_plans.js b/admin/tool/lp/amd/src/competencypicker_user_plans.js
index 8046c2935c6..6565ed2ca41 100644
--- a/admin/tool/lp/amd/src/competencypicker_user_plans.js
+++ b/admin/tool/lp/amd/src/competencypicker_user_plans.js
@@ -100,7 +100,8 @@ define(['jquery',
         ])[0].done(function(competencies) {
 
             // Expand the list of competencies into a fake tree.
-            var i, tree = [], comp;
+            var i, comp;
+            var tree = [];
             for (i = 0; i < competencies.length; i++) {
                 comp = competencies[i].competency;
                 if (comp.shortname.toLowerCase().indexOf(searchText.toLowerCase()) < 0) {
diff --git a/lib/amd/src/form-course-selector.js b/lib/amd/src/form-course-selector.js
index f34fb3db6de..0914918b715 100644
--- a/lib/amd/src/form-course-selector.js
+++ b/lib/amd/src/form-course-selector.js
@@ -29,7 +29,8 @@ define(['core/ajax', 'jquery'], function(ajax, $) {
         // Public variables and functions.
         processResults: function(selector, data) {
             // Mangle the results into an array of objects.
-            var results = [], i = 0;
+            var results = [];
+            var i = 0;
             var excludelist = String($(selector).data('exclude')).split(',');
 
             for (i = 0; i < data.courses.length; i++) {
diff --git a/lib/amd/src/localstorage.js b/lib/amd/src/localstorage.js
index 27da8e4f0bd..adb2c807d08 100644
--- a/lib/amd/src/localstorage.js
+++ b/lib/amd/src/localstorage.js
@@ -106,7 +106,8 @@ define(['core/config'], function(config) {
         // From http://stackoverflow.com/questions/7616461/generate-a-hash-from-string-in-javascript-jquery.
         /* jshint bitwise: false */
         /* eslint no-bitwise: "off" */
-        var hash = 0, i, chr, len;
+        var hash = 0;
+        var i, chr, len;
         if (source.length === 0) {
             return hash;
         }
diff --git a/mod/assign/amd/src/grading_navigation.js b/mod/assign/amd/src/grading_navigation.js
index b2844353360..97087e0c4ad 100644
--- a/mod/assign/amd/src/grading_navigation.js
+++ b/mod/assign/amd/src/grading_navigation.js
@@ -332,7 +332,8 @@ define(['jquery', 'core/notification', 'core/str', 'core/form-autocomplete',
         e.preventDefault();
         var select = this._region.find('[data-action=change-user]');
         var currentUserId = select.attr('data-selected');
-        var i = 0, currentIndex = 0;
+        var i = 0;
+        var currentIndex = 0;
 
         for (i = 0; i < this._filteredUsers.length; i++) {
             if (this._filteredUsers[i].id == currentUserId) {
@@ -361,7 +362,8 @@ define(['jquery', 'core/notification', 'core/str', 'core/form-autocomplete',
         e.preventDefault();
         var select = this._region.find('[data-action=change-user]');
         var currentUserId = select.attr('data-selected');
-        var i = 0, currentIndex = 0;
+        var i = 0;
+        var currentIndex = 0;
 
         for (i = 0; i < this._filteredUsers.length; i++) {
             if (this._filteredUsers[i].id == currentUserId) {
diff --git a/mod/assign/amd/src/participant_selector.js b/mod/assign/amd/src/participant_selector.js
index d8094b79b63..71312a68acc 100644
--- a/mod/assign/amd/src/participant_selector.js
+++ b/mod/assign/amd/src/participant_selector.js
@@ -35,7 +35,8 @@ define(['core/ajax', 'jquery', 'core/templates'], function(ajax, $, templates) {
          * @return {Array}
          */
         processResults: function(selector, data) {
-            var results = [], i = 0;
+            var results = [];
+            var i = 0;
             for (i = 0; i < data.length; i++) {
                 results[i] = {value: data[i].id, label: data[i].label};
             }