diff options
author | Dico200 <dico.karssiens@gmail.com> | 2016-05-25 03:05:59 +0200 |
---|---|---|
committer | Dico200 <dico.karssiens@gmail.com> | 2016-05-25 03:05:59 +0200 |
commit | 9a1006e7114e9ed63b67e1197a3ec934a9089814 (patch) | |
tree | e655e403e3cd5bebe05a3f57804831100b9785fe | |
parent | 29e7ce174b5f353d79773f1e73f047bb0007acbc (diff) |
Imbusy command: status <player> tweaked, cleaned up return statements
-rw-r--r-- | imbusy.py | 26 |
1 files changed, 11 insertions, 15 deletions
@@ -44,49 +44,45 @@ def on_busy_command(sender, cmd, label, args): msg(sender, "/busy on: turns on busy mode") msg(sender, "/busy off: turns off busy mode") msg(sender, "/busy status [player]: shows your or [player]'s current busy status.") - return True elif len(args) == 1: if args[0] == "on": if sender.getName() in busy_players: msg(sender, "You cannot be even more focused than this without being a jedi!") - return True - busy_players.append(sender.getName()) - broadcast(None, "&c[&2Busy&c] &fNow busy: %s&f, don't even TRY bothering them!" % sender.getDisplayName()) - return True + else: + busy_players.append(sender.getName()) + broadcast(None, "&c[&2Busy&c] &fNow busy: %s&r, don't even TRY bothering them!" % sender.getDisplayName()) elif args[0] == "off": try: busy_players.remove(sender.getName()) - msg(sender, "Master has sent /busy command, %s&f is freeee of bothering!" % sender.getDisplayName()) - return True + msg(sender, "Master has sent /busy command, %s&r is freeee of bothering!" % sender.getDisplayName()) except ValueError: msg(sender, "You are not busy! You cannot be even less busy! Are you perhaps bored?") - return True elif args[0] == "status": if sender.getName() in busy_players: msg(sender, "You are super-duper busy and concentrated right now. Think, think, think!") - return True else: msg(sender, "You are completely unable to focus right now.") - return True else: unclear(sender) return False elif len(args) == 2 and args[0] == "status": - if args[1] in busy_players: - msg(sender, "Yes, %s is busy. Shhh..." % args[1]) - return True + target = server.getPlayer(args[1]) + if target is None: + msg(sender, "That player is not online, I doubt they are busy.") + elif target.getName() in busy_players: + msg(sender, "Yes, %s&r is busy. Shhh..." % target.getDisplayName()) else: - msg(sender, "No, you're good. Feel free to chat with %s!" % args[1]) - return True + msg(sender, "No, you're good. Feel free to chat with %s&r!" % target.getDisplayName()) else: unclear(sender) return False + return True @hook.event("player.PlayerCommandPreprocessEvent", "monitor") |