Merge pull request #47 from ClassConnect-org/admin-refactor

filter per email admins

Author: joacoalvarez

src/app/controller/admin_controller.py

@@ -274,9 +274,10 @@ def get_admins(
     request: Request,
     skip: int = Query(0, ge=0, description="Number of records to skip"),
     limit: int = Query(10, ge=1, le=100, description="Max records to return"),
+    email: str = Query(None, description="Filter admins by email"),
 ):
     """
     Retrieve all admins from the database. Requires admin session.
     """
     service.get_current_session_admin(request)
-    return service.get_admins(skip=skip, limit=limit)
+    return service.get_admins(skip=skip, limit=limit, email=email)

src/app/service/admin_service.py

@@ -11,7 +11,6 @@
 
 from app.db.db import db
 from app.models.admin import AdminCreate, AdminLogin
-from app.schemas.admins import admins_entity
 
 logger = logging.getLogger(__name__)
 
@@ -94,10 +93,16 @@ def get_current_session_admin(self, request: Request):
 
         return session["email"]
 
-    def get_admins(self, skip: int = 0, limit: int = 10):
-        admins = admins_entity(self.admins_collection.find().skip(skip).limit(limit))
-        logger.info(f"[SUCCESSFUL] GET: get_admins.\n\tFetched {len(admins)} admins.")
-        total = self.admins_collection.count_documents({})
+    def get_admins(self, skip=0, limit=10, email=None):
+        query = {}
+        if email:
+            query["email"] = {"$regex": email, "$options": "i"}  # busqueda no-exacta
+        admins_cursor = self.admins_collection.find(query).skip(skip).limit(limit)
+        admins = []
+        for admin in admins_cursor:
+            admin["_id"] = str(admin["_id"])  # <- conversión
+            admins.append(admin)
+        total = self.admins_collection.count_documents(query)
         return {"total": total, "admins": admins}
 
     @staticmethod