Recent commits have trimmed down this object until it has a single
property, so we can drop the object and store the value of that
property directly.
This makes a bunch of code much easier to read by replacing all
occurences of "game.game" with just "game". The old code was really
confusing because it was so unclear what the top-level game container
even was.
const game = new engine.Game(id);
const game = new engine.Game(id);
- this.games[id] = {
- game: game
- };
response.sendStatus(404);
return;
}
response.sendStatus(404);
return;
}
- response.redirect(301, `/${game.game.meta.identifier}/${game.game.id}/`);
+ response.redirect(301, `/${game.meta.identifier}/${game.id}/`);
});
/* LMNO middleware to lookup the game. */
});
/* LMNO middleware to lookup the game. */
}
/* Stash the game onto the request to be used by the game-specific code. */
}
/* Stash the game onto the request to be used by the game-specific code. */
- request.game = game.game;
let idle = [];
for (let id in lmno.games) {
let idle = [];
for (let id in lmno.games) {
- if (lmno.games[id].game.clients.length)
+ if (lmno.games[id].clients.length)
active.push(lmno.games[id]);
else
idle.push(lmno.games[id]);
active.push(lmno.games[id]);
else
idle.push(lmno.games[id]);
<ul>
{% for game in games.active %}
<li>
<ul>
{% for game in games.active %}
<li>
- {{ game.game.id }} ({{ game.game.clients.length }}/{{ game.game._players.length }} active)
- {% for player in game.game._players %}
+ {{ game.id }} ({{ game.clients.length }}/{{ game._players.length }} active)
+ {% for player in game._players %}
{{ player.name }}
{% endfor %}
</li>
{{ player.name }}
{% endfor %}
</li>
<ul>
{% for game in games.idle %}
<li>
<ul>
{% for game in games.idle %}
<li>
- {{ game.game.id }} ({{ game.game._players.length }})
- {% for player in game.game._players %}
+ {{ game.id }} ({{ game._players.length }})
+ {% for player in game._players %}
{{ player.name }}
{% endfor %}
</li>
{{ player.name }}
{% endfor %}
</li>