Removed unnessecary debug outputs

Fixed naming error in rank trainingmod

blockplacemods.py

@@ -11,6 +11,17 @@ import org.bukkit.event.block.Action as Action
 import org.bukkit.block.BlockFace as BlockFace
 import org.bukkit.scheduler.BukkitRunnable as Runnable
 
+"""
+  # Permissions:
+  # - utils.toggle: for use of the command
+  # - utils.toggle.cauldron
+  # - utils.toggle.slab
+  # - utils.toggle.furnace
+  # - utils.toggle.piston
+  # - utils.toggle.dropper
+  # - utils.toggle.hopper
+"""
+
 settingInformation = dict( #[setting type, identifying description, detailed description, aliases, (optional) max slot id], setting types: 0 = toggle, default on. 1 = Set your setting to held itemstack, 2 = toggle, default off
     cauldron = [0,
         "easy cauldron water level control",

chatalias.py

@@ -1,138 +1,319 @@
-# Chat Aliasing plugin by Curs3d #
+############################################
-##################################
+# Alias v2.0 by Pepich                     #
-# Allows users to alias words,
+# Changes to previous version from curs3d: #
-# so that when they send a
+# Dynamic alias limit from permissions     #
-# message in chat, it gets
+# AC/CG/MSG support                        #
-# replaced by their specified
+# Color support                            #
-# word. The JSON file for this
+# Bugfixes                                 #
-# plugin is generated if not
+#                                          #
-# present. Set values to -1
+#                  TODO:                   #
-# for "unlimited" setting.
+# Add command support...                   #
+############################################
 
+import os
+import org.bukkit as bukkit
+from org.bukkit import *
 from helpers import *
-import re
 from traceback import format_exc as trace
+from secrets import *
 
-data = None
+# Version number and requirements
 
-max_entries = 10
+alias_version = "2.0.0"
-max_alias_length = 35
+helpers_versions = ["1.1.0", "2.0.0"]
-# Minecraft message limit is 100 so I decided to give a little tolerance (and I added a bit more)
+enabled = False
-max_overall_length = 100 + max_alias_length
+error = colorify("&cUnspecified error")
+commands_per_page = 5
+global_aliases = {"./":"/"}
+data = {}
+use_mysql = False
 
-alias_perm = "utils.alias.allowed"
+# Permissions:
-exceed_length = "utils.alias.exceedlimit"
+
-exceed_entries = "utils.alias.exceedlimit"
+# Grants full access immediately
-exceed_overall_length = "utils.alias.exceedlimit"
+permission_ALL = "utils.alias.*"
+# Access to the command to display the help screen
+permission_BASE = "utils.alias"
+# Make replacements only when the user has this permission
+permission_USE = "utils.alias.use"
+# Modify aliases
+permission_MODIFY = "utils.alias.modify"
+permission_MODIFY_OTHERS = "utils.alias.modify.others"
+# List aliases
+permission_LIST = "utils.alias.list"
+permission_LIST_OTHERS = "utils.alias.list.others"
+# Set alias amounts/length limits, e.g. utils.alias.amount.420
+permission_AMOUNT = "utils.alias.amount."
+permission_LENGTH = "utils.alias.length."
+# See when the plugin was disabled due to version errors
+permission_INFO = "utils.alias.info"
+permission_FINFO = "utils.alias.finfo"
+
+########
+# CODE #
+########
+
+enabled = helpers_version in helpers_versions
+if not enabled:
+    error = colorify("&6Incompatible versions detected (&chelpers.py&6)")
 
 
-def safe_open_json():
+def safe_open_json(uuid):
-    global data
+    if not os.path.exists("plugins/redstoner-utils.py.dir/files/aliases"):
-    if data is not None:
+        os.makedirs("plugins/redstoner-utils.py.dir/files/aliases")
-        return data
+    value = open_json_file("aliases/" + uuid)
-    data = open_json_file("aliases")
+    if value is None:
-    if data is None:
+        value = global_aliases
-        data = {}
+    save_json_file("aliases/" + uuid, value)
-    save_json_file("aliases", data)
+    return value
-    return data
-
-def multiple_replace(aliases, text):
-    regex = re.compile("|".join(map(re.escape, aliases.keys())))
-    return regex.sub(lambda mo: aliases[mo.group(0)], text)
 
 
 @hook.command("alias",
-              usage="/<command> [to_alias] [alias...]",
+              usage="/<command> <add, remove, list, help> [...]",
-              desc="Aliases words in chat")
+              desc="Allows aliasing of words")
 def on_alias_command(sender, cmd, label, args):
-
+    try:
-    if not is_player(sender):
+        args = array_to_list(args)
-        msg(sender, "Sorry, non-players cannot alias words")
+        if not enabled:
+            disabled_fallback(sender)
             return True
-    if not sender.hasPermission(alias_perm):
+        if not hasPerm(sender, permission_BASE):
-        plugin_header(recipient=sender, name="Chat Alias")
+            plugin_header(recipient=sender, name="Alias")
             noperm(sender)
             return True
-    if len(args) == 0:
+        return subcommands[args[0].lower()](sender, args[1:])
-        plugin_header(recipient=sender, name="Chat Alias")
+    except:
-        msg(sender, "&7This is a plugin that allows you to get words" +
+        return subcommands["help"](sender, "1")
-            "replaced by other ones automatically!")
+
-        msg(sender, "&7\nCommands:")
+
-        msg(sender, "&e/alias <word> &7removes <word> from your aliases. " +
+def help(sender, args):
-            "Use &e/alias * &7to remove all aliases.")
+    commands = [colorify("&e/alias help [page]")]
-        msg(sender, "&e/alias <word> <replacement> &7will change &e<word> " +
+    if hasPerm(sender, permission_LIST):
-            "&7to &e<replacement> &7in chat")
+        commands += [colorify("&e/alias list &7- Lists all your aliases")]
-        msg(sender, "&7\nYour Aliases:")
+    if hasPerm(sender, permission_MODIFY):
-        data = safe_open_json()
+        commands += [colorify("&e/alias add <word> <alias> &7- Add an alias")]
+        commands += [colorify("&e/alias remove <word> &7- Remove an alias")]
+    if can_remote(sender):
+        while len(commands) < commands_per_page:
+            commands += [""]
+        commands += [colorify("&7Following commands will be executed on <player> yet all output will be redirected to you, except when you set silent to false, then <player> will see it too.")]
+    if hasPerm(sender, permission_LIST_OTHERS):
+        commands += [colorify("&e/alias player <name> list [silent]")]
+    if hasPerm(sender, permission_MODIFY_OTHERS):
+        commands += [colorify("&e/alias player <name> add <word> <alias> [silent]")]
+        commands += [colorify("&e/alias player <name> remove <word> [silent]")]
+    pages = (len(commands)-1)/commands_per_page + 1
+    page = 1
+    if len(args) != 0:
+        page = int(args[0])
+    if (page > pages):
+        page = pages
+    if page < 1:
+        page = 1
+    msg(sender, colorify("&e---- &6Help &e-- &6Page &c" + str(page) + "&6/&c" + str(pages) + " &e----"))
+    page -= 1
+    to_display = commands[5*page:5*page+5]
+    for message in to_display:
+        msg(sender, message)
+    if page+1 < pages:
+        msg(sender, colorify("&6To display the next page, type &c/help " + str(page+2)))
+    return True
+
+
+@hook.event("player.PlayerJoinEvent", "high")
+def on_join(event):
     try:
-            for alias, value in data[str(sender.getUniqueId())].items():
+        if enabled:
-                msg(sender, "&7%s &7==> %s" % (alias, value))
+            t = threading.Thread(target=load_data, args=(uid(event.getPlayer()), ))
-        except KeyError:
+            t.daemon = True
-            pass
+            t.start()
-        return True
-    elif len(args) == 1:
-        data = safe_open_json()
-        if args[0] == "*":
-            try:
-                del data[str(sender.getUniqueId())]
-            except KeyError:
-                plugin_header(recipient=sender, name="Chat Alias")
-                msg(sender, "&7No alias data to remove!")
-                return True
-            save_json_file("aliases", data)
-            plugin_header(recipient=sender, name="Chat Alias")
-            msg(sender, "&cALL &7alias data successfuly removed!")
-            return True
-        try:
-            if data[str(sender.getUniqueId())].pop(args[0], None) is None:
-                plugin_header(recipient=sender, name="Chat Alias")
-                msg(sender, "&7Could not remove: alias not present!")
-                return True
-        except KeyError:
-            plugin_header(recipient=sender, name="Chat Alias")
-            msg(sender, "&7Could not remove: you do not have any aliases!")
-            return True
-        save_json_file("aliases", data)
-        plugin_header(recipient=sender, name="Chat Alias")
-        msg(sender, "&7Alias for %s &7successfuly removed" % args[0])
-        return True
-    elif len(args) >= 2:
-        data = safe_open_json()
-        alias = " ".join(args[1:])
-        try:
-            if (len(alias) > max_alias_length) and (max_alias_length >= 0) and (not sender.hasPermission(exceed_length)):
-                plugin_header(recipient=sender, name="Chat Alias")
-                msg(sender, "&7Please do not alias long words/sentences.")
-                return True
-            if (len(data[str(sender.getUniqueId())]) >= max_entries) and (max_entries >= 0) and (not sender.hasPermission(exceed_entries)):
-                plugin_header(recipient=sender, name="Chat Alias")
-                msg(sender, "&7You have reached your alias limit!")
-                return True
-        except KeyError:
-            data[str(sender.getUniqueId())] = {}
-        data[str(sender.getUniqueId())][args[0]] = alias
-        save_json_file("aliases", data)
-        plugin_header(recipient=sender, name="Chat Alias")
-        msg(sender, "&7Chat Alias %s &7==> %s &7successfully created!" % (args[0], alias))
-        return True
         else:
-        return False
+            if event.getPlayer().hasPermission(permission_FINFO):
+                disabled_fallback(event.getPlayer())
+    except:
+        print(trace())
 
 
 @hook.event("player.AsyncPlayerChatEvent", "high")
 def on_player_chat(event):
-    playerid = str(event.getPlayer().getUniqueId())
+    try:
-    data = safe_open_json()
+        if enabled:
             if event.isCancelled():
                 return
-    if not playerid in data:
+            if not hasPerm(event.getPlayer(), permission_USE):
+                return
+            for alias, value in data[str(uid(event.getPlayer()))].items():
+                if not event.getPlayer().hasPermission(permission_ALL) and len(event.getMessage()) > int(get_permission_content(event.getPlayer(), permission_LENGTH)):
+                    event.setCanceled(True)
+                    plugin_header(event.getPlayer, "Alias")
+                    msg(event.getPlayer(), "The message you wanted to generate would exceed your limit. Please make it shorter!")
+                    return
+                if event.getPlayer().hasPermission("essentials.chat.color"):
+                    event.setMessage(event.getMessage().replace(colorify(alias), colorify(value)))
+                else:
+                    event.setMessage(event.getMessage().replace(alias, value))
+    except:
         return
 
-    event.setMessage(multiple_replace(data[playerid], event.getMessage()))
+def hasPerm(player, permission):
+    return (player.hasPermission(permission)) or (player.hasPermission(permission_ALL))
 
-    if (event.getPlayer().hasPermission("essentials.chat.color")):
+
-        event.setMessage(colorify(event.getMessage()))
+def disabled_fallback(receiver):
-    if (max_overall_length >= 0) and (len(event.getMessage()) > max_overall_length) and (not event.getPlayer().hasPermission(exceed_overall_length)):
+    if not hasPerm(receiver, permission_INFO):
-        event.setCancelled(True)
+        msg(receiver, colorify("&cUnknown command. Use &e/help&c, &e/plugins &cor ask a mod."))
-        plugin_header(recipient=event.getPlayer(), name="Chat Alias")
+    else:
-        msg(event.getPlayer(), "&7The message generated was too long and was not sent. :/")
+        msg(receiver, colorify("&cPlugin alias v" + alias_version + " has experienced an &eEMERGENCY SHUTDOWN:"))
+        msg(receiver, error)
+        msg(receiver, colorify("&cPlease contact a dev/admin (especially pep :P) about this to take a look at it."))
+
+
+def can_remote(player):
+    return hasPerm(player, permission_LIST_OTHERS) or hasPerm(player, permission_MODIFY_OTHERS)
+
+
+def add(sender, args):
+    plugin_header(sender, "Alias")
+    if not sender.hasPermission(permission_ALL) and len(data[uid(sender)]) >= int(get_permission_content(sender, permission_AMOUNT)):
+        msg(sender, "&cCould not create alias: Max_limit reached!")
+        return True
+    args = [args[0]] + [" ".join(args[1:])]
+    data[str(uid(sender))][str(args[0])] = args[1]
+    save_data(uid(sender))
+    msg(sender, colorify("&7Alias: ") + args[0] + colorify("&7 -> " + args[1] + colorify("&7 was succesfully created!")), usecolor=sender.hasPermission("essentials.chat.color"))
+    return True
+
+
+def radd(sender, args):
+    args = [args[0:1]] + [" ".join([args[2:len(args)-2]])] + [args[len(args)-1]]
+    plugin_header(sender, "Alias")
+    if args[3].lower() == "false":
+        plugin_header(target, "Alias")
+        msg(target, "&cPlayer " + sender_name + " &cis creating an alias for you!")
+    if not sender.hasPermission(permission_ALL) and len(data[uid(sender)]) >= int(get_permission_content(target, permission_AMOUNT)):
+        msg(sender, "&cCould not create alias: Max_limit reached!")
+        if args[3].lower() == "false":
+            msg(target, "&cCould not create alias: Max_limit reached!")
+        return True
+    
+    target = get_player(args[0])
+    if is_player(sender):
+        sender_name = colorify(sender.getDisplayName)
+    else:
+        sender_name = colorify("&6Console")
+    if len(args) == 3:
+        args += ["true"]
+    data[str(uid(target))][str(args[1])] = str(args[2])
+    save_data(uid(target))
+    msg(sender, colorify("&7Alias: ") + args[1] + colorify("&7 -> " + args[2] + colorify("&7 was succesfully created!")), usecolor=target.hasPermission("essentials.chat.color"))
+    if args[3].lower() == "false":
+        msg(target, colorify("&7Alias: ") + args[1] + colorify("&7 -> " + args[2] + colorify("&7 was succesfully created!")), usecolor=target.hasPermission("essentials.chat.color"))
+    return True
+
+
+def remove(sender, args):
+    plugin_header(sender, "Alias")
+    try:
+        msg(sender, colorify("&7Successfully removed alias ") + args[0] + colorify(" &7-> ") + data[uid(sender)].pop(args[0]) + colorify("&7!"), usecolor=sender.hasPermission("essentials.chat.color"))
+        save_data(uid(sender))
+    except:
+        msg(sender, colorify("&cCould not remove alias ") + args[0] + colorify(", it does not exist."), usecolor=sender.hasPermission("essentials.chat.color"))
+    return True
+
+
+def rremove(sender, args):
+    plugin_header(sender, "Alias")
+    target = get_player(args[0])
+    if is_player(sender):
+        sender_name = colorify(sender.getDisplayName)
+    else:
+        sender_name = colorify("&6Console")
+    if args[2].lower() == "false":
+        print("WTF")
+        plugin_header(target, "Alias")
+        msg(target, "&cPlayer " + sender_name + " &cis removing an alias for you!")
+    try:
+        alias = data[uid(target)].pop(args[1])
+        msg(sender, colorify("&7Successfully removed alias ") + args[1] + colorify(" &7-> ") + alias + colorify("&7!"), usecolor=sender.hasPermission("essentials.chat.color"))
+        if args[2].lower() == "false":
+            msg(target, colorify("&7Successfully removed alias ") + args[1] + colorify(" &7-> ") + alias + colorify("&7!"), usecolor=sender.hasPermission("essentials.chat.color"))
+        save_data(uid(target))
+    except:
+        msg(sender, colorify("&cCould not remove alias ") + args[1] + colorify(", it does not exist."), usecolor=sender.hasPermission("essentials.chat.color"))
+        if args[2].lower() == "false":
+            msg(target, colorify("&cCould not remove alias ") + args[1] + colorify(", it does not exist."), usecolor=sender.hasPermission("essentials.chat.color"))
+    return True
+
+
+def list_alias(sender, args):
+    plugin_header(sender, "Alias")
+    msg(sender, "&7You have a total of " + str(len(data[uid(sender)])) + " aliases:")
+    for word, alias in data[str(uid(sender))].items():
+        msg(sender, colorify("&7") + word + colorify("&7 -> ") + alias, usecolor=sender.hasPermission("essentials.chat.color"))
+    return True
+
+
+def rlist_alias(sender, args):
+    plugin_header(sender, "Alias")
+    target = get_player(args[0])
+    if is_player(sender):
+        sender_name = colorify(sender.getDisplayName)
+    else:
+        sender_name = colorify("&6Console")
+    if len(args) == 1:
+        args += ["true"]
+    msg(sender, "Player " + args[0] + " has following aliases (" + str(len(data[uid(target)])) + " in total):")
+    if args[1].lower() == "false":
+        plugin_header(target, "Alias")
+        msg(target, "&cPlayer " + sender_name + " &cis listing your aliases (" + str(len(data[uid(target)])) + " in total):")
+    for word, alias in data[str(uid(target))].items():
+        msg(sender, colorify("&7") + word + colorify("&7 -> ") + alias, usecolor=target.hasPermission("essentials.chat.color"))
+        if args[1].lower() == "false":
+            msg(target, colorify("&7") + word + colorify("&7 -> ") + alias, usecolor=target.hasPermission("essentials.chat.color"))
+    return True
+
+
+def remote(sender, args):
+    try:
+        return remotes[args[1].lower()](sender, [args[0]] + [args[2:]])
+    except:
+        return subcommands["help"](sender, ["2"])
+
+
+def load_data(uuid):
+    if use_mysql:
+        conn = zxJDBC.connect(mysql_database, mysql_user, mysql_pass, "com.mysql.jdbc.Driver")
+        curs = conn.cursor()
+        curs.execute("SELECT alias FROM alias WHERE uuid = ?", (uuid, ))
+        results = curs.fetchall()
+        if len(results) == 0:
+            results = global_aliases
+            curs.execute("INSERT INTO alias VALUES (?,?)", (uuid, results, ))
+        data[uuid] = results
+    else:
+        data[uuid] = safe_open_json(uuid)
+
+
+def save_data(uuid):
+    if use_mysql:
+        conn = zxJDBC.connect(mysql_database, mysql_user, mysql_pass, "com.mysql.jdbc.Driver")
+        curs = conn.cursor()
+        curs.execute("UPDATE alias SET alias = ? WHERE uuid = ?", (data[uuid], uuid, ))
+    else:
+        save_json_file("aliases/" + uuid, data[uuid])
+
+# Subcommands:
+
+subcommands = {
+    "help": help,
+    "add": add,
+    "remove": remove,
+    "player": remote,
+    "list": list_alias
+}
+
+remotes = {
+    "add": radd,
+    "remove": rremove,
+    "list": rlist_alias,
+}

chatgroups.py

@@ -21,6 +21,8 @@ def on_chatgroup_command(sender, command, label, args):
     if len(args) == 1 and args[0] == "leave":
         if sender_id in groups.keys():
             groupchat(sender, "left the group", True)
+            if sender in cg_toggle_list:
+                cg_toggle_list.remove(p)
             group = groups[sender_id]
             del(groups[sender_id])
             save_groups()

helpers.py

@@ -1,32 +1,28 @@
 #pylint: disable = F0401
-
-import org.bukkit as bukkit
-import org.bukkit.block as bblock
-import org.bukkit.Location as Location
-import org.bukkit.event.entity as entity
-import org.bukkit.entity.Player as Player
-import org.bukkit.command.ConsoleCommandSender
-import org.bukkit.event.block.BlockBreakEvent as BlockBreakEvent
-import org.bukkit.event.player.PlayerTeleportEvent.TeleportCause as TeleportCause
-
 from re import sub
-from thread_utils import *
-from player import py_players
-from org.bukkit.entity import *
-from player import get_py_player
-from traceback import format_exc as trace
 from java.util.UUID import fromString as juuid
 from json import dumps as json_dumps, loads as json_loads
+import org.bukkit as bukkit
+import org.bukkit.Location as Location
+import org.bukkit.entity.Player as Player
+import org.bukkit.event.player.PlayerTeleportEvent.TeleportCause as TeleportCause
+import org.bukkit.event.block.BlockBreakEvent as BlockBreakEvent
+import org.bukkit.block as bblock
+import org.bukkit.event.entity as entity
+import org.bukkit.command.ConsoleCommandSender
+from org.bukkit.entity import *
+from player import get_py_player
+from player import py_players
 
 #Imports for async query
-import threading
-import mysqlhack
-
 from secrets import *
+import mysqlhack
 from com.ziclix.python.sql import zxJDBC
+import threading
 
+from traceback import format_exc as trace
 
-
+helpers_version = "2.0.0"
 shared = {} # this dict can be used to share stuff across modules
 
 server = bukkit.Bukkit.getServer()
@@ -91,7 +87,7 @@ def colorify(text):
     """
     replace &-codes with real color codes
     """
-    return sub("&" + u"\u00A7", "&", "%s" % sub("&(?=[?\\da-fk-or])", u"\u00A7", "%s" % text))
+    return sub("&(?=[?\\da-fk-or])", u"\u00A7", "%s" % text)
 
 
 def stripcolors(text):
@@ -185,16 +181,6 @@ def is_creative(player):
     return str(player.getGameMode()) == "CREATIVE"
 
 
-def is_rank(player, rank):
-    """
-    rank: a string equal to the PEX group name found in /pex groups
-    returns True if one of the following conditions are met:
-    - the player is of the given rank,
-    - their rank inherits the given rank.
-    """
-    return player.hasPermission("groups." + rank)
-
-
 def uid(player):
     """
     returns the player's UUID
@@ -218,6 +204,31 @@ def known_player(player):
     """
     return player.hasPlayedBefore()
 
+"""
+Runs a async query, calls target function with fetchall as an argument, it will be an empty list if there is nothing to fetch.
+(So make sure your function takes that argument.)
+
+If you want your function to run sync in the case you are doing something spigot wouldn't like to be async use the bukkit scheduler.
+Example can be found in loginsecurity.py
+
+"""
+def async_query(mysql_database,query,query_args,target,*target_args,**target_kwargs):
+
+    def async_query_t(mysql_database,query,query_args,target,target_args,target_kwargs):
+        db_conn = zxJDBC.connect(mysql_database, mysql_user, mysql_pass, "com.mysql.jdbc.Driver")
+        db_curs = db_conn.cursor()
+        db_curs.execute(query,query_args)
+        db_conn.commit()
+        fetchall = db_curs.fetchall()
+        db_curs.close()
+        db_conn.close()
+        target(fetchall,target_args,target_kwargs)
+
+    t = threading.Thread(target=async_query_t,args=(mysql_database,query,query_args,target,target_args,target_kwargs))
+    t.daemon = True
+    t.start()
+
+
 def open_json_file(filename, default = None):
     """
     opens the given json file and returns an object or returns None on error
@@ -260,7 +271,8 @@ def send_JSON_message(playername, message):
     bukkit.Bukkit.getServer().dispatchCommand(bukkit.Bukkit.getServer().getConsoleSender(), "tellraw " + playername + " " + message)
 
 
-def isIP(tocheck):
+# Allows to check if a String is a valid IPv4 or not. Accepts any string, returns true if the String is a valid IPv4
+def is_ip(tocheck):
     subsets = ["","","",""]
     i = 0
     for j in range(0,len(tocheck)):
@@ -278,3 +290,27 @@ def isIP(tocheck):
         if not ((int(subsets[j]) >= 0) & (int(subsets[j]) <= 255)):
             return False
     return True
+
+
+# Allows the use of e.g. numeric permission nodes like "permission.amount.5" and similar.
+# To get the data fetch the player and the start of the permission node, looking like "permission.amount."
+def get_permission_content(player, permnode):
+    perms = player.getEffectivePermissions()
+    for perm in perms:
+        if str(perm.getPermission()).startswith(permnode):
+               return str(perm.getPermission()).replace(permnode, "")
+
+
+# Gets an online player from their name
+def get_player(name):
+    for p in bukkit.Bukkit.getServer().getOnlinePlayers():
+        if p.getName().lower() == name.lower():
+            return p
+    return None
+
+
+def array_to_list(array):
+    return_list = []
+    for a in array:
+        return_list += [a]
+    return return_list

imbusy.py

@@ -105,3 +105,10 @@ def on_cmd_preprocess_event(event):
             plugin_header(recipient = event.getPlayer(), name = "I'M BUSY!")
             msg(event.getPlayer(), "We are sorry, but %s is currently busy. Please try again later." % message[1])
             event.setCancelled(True)
+
+@hook.event("player.PlayerQuitEvent", "lowest")
+def on_player_leave(event):
+    try:
+        busy_players.remove(event.getPlayer().getName())
+    except:
+        pass

iptracker.py

@@ -9,6 +9,7 @@ from iptracker_secrets import *
 
 
 iptrack_permission = "utils.iptrack"
+iptrack_version = "1.1.0"
 
 
 @hook.event("player.PlayerJoinEvent", "low")
@@ -42,13 +43,13 @@ def on_player_join_thread(event):
         if new_ip_entry:
             curs.execute("INSERT INTO uuid2ips VALUES (?,?)", (uuid, json.dumps(ips), ))
         else:
-            curs.execute("UPDATE uuid2ips SET ips = ? WHERE uuid = ?", (uuid, json.dumps(ips), ))
+            curs.execute("UPDATE uuid2ips SET ips = ? WHERE uuid = ?", (json.dumps(ips), uuid, ))
     if uuid not in uuids:
         uuids.append(uuid)
         if new_uuid_entry:
             curs.execute("INSERT INTO ip2uuids VALUES (?,?)", (ip, json.dumps(uuids), ))
         else:
-            curs.execute("UPDATE ip2uuids SET uuids = ? WHERE uuid = ?", (ip, json.dumps(uuids), ))
+            curs.execute("UPDATE ip2uuids SET uuids = ? WHERE ip = ?", (json.dumps(uuids), ip, ))
         conn.commit()
     curs.close()
     conn.close()
@@ -56,7 +57,7 @@ def on_player_join_thread(event):
 
 @hook.command("getinfo")
 def on_getinfo_command(sender, args):
-    t = threading.Thread(target=on_player_join_thread, args=(sender, args))
+    t = threading.Thread(target=on_getinfo_command_thread, args=(sender, args))
     t.daemon = True
     t.start()
 
@@ -65,7 +66,7 @@ def on_getinfo_command_thread(sender, args):
         if not checkargs(sender, args, 1, 1):
             return False
         else:
-            if isIP(args[0]):
+            if is_ip(args[0]):
                 conn = zxJDBC.connect(mysql_database, mysql_user, mysql_pass, "com.mysql.jdbc.Driver")
                 curs = conn.cursor()
                 curs.execute("SELECT uuids FROM ip2uuids WHERE ip = ?", (args[0], ))

login.py

@@ -1,129 +0,0 @@
-from wrapper import *
-from passlib.hash import pbkdf2_sha256 as crypt
-
-@event_handler("player_login","normal", utils = True)
-def player_join(*args):
-    player = args[1]
-    if not player.registered:
-        player.authenticated = True
-        player.msg("Successfully logged in!")
-
-"""
-@event_handler("player.PlayerCommandPreprocessEvent", "lowest")
-def on_login_command(event):
-    player = event.player
-    password = event.message.replace("/login", "").replace(" ", "")
-    event.message = event.player.name + " Attempting to login!"
-    event.cancelled = True
-
-    @async(daemon = True)
-    def check_pass(player, password):
-        print password
-        if not player.registered:
-            player.msg("You are not registered! use /register <password> to register!")
-            return
-        else:
-            if crypt.verify(password, player.password):
-                player.authenticated = True
-                player.msg("Successfully logged in!")
-            else:
-                print event.message
-                player.msg("Wrong password!")
-    check_pass(player, password)"""
-
-@command("login")
-@async(daemon = True)
-def on_login_command(**kwargs):
-    player = kwargs["sender"]
-    args = kwargs["args"]
-
-    if not player.registered:
-        player.msg("You are not registered! use /register <password> to register!")
-        return
-    if len(args) > 1:
-        player.msg("The syntax is /login <password>")
-        return
-    elif len(args) is 1:
-        if crypt.verify(args[0], player.password):
-            player.authenticated = True
-            player.msg("Successfully logged in!")
-        else:
-            player.msg("Wrong password!")
-
-@command("changepass")
-@async(daemon = True)
-def on_changepass_command(**kwargs):
-    player = kwargs["sender"]
-    args = kwargs["args"]
-
-    if not player.registered:
-        player.msg("You are not registered! use /register <password> to register!")
-        return
-
-    if len(args) < 2:
-        player.msg("The syntax is /login <current_password> <new_password>")
-        return
-    elif len(args) is 2:
-        if crypt.verify(args[0], player.password):
-            player.password = crypt.encrypt(args[1], rounds=200000, salt_size=16)
-            player.msg("Successfully changed your password!")
-            player.save()
-        else:
-            player.msg("You have entered an incorrect current password!")
-
-@command("removepass")
-@async(daemon = True)
-def on_removepass_command(**kwargs):
-    player = kwargs["sender"]
-    args = kwargs["args"]
-
-    if not player.registered:
-        player.msg("You are not registered! use /register <password> to register!")
-        return
-
-    if len(args) < 1:
-        player.msg("The syntax is /removepass <current_password>")
-        return
-
-    elif len(args) is 1:
-        if crypt.verify(args[0], player.password):
-            player.password = "None"
-            player.registered = False
-            player.save()
-            player.msg("Successfully removed your password!")
-        else:
-            player.msg("You have entered an incorrect current password!")
-
-
-
-
-@command("register")
-@async(daemon = True)
-def on_register_command(**kwargs):
-    player = kwargs["sender"]
-    args = kwargs["args"]
-    if len(args) > 1:
-        player.msg("The syntax is /register <password>")
-        return
-    elif len(args) is 1:
-        if player.registered:
-            player.msg("You are already registered!")
-            return 
-        player.password = crypt.encrypt(args[0], rounds=200000, salt_size=16)
-        player.registered = True
-        print player.password
-        player.save()
-        player.msg("Successfully registered!")
-
-
-blocked_events = ["block.BlockBreakEvent", "block.BlockPlaceEvent", "player.PlayerMoveEvent",
-                    "player.AsyncPlayerChatEvent","player.PlayerTeleportEvent",
-                     "player.PlayerInteractEvent"]
-
-for event in blocked_events:
-    @event_handler(event_name = event, priority = "highest")
-    def on_blocked_event(event):
-        if not event.player.authenticated:
-            event.cancelled = True
-
-

loginsecurity.py

@@ -0,0 +1,275 @@
+from helpers import *
+from passlib.hash import pbkdf2_sha256 as crypt
+from basecommands import simplecommand
+import time
+import threading
+from login_secrets import * #Don't forget to make login_secrets aswell
+import mysqlhack
+from com.ziclix.python.sql import zxJDBC
+from java.lang import Runnable
+
+wait_time = 30 #seconds
+admin_perm = "utils.loginsecurity.admin"
+min_pass_length = 8
+blocked_events = ["block.BlockBreakEvent", "block.BlockPlaceEvent", "player.PlayerMoveEvent","player.AsyncPlayerChatEvent"]
+
+
+
+
+def matches(password,user):
+    thread = threading.Thread(target=matches_thread, args = (password,user))
+    thread.start()
+
+
+def matches_thread(password, user):
+    hashed = get_pass(uid(user))
+    py_player = get_py_player(user)
+    if crypt.verify(password, hashed):
+        if py_player.logging_in:
+            py_player.logging_in = False
+            msg(user, "&aLogged in successfully!")
+    else:
+        if py_player.logging_in:
+            msg(user, "&cInvalid password!")
+
+
+
+
+
+@simplecommand("cgpass",
+    usage       = "<password> <new password>",
+    description = "Changes your password",
+    senderLimit = 0,
+    helpNoargs  = True)
+def change_pass_command(sender, command, label, args):
+    
+    py_player = get_py_player(sender)
+
+    if py_player.logging_in:
+        return "&cYou are not logged in"
+    if not len(args) == 2:
+        return "&cInvalid arguments"
+
+    password = args[0]
+    new_password = args[1]
+    uuid = uid(sender)
+
+    if is_registered(uuid):
+        change_pass(uuid, crypt.encrypt(new_password, rounds=200000, salt_size=16))
+        return "&aPassword changed"
+    return "&cYou are not registered"
+
+
+
+
+@simplecommand("login",
+        usage       = "<password>",
+        description = "Logs you in if <password> matches your password.",
+        senderLimit = 0,
+        helpNoargs  = True)
+def login_command(sender, command, label, args):
+    py_player = get_py_player(sender)
+
+    if not py_player.logging_in:
+        msg(sender,"&cAlready logged in!")
+
+    password = args[0]
+    matches(password, sender)
+
+
+
+
+@simplecommand("register",
+        usage       = "<password>",
+        description = "Registers you with <password>. Next time you join, log in with /login",
+        senderLimit = 0,
+        helpNoargs  = True)
+def register_command(sender, command, label, args):
+
+    py_player = get_py_player(sender)
+
+    if len(args) > 1:
+        return "&cPassword can only be one word!"
+
+    uuid = uid(sender)
+    if is_registered(uuid):
+        return "&cYou are already registered!"
+
+    password = args[0]
+
+    if len(password) < min_pass_length:
+        return "&cThe password has to be made up of at least %s characters!" % min_pass_length
+
+    create_pass(uuid, password)
+    return "&cPassword set. Use /login <password> upon join."
+
+
+
+
+@simplecommand("rmpass",
+        description = "Removes your password if the password matches",
+        senderLimit = 0,
+        amax = 0,
+        helpNoargs  = False)
+def rmpass_command(sender, command, label, args):
+
+    py_player = get_py_player(sender)
+
+    if py_player.logging_in:
+        return "&cYou are not logged in"
+
+    if not is_registered(uid(sender)):
+        return "&cYou are not registered!"
+
+    if py_player.logging_in == False:
+        delete_pass(uid(sender))
+        return "&aPassword removed successfully. You will not be prompted anymore."
+    return "&cFailed to remove password, please contact a staff member"
+
+
+
+
+@simplecommand("rmotherpass",
+        aliases     = ["lacrmpass"],
+        usage       = "<user>",
+        senderLimit = -1,
+        description = "Removes password of <user> and sends them a notification",
+        helpNoargs  = True)
+def rmotherpass_command(sender, command, label, args):
+    
+    py_player = get_py_player(sender)
+
+    if py_player.logging_in:
+        return "&cYou are not logged in"
+    
+    if not sender.hasPermission(admin_perm):
+        noperm(sender)
+        return
+
+    user = server.getOfflinePlayer(args[0])
+    
+    if is_registered(uid(user)):
+        delete_pass(uid(user))
+        runas(server.getConsoleSender(), colorify("mail send %s &cYour password was reset by a staff member. Use &6/register&c to set a new one." % user.getName()))
+        return "&aPassword of %s reset successfully" % user.getName()
+    return "&cThat player could not be found (or is not registered)"
+
+def change_pass(uuid, pw):
+    conn = zxJDBC.connect(mysql_database, mysql_user, mysql_pass, "com.mysql.jdbc.Driver")
+    curs = conn.cursor()
+    curs.execute("UPDATE secret SET pass = ? WHERE uuid = ?", (pw,uuid,))
+    conn.commit()
+    curs.close()
+    conn.close()
+
+def get_pass(uuid):
+    conn = zxJDBC.connect(mysql_database, mysql_user, mysql_pass, "com.mysql.jdbc.Driver")
+    curs = conn.cursor()
+    curs.execute("SELECT pass FROM secret WHERE uuid = ?", (uuid,))
+    results = curs.fetchall()
+    curs.close()
+    conn.close()
+    return results[0][0]
+
+def create_pass(uuid,pw):
+    thread = threading.Thread(target=create_pass_thread, args=(uuid,pw))
+    thread.start()
+
+def create_pass_thread(uuid, pw):
+    pw = crypt.encrypt(pw, rounds=200000, salt_size=16)
+    conn = zxJDBC.connect(mysql_database, mysql_user, mysql_pass, "com.mysql.jdbc.Driver")
+    curs = conn.cursor()
+    curs.execute("INSERT INTO secret VALUES (?,?)", (uuid,pw,))
+    conn.commit()
+    curs.close()
+    conn.close()
+
+def is_registered(uuid):
+    conn = zxJDBC.connect(mysql_database, mysql_user, mysql_pass, "com.mysql.jdbc.Driver")
+    curs = conn.cursor()
+    curs.execute("SELECT EXISTS(SELECT * FROM secret WHERE uuid = ?)", (uuid,))
+    results = curs.fetchall()
+    curs.close()
+    conn.close()
+    if results[0][0] == 1:
+        return True
+    return False
+
+def delete_pass(uuid):
+    conn = zxJDBC.connect(mysql_database, mysql_user, mysql_pass, "com.mysql.jdbc.Driver")
+    curs = conn.cursor()
+    curs.execute("DELETE FROM secret WHERE uuid = ?", (uuid,))
+    conn.commit()
+    curs.close()
+    conn.close()
+
+@hook.event("player.PlayerJoinEvent", "highest")
+def on_join(event):
+    user = event.getPlayer()
+    py_player = get_py_player(event.getPlayer())
+    if is_registered(uid(user)):
+        msg(event.getPlayer(), "&4You will be disconnected after 60 seconds if you don't &alogin")
+        msg(user, "&cUse /login <password>")
+        py_player.logging_in = True
+        py_player.login_time = time.time()
+        return
+    elif user.hasPermission(admin_perm):
+        pass #Do what? force them to make a password, lots of code, maybe just message us on slack?
+
+#This shouldn't be needed anymore as py_player gets removed anyway.
+"""
+
+@hook.event("player.PlayerQuitEvent", "high")
+def on_quit(event):
+    if event.getPlayer().getName() in logging_in:
+        del logging_in[event.getPlayer().getName()]
+"""
+
+##Threading start
+class kick_class(Runnable):
+
+    def __init__(self, player):
+        self.player = player
+
+    def run(self):
+        if self.player.isOnline():
+            self.player.kickPlayer(colorify("&aLogin timed out"))
+
+def kick_thread():
+    while True:
+        time.sleep(1)
+        now = time.time()
+        for py_player in py_players:
+            if py_player.logging_in:
+                if now - py_player.login_time > wait_time:
+                    player = py_player.player
+                    kick = kick_class(player)
+                    server.getScheduler().runTask(server.getPluginManager().getPlugin("RedstonerUtils"), kick)
+                    
+
+                    """if name in logging_in:
+                        del logging_in[name]
+                        break
+                    """
+
+
+thread = threading.Thread(target = kick_thread)
+thread.daemon = True
+thread.start()
+##Threading end
+
+for blocked_event in blocked_events:
+    @hook.event(blocked_event, "high")
+    def on_blocked_event(event):
+        user = get_py_player(event.getPlayer())
+        if user.logging_in:
+            event.setCancelled(True)
+
+@hook.event("player.PlayerCommandPreprocessEvent","normal")
+def pre_command_proccess(event):
+    player = get_py_player(event.getPlayer())
+    if player.logging_in:
+        args = event.getMessage().split(" ")
+        if not args[0].lower() == "/login":
+            msg(player.player, "&4You need to login before you do that!")
+            event.setCancelled(True)

main.py

@@ -11,10 +11,11 @@ sys.path += ['', '/usr/lib/python2.7', '/usr/lib/python2.7/plat-linux2', '/usr/l
 try:
     # Library that adds a bunch of re-usable methods which are used in nearly all other modules
     from helpers import *
-    #from wrapper import *
 except:
-    error("[RedstonerUtils] ERROR: Failed to import Wrapper:")
+    print("[RedstonerUtils] ERROR: Failed to import helpers:")
-    error(print_traceback())
+    print(print_traceback())
+
+
 
 @hook.enable
 def on_enable():
@@ -23,20 +24,87 @@ def on_enable():
 
 @hook.disable
 def on_disable():
-    #shared["modules"]["reports"].stop_reporting()
+    shared["modules"]["reports"].stop_reporting()
     info("RedstonerUtils disabled!")
 
 
 info("Loading RedstonerUtils...")
 
-
-
 # Import all modules, in this order
 shared["load_modules"] = [
-    "test", 
+    # Collection of tiny utilities
-#    "login",
+    "misc",
+    # Adds chat for staff using /ac <text or ,<text>
+    "adminchat",
+    # Adds /badge, allows to give players achievements
+    "badges",
+    # Adds a few block placement corrections/mods
+    "blockplacemods",
+    # Adds /calc, toggles automatic solving of Math expressions in chat
+    "calc",
+    # Adds aliasing of chat words
+    #"chatalias",
+    # Plugin to locate laggy chunks. /lc <n> lists chunks with more than n entities
+    "lagchunks",
+    # Adds /report and /rp, Stores reports with time and location
+    "reports",
+    # Adds group-chat with /chatgroup and /cgt to toggle normal chat into group mode
+    "chatgroups",
+    # Adds /token, reads and writes from the database to generate pronouncable (and thus memorable) registration-tokens for the website
+    "webtoken",
+    # Adds /lol, broadcasts random funyy messages. A bit like the splash text in the menu
+    "saylol",
+    # Adds /signalstrength, lets you request a signal strength and an amount of items will be inserted into target container to meet that strength.
+    "signalstrength",
+    # Shows the owner of a skull when right-clicked
+    "skullclick",
+    # Adds /listen, highlights chat and plays a sound when your name was mentioned
+    "mentio",
+    # Adds /cycler, swaps the hotbar with inventory when player changes slot from right->left or left->right
+    "cycle",
+    # Adds /getmotd & /setmotd to update the motd on the fly (no reboot)
+    "motd",
+    # AnswerBot. Hides stupid questions from chat and tells the sender about /faq or the like
+    "abot",
+    # Adds '/forcefield', creates forcefield for players who want it.
+    "forcefield",
+    # Adds /damnspam, creates timeout for buttons/levers to mitigate button spam.
+    "damnspam",
+    # Adds /check, useful to lookup details about a player
+    "check",
+    # Adds /an, a command you can use to share thoughts/plans/news
+    "adminnotes",
+    # Adds busy status to players
+    "imbusy",
+    # Adds /imout, displays fake leave/join messages
+    "imout",
+    #adds snowbrawl minigame
+    "snowbrawl",
+    # Adds /tm [player] for a messages to be sent to this player via /msg
+    "pmtoggle",
+    # Replacement for LoginSecurity
+    "loginsecurity",
+    # Centralized Player class
+    "player",
+    # Servercontrol extension for telnet access to logs/AC
+    #"servercontrol",
+    # Script helper plugin
+    "scriptutils",
+    # Per-player notes
+    "tag",
+    # vanish toggle module - temporary fix
+    #"vanishfix",
+    # obisidian mining punishment plugin
+    "punishments",
+    # a simple replacement for the buggy essentials /vanish
+    "vanish",
+    # ip-tracking utility - disabled as of instability
+    #"iptracker",
+    #server signs for everyone
+    "serversigns",
+    # Makes player's names colored, sorts tab list by rank
+    "nametags"
 ]
-
 shared["modules"] = {}
 for module in shared["load_modules"]:
     try:

misc.py

@@ -114,9 +114,20 @@ def rs_material_broken_by_flow(material):
     length = len(parts)
     return length > 1 and (parts[0] == "DIODE" or parts[1] in ("TORCH", "WIRE", "BUTTON", "HOOK") or (length == 3 and parts[1] == "COMPARATOR"))
 
+
+@hook.event("player.PlayerInteractEvent")
+def on_interact(event):
+    if (not event.isCancelled()
+        and str(event.getAction()) == "RIGHT_CLICK_BLOCK"
+        and str(event.getMaterial()) in ("REDSTONE_COMPARATOR_OFF", "REDSTONE_COMPARATOR_ON")
+        and not can_build(player, event.getClickedBlock())
+    ):
+        event.setCancelled(True)
+
+
 sudo_blacklist = ["pyeval", "script_backup_begin", "script_backup_end", "script_backup_error", "script_backup_database_begin", "script_backup_database_dumps", "script_backup_database_end",
 "script_backup_database_error", "script_backup_database_abort", "script_trim", "script_trim_result", "script_spigot_update", "script_disk_filled", "script_restart", "script_restart_abort",
-"script_stop", "script_stop_abort", "script_shutdown", "stop", "esudo", "essentials:sudo"]
+"script_stop", "script_stop_abort", "script_shutdown", "stop", "esudo", "essentials:sudo", "sudo"]
 
 @simplecommand("sudo",
         usage        = "<player> [cmd..]",

mysql_utils.py

@@ -1,45 +0,0 @@
-import mysqlhack
-from secrets import *
-from thread_utils import *
-from com.ziclix.python.sql import zxJDBC
-from traceback import format_exc as trace
-
-class mysql_connect:
-    def __init__(self):
-        self.conn = zxJDBC.connect(mysql_database, mysql_user, mysql_pass, "com.mysql.jdbc.Driver")
-        self.curs = self.conn.cursor()
-
-    def execute(self, query, args=None):
-        if args is None:
-            return self.curs.execute(query)
-        else:
-            print query
-            print args
-            return self.curs.execute(query, args)
-
-    def fetchall(self):
-        return self.curs.fetchall()
-
-    @property
-    def columns(self):
-        self.execute("SHOW COLUMNS FROM utils_players")
-        fetched = self.fetchall()
-        columns = []
-
-        for row in fetched:
-            columns.append(row[0])
-        return columns
-
-    def __enter__(self):
-        return self
-
-    def __exit__(self, exc_type, exc_inst, exc_tb):
-        if exc_type is None:
-            try:        
-                self.conn.commit()
-                self.curs.close()
-                self.conn.close()
-            except:
-                print(trace())
-        else:
-            print(exc_tb)

mysqlhack.py

@@ -4,12 +4,12 @@ A library that makes use of the so called ClassPathHack for jython
 to allow proper loading of mysql-connector.jar at runtime.
 Import only, no methods.
 """
-import jarray
-import java.net.URL
-import java.io.File
-from java.lang import Class
-import java.net.URLClassLoader
 import java.lang.reflect.Method
+import java.io.File
+import java.net.URL
+import java.net.URLClassLoader
+import jarray
+from java.lang import Class
 
 
 # hacky code to add mysql-connector to java's classpath ('classPathHack')

nametags.py

@@ -0,0 +1,37 @@
+from helpers import *
+
+##############################################################
+#                                                            #
+# This module automatically puts people in the corresponding #
+# scoreboard team so that their name is colored properly and #
+# tab will be nicely sorted.                                 #
+#                                                            #
+##############################################################
+
+ranks = ["visitor", "member", "builder", "trusted", "trainingmod", "mod", "admin"]
+# prefixes are used for sorting in the tab list
+prefixes = {"admin":"a", "mod":"b", "trainingmod":"c", "trusted":"d", "builder":"e", "member":"f","visitor":"g"}
+
+@hook.event("player.PlayerJoinEvent", "low")
+def on_player_join(event):
+    player = event.getPlayer()
+    team = get_team(player)
+    if team:
+        cmd = "scoreboard teams join %s %s" % (team, player.getName())
+        server.dispatchCommand(server.getConsoleSender(), cmd)
+
+def get_rank(player):
+    player_rank = None
+    for rank in ranks:
+        if not player.hasPermission("group.%s" % rank):
+            break
+        player_rank = rank
+    if not player_rank:
+        warn("Couldn't find rank for player %s" % player.getName())
+    return player_rank
+
+def get_team(player):
+    rank = get_rank(player)
+    if rank:
+        prefix = prefixes.get(rank)
+        return "_".join([prefix, rank])

pmtoggle.py

@@ -5,7 +5,11 @@ from java.util.UUID import fromString as juuid
 toggle_dict = {}
 permission = "utils.pmtoggle"
 
-@hook.command("tm")
+@hook.command("pmtoggle",
+              aliases = ["tm", "mt", "tmsg", "msgt", "pmt", "tpm"],
+              usage = "/<command> [player]",
+              description = "Toggle automatic sending of messages"
+              )
 def on_toggle_message_command(sender, command, label, args):
     if not sender.hasPermission(permission) or not is_player(sender):
         noperm(sender)

serversigns.py

@@ -5,15 +5,7 @@ import java.util.UUID as UUID
 import org.bukkit.Material as Material
 import org.bukkit.block.BlockFace as BlockFace
 
-"""
+blocked_cmds = tuple(shared["modules"]["misc"].sudo_blacklist) + ("pex", "kick", "ban", "tempban", "reload", "op", "deop", "whitelist")
-  # About permissions:
-  # To use the command, the user needs to have utils.serversigns.
-  # To use ANY subcommand, the user needs to have utils.serversigns.<subcommand> IN ADDITION to the previously mentioned node.
-  # To be able to add commands as messages to a sign, a user will need the node utils.serversigns.command.
-  # To be able to claim a sign for another player or to edit signs that the user doesn't own, they will need utils.serversigns.admin.
-"""
-
-blocked_cmds = ("pex", "kick", "ban", "tempban", "pyeval", "sudo", "stop", "reload", "op", "deop", "whitelist")
 
 def load_signs():
     signs_obj = open_json_file("serversigns", [])
@@ -32,18 +24,6 @@ signs = load_signs() # {("world", x, y, z): ["owner_id", "msg1", "msg2"]}
 
 lines = {} # Accumulated messages so players can have longer messages: {"Dico200": "Message...........", ""}
 
-@hook.enable
-def check_all_signs(): 
-    # Check if all saved signs actually represent a sign block. There are ways to break the signs without the plugin knowing.
-    for loc in dict(signs): # Can't change dict size during iteration, using a copy
-        world = server.getWorld(loc[0])
-        if world and world.getBlockAt(loc[1], loc[2], loc[3]).getType() in (Material.SIGN_POST, Material.WALL_SIGN):
-            continue
-        del signs[loc]
-        info("[Server Signs] Couldn't find a %s, removed the data for the sign that was once there." % identifySign(loc))
-    save_signs()
-
-
 def fromLoc(bLoc):
     """
       # Returns a tuple containing the (bukkit)location's world's name and its x, y and z coordinates
@@ -242,6 +222,9 @@ def svs_command(sender, command, label, args):
         return signsMsg("Removed all messages and the owner from the %s, it can now be claimed" % signName, 'a')
     #-------------------------------------------------------------------------------------------------------
 
+
+
+
 @hook.event("player.PlayerInteractEvent")
 def on_click(event):
     if str(event.getAction()) != "RIGHT_CLICK_BLOCK":
@@ -271,6 +254,7 @@ faces = {
 
 @hook.event("block.BlockBreakEvent", "monitor")
 def on_break(event):
+    try:
         global checking_block
         if checking_block or event.isCancelled():
             return
@@ -287,6 +271,8 @@ def on_break(event):
         block3 = block.getRelative(BlockFace.UP)
         if block3.getType() == Material.SIGN_POST:
             check_sign(event, block3)
+    except:
+        error(trace())
 
 
 def check_sign(event, block, attached = True):
@@ -301,7 +287,6 @@ def check_sign(event, block, attached = True):
         save_signs()
         msg(player, signsMsg("Reset the %s which you just broke" % identifySign(loc)))
 
-
 def can_build2(player, block):
     global checking_block
     event = BlockBreakEvent(block, player)

setup.sh

@@ -25,7 +25,7 @@ mkdir -v "build"
 cd "build"
 
 echo -e "\n> Downloading Spigot build tools"
-curl --progress-bar -Lo "buildtools.jar" "https://hub.spigotmc.org/jenkins/job/BuildTools/lastSuccessfulBuild/artifact/target/BuildTools.jar"
+curl --progress-bar -Lo "BuildTools.jar" "https://hub.spigotmc.org/jenkins/job/BuildTools/lastSuccessfulBuild/artifact/target/BuildTools.jar"
 
 echo -e "\n> Building Spigot, this will take a while ..."
 java -jar BuildTools.jar > /dev/null

thread_utils.py

@@ -1,24 +0,0 @@
-import threading
-"""
-Quick implementation of a @synchronized and @asynchronized decorators
-"""
-
-#To be replaced by bukkit scheduler.
-"""
-def sync(lock=None):
-    def decorator(wrapped):
-        def wrapper(*args, **kwargs):
-            with lock:
-                return wrapped(*args, **kwargs)
-        return wrapper
-    return decorator
-"""
-
-def async(daemon = True):
-    def decorator(function):
-        def wrapper(*args,**kwargs):
-            thread = threading.Thread(target=function,args=args,kwargs=kwargs)
-            thread.daemon = daemon
-            thread.start()
-        return wrapper
-    return decorator

util_events.py

@@ -1,80 +0,0 @@
-from thread_utils import *
-
-MONITOR_PRIORITY = "monitor"
-HIGHEST_PRIORITY = "highest"
-HIGH_PRIORITY    = "high"
-NORMAL_PRIORITY  = "normal"
-LOW_PRIORITY     = "low"
-LOWEST_PRIORITY  = "lowest"
-
-priorities = ["lowest","low","normal","high","highest","monitor"]
-events = []
-
-class base_event():
-    def __init__(self,event_name):
-        self.name = event_name
-        self.cancelled = False
-        self._handlers = [ [],[],[],[],[],[] ]
-
-        self.cancelled_lock = threading.Lock()
-
-    def add_handler(self,function,priority):
-        for prior in priorities:
-            if prior == priority:
-                self._handlers[priorities.index(prior)].append(function)
-
-    def fire(self,*args):
-        for priority in self._handlers:
-            for handler in priority:
-                handler(self,*args)
-
-    def set_cancelled(self,state):
-        with self.cancelled_lock:
-            self.cancelled = state
-
-
-class utils_events(base_event):
-    def __init__(self,event_name):
-        base_event.__init__(self,event_name)
-
-
-def add_event(event_name, event_class = base_event):
-    """
-      # Adds a new event type of the given class with the given name
-    """
-    event = event_class(event_name)
-    events.append(event)
-
-def fire_event(event_name,*args):
-    """
-      # Calls the ehe event with the given arguments
-    """
-    for event in events:
-        if event.name == event_name:
-            event.call(*args)
-            return event
-
-def check_events(event_name):
-    """
-      # Returns whether there is an event with the name event_name
-    """
-    for event in events:
-        if event.name == event_name:
-            return True
-    return False
-
-
-#Decorator
-def utils_event(event_name, priority, create_event=base_event):
-    def event_decorator(function):
-        def wrapper(*args, **kwargs):
-            pass
-
-        if not check_events(event_name): #Check if the event exists, if not create it.
-            add_event(event_name,create_event)
-
-        for event in events: #Go through the list of events, find the one we need and call all of its handlers
-            if event.name == event_name:
-                event.add_handler(function,priority)
-        return wrapper
-    return event_decorator

vanish.py

@@ -13,7 +13,7 @@ def is_vanished(player):
     return uid(player) in vanished
 
 
-#this can be used to silently set the vanished state of a player I guess.
+#this can be used to silently set the vanished state of a player
 def set_state(player, state):
     if state == is_vanished(player):
         return
@@ -37,10 +37,6 @@ def disable_vanish(target):
         player.showPlayer(target)
 
 
-def get_online_vanished_players():
-    return (player.getPlayer() for player in (retrieve_player(uuid) for uuid in vanished) if player.isOnline())
-
-
 @simplecommand("vanish", 
         aliases = ["v"],
         usage = "[on/off]",
@@ -52,6 +48,7 @@ def get_online_vanished_players():
         helpSubcmd = True
 )
 def vanish_command(sender, command, label, args):
+    try:
         current_state = is_vanished(sender)
         new_state = not current_state
 
@@ -62,28 +59,34 @@ def vanish_command(sender, command, label, args):
             elif arg == "off":
                 new_state = False
 
-    Validate.isTrue(current_state != new_state, "&cYou were %s vanished!" % ("already" if current_state else "not yet"))
+        if current_state == new_state:
+            return "&cYou were %s vanished!" % ("already" if current_state else "not yet")
+
         set_state(sender, new_state)
         return "&a%s vanish mode!" % ("Enabled" if new_state else "Disabled")
+    except:
+        error(trace())
 
 
 @hook.event("player.PlayerJoinEvent")
 def on_player_join(event):
     player = event.getPlayer()
+
     if not is_authorized(player):
-        for vanished in get_online_vanished_players():
+        for uuid in vanished:
-            player.hidePlayer(vanished)
+            player.hidePlayer(retrieve_player(uuid))
+
+    elif is_vanished(player):
+        msg(player, "&cKeep in mind that you are still vanished! Use /vanish to disable.")
 
 
 @hook.event("player.PlayerQuitEvent")
 def on_player_quit(event):
     player = event.getPlayer()
-    if not is_authorized(player):
-        for vanished in get_online_vanished_players():
-            player.showPlayer(vanished)
 
-    elif is_vanished(player):
+    if not is_authorized(player):
-        disable_vanish(player)
+        for uuid in vanished:
+            player.showPlayer(retrieve_player(uuid))
 
 
 @simplecommand("vanishother",
@@ -107,7 +110,9 @@ def vanishother_command(sender, command, label, args):
         elif arg == "off":
             new_state = False
 
-    Validate.isTrue(current_state != new_state, "&cThat player was %s vanished!" % ("already" if current_state else "not yet"))
+    if current_state == new_state:
+        return "&cThat player was already vanished!" if current_state else "&cThat player was not yet vanished!"
+
     set_state(target, new_state)
 
     enabled_str = "enabled" if new_state else "disabled"

wrapper.py

@@ -1,12 +0,0 @@
-"""
-Adapter classes for spigot api for more idiomatic python code.
-
-Before you run away from this if the class you need to use isn't here, please create it.
-
-
-"""
-from helpers import *
-from wrapper_event import *
-from wrapper_player import *
-from wrapper_command import *
-from util_events import utils_event, fire_event, utils_events

wrapper_command.py

@@ -1,328 +0,0 @@
-from wrapper_player import *
-from helpers import *
-
-class Command(object):
-    """
-      # Documentation to come.s
-    """
-
-    SENDER_ANY = 0
-    SENDER_PLAYER = 1
-    SENDER_CONSOLE = 2
-
-    ACTION_IGNORE = 3
-    ACTION_SYNTAXERROR = 4
-    ACTION_DISPLAYSYNTAX = 5
-    ACTION_DISPLAYHELP = 6
-
-    def __init__(self,
-                command, 
-                parent = None,
-                aliases = tuple(), 
-                permission = None,
-                description = "Description",
-                type = 0, 
-                no_arg_action = 3,
-                help_request_action = 3,
-                arguments = tuple(), 
-        ):
-
-        self.command = command.lower()
-        self.aliases = tuple(alias.lower() for alias in aliases)
-        self.description = description
-        self.type = type
-        self.no_arg_action = no_arg_action
-        self.help_request_action = help_request_action
-        self.arguments = arguments
-        self.parent = parent
-        self.sub_commands = Command_dict()
-
-        # ---- Check if argument layout is valid ----
-        prev_arg = arguments[0] if len(arguments) > 0 else None
-        for arg_info in arguments[1:]:
-
-            if not prev_arg.required and arg_info.required:
-                raise Argument_exception("Command: %s; There may not be required arguments after non-required arguments" % command)
-
-            if prev_arg.type == Argument.MESSAGE:
-                raise Argument_exception("Command: %s; An argument of type MESSAGE may not be followed by other arguments" % command)
-
-            prev_arg = arg_info
-
-        # ---- Add self to parent sub_commands and set permission node ----
-        perm_builder = "utils"
-        if self.parent == None:
-            root_commands[self.command] = self
-        else:
-            try:
-                parent_route = self.parent.split(" ")
-                parent_sub_commands = root_commands
-                parent_obj = None
-                for cmd_name in parent_route:
-                    parent_obj = parent_sub_commands[cmd_name]
-                    parent_sub_commands = parent_obj.sub_commands
-
-                parent_obj.sub_commands[self.command] = self
-
-            except KeyError as e:
-                raise Argument_exception("Error occurred while setting up command hierarchy: " + e.message + "\n" + trace())
-
-        perm_builder += "." + (permission if permission else command).lower()
-
-    def __call__(self, handler):
-        """
-          # To clarify: This function is called when you 'call' an instance of a class.
-          # This means, that Command() calls __init__() and Command()() calls __call__().
-          # This makes it possible to use class instances for decoration. The decorator is this function.
-        """
-        self.handler = handler
-
-        if self.parent == None:
-            @hook.command(self.command, aliases = self.aliases)
-            def run(sender, command, label, args):
-                """
-                  # This function will take care of prefixing and colouring of messages in the future.
-                  # So it's very much WIP.
-                """
-                try:
-                    message = self.execute(sender, command, label, args)
-                except Command_exception as e:
-                    message = e.message
-                except Argument_exception as e:
-                    message = e.message
-                except Exception:
-                    error(trace())
-                    return True
-                if message:
-                    sender.sendMessage(message)
-                return True
-
-        return handler
-
-    def execute(self, sender, command, label, args):
-        try:
-            return self.sub_commands[args[0].lower()].execute(sender, command, label, args[1:])
-        except (KeyError, IndexError):
-            return self.execute_checks(sender, command, label, args)
-
-    def execute_checks(self, sender, command, label, args):
-
-        # ---- Check sender type ----
-        if is_player(sender):
-            Validate.is_true(self.type != Command.SENDER_CONSOLE, "That command can only be used by the console")
-            #sender = py_players.__getattr__(sender)
-        else:
-            Validate.is_true(self.type != Command.SENDER_PLAYER, "That command can only be used by players")
-
-        # ---- Check permission ----
-        Validate.is_authorized(sender, self.permission)
-
-        # ---- Check if a help message is expected ----
-        if len(args) == 0:
-            action = self.no_arg_action
-        elif args[0].lower() == "help":
-            action = self.help_request_action
-        else:
-            action = Command.ACTION_IGNORE
-
-        if action != Command.ACTION_IGNORE:
-            if action == Command.ACTION_SYNTAXERROR:
-                return "&cInvalid syntax, please try again."
-            if action == Command.ACTION_DISPLAYSYNTAX:
-                return self.syntax()
-            if action == Command.ACTION_DISPLAYHELP:
-                return self.help()
-
-        # ---- Set up passed arguments, prepare for handler call ----
-        scape = Command_scape(args, self.arguments, command, label)
-        if is_player(sender):
-            #sender = py_players[sender]
-            pass
-
-        return self.handler(sender, self, scape)
-        # @Command("hello") def on_hello_command(sender, command, scape/args)
-
-    def syntax(self):
-        return " ".join(tuple(arg_info.syntax() for arg_info in self.arguments))
-
-    def help(self):
-        syntax = self.syntax()
-        return syntax #WIP...
-
-class Argument():
-
-    """
-      # A more advanced implementation of amin and amax, though it doesn't do exactly the same.
-      # You can now pass a list of Argument objects which define what the argument represents.
-      # In the process of doing so, you can set an argument type, one of the ones mentioned below.
-      # For example, if Argument.PLAYER is given, the server will be searched for the given player, and
-      # they will be passed as the argument, instead of a string representing their name.
-      #
-      # Feel free to add your own argument types. If you want to make a change to the API to make it different,
-      # please don't do so on your own behalf.
-    """
-
-    STRING = 0
-    INTEGER = 1
-    FLOAT = 2
-    PLAYER = 3
-    OFFLINE_PLAYER = 4
-    MESSAGE = 5
-
-    def __init__(self, name, type, definition, required = True):
-        self.name = name
-        self.type = type
-        self.definition = definition
-        self.required = required
-
-    def syntax(self):
-        syntax = self.name
-        if self.type == Argument.MESSAGE:
-            syntax += "..."
-        return (("<%s>" if self.required else "[%s]") % syntax)
-
-class Validate():
-
-    """
-      # Much like what you often see in Java.
-      # Instead of having to check if a condition is met, and if not,
-      # sending the player a message and returning true,
-      # You can use one of these methods to check the condition, and
-      # pass a message if it's not met.
-      #
-      # For example:
-      # > if not sender.hasPermission("utils.smth"):
-      #       noperm(sender)
-      #       return True
-      #
-      # Can be replaced with:
-      # > Validate.is_authorized(sender, "utils.smth")
-      #
-    """
-
-    @staticmethod
-    def is_true(expression, fail_message):
-        if not expression:
-            raise Command_exception(fail_message)
-
-    @staticmethod
-    def not_none(obj, fail_message):
-        if obj == None:
-            raise Command_exception(fail_message)
-
-    @staticmethod
-    def is_authorized(player, permission, msg = "You do not have permission to use that command"):
-        if not player.hasPermission(permission):
-            raise Command_exception(msg)
-
-    @staticmethod
-    def is_player(sender):
-        if not is_player(sender):
-            raise Command_exception("That command can only be used by players")
-
-    @staticmethod
-    def is_console(sender):
-        if is_player(sender):
-            raise Command_exception("That command can only be used by the console")
-
-"""
-  # ---------- API classes ----------
-"""
-
-class Command_dict(dict):
-    #{"cmd1" : cmd_object}
-    def __getattr__(self, alias):
-        for cmd_name, cmd_obj in self.iteritems():
-            if alias == cmd_name or alias in cmd_obj.aliases:
-                return cmd_obj
-        raise KeyError("Subcommand '%s' was not found" % alias)
-
-root_commands = Command_dict() # {"command": command_object}
-
-class Command_exception(Exception):
-
-    def __init__(self, message):
-        self.message = message
-
-class Command_scape(list):
-
-    def __init__(self, args, arg_layout, command, label):
-        super(list, self).__init__()
-        self.raw = args
-        self.arg_layout = arg_layout
-        self.command = command
-        self.label = label
-
-        has_message = False
-        for i in range(len(arg_layout)):
-            arg_info = arg_layout[i]
-
-            given = (len(args) >= i + 1)
-            if arg_info.required and not given:
-                raise Argument_exception("You must specify the " + arg_info.name)
-
-            if not given:
-                self.append(None)
-                continue
-
-            given_arg = args[i]
-            arg_type = arg_info.type
-
-            if arg_type == Argument.STRING:
-                self.append(given_arg)
-
-            elif arg_type == Argument.INTEGER:
-                try:
-                    value = int(given_arg)
-                except ValueError:
-                    raise Argument_exception("The %s has to be a round number" % arg_info.name)
-                self.append(value)
-
-            elif arg_type == Argument.FLOAT:
-                try:
-                    value = float(given_arg)
-                except ValueError:
-                    raise Argument_exception("The %s has to be a number" % arg_info.name)
-                self.append(value)
-
-            elif arg_type == Argument.PLAYER:
-                target = server.getPlayer(given_arg)
-                if target == None:
-                    raise Argument_exception("The %s has to be an online player" % arg_info.name)
-                self.append(target)
-                #self.append(py_players[target])
-
-            elif arg_type == Argument.OFFLINE_PLAYER:
-                try:
-                    # Code to get the PY PLAYER by name. Possibly, uid(server.getOfflinePlayer(given_arg)) can be used?
-                    pass
-                except KeyError:
-                    raise Argument_exception("The %s has to be an existing player" % arg_info.name)
-                self.append(None)
-
-            elif arg_type == Argument.MESSAGE:
-                self.append(" ".join(args[i:]))
-                has_message = True
-            else:
-                error("Argument type not found: %d" % arg_type)
-                raise Argument_exception("A weird thing has happened, please contact an administrator")
-
-        if not has_message:
-            self.remainder = args[len(arg_layout):]
-        else:
-            self.remainder = None
-
-    def has_flag(self, flag, check_all = False):
-        return (("-" + flag) in self.raw) if check_all else (("-" + flag) in self.remainder)
-
-    def get_raw(self):
-        return self.raw
-
-    def get_arg_layout(self):
-        return self.arg_layout
-
-class Argument_exception(Exception):
-
-    def __init__(self, message):
-        self.message = message
-

wrapper_event.py

@@ -1,56 +0,0 @@
-from wrapper import *
-from util_events import utils_event, utils_events
-from wrapper_player import *
-from traceback import format_exc as print_traceback
-
-class py_event(object):
-    def __init__(self,event):
-        self.event = event
-        try:
-            self.player = py_players[event.getPlayer()]
-        except:
-            warn("Player doesn't exist")
-
-    @property
-    def cancelled(self):
-        return self.event.isCancelled()
-
-    @cancelled.setter
-    def cancelled(self, value):
-        self.event.setCancelled(value)
-
-    @property
-    def message(self):
-        try:
-            return self.event.getMessage()
-        except:
-            raise AttributeError
-    
-    @message.setter
-    def message(self, msg):
-        try:
-            self.event.setMessage(msg)
-        except:
-            raise AttributeError
-
-def event_handler(event_name = None, priority = "normal", utils = False):
-    if not utils:
-        def decorator(wrapped):
-            @hook.event(event_name, priority)
-            def wrapper(event):
-                try:
-                    wrapped(py_event(event))
-                except:
-                    print(print_traceback())
-        return decorator
-    elif utils:
-        def decorator(wrapped):
-            @utils_event(event_name, priority, create_event = utils_events)
-            def wrapper(*args):
-                try:
-                    wrapped(*args)
-                except:
-                    print(print_traceback())
-        return decorator
-
-

wrapper_player.py

@@ -1,186 +0,0 @@
-import time
-import mysqlhack
-from mysql_utils import *
-from util_events import fire_event
-from thread_utils import *
-#from players_secret import *
-from datetime import datetime
-from com.ziclix.python.sql import zxJDBC
-from traceback import format_exc as print_traceback
-
-class py_player(object):
-    def __init__(self,player):
-        self.player = player
-        self.logging_in = True
-        self.authenticated = False
-        self.login_time = time.time()
-        
-        self.props = {
-            "uuid":self.uuid,
-            "name":self.name,
-            "nickname":self.name,
-            "registered":False,
-            "password":"None",
-            "banned":False,
-            "banned_reason":None,
-            "played_time":time.time() - self.login_time,
-            "last_login":datetime.now(),
-            "first_seen":datetime.now(),
-        }
-
-    def __setattr__(self, attribute, value):
-        if not attribute in dir(self):
-            if not 'props' in self.__dict__:
-                self.__dict__[attribute] = value
-            else:
-                self.props[attribute] = value
-        else:
-            object.__setattr__(self, attribute, value)
-
-    def __getattr__(self, attribute):
-        try:
-            return self.props[attribute]
-        except:
-            raise AttributeError("Attribute %s does not exist on this py_player" % attribute)
-            
-    def save(self):
-        properties = []
-        keys = []
-        columns = []
-
-        with mysql_connect() as sql:
-            columns = sql.columns
-        
-        for key, value in self.props.items():
-            if key not in columns:
-                with mysql_connect() as sql:
-                    if isinstance(value, int):
-                        sql.execute("ALTER TABLE utils_players ADD %s INT" % key)
-                    elif isinstance(value, str):
-                        sql.execute("ALTER TABLE utils_players ADD %s TEXT" % key)
-                    elif isinstance(value, bool):
-                        sql.execute("ALTER TABLE utils_players ADD %s TINYINT(1)" % key)
-            if key == "uuid":
-                continue
-            keys.append(key+"=?")
-            properties.append(value)
-            print value
-        properties.append(self.props["uuid"])
-        keys = str(tuple(keys)).replace("\'","").replace("(","").replace(")","")
-
-
-        with mysql_connect() as sql:
-            sql.execute("UPDATE utils_players set %s WHERE uuid = ?" % keys, properties)
-
-
-    def kick(self, kick_message = "You have been kicked from the server!"):
-        self.player.KickPlayer(kick_message)
-
-    def msg(self, message):
-        self.player.sendMessage(message)
-
-    @property
-    def name(self):
-        return self.player.getName()
-
-    @property
-    def uuid(self):
-        return str(self.player.getUniqueId())
- 
-    
-
-class Py_players:
-    def __init__(self):
-        self.players = []
-
-    def __len__(self):
-        return len(self.players)
-
-    def __getitem__(self, player):
-        for py_player in self.players:
-            if py_player.name == player.getName():
-                return py_player
-        else:
-            return None
-
-    def remove(self, player):
-        self.players.remove(player)
-
-    def append(self, player):
-        self.players.append(player)
-
-py_players = Py_players()
-
-@async(daemon=True)
-def fetch_player(player):
-    properties = []
-    keys = []
-    for key, value in player.props.iteritems():
-        keys.append(key)
-        properties.append(value)
-
-    with mysql_connect() as sql:
-        sql.execute("SELECT * FROM utils_players WHERE uuid = ?", (player.uuid,))
-        result = sql.fetchall()
-
-    if len(result) is 0:
-        with mysql_connect() as sql:
-            sql.execute("INSERT INTO utils_players %s \
-                VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?)" % str(tuple(keys)).replace("\'","")
-                ,args=tuple(properties))
-
-    elif len(result) is 1:
-        keys = []
-        props = result[0]
-        with mysql_connect() as sql:
-            sql.execute("SHOW COLUMNS FROM utils_players")
-            result = sql.fetchall()
-            for row in result:
-                keys.append(row[0]) 
-
-        for key in keys:
-            player.props[key] = props[keys.index(key)]
-
-        for prop in properties:
-            print str(prop)
-
-    else:
-        player.kick("Something went wrong while loading your player data, please contact an admin")
-        return
-    
-    player.logging_in = False
-    player.msg("You have succesfully logged into redstoner!")
-    fire_event("player_login", player)
-
-
-
-
-blocked_events = ["block.BlockBreakEvent", "block.BlockPlaceEvent", "player.PlayerMoveEvent",
-                    "player.AsyncPlayerChatEvent","player.PlayerTeleportEvent",
-                    "player.PlayerCommandPreprocessEvent", "player.PlayerInteractEvent"]
-
-for event in blocked_events:
-    @hook.event(event,"highest")
-    def on_blocked_event(event):
-        """player = py_players[event.getPlayer()]
-                                if player.logging_in:
-                                    event.setCancelled(True)"""
-        pass
-
-
-
-@hook.event("player.PlayerJoinEvent","lowest")
-def on_join(event):
-    try:
-        player = py_player(event.getPlayer())
-    except:
-        error(print_traceback())
-        time.sleep(10)
-    py_players.append(player)
-    player.msg("Your input will be blocked for a short while")
-    #fetch_player(player)
-
-
-@hook.event("player.PlayerQuitEvent","highest")
-def on_leave(event):
-    py_players.remove(py_players[event.getPlayer()])