From 65c57618b95b9111b07f8fbc4c38c99d942a272a Mon Sep 17 00:00:00 2001
From: Christian Heller <c.heller@plomlompom.de>
Date: Fri, 26 Feb 2016 13:39:55 +0100
Subject: [PATCH] Server, plugin: Abort command_worldactive checks as early as
 possible.

---
 plugins/server/PleaseTheIslandGod.py | 45 +++++++++++++++-------------
 server/commands.py                   | 15 ++++++----
 2 files changed, 33 insertions(+), 27 deletions(-)

diff --git a/plugins/server/PleaseTheIslandGod.py b/plugins/server/PleaseTheIslandGod.py
index f4f5e6e..a4238e5 100644
--- a/plugins/server/PleaseTheIslandGod.py
+++ b/plugins/server/PleaseTheIslandGod.py
@@ -580,41 +580,44 @@ def command_worldactive(worldactive_string):
             else:
                 print("World already active.")
         elif 0 == world_db["WORLD_ACTIVE"]:
-            wait_exists = False
             for ThingAction in world_db["ThingActions"]:
                 if "wait" == world_db["ThingActions"][ThingAction]["TA_NAME"]:
-                    wait_exists = True
                     break
-            player_exists = False
+            else:
+                print("Ignored: No wait action defined for world to activate.")
+                return
             for Thing in world_db["Things"]:
                 if 0 == Thing:
-                    player_exists = True
                     break
-            altar_found = False
+            else:
+                print("Ignored: No player defined for world to activate.")
+                return
             if world_db["MAP"]:
                 pos = world_db["MAP"].find(b'_')
                 if pos > 0:
                     y = int(pos / world_db["MAP_LENGTH"])
                     x = pos % world_db["MAP_LENGTH"]
                     world_db["altar"] = (y, x)
-                    altar_found = True
-            specials_set = True
+                else:
+                    print("Ignored: No altar defined for world to activate.")
+                    return
+            else:
+                print("Ignored: No map defined for world to activate.")
+                return
             for name in world_db["specials"]:
                 if world_db[name] not in world_db["ThingTypes"]:
-                    specials_set = False
-            if altar_found and wait_exists and player_exists and \
-                    world_db["MAP"] and specials_set:
-                for id in world_db["Things"]:
-                    if world_db["Things"][id]["T_LIFEPOINTS"]:
-                        build_fov_map(world_db["Things"][id])
-                        if 0 == id:
-                            update_map_memory(world_db["Things"][id], False)
-                if not world_db["Things"][0]["T_LIFEPOINTS"]:
-                    empty_fovmap = bytearray(b" " * world_db["MAP_LENGTH"] ** 2)
-                    world_db["Things"][0]["fovmap"] = empty_fovmap
-                world_db["WORLD_ACTIVE"] = 1
-            else:
-                print("Ignoring: Not all conditions for world activation met.")
+                    print("Ignored: Not all specials set for world to "
+                          "activate.")
+                    return
+            for id in world_db["Things"]:
+                if world_db["Things"][id]["T_LIFEPOINTS"]:
+                    build_fov_map(world_db["Things"][id])
+                    if 0 == id:
+                        update_map_memory(world_db["Things"][id], False)
+            if not world_db["Things"][0]["T_LIFEPOINTS"]:
+                empty_fovmap = bytearray(b" " * world_db["MAP_LENGTH"] ** 2)
+                world_db["Things"][0]["fovmap"] = empty_fovmap
+            world_db["WORLD_ACTIVE"] = 1
 
 def play_move(str_arg):
     if action_exists("move"):
diff --git a/server/commands.py b/server/commands.py
index d002ab3..1ce6464 100644
--- a/server/commands.py
+++ b/server/commands.py
@@ -118,17 +118,19 @@ def command_worldactive(worldactive_string):
             else:
                 print("World already active.")
         elif 0 == world_db["WORLD_ACTIVE"]:
-            wait_exists = False
             for ThingAction in world_db["ThingActions"]:
                 if "wait" == world_db["ThingActions"][ThingAction]["TA_NAME"]:
-                    wait_exists = True
                     break
-            player_exists = False
+            else:
+                print("Ignored: No wait action defined for world to activate.")
+                return
             for Thing in world_db["Things"]:
                 if 0 == Thing:
-                    player_exists = True
                     break
-            if wait_exists and player_exists and world_db["MAP"]:
+            else:
+                print("Ignored: No player defined for world to activate.")
+                return
+            if world_db["MAP"]:
                 for id in world_db["Things"]:
                     if world_db["Things"][id]["T_LIFEPOINTS"]:
                         build_fov_map(world_db["Things"][id])
@@ -139,7 +141,8 @@ def command_worldactive(worldactive_string):
                     world_db["Things"][0]["fovmap"] = empty_fovmap
                 world_db["WORLD_ACTIVE"] = 1
             else:
-                print("Ignoring: Not all conditions for world activation met.")
+                print("Ignoring: No map defined for world to activate.")
+                return
 
 
 def command_tid(id_string):
-- 
2.30.2