Fix weird BECOME_ADMIN code.

[plomrogue2] / plomrogue / commands.py

diff --git a/plomrogue/commands.py b/plomrogue/commands.py
index 026f99603de4b90562c1d4f1d639481a3b3bddaf..ca5c37c60ab635056cfd0bbdd11b662cc87232b7 100644 (file)
--- a/plomrogue/commands.py
+++ b/plomrogue/commands.py
@@ -1,5 +1,6 @@
 from plomrogue.misc import quote
 from plomrogue.errors import GameError, ArgError
+from plomrogue.misc import Terrain
 
 
 
@@ -16,11 +17,17 @@ def cmd_THING_TYPES(game, connection_id):
 cmd_THING_TYPES.argtypes = ''
 
 def cmd_TERRAINS(game, connection_id):
-    for t in game.terrains.keys():
-        game.io.send('TERRAIN %s %s' % (quote(t), quote(game.terrains[t])),
-                     connection_id)
+    for t in game.terrains.values():
+        game.io.send('TERRAIN %s %s' % (quote(t.character),
+                                        quote(t.description)), connection_id)
 cmd_TERRAINS.argtypes = ''
 
+def cmd_TERRAIN(game, character, description,
+                blocks_light, blocks_sound, blocks_movement):
+    game.terrains[character] = Terrain(character, description, blocks_light,
+                                       blocks_sound, blocks_movement)
+cmd_TERRAIN.argtypes = 'char string bool bool bool'
+
 def cmd_ALL(game, msg, connection_id):
     speaker = game.get_player(connection_id)
     if not speaker:
@@ -61,7 +68,7 @@ def cmd_LOGIN(game, nick, connection_id):
 cmd_LOGIN.argtypes = 'string'
 
 def cmd_BECOME_ADMIN(game, password, connection_id):
-    player = game.thing_types['Player'](game)
+    player = game.get_player(connection_id)
     if not player:
         raise GameError('need to be logged in for this')
     if password in game.admin_passwords:
@@ -85,7 +92,7 @@ def cmd_SET_TILE_CONTROL(game, yx, control_char, connection_id):
     map_control = game.get_map(big_yx, 'control')
     map_control[little_yx] = control_char
     game.changed = True
-    game.record_fov_change((big_yx, little_yx))
+    game.record_change((big_yx, little_yx), 'fov')
 cmd_SET_TILE_CONTROL.argtypes = 'yx_tuple:nonneg char'
 
 def cmd_THING_PROTECTION(game, thing_id, protection_char, connection_id):
@@ -99,8 +106,7 @@ def cmd_THING_PROTECTION(game, thing_id, protection_char, connection_id):
         raise GameError('thing of ID %s not found' % thing_id)
     t.protection = protection_char
     game.changed = True
-    # FIXME: pseudo-FOV-change actually
-    game.record_fov_change(t.position)
+    game.record_change(t.position, 'other')
 cmd_THING_PROTECTION.argtypes = 'int:pos char'
 
 def cmd_SET_MAP_CONTROL_PASSWORD(game, tile_class, password, connection_id):
@@ -128,8 +134,7 @@ def cmd_NICK(game, nick, connection_id):
     t.name = nick
     game.io.send('CHAT ' + quote(old_nick + ' renamed themselves to ' + nick))
     game.changed = True
-    # FIXME: pseudo-FOV-change actually
-    game.record_fov_change(t.position)
+    game.record_change(t.position, 'other')
 cmd_NICK.argtypes = 'string'
 
 def cmd_GET_GAMESTATE(game, connection_id):
@@ -176,8 +181,7 @@ def cmd_ANNOTATE(game, yx, msg, pw, connection_id):
         if big_yx not in game.annotations:
             game.annotations[big_yx] = {}
         game.annotations[big_yx][little_yx] = msg
-    # FIXME: pseudo-FOV-change actually
-    game.record_fov_change([big_yx, little_yx])
+    game.record_change([big_yx, little_yx], 'other')
     game.changed = True
 cmd_ANNOTATE.argtypes = 'yx_tuple:nonneg string string'
 
@@ -196,8 +200,7 @@ def cmd_PORTAL(game, yx, msg, pw, connection_id):
         if big_yx not in game.portals:
             game.portals[big_yx] = {}
         game.portals[big_yx][little_yx] = msg
-    # FIXME: pseudo-FOV-change actually
-    game.record_fov_change([big_yx, little_yx])
+    game.record_change([big_yx, little_yx], 'other')
     game.changed = True
 cmd_PORTAL.argtypes = 'yx_tuple:nonneg string string'
 
@@ -226,6 +229,7 @@ def cmd_MAP(game, geometry, size):
     if geometry == 'Hex':
         map_geometry_class = MapGeometryHex
     game.new_world(map_geometry_class(size))
+    game.terrains = {}
 cmd_MAP.argtypes = 'string:map_geometry yx_tuple:pos'
 
 def cmd_MAP_CONTROL_LINE(game, big_yx, y, line):
@@ -260,8 +264,7 @@ def cmd_THING_NAME(game, thing_id, name, pw, connection_id):
     else:
         t.name = name
     game.changed = True
-    # FIXME: pseudo-FOV-change actually
-    game.record_fov_change(t.position)
+    game.record_change(t.position, 'other')
 cmd_THING_NAME.argtypes = 'int:pos string string'
 
 def cmd_GOD_THING_NAME(game, thing_id, name):
@@ -284,9 +287,7 @@ def cmd_THING_DOOR_CLOSED(game, thing_id):
         raise GameError('thing of ID %s not found' % thing_id)
     if not t.type_ == 'Door':
         raise GameError('thing of ID %s not door' % thing_id)
-    t.blocking = True
-    t.portable = False
-    t.thing_char = '#'
+    t.close()
 cmd_THING_DOOR_CLOSED.argtypes = 'int:pos'
 
 def cmd_THING_MUSICPLAYER_SETTINGS(game, thing_id, playing, index, repeat):
@@ -335,10 +336,28 @@ def cmd_PLAYER_FACE(game, face, connection_id):
         raise GameError('wrong face string length')
     game.faces[t.name] = face
     game.changed = True
-    # FIXME: pseudo-FOV-change actually
-    game.record_fov_change(t.position)
+    game.record_change(t.position, 'other')
 cmd_PLAYER_FACE.argtypes = 'string'
 
+def cmd_PLAYER_HAT(game, hat, connection_id):
+    t = game.get_player(connection_id)
+    if not t:
+        raise GameError('can only edit hat when already logged in')
+    if not t.name in game.hats:
+        raise GameError('not currently wearing an editable hat')
+    if len(hat) != 18:
+        raise GameError('wrong hat string length')
+    legal_chars = t.get_cookie_chars()
+    for c in hat:
+        if c not in legal_chars:
+            raise GameError('used illegal character: "%s" – '
+                            'allowed characters: %s'
+                            % (c, legal_chars))
+    game.hats[t.name] = hat
+    game.changed = True
+    game.record_change(t.position, 'other')
+cmd_PLAYER_HAT.argtypes = 'string'
+
 def cmd_GOD_PLAYER_FACE(game, name, face):
     if len(face) != 18:
         raise GameError('wrong face string length')
@@ -351,6 +370,10 @@ def cmd_GOD_PLAYER_HAT(game, name, hat):
     game.hats[name] = hat
 cmd_GOD_PLAYER_HAT.argtypes = 'string string'
 
+def cmd_GOD_PLAYERS_HAT_CHARS(game, name, hat_chars):
+    game.players_hat_chars[name] = hat_chars
+cmd_GOD_PLAYERS_HAT_CHARS.argtypes = 'string string'
+
 def cmd_THING_HAT_DESIGN(game, thing_id, design):
     if len(design) != 18:
         raise GameError('hat design of wrong length')