Drop the "name" entry from the game-engine exports

[empires-server] / lmno.js

diff --git a/lmno.js b/lmno.js
index d818cd12fb16de02af766b02b995d5eac3f43886..69e445a24e3ab1fde24bdc4df4a0e50c059ff993 100644 (file)
--- a/lmno.js
+++ b/lmno.js
@@ -54,7 +54,7 @@ const engines = {
 
 class LMNO {
   constructor() {
-    this.ids = {};
+    this.games = {};
   }
 
   generate_id() {
@@ -64,15 +64,14 @@ class LMNO {
   create_game(engine_name) {
     do {
       var id = this.generate_id();
-    } while (id in this.ids);
+    } while (id in this.games);
 
     const engine = engines[engine_name];
 
     const game = new engine.Game();
 
-    this.ids[id] = {
+    this.games[id] = {
       id: id,
-      engine: engine.name,
       game: game
     };
 
@@ -132,12 +131,12 @@ app.get('/[a-zA-Z0-9]{4}', (request, response) => {
     return;
   }
 
-  const game = lmno.ids[game_id];
+  const game = lmno.games[game_id];
   if (game === undefined) {
       response.sendStatus(404);
       return;
   }
-  response.redirect(301, `/${game.engine}/${game.id}/`);
+  response.redirect(301, `/${game.game.meta.identifier}/${game.id}/`);
 });
 
 /* LMNO middleware to lookup the game. */
@@ -155,13 +154,12 @@ app.use('/:engine([^/]+)/:game_id([a-zA-Z0-9]{4})', (request, response, next) =>
     const old_path = new RegExp(`/${engine}/${game_id}/?`);
     const new_path = `/${engine}/${canon_id}/`;
     const new_url = request.originalUrl.replace(old_path, new_path);
-    console.log("Redirecting from " + request.originalUrl + " to " + new_url);
     response.redirect(301, new_url);
     return;
   }
 
   /* See if there is any game with this ID. */
-  const game = lmno.ids[game_id];
+  const game = lmno.games[game_id];
   if (game === undefined) {
     response.sendStatus(404);
     return;
@@ -239,11 +237,11 @@ app.get('/admin/', auth_admin, (request, response) => {
   let active = [];
   let idle = [];
 
-  for (let id in lmno.ids) {
-    if (lmno.ids[id].game.clients.length)
-      active.push(lmno.ids[id]);
+  for (let id in lmno.games) {
+    if (lmno.games[id].game.clients.length)
+      active.push(lmno.games[id]);
     else
-      idle.push(lmno.ids[id]);
+      idle.push(lmno.games[id]);
   }
   response.render('admin.html', { test: "foobar", games: { active: active, idle: idle}});
 });
@@ -252,7 +250,19 @@ app.get('/admin/', auth_admin, (request, response) => {
 /* Mount sub apps. only _after_ we have done all the middleware we need. */
 for (let key in engines) {
   const engine = engines[key];
-  app.use(`/${engine.name}/[a-zA-Z0-9]{4}/`, engine.app);
+
+  /* Add routes that are common to all games. */
+  engine.router.get('/', (request, response) => {
+    const game = request.game;
+
+    if (! request.session.nickname)
+      response.render('choose-nickname.html', { game_name: game.meta.name });
+    else
+      response.render(`${game.meta.identifier}-game.html`);
+  });
+
+  /* And mount the whole router at the path for the game. */
+  app.use(`/${engine.Game.meta.identifier}/[a-zA-Z0-9]{4}/`, engine.router);
 }
 
 app.listen(4000, function () {