fix: use attribute instead of item getter; better admin filtering

src/bot/routers/admin.py

@@ -22,34 +22,31 @@ async def __call__(self, event: TelegramObject, event_from_user: User) -> bool |
         if self._status is None:
             return {"status": user.status}
 
-        if user["status"] == self._status:
+        if user.status == self._status:
             return True
         return False
 
 
-@router.message(Command("admin"), StatusFilter())
-async def enable_admin_mode(message: types.Message, bot: Bot, status: str):
-    if status == UserStatus.LORD:
-        text = "You are the Lord of the Music Room! You can use the following commands:"
+@router.message(Command("admin"), StatusFilter(UserStatus.LORD))
+async def enable_admin_mode(message: types.Message, bot: Bot):
+    text = "You are the Lord of the Music Room! You can use the following commands:"
 
-        for command in admin_commands:
-            text += f"\n{command.command} - {command.description}"
+    for command in admin_commands:
+        text += f"\n{command.command} - {command.description}"
 
-        await message.answer(text)
-        await bot.set_my_commands(
-            bot_commands + admin_commands,
-            scope=BotCommandScopeChat(chat_id=message.from_user.id),
-        )
-    else:
-        await bot.set_my_commands(bot_commands, scope=BotCommandScopeChat(chat_id=message.from_user.id))
+    await message.answer(text)
+    await bot.set_my_commands(
+        bot_commands + admin_commands,
+        scope=BotCommandScopeChat(chat_id=message.from_user.id),
+    )
 
 
-@router.message(Command("export_users"))
+@router.message(Command("export_users"), StatusFilter(UserStatus.LORD))
 async def export_users(message: types.Message):
     response = await api_client.export_users(message.from_user.id)
     if response:
         bytes_, filename = response
         document = BufferedInputFile(bytes_, filename)
         await message.answer_document(document, caption="Here is the list of users.")
     else:
-        await message.answer("You don't have access to this command.")
+        await message.answer("Failed to export users.")