fix permissions and help

Author: Mapy542

Bot/Commands.txt

@@ -1,5 +1,5 @@
 command,permission,return  (perms = Admin, Trustee, User, Guest) Minnimum permission required to execute, return is posted to the channel. *par1* is the first user parameter after command and so on
-alwaysday,User,Alwaysday set to *par1
+alwaysday,Admin,Alwaysday set to *par1
 clear,Admin,Cleared inventory of *par1
 deop,Admin,Deopped *par1
 difficulty,Trustee,Set difficulty to *par1

Bot/Minceraft.py

@@ -1,5 +1,8 @@
 # bot.py
-import os, glob, shlex
+import os
+import glob
+import shlex
+import time
 try:
     import discord
 except ImportError:
@@ -9,12 +12,13 @@
 
 def Parsecommand(message, permission):
     command = shlex.split(message.content, posix=False)
-    #remove slash from command
+    # remove slash from command
     command[0] = command[0][1:]
-    #important this command is not common use as the return value is important
+    # important this command is not common use as the return value is important
     if command[0] == "list":
         global PATH
         minecraft_inject("list^M")
+        time.sleep(2)
         log = get_latest_file(str(PATH) + "/logs/*")
         try:
             with open(log, "r") as f:
@@ -31,42 +35,43 @@ def Parsecommand(message, permission):
             return "log file not found"
 
     elif command[0] == "help":
-        with open("Commands.txt", "r") as f:
-            text = f.read().splitlines()
-            text.pop(0)
-            return text
-
-    else:
         global COMMANDS
         global PERMISSIONS
+        text = ""
+        for i in range(len(COMMANDS)):
+            text = text + COMMANDS[i] + " - " + PERMISSIONS[i] + "\n"
+        return text
+
+    else:
         global RETURNS
         Permission_Ranking = ["Guest", "User", "Trustee", "Admin"]
-        
-        if not command[0] in COMMANDS: #checks for valid command
+
+        if not command[0] in COMMANDS:  # checks for valid command
             return "Command not found. Try /help"
         index = COMMANDS.index(command[0])
 
-        #check for acceptable permissions
-        if Permission_Ranking.index(PERMISSIONS[index]) >  Permission_Ranking.index(permission):
+        # check for acceptable permissions
+        if Permission_Ranking.index(PERMISSIONS[index]) > Permission_Ranking.index(permission):
             return "You do not have permission to use this command"
 
-        #actuate command
+        # actuate command
         minecraft_inject(" ".join(command) + "^M")
 
-        #return value
+        # return value
         returnval = RETURNS[index]
 
         returnval = returnval.split(" ")
         for i in range(len(returnval)):
             if "*par" in returnval[i]:
-                if(len(command) - 1 >= int(returnval[i][4:])):
+                if (len(command) - 1 >= int(returnval[i][4:])):
                     returnval[i] = command[int(returnval[i][4:])]
                 else:
                     returnval[i] = "default"
 
         returnval = " ".join(returnval)
         return returnval
 
+
 def minecraft_inject(message):
     os.system("screen -S Bedrock -p 0 -X stuff \"" + str(message) + "\"")
 
@@ -76,7 +81,8 @@ def get_latest_file(path):
     latest_file = max(list_of_files, key=os.path.getctime)
     return latest_file
 
-try: #pull oauth token from file and other constants data
+
+try:  # pull oauth token from file and other constants data
     with open("Creds.txt", "r") as f:
         data = f.read().splitlines()
         TOKEN = data[0].split("=")[1]
@@ -86,7 +92,7 @@ def get_latest_file(path):
 except FileNotFoundError:
     TOKEN = input("Enter token: ")
 
-try: #import commands and permissions
+try:  # import commands and permissions
     with open("Commands.txt", "r") as f:
         commands = f.read().splitlines()
         commands.pop(0)
@@ -106,11 +112,11 @@ def get_latest_file(path):
     commands = []
     print("Commands.txt not found")
 
-#initiate discord bot
+# initiate discord bot
 client = discord.Client(intents=discord.Intents.all())
 
 
-@client.event #on ready post to console
+@client.event  # on ready post to console
 async def on_ready():
     for guild in client.guilds:
         if guild.name == GUILD:
@@ -125,7 +131,7 @@ async def on_ready():
     print(f'Guild Members:\n - {members}')
 
 
-@client.event #on join guild DM them
+@client.event  # on join guild DM them
 async def on_member_join(member):
     await member.create_dm()
     await member.dm_channel.send(
@@ -134,18 +140,18 @@ async def on_member_join(member):
     await member.dm_channel.send("Join the server at: remotehost.ddns.net")
 
 
-@client.event #on message
-async def on_message(message): #ignore self
+@client.event  # on message
+async def on_message(message):  # ignore self
     if message.author == client.user:
         return
 
     if "/" in message.content:
         roles = [role.name for role in message.author.roles]
-        if("Admin" in roles):
+        if ("Admin" in roles):
             permission = 'Admin'
-        elif("Trustee" in roles):
+        elif ("Trustee" in roles):
             permission = 'Trustee'
-        elif("User" in roles):
+        elif ("User" in roles):
             permission = 'User'
         statement = Parsecommand(message, permission)
         if statement != None: