From b5276f7f83f8e6d57434964f64b4dfa53694562d Mon Sep 17 00:00:00 2001 From: Lucas Lois Date: Sun, 2 Oct 2016 16:48:27 -0300 Subject: [PATCH 1/9] Made translate.js jshint compliant --- lib/translate.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/translate.js b/lib/translate.js index 44839a724..8c1395b5f 100644 --- a/lib/translate.js +++ b/lib/translate.js @@ -63,7 +63,7 @@ return target; } - if (target in translation == false) + if (target in translation === false) { if (dynoTrans != null) { @@ -79,7 +79,7 @@ } return result; - }; + } window._ = translate; From 90aa1ca4fdb1859a00ef65d327ea0ddf391ef32a Mon Sep 17 00:00:00 2001 From: Lucas Lois Date: Sun, 2 Oct 2016 17:00:57 -0300 Subject: [PATCH 2/9] Made Button.js jshint compliant --- script/Button.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/script/Button.js b/script/Button.js index df1f0e9fb..83aac0257 100644 --- a/script/Button.js +++ b/script/Button.js @@ -109,8 +109,8 @@ var Button = { } }, - clearCooldown: function(btn, ended) { - var ended = ended || false; + clearCooldown: function(btn, cooldownEnded) { + var ended = cooldownEnded || false; if(!ended){ $('div.cooldown', btn).stop(true, true); } From 4ced5337daf901b3951e097b2725752b5bb4143b Mon Sep 17 00:00:00 2001 From: Lucas Lois Date: Sun, 2 Oct 2016 17:22:32 -0300 Subject: [PATCH 3/9] Made engine.js jshint compliant. Error on /script/engine.js at 775:128 was ignored --- script/engine.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/script/engine.js b/script/engine.js index 44c836ab0..d1c7381f6 100644 --- a/script/engine.js +++ b/script/engine.js @@ -427,7 +427,7 @@ } } } - }) + }); }, share: function() { From 8370eac8aaac16a8bbe0849e7fcbd10d6d1aec0f Mon Sep 17 00:00:00 2001 From: Lucas Lois Date: Sun, 2 Oct 2016 17:40:54 -0300 Subject: [PATCH 4/9] Made events.js jshint compliant --- script/events.js | 26 +++++++++++++++----------- 1 file changed, 15 insertions(+), 11 deletions(-) diff --git a/script/events.js b/script/events.js index fd2d898bf..9e111aac3 100644 --- a/script/events.js +++ b/script/events.js @@ -153,7 +153,7 @@ var Events = { string = 'resume.'; log = 'paused'; } - btn.children('.text').first().text( _(string) ) + btn.children('.text').first().text( _(string) ); Events.paused = (state == 'auto') ? 'auto' : true; event.addClass('paused'); Button.clearCooldown(btn); @@ -189,6 +189,7 @@ var Events = { break; case 'auto': Button.cooldown(btn); + /* falls through */ default: log = 'resumed'; time = Events._PAUSE_COOLDOWN * 1000; @@ -373,8 +374,8 @@ var Events = { out = true; } } - for(var k in mod) { - Path.outfit[k] += mod[k]; + for(var m in mod) { + Path.outfit[m] += mod[m]; } if(out) { Button.setDisabled(btn, true); @@ -596,11 +597,12 @@ var Events = { if(weight > freeSpace) { // Draw the drop menu Engine.log('drop menu'); + var dropMenu; if($('#dropMenu').length){ - var dropMenu = $('#dropMenu'); + dropMenu = $('#dropMenu'); $('#dropMenu').empty(); } else { - var dropMenu = $('
').attr({'id': 'dropMenu', 'data-legend': _('drop:')}); + dropMenu = $('
').attr({'id': 'dropMenu', 'data-legend': _('drop:')}); needsAppend = true; } for(var k in Path.outfit) { @@ -676,9 +678,10 @@ var Events = { } } lootButtons.appendTo(desc); + var takeET = null; if(lootButtons.children().length > 0) { var takeETrow = $('
').addClass('takeETrow'); - var takeET = new Button.Button({ + takeET = new Button.Button({ id: 'loot_takeEverything', text: '', cooldown: Events._LEAVE_COOLDOWN, @@ -773,10 +776,10 @@ var Events = { World.updateSupplies(); }, - getLoot: function(btn, skipButtonSet) { + getLoot: function(btn, stateSkipButtonSet) { var name = btn.attr('id').substring(5).replace('-', ' '); if(btn.data('numLeft') > 0) { - var skipButtonSet = skipButtonSet || false; + var skipButtonSet = stateSkipButtonSet || false; var weight = Path.getWeight(name); var freeSpace = Path.getFreeSpace(); if(weight <= freeSpace) { @@ -857,8 +860,9 @@ var Events = { } // Draw any loot + var takeETbtn; if(scene.loot) { - var takeETbtn = Events.drawLoot(scene.loot); + takeETbtn = Events.drawLoot(scene.loot); } // Draw the buttons @@ -1113,7 +1117,7 @@ var Events = { if(typeof target[i] == 'function'){ target[i](); } else { - $SM.remove(stateName) + $SM.remove(stateName); } } } @@ -1127,7 +1131,7 @@ var Events = { if(delay){ $SM.set(state, delay); } else { - var delay = $SM.get(state, true) + delay = $SM.get(state, true); } var time = Engine.setInterval(function(){ // update state every half second From a903a1c9fe15a6ad82881e9656b0aa04958eef8b Mon Sep 17 00:00:00 2001 From: Lucas Lois Date: Sun, 2 Oct 2016 17:45:09 -0300 Subject: [PATCH 5/9] Made outside.js jshint compliant --- script/localization.js | 2 +- script/outside.js | 10 +++++----- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/script/localization.js b/script/localization.js index dd1aa9de4..5e9c3d4cd 100644 --- a/script/localization.js +++ b/script/localization.js @@ -65,5 +65,5 @@ _('the compass points southwest') ]; - delete keywords; + keywords = null; })(); diff --git a/script/outside.js b/script/outside.js index 4fdfa0e69..881bc43ed 100644 --- a/script/outside.js +++ b/script/outside.js @@ -619,14 +619,14 @@ var Outside = { } /// TRANSLATORS : Mind the whitespace at the end. var s = _('the traps contain '); - for(var i = 0, len = msg.length; i < len; i++) { - if(len > 1 && i > 0 && i < len - 1) { + for(var l = 0, len = msg.length; l < len; l++) { + if(len > 1 && l > 0 && l < len - 1) { s += ", "; - } else if(len > 1 && i == len - 1) { + } else if(len > 1 && l == len - 1) { /// TRANSLATORS : Mind the whitespaces at the beginning and end. s += _(" and "); } - s += msg[i]; + s += msg[l]; } var baitUsed = numBait < numTraps ? numBait : numTraps; @@ -643,7 +643,7 @@ var Outside = { Outside.updateVillage(); Outside.updateWorkersView(); Outside.updateVillageIncome(); - }; + } }, scrollSidebar: function(direction, reset) { From 05edb2a98a7bc582284e19656cd00f272da29cf1 Mon Sep 17 00:00:00 2001 From: Lucas Lois Date: Sun, 2 Oct 2016 17:52:22 -0300 Subject: [PATCH 6/9] Made path.js jshint compliant --- script/path.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/script/path.js b/script/path.js index d5dc04259..e4ad87c03 100644 --- a/script/path.js +++ b/script/path.js @@ -339,7 +339,7 @@ var Path = { var momentum = 10; if( direction == 'up' ) - momentum = momentum * -1 + momentum = momentum * -1; if( direction == 'down' && inView( direction, $('#perks') ) ){ From c1ce1a89d97198c17fcd109285a9a37c0119e49d Mon Sep 17 00:00:00 2001 From: Lucas Lois Date: Sun, 2 Oct 2016 17:52:39 -0300 Subject: [PATCH 7/9] Made room.js jshint compliant --- script/room.js | 36 ++++++++++++++++++------------------ 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/script/room.js b/script/room.js index d277ca24d..fa5aa910d 100644 --- a/script/room.js +++ b/script/room.js @@ -1092,9 +1092,9 @@ var Room = { var costTooltip = $('.tooltip', craftable.button); costTooltip.empty(); var cost = craftable.cost(); - for(var k in cost) { - $("
").addClass('row_key').text(_(k)).appendTo(costTooltip); - $("
").addClass('row_val').text(cost[k]).appendTo(costTooltip); + for(var c in cost) { + $("
").addClass('row_key').text(_(c)).appendTo(costTooltip); + $("
").addClass('row_val').text(cost[c]).appendTo(costTooltip); } if(max && !craftable.button.hasClass('disabled')) { Notifications.notify(Room, craftable.maxMsg); @@ -1107,33 +1107,33 @@ var Room = { } } - for(var k in Room.TradeGoods) { - good = Room.TradeGoods[k]; - var max = $SM.num(k, good) + 1 > good.maximum; + for(var g in Room.TradeGoods) { + good = Room.TradeGoods[g]; + var goodsMax = $SM.num(g, good) + 1 > good.maximum; if(good.button == null) { - if(Room.buyUnlocked(k)) { + if(Room.buyUnlocked(g)) { good.button = new Button.Button({ - id: 'build_' + k, + id: 'build_' + g, cost: good.cost(), - text: _(k), + text: _(g), click: Room.buy, width: '80px' - }).css('opacity', 0).attr('buildThing', k).appendTo(buySection).animate({opacity:1}, 300, 'linear'); + }).css('opacity', 0).attr('buildThing', g).appendTo(buySection).animate({opacity:1}, 300, 'linear'); } } else { // refresh the tooltip - var costTooltip = $('.tooltip', good.button); - costTooltip.empty(); - var cost = good.cost(); - for(var k in cost) { - $("
").addClass('row_key').text(_(k)).appendTo(costTooltip); - $("
").addClass('row_val').text(cost[k]).appendTo(costTooltip); + var goodsCostTooltip = $('.tooltip', good.button); + goodsCostTooltip.empty(); + var goodCost = good.cost(); + for(var gc in goodCost) { + $("
").addClass('row_key').text(_(gc)).appendTo(goodsCostTooltip); + $("
").addClass('row_val').text(goodCost[gc]).appendTo(goodsCostTooltip); } - if(max && !good.button.hasClass('disabled')) { + if(goodsMax && !good.button.hasClass('disabled')) { Notifications.notify(Room, good.maxMsg); } } - if(max) { + if(goodsMax) { Button.setDisabled(good.button, true); } else { Button.setDisabled(good.button, false); From 36b01a78b628e5c0de49958c626546c84fa1b451 Mon Sep 17 00:00:00 2001 From: Lucas Lois Date: Sun, 2 Oct 2016 17:57:17 -0300 Subject: [PATCH 8/9] Made scoring.js, space.js and world.js jshint compliant --- script/scoring.js | 2 +- script/space.js | 4 ++-- script/world.js | 12 ++++++------ 3 files changed, 9 insertions(+), 9 deletions(-) diff --git a/script/scoring.js b/script/scoring.js index e9f957de5..1e50ba913 100644 --- a/script/scoring.js +++ b/script/scoring.js @@ -13,7 +13,7 @@ var Score = { var fullScore = 0; var factor = [1, 1.5, 1, 2, 2, 3, 3, 2, 2, 2, 2, 1.5, 1, - 1, 10, 30, 50, 100, 150, 150, 3, 3, 5, 4] + 1, 10, 30, 50, 100, 150, 150, 3, 3, 5, 4]; for(var i = 0; i< factor.length; i++){ fullScore += scoreUnadded[i] * factor[i]; } diff --git a/script/space.js b/script/space.js index c12ef0dea..d5b72cdae 100644 --- a/script/space.js +++ b/script/space.js @@ -372,8 +372,8 @@ var Space = { clearTimeout(Events._eventTimeout); clearTimeout(Room._fireTimer); clearTimeout(Room._tempTimer); - for(var k in Room.Craftables) { - Room.Craftables[k].button = null; + for(var j in Room.Craftables) { + Room.Craftables[j].button = null; } for(var k in Room.TradeGoods) { Room.TradeGoods[k].button = null; diff --git a/script/world.js b/script/world.js index d40e6293a..7c592784a 100644 --- a/script/world.js +++ b/script/world.js @@ -685,7 +685,7 @@ var World = { // Place landmarks for(var k in World.LANDMARKS) { var landmark = World.LANDMARKS[k]; - for(var i = 0; i < landmark.num; i++) { + for(var l = 0; l < landmark.num; l++) { var pos = World.placeLandmark(landmark.minRadius, landmark.maxRadius, k, map); } } @@ -713,7 +713,7 @@ var World = { targets[index] = { x : i - World.RADIUS, y : j - World.RADIUS, - } + }; index++; if(index === max){ // optimisation: stop the research if maximum number of items has been reached @@ -796,8 +796,8 @@ var World = { } var list = []; - for(var t in chances) { - list.push(chances[t] + '' + t); + for(var j in chances) { + list.push(chances[j] + '' + j); } list.sort(function(a, b) { var n1 = parseFloat(a.substring(0, a.length - 1)); @@ -807,8 +807,8 @@ var World = { var c = 0; var r = Math.random(); - for(var i in list) { - var prob = list[i]; + for(var l in list) { + var prob = list[l]; c += parseFloat(prob.substring(0,prob.length - 1)); if(r < c) { return prob.charAt(prob.length - 1); From 69267045a8133898ccd5db35a0f722c564a660d5 Mon Sep 17 00:00:00 2001 From: Lucas Lois Date: Sun, 2 Oct 2016 18:07:02 -0300 Subject: [PATCH 9/9] Made events/room.js and state_manager.js jshinthint compliant --- script/events/room.js | 16 ++++++++-------- script/state_manager.js | 10 +++++----- 2 files changed, 13 insertions(+), 13 deletions(-) diff --git a/script/events/room.js b/script/events/room.js index 6710b25c6..64f49372a 100644 --- a/script/events/room.js +++ b/script/events/room.js @@ -292,8 +292,8 @@ Events.Room = [ text: [ _('the wanderer leaves, cart loaded with wood') ], - action: function(delay) { - var delay = delay || false; + action: function(inputDelay) { + var delay = inputDelay || false; Events.saveDelay(function() { $SM.add('stores.wood', 300); Notifications.notify(Room, _('the mysterious wanderer returns, cart piled high with wood.')); @@ -315,8 +315,8 @@ Events.Room = [ text: [ _('the wanderer leaves, cart loaded with wood') ], - action: function(delay) { - var delay = delay || false; + action: function(inputDelay) { + var delay = inputDelay || false; Events.saveDelay(function() { $SM.add('stores.wood', 1500); Notifications.notify(Room, _('the mysterious wanderer returns, cart piled high with wood.')); @@ -371,8 +371,8 @@ Events.Room = [ text: [ _('the wanderer leaves, cart loaded with furs') ], - action: function(delay) { - var delay = delay || false; + action: function(inputDelay) { + var delay = inputDelay || false; Events.saveDelay(function() { $SM.add('stores.fur', 300); Notifications.notify(Room, _('the mysterious wanderer returns, cart piled high with furs.')); @@ -394,8 +394,8 @@ Events.Room = [ text: [ _('the wanderer leaves, cart loaded with furs') ], - action: function(delay) { - var delay = delay || false; + action: function(inputDelay) { + var delay = inputDelay || false; Events.saveDelay(function() { $SM.add('stores.fur', 1500); Notifications.notify(Room, _('the mysterious wanderer returns, cart piled high with furs.')); diff --git a/script/state_manager.js b/script/state_manager.js index 6d31aacb7..94624ae94 100644 --- a/script/state_manager.js +++ b/script/state_manager.js @@ -54,10 +54,10 @@ var StateManager = { createState: function(stateName, value) { var words = stateName.split(/[.\[\]'"]+/); //for some reason there are sometimes empty strings - for (var i = 0; i < words.length; i++) { - if (words[i] === '') { - words.splice(i, 1); - i--; + for (var j = 0; j < words.length; j++) { + if (words[j] === '') { + words.splice(j, 1); + j--; } } var obj = State; @@ -218,7 +218,7 @@ var StateManager = { fireUpdate: function(stateName, save){ var category = $SM.getCategory(stateName); - if(stateName == undefined) stateName = category = 'all'; //best if this doesn't happen as it will trigger more stuff + if(stateName === undefined) stateName = category = 'all'; //best if this doesn't happen as it will trigger more stuff $.Dispatch('stateUpdate').publish({'category': category, 'stateName':stateName}); if(save) Engine.saveGame(); },