X-Git-Url: https://mudpy.org/gitweb?p=mudpy.git;a=blobdiff_plain;f=mudpy%2Fmisc.py;h=c0f73a1faf6af70d44b7b5620de8acda50811c28;hp=044614fb2aaec370376c2189514bd6498cbe9714;hb=3b59b8a74388427be4e06d0dad558d597ee34ef4;hpb=6841e0ca1d916a31178ceab9bb7802f7a055bd52 diff --git a/mudpy/misc.py b/mudpy/misc.py index 044614f..c0f73a1 100644 --- a/mudpy/misc.py +++ b/mudpy/misc.py @@ -184,27 +184,35 @@ class Element: prepend_padding ) + def is_restricted(self): + """Boolean check whether command is administrative or debugging.""" + 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() + 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(): - result = False + return False + + # debugging commands are not allowed outside debug mode + if command.get("debugging") and not self.universe.debug_mode(): + return False # avatars of administrators can run any command - elif self.owner and self.owner.account.get("administrator"): - result = True + if self.is_admin(): + return True # everyone can run non-administrative commands - elif not command.get("administrative"): - result = True + if not command.is_restricted(): + return True # otherwise the command cannot be run by this actor - else: - result = False - - # pass back the result - return result + return False def update_location(self): """Make sure the location's contents contain this element.""" @@ -477,6 +485,10 @@ class Universe: if fallback not in self.files: mudpy.data.Data(fallback, self, flags=flags) + def debug_mode(self): + """Boolean method to indicate whether unsafe debugging is enabled.""" + return self.groups["mudpy"]["limit"].get("debug", False) + class User: @@ -565,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() @@ -629,7 +641,7 @@ class User: # take this one out of the list and delete self.remove() - del(self) + del self return_value = True break @@ -999,6 +1011,10 @@ class User: avatar, 6) return avatars + def is_admin(self): + """Boolean check whether user's account is an admin.""" + return self.account.get("administrator", False) + def broadcast(message, add_prompt=True): """Send a message to all connected users.""" @@ -1053,9 +1069,10 @@ def log(message, level=0): # display to connected administrators for user in universe.userlist: - if user.state == "active" and user.account.get( - "administrator" - ) and user.account.get("loglevel", 0) <= level: + if ( + user.state == "active" + and user.is_admin() + and user.account.get("loglevel", 0) <= level): # iterate over every line in the message full_message = "" for line in lines: @@ -1501,7 +1518,7 @@ def find_command(command_name): else: for candidate in sorted(universe.groups["command"]): if candidate.startswith(command_name) and not universe.groups[ - "command"][candidate].get("administrative"): + "command"][candidate].is_restricted(): # the command matches the start of a command word and is not # restricted to administrators command = universe.groups["command"][candidate] @@ -2103,6 +2120,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