X-Git-Url: https://mudpy.org/gitweb?a=blobdiff_plain;f=mudpy%2Fmisc.py;h=81b43eff633fded863b1515b88178bfa15804960;hb=1a321872691a7d8bbac0a8048d361134f5136f88;hp=3968039fff7d240a8ca2b05e5aa91bd69984b4bc;hpb=2c533eb48672110a04d6c4850d3e103d6294ee8d;p=mudpy.git

diff --git a/mudpy/misc.py b/mudpy/misc.py
index 3968039..81b43ef 100644
--- a/mudpy/misc.py
+++ b/mudpy/misc.py
@@ -186,34 +186,33 @@ class Element:
 
     def is_restricted(self):
         """Boolean check whether command is administrative or debugging."""
-        return(
-            self.get("administrative", False) or self.get("debugging", False))
+        return bool(self.get("administrative") or self.get("debugging"))
 
     def is_admin(self):
         """Boolean check whether an actor is controlled by an admin owner."""
-        return(self.owner and self.owner.is_admin())
+        return self.owner and self.owner.is_admin()
 
     def can_run(self, command):
         """Check if the user can run this command object."""
 
         # has to be in the commands group
         if command not in self.universe.groups["command"].values():
-            return(False)
+            return False
 
         # debugging commands are not allowed outside debug mode
         if command.get("debugging") and not self.universe.debug_mode():
-            return(False)
+            return False
 
         # avatars of administrators can run any command
         if self.is_admin():
-            return(True)
+            return True
 
         # everyone can run non-administrative commands
         if not command.is_restricted():
-            return(True)
+            return True
 
         # otherwise the command cannot be run by this actor
-        return(False)
+        return False
 
     def update_location(self):
         """Make sure the location's contents contain this element."""
@@ -578,7 +577,7 @@ class User:
         self.remove()
 
         # get rid of the old user object
-        del(self)
+        del self
 
         # create a new user object
         new_user = User()
@@ -642,7 +641,7 @@ class User:
 
                 # take this one out of the list and delete
                 self.remove()
-                del(self)
+                del self
                 return_value = True
                 break
 
@@ -1014,7 +1013,7 @@ class User:
 
     def is_admin(self):
         """Boolean check whether user's account is an admin."""
-        return(self.account.get("administrator", False))
+        return self.account.get("administrator", False)
 
 
 def broadcast(message, add_prompt=True):
@@ -1086,14 +1085,14 @@ def log(message, level=0):
     for line in lines:
         while 0 < len(universe.loglines) >= max_log_lines:
             del universe.loglines[0]
-        universe.loglines.append((level, timestamp + " " + line))
+        universe.loglines.append((timestamp + " " + line, level))
 
 
 def get_loglines(level, start, stop):
     """Return a specific range of loglines filtered by level."""
 
     # filter the log lines
-    loglines = [x for x in universe.loglines if x[0] >= level]
+    loglines = [x for x in universe.loglines if x[1] >= level]
 
     # we need these in several places
     total_count = str(len(universe.loglines))
@@ -1115,11 +1114,10 @@ def get_loglines(level, start, stop):
             stop = 1
 
         # some preamble
-        message = "There are " + str(total_count)
-        message += " log lines in memory and " + str(filtered_count)
-        message += " at or above level " + str(level) + "."
-        message += " The matching lines from " + str(stop) + " to "
-        message += str(start) + " are:$(eol)$(eol)"
+        message = (
+            "There are %s log lines in memory and %s at or above level %s. "
+            "The matching lines from %s to %s are:$(eol)$(eol)" %
+            (total_count, filtered_count, level, stop, start))
 
         # add the text from the selected lines
         if stop > 1:
@@ -1127,14 +1125,13 @@ def get_loglines(level, start, stop):
         else:
             range_lines = loglines[-start:]
         for line in range_lines:
-            message += "   (" + str(line[0]) + ") " + line[1].replace(
-                "$(", "$_("
-            ) + "$(eol)"
+            message += "   (%s) %s$(eol)" % (
+                line[1], line[0].replace("$(", "$_("))
 
     # there were no lines
     else:
-        message = "None of the " + str(total_count)
-        message += " lines in memory matches your request."
+        message = "None of the %s lines in memory matches your request." % (
+            total_count)
 
     # pass it back
     return message
@@ -2121,6 +2118,8 @@ def setup():
     log("Running version: %s" % universe.versions.version, 1)
     log("Initial directory: %s" % universe.startdir, 1)
     log("Command line: %s" % " ".join(sys.argv), 1)
+    if universe.debug_mode():
+        log("WARNING: Unsafe debugging mode is enabled!", 6)
 
     # pass the initialized universe back
     return universe