Update modules/database/sql_db.py

modules/database/sql_db.py

@@ -8,6 +8,7 @@ import uuid
 
 logger = logging.getLogger(__name__)
 
+#########################################
 def get_user(username, role=None):
     container = get_container("users")
     try:
@@ -20,15 +21,23 @@ def get_user(username, role=None):
         logger.error(f"Error al obtener usuario {username}: {str(e)}")
         return None
 
+
+#########################################
 def get_admin_user(username):
     return get_user(username, role='Administrador')
 
+
+#########################################
 def get_student_user(username):
     return get_user(username, role='Estudiante')
 
+
+#########################################
 def get_teacher_user(username):
     return get_user(username, role='Profesor')
 
+
+#########################################
 def create_user(username, password, role, additional_info=None):
     """Crea un nuevo usuario"""
     container = get_container("users")
@@ -55,15 +64,19 @@ def create_user(username, password, role, additional_info=None):
         logger.error(f"Error al crear usuario {role}: {str(e)}")
         return False
 
+#########################################
 def create_student_user(username, password, additional_info=None):
     return create_user(username, password, 'Estudiante', additional_info)
 
+#########################################
 def create_teacher_user(username, password, additional_info=None):
     return create_user(username, password, 'Profesor', additional_info)
 
+#########################################
 def create_admin_user(username, password, additional_info=None):
     return create_user(username, password, 'Administrador', additional_info)
 
+#########################################
 def record_login(username):
     """Registra el inicio de sesión de un usuario"""
     try:
@@ -89,6 +102,7 @@ def record_login(username):
         logger.error(f"Error registrando login: {str(e)}")
         return None
 
+#########################################
 def record_logout(username, session_id):
     """Registra el cierre de sesión y calcula la duración"""
     try:
@@ -130,6 +144,7 @@ def record_logout(username, session_id):
         logger.error(f"Error registrando logout: {str(e)}")
         return False
 
+#########################################
 def get_recent_sessions(limit=10):
     """Obtiene las sesiones más recientes"""
     try:
@@ -169,6 +184,7 @@ def get_recent_sessions(limit=10):
         logger.error(f"Error obteniendo sesiones recientes: {str(e)}")
         return []
 
+#########################################
 def get_user_total_time(username):
     """Obtiene el tiempo total que un usuario ha pasado en la plataforma"""
     try:
@@ -194,6 +210,7 @@ def get_user_total_time(username):
         logger.error(f"Error obteniendo tiempo total: {str(e)}")
         return 0
 
+#########################################
 def update_student_user(username, new_info):
     container = get_container("users") 
     try:
@@ -211,6 +228,7 @@ def update_student_user(username, new_info):
         logger.error(f"Error al actualizar información del estudiante {username}: {str(e)}")
         return False
 
+#########################################
 def delete_student_user(username):
     container = get_container("users")
     try:
@@ -227,9 +245,18 @@ def delete_student_user(username):
         logger.error(f"Error al eliminar estudiante {username}: {str(e)}")
         return False
 
+#########################################
 def store_application_request(name, lastname, email, institution, current_role, desired_role, reason):
-    container = get_container("application_requests")
+    """Almacena una solicitud de aplicación"""
     try:
+        # Obtener el contenedor usando get_container() que sí funciona
+        container = get_container("application_requests")
+        if not container:
+            logger.error("No se pudo obtener el contenedor de solicitudes")
+            return False
+
+        # Crear documento con la solicitud
+        # Nótese que incluimos email como partition key en el cuerpo del documento
         application_request = {
             "id": str(uuid.uuid4()),
             "name": name,
@@ -240,31 +267,56 @@ def store_application_request(name, lastname, email, institution, current_role,
             "desired_role": desired_role,
             "reason": reason,
             "requestDate": datetime.utcnow().isoformat(),
-            "partitionKey": email  # Usar email como partition key
+            # El campo para partition key debe estar en el documento
+            "partitionKey": email
         }
-        container.create_item(body=application_request)
-        logger.info(f"Solicitud de aplicación almacenada para el email: {email}")
+        
+        # Crear el item en el contenedor - sin el parámetro enable_cross_partition_query
+        container.create_item(
+            body=application_request  # Solo pasamos el body
+        )
+        logger.info(f"Solicitud de aplicación almacenada para: {email}")
         return True
+        
     except Exception as e:
         logger.error(f"Error al almacenar la solicitud de aplicación: {str(e)}")
+        logger.error(f"Detalles del error: {str(e)}")
         return False
 
+
+################################################################
 def store_student_feedback(username, name, email, feedback):
-    container = get_container("user_feedback")
+    """Almacena el feedback de un estudiante"""
     try:
+        # Obtener el contenedor - verificar disponibilidad
+        logger.info(f"Intentando obtener contenedor user_feedback para usuario: {username}")
+        container = get_container("user_feedback")
+        if not container:
+            logger.error("No se pudo obtener el contenedor user_feedback")
+            return False
+
+        # Crear documento de feedback - asegurar que el username esté como partition key
         feedback_item = {
             "id": str(uuid.uuid4()),
-            "username": username,
+            "username": username,  # Campo regular
             "name": name,
             "email": email,
             "feedback": feedback,
             "role": "Estudiante",
             "timestamp": datetime.now(timezone.utc).isoformat(),
-            "partitionKey": username  # Usar username como partition key
+            "partitionKey": username  # Campo de partición
         }
-        result = container.create_item(body=feedback_item)
-        logger.info(f"Feedback de estudiante almacenado con ID: {result['id']} para el usuario: {username}")
+
+        # Crear el item - sin el parámetro enable_cross_partition_query
+        logger.info(f"Intentando almacenar feedback para usuario: {username}")
+        result = container.create_item(
+            body=feedback_item  # Solo el body, no parámetros adicionales
+        )
+        
+        logger.info(f"Feedback almacenado exitosamente para el usuario: {username}")
         return True
+
     except Exception as e:
-        logger.error(f"Error al almacenar el feedback del estudiante {username}: {str(e)}")
+        logger.error(f"Error al almacenar el feedback del estudiante {username}")
+        logger.error(f"Detalles del error: {str(e)}")
         return False