home · contact · privacy
Don't use LOG for meta messages, errors.
authorChristian Heller <c.heller@plomlompom.de>
Sun, 25 Oct 2020 02:19:17 +0000 (03:19 +0100)
committerChristian Heller <c.heller@plomlompom.de>
Sun, 25 Oct 2020 02:19:17 +0000 (03:19 +0100)
new2/plomrogue/commands.py
new2/rogue_chat.html

index a68efc230a8791aa8ee4294bd07dea2ccbb42788..79a4678b1b8133b969e589729d90498e235a2dc8 100644 (file)
@@ -1,29 +1,28 @@
 from plomrogue.misc import quote
+from plomrogue.errors import GameError
 
 
 
 def cmd_ALL(game, msg, connection_id):
     if not connection_id in game.sessions:
-        game.io.send('LOG' + quote('need to be logged in for this'), connection_id)
-        return
+        raise GameError('need to be logged in for this')
     t = game.get_thing(game.sessions[connection_id], False)
     game.io.send('LOG ' + quote(t.nickname + ': ' + msg))
 cmd_ALL.argtypes = 'string'
 
 def cmd_LOGIN(game, nick, connection_id):
     for t in [t for t in game.things if t.type_ == 'player' and t.nickname == nick]:
-        game.io.send('LOG ' + quote('name already in use'), connection_id)
-        return
+        raise GameError('name already in use')
     t = game.thing_types['player'](game)
     t.nickname = nick
     game.things += [t]  # TODO refactor into Thing.__init__?
     game.sessions[connection_id] = t.id_ 
-    game.io.send('LOG ' + quote('your are now: ' + nick), connection_id)
+    game.io.send('META ' + quote('you are now: ' + nick), connection_id)
 cmd_LOGIN.argtypes = 'string'
 
 def cmd_QUERY(game, target_nick, msg, connection_id):
     if not connection_id in game.sessions:
-        game.io.send('LOG ' + quote('can only query when logged in'), connection_id)
+        raise GameError('can only query when logged in')
     t = game.get_thing(game.sessions[connection_id], False)
     source_nick = t.nickname
     for t in [t for t in game.things if t.type_ == 'player' and t.nickname == target_nick]:
@@ -32,6 +31,6 @@ def cmd_QUERY(game, target_nick, msg, connection_id):
                 game.io.send('LOG ' + quote(source_nick+ '->' + target_nick + ': ' + msg), c_id)
                 game.io.send('LOG ' + quote(source_nick+ '->' + target_nick + ': ' + msg), connection_id)
                 return
-        game.io.send('LOG ' + quote('target user offline?'))
-    game.io.send('LOG ' + quote('can only query with registered nicknames'))
+        raise GameError('target user offline')
+    raise GameError('can only query with registered nicknames')
 cmd_QUERY.argtypes = 'string string'
index 10e5ee2f5fe0c024a871643fb475aea41ae96a12..bc90c07e72d09ac8b6dc5a51f0c7b26911bbce59 100644 (file)
@@ -217,6 +217,8 @@ websocket.onmessage = function (event) {
     tui.draw_map();
   } else if (tokens[0] === 'LOG') {
      tui.log_msg(tokens[1], 1);
+  } else if (tokens[0] === 'META') {
+     tui.log_msg(tokens[1]);
   } else if (tokens[0] === 'UNHANDLED_INPUT') {
      tui.log_msg('unknown command');
   } else if (tokens[0] === 'GAME_ERROR') {