aboutsummaryrefslogtreecommitdiff
path: root/webAO/client.js
diff options
context:
space:
mode:
authorsD <stoned@derpymail.org>2020-04-13 18:20:56 +0200
committersD <stoned@derpymail.org>2020-04-13 18:20:56 +0200
commit97caca41dfa78158fb64a0229f173d5d6f45a409 (patch)
treec14ad486aae2a4b05ccf26af0aba42ca8772d186 /webAO/client.js
parente2c97a3ac234370c083bb3117c5ad25f4bef4033 (diff)
fix off-by-one in ARUP code
Diffstat (limited to 'webAO/client.js')
-rw-r--r--webAO/client.js10
1 files changed, 5 insertions, 5 deletions
diff --git a/webAO/client.js b/webAO/client.js
index 9390634..9be4529 100644
--- a/webAO/client.js
+++ b/webAO/client.js
@@ -1029,23 +1029,23 @@ class Client extends EventEmitter {
*/
handleARUP(args) {
args = args.slice(1);
- for (let i = 1; i < args.length - 1; i++) {
+ for (let i = 0; i < args.length - 2; i++) {
if (this.areas[i]) { // the server sends us ARUP before we even get the area list
const thisarea = document.getElementById("area" + i);
switch (Number(args[0])) {
case 0: // playercount
- this.areas[i].players = Number(args[i]);
+ this.areas[i].players = Number(args[i+1]);
thisarea.innerText = `${this.areas[i].name} (${this.areas[i].players})`;
break;
case 1: // status
- this.areas[i].status = safe_tags(args[i]);
+ this.areas[i].status = safe_tags(args[i+1]);
thisarea.classList = "area-button area-" + this.areas[i].status.toLowerCase();
break;
case 2:
- this.areas[i].cm = safe_tags(args[i]);
+ this.areas[i].cm = safe_tags(args[i+1]);
break;
case 3:
- this.areas[i].locked = safe_tags(args[i]);
+ this.areas[i].locked = safe_tags(args[i+1]);
break;
}