X-Git-Url: https://git.cworth.org/git?a=blobdiff_plain;f=lmno.js;h=b85041ec74d13be48abcbca9a5f7e3e8b900ba4e;hb=76232144e294f384cde30933687de514fd90584a;hp=b6c5b78e04e6b83a6700c19d623a6e55d625f1c8;hpb=00507589b4984d50a34a383975fc6323db19bcfa;p=empires-server diff --git a/lmno.js b/lmno.js index b6c5b78..b85041e 100644 --- a/lmno.js +++ b/lmno.js @@ -69,6 +69,8 @@ nunjucks.configure("templates", { * * / Serves -game.html template * + * /player Allows client to set name or team + * * /events Serves a stream of events. Game can override * the handle_events method, call super() first, * and then have code to add custom events. @@ -289,6 +291,12 @@ for (let key in engines) { response.render(`${game.meta.identifier}-game.html`); }); + router.put('/player', (request, response) => { + const game = request.game; + + game.handle_player(request, response); + }); + router.get('/events', (request, response) => { const game = request.game; @@ -297,12 +305,27 @@ for (let key in engines) { /* Further, add some routes conditionally depending on whether the * engine provides specific, necessary methods for the routes. */ - if (engine.prototype.add_move) { + + /* Note: We have to use hasOwnProperty here since the base Game + * class has a geeric add_move function, and we don't want that to + * have any influence on our decision. Only if the child has + * overridden that do we want to create a "/move" route. */ + if (engine.prototype.hasOwnProperty("add_move")) { router.post('/move', (request, response) => { const game = request.game; const move = request.body.move; + const player = game.players_by_session[request.session.id]; + + /* Reject move if there is no player for this session. */ + if (! player) { + response.json({legal: false, message: "No valid player from session"}); + return; + } + + const result = game.add_move(player, move); - const result = game.add_move(move); + /* Take care of any generic post-move work. */ + game.post_move(player, result); /* Feed move response back to the client. */ response.json(result);