feat(admin): add /movergrupo command to recategorize groups

- Created /movergrupo <name> command for admins
- Displays an inline keyboard to choose the new Listable category
- Added 'MoverGrupo' callback logic to update the group's type directly in DB

Author: rozen03

bot_logic.py

@@ -1,7 +1,7 @@
 from utils.orga2Utils import noitip, asm
 from handlers.basic import start, help_command, estasvivo, colaborar
 from handlers.info import campusvivo, flan, flanviejo, aulas, cuandovence, listarlabos
-from handlers.admin import checodepers, checodeppers, sugerirNoticia, get_logs, joder
+from handlers.admin import checodepers, checodeppers, sugerirNoticia, get_logs, joder, movergrupo
 from handlers.groups import listar, listaroptativa, listareci, listarotro, cubawiki, agregargrupo, agregaroptativa, agregarotros, agregareci, sugerirgrupo, sugeriroptativa, sugerireci, sugerirotro, actualizar_grupos, _update_groups
 from handlers.callbacks import button
 from handlers.crons import felizdia, actualizarRiver
@@ -11,6 +11,10 @@
         'handler': joder,
         'description': '(Admin) Enviar un mensaje al grupo general'
     },
+    'movergrupo': {
+        'handler': movergrupo,
+        'description': '(Admin) Cambiar la categoría de un grupo'
+    },
     'logs': {
         'handler': get_logs,
         'description': '(Admin) Revisa los últimos 10 errores en GCP.'

handlers/admin.py

@@ -29,6 +29,55 @@ async def joder(update: Update, context: ContextTypes.DEFAULT_TYPE):
         logger.error(f"Failed to send joder message: {e}")
         await update.message.reply_text(f"Error al enviar el mensaje: {e}")
 
+async def movergrupo(update: Update, context: ContextTypes.DEFAULT_TYPE):
+    user_id = update.effective_user.id
+    if user_id not in admin_ids and str(user_id) not in admin_ids:
+        logger.warning(f"Unauthorized user {user_id} tried to access /movergrupo")
+        return
+        
+    if not context.args:
+        await update.message.reply_text("Uso: /movergrupo <nombre exacto del grupo>")
+        return
+        
+    name = " ".join(context.args)
+    with get_session() as session:
+        from models import Listable
+        groups = session.query(Listable).filter(Listable.name.ilike(f"%{name}%")).all()
+        
+        if not groups:
+            await update.message.reply_text("No se encontró ningún grupo con ese nombre.")
+            return
+            
+        if len(groups) > 1:
+            names = "\n".join([f"- {g.name} ({g.type})" for g in groups])
+            await update.message.reply_text(f"Se encontraron múltiples grupos:\n{names}\n\nPor favor, sé más específico.")
+            return
+            
+        group = groups[0]
+        
+        keyboard = [
+            [
+                InlineKeyboardButton("Grupo (Oblig.)", callback_data=f"MoverGrupo|{group.id}|Grupo"),
+                InlineKeyboardButton("GrupoOptativa", callback_data=f"MoverGrupo|{group.id}|GrupoOptativa"),
+                InlineKeyboardButton("ECI", callback_data=f"MoverGrupo|{group.id}|ECI")
+            ],
+            [
+                InlineKeyboardButton("Otro", callback_data=f"MoverGrupo|{group.id}|Otro"),
+                InlineKeyboardButton("GrupoOtros", callback_data=f"MoverGrupo|{group.id}|GrupoOtros"),
+                InlineKeyboardButton("Obligatoria (viejo)", callback_data=f"MoverGrupo|{group.id}|Obligatoria"),
+            ],
+            [
+                InlineKeyboardButton("Optativa (viejo)", callback_data=f"MoverGrupo|{group.id}|Optativa"),
+                InlineKeyboardButton("Cancelar", callback_data=f"MoverGrupo|{group.id}|Cancelar")
+            ]
+        ]
+        reply_markup = InlineKeyboardMarkup(keyboard)
+        await update.message.reply_text(
+            f"Seleccioná la nueva categoría para el grupo:\n\n*Nombre:* {group.name}\n*Categoría Actual:* {group.type}",
+            reply_markup=reply_markup,
+            parse_mode=ParseMode.MARKDOWN
+        )
+
 async def checodepers(update: Update, context: ContextTypes.DEFAULT_TYPE):
     if not context.args:
         ejemplo = """ Ejemplo de uso:

handlers/callbacks.py

@@ -9,7 +9,12 @@ async def button(update: Update, context: ContextTypes.DEFAULT_TYPE):
     query = update.callback_query
     await query.answer()
     message = query.message
-    buttonType, id_val, action = query.data.split("|")
+    
+    # Parse button data safely
+    data_parts = query.data.split("|")
+    buttonType = data_parts[0]
+    id_val = data_parts[1] if len(data_parts) > 1 else None
+    action = data_parts[2] if len(data_parts) > 2 else None
 
     with get_session() as session:
         if buttonType == "Listable":
@@ -24,6 +29,23 @@ async def button(update: Update, context: ContextTypes.DEFAULT_TYPE):
                 await query.edit_message_text(text=message.text + action_text)
             else:
                 await query.edit_message_text(text=message.text + "\n[Botón huérfano: El grupo ya no existe en la base de datos]")
+                
+        elif buttonType == "MoverGrupo":
+            if action == "Cancelar":
+                await query.edit_message_text(text=f"❌ Operación cancelada para el grupo ID {id_val}.")
+                return
+                
+            group = session.query(Listable).filter_by(id=int(id_val)).first()
+            if group:
+                old_type = group.type
+                new_type = action
+                # Update the polymorphic identity directly via the type column
+                group.type = new_type
+                # Depending on SQLAlchemy, modifying the polymorphic column directly might require 
+                # a raw SQL update or session.flush() to be safe. We'll rely on SQLAlchemy's update.
+                await query.edit_message_text(text=f"✅ Grupo *{group.name}* movido exitosamente:\nDe `{old_type}` ➡️ a `{new_type}`", parse_mode=ParseMode.MARKDOWN)
+            else:
+                await query.edit_message_text(text=message.text + "\n[Botón huérfano: El grupo ya no existe]")
 
         elif buttonType == "Noticia":
             noticia = session.query(Noticia).filter_by(id=int(id_val)).first()