From ebc9d65e419cc70d12fc40bb0134255d5b39f497 Mon Sep 17 00:00:00 2001 From: Kevin Zheng Date: Sat, 4 Jun 2022 15:13:46 -0700 Subject: [PATCH] Add error handling to 'status' command --- python/guilds/guildoracle.py | 23 +++++++++++++---------- 1 file changed, 13 insertions(+), 10 deletions(-) diff --git a/python/guilds/guildoracle.py b/python/guilds/guildoracle.py index 812bd7f80..ffb501e8f 100644 --- a/python/guilds/guildoracle.py +++ b/python/guilds/guildoracle.py @@ -118,20 +118,23 @@ if (Access ==1) or (isDM == 1): else: message = 'Usage "demote "' elif text[0] == 'status': - record = guild.info(text[1]) - if record: - if len(text)==3: - if guild.change_status(text[1],text[2]): - record = guild.info(text[1]) #refresh record - message = '%s now has status of %s' %(text[1], record['Status']) + if len(text) > 1: + record = guild.info(text[1]) + if record: + if len(text)==3: + if guild.change_status(text[1],text[2]): + record = guild.info(text[1]) #refresh record + message = '%s now has status of %s' %(text[1], record['Status']) + else: + status = ', '.join(guild.status) + message = '%s is not a valid status, valid values are %s.' %(text[2],status) else: status = ', '.join(guild.status) - message = '%s is not a valid status, valid values are %s.' %(text[2],status) + message = 'Current status for %s is %s.\nTo change use "status %s " where status is one of %s.' %(text[1],record['Status'],text[1],status) else: - status = ', '.join(guild.status) - message = 'Current status for %s is %s.\nTo change use "status %s " where status is one of %s.' %(text[1],record['Status'],text[1],status) + message = '%s is not a member' %text[1] else: - message = '%s is not a member' %text[1] + message = "Say status to query the member's status." elif text[0] == 'add' and isDM: if len(text)==2: if log.info(text[1]):