mirror_php/moodle
1
0
Fork 0
mirror of https://github.com/moodle/moodle.git synced 2025-04-22 08:55:15 +02:00
Code Issues Projects Releases Wiki Activity

MDL-54944 js: split variable intialisations in multiple lines

Browse Source 
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
This commit is contained in:
Dan Poltawski 2016-06-29 14:51:07 +01:00
parent 2d2b6c9d22
commit 7b7d55e11a
7 changed files with 16 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
admin/tool/lp/amd/src/competencyactions.js
View File

@ -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") {

3
admin/tool/lp/amd/src/competencypicker.js
View File

@ -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.

3
admin/tool/lp/amd/src/competencypicker_user_plans.js
View File

@ -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) {

3
lib/amd/src/form-course-selector.js
View File

@ -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++) {

3
lib/amd/src/localstorage.js
View File

@ -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;
}

6
mod/assign/amd/src/grading_navigation.js
View File

@ -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) {

3
mod/assign/amd/src/participant_selector.js
View File

@ -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};
}