Skip to content

Commit 6fb36bf

Browse files
1ukastesarsolumath
authored andcommitted
fix: teacher listing not changed after update
1 parent 69135b6 commit 6fb36bf

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

cogs/fitwide/features.py

+5-5
Original file line numberDiff line numberDiff line change
@@ -107,16 +107,16 @@ async def get_teacher_perms_list(
107107
perms_list = None
108108

109109
if channel_teachers:
110-
perms_list = f"**===== {channel.name.upper()}: {channel.mention} =====**\n"
110+
perms_list = f"**===== {channel.name.upper()}: {channel.mention} =====**"
111111
for teacher in channel_teachers:
112+
perms_list += "\n"
112113
perms_list += f"- {teacher.mention} ({teacher.display_name})"
113114
user = ValidPersonDB.get_user_by_id(teacher.id)
114115
if user:
115116
# Get Full Name in reverse order
116117
last_name, first_name = user.name.split(" ")
117118
if first_name and last_name:
118119
perms_list += f" - {first_name} {last_name}"
119-
perms_list += "\n"
120120

121121
return perms_list
122122

@@ -142,14 +142,14 @@ async def update_teacher_info(
142142
async for message in teacher_info_channel.history():
143143
if channel.name.upper() in message.content:
144144
old_perms_list = message.content
145-
if new_perms_list:
145+
if new_perms_list is None: # Channel had listing but now it's empty
146+
await message.delete()
147+
elif new_perms_list != old_perms_list: # Channel had listing and it's different
146148
if message.author == channel.guild.me: # Can edit only my own messages
147149
await message.edit(content=new_perms_list)
148150
else: # If not mine, delete and resend
149151
await message.delete()
150152
await send_teacher_info(new_perms_list, teacher_info_channel)
151-
else: # Channel had listing but now it's empty
152-
await message.delete()
153153
return old_perms_list, new_perms_list
154154

155155
# Channel had no listing yet

0 commit comments

Comments
 (0)