refactor: Mejora de funciones de chat y optimizado la carga de mensajes, para que ahora sea bajo demanda y no cargen todos los mensajes al entrar al panel de administración.

Author: alejandroGM0

codigo/chat/_utils.py

@@ -2,7 +2,8 @@
 # Autor: Alejandro Gasca Mediel
 # ==========================================
 """
-Funciones de utilidad interna para la aplicación de chat."""
+Utilidades para la aplicación de chat.
+"""
 from django.contrib.auth.models import User
 from django.utils import timezone
 from .models import Message, Chat
@@ -24,12 +25,30 @@ def format_message_for_api(message: Message) -> dict:
         'date': message.created_at.strftime('%Y-%m-%d')
     }
 
-def get_messages_for_chat(chat: Chat) -> list:
+def get_messages_for_chat(chat):
     """
-    Obtiene todos los mensajes de un chat formateados para la API.
+    Obtiene los mensajes de un chat formateados para la API.
+    Solo se llama cuando se visualiza un chat específico.
     """
-    messages = chat.messages.all().order_by('created_at')
-    return [format_message_for_api(msg) for msg in messages]
+    messages = Message.objects.filter(chat=chat).order_by('created_at')
+    messages_data = []
+    
+    for message in messages:
+        # Format date as day/month/year
+        date_str = message.created_at.strftime('%d/%m/%Y')
+        # Format time as hour:minute
+        time_str = message.created_at.strftime('%H:%M')
+        
+        messages_data.append({
+            'id': message.id,
+            'sender': message.sender.username,
+            'content': message.content,
+            'timestamp': time_str,
+            'date': date_str,
+            'is_read': message.is_read,
+        })
+    
+    return messages_data
 
 def can_send_message(chat: Chat) -> bool:
     """

codigo/chat/public.py

@@ -1,12 +1,13 @@
-
-
-
+# ==========================================
+# Autor: Alejandro Gasca Mediel
+# ==========================================
 """
 API pública para la aplicación de chat.
 """
 from django.contrib.auth.models import User
 from django.utils import timezone
 from django.db.models import Q, Count
+from django.db.models import Count, Max, Subquery, OuterRef, F
 
 from .models import Chat, Message
 from accounts.public import update_last_activity
@@ -159,6 +160,67 @@ def filter_chats_by_criteria(search=None, chat_type=None, from_date=None, to_dat
                 key=lambda x: x['last_message'].created_at if x['last_message'] else chat.created_at,
                 reverse=True)
 
+def filter_chats_by_criteria_optimized(search=None, chat_type=None, from_date=None, to_date=None):
+    """
+    Versión optimizada de filter_chats_by_criteria que no carga todos los mensajes
+    sino solo datos esenciales para mostrar en la lista.
+    """    
+    
+    chats_query = Chat.objects.all()
+    
+    
+    if search:
+        matching_users = User.objects.filter(username__icontains=search)
+        chats_query = chats_query.filter(participants__in=matching_users).distinct()
+    
+    
+    if chat_type == 'ride':
+        chats_query = chats_query.filter(ride__isnull=False)
+    elif chat_type == 'direct':
+        chats_query = chats_query.filter(ride__isnull=True)
+    
+    
+    if from_date:
+        chats_query = chats_query.filter(created_at__gte=from_date)
+    if to_date:
+        chats_query = chats_query.filter(created_at__lte=to_date)
+    
+    
+    chats_query = chats_query.annotate(
+        messages_count=Count('messages', distinct=True),
+        has_ride=Count('ride', distinct=True)
+    )
+    
+    
+    last_message_subquery = Message.objects.filter(
+        chat=OuterRef('pk')
+    ).order_by('-created_at').values('id', 'created_at')[:1]
+    
+    chats_query = chats_query.annotate(
+        last_message_id=Subquery(last_message_subquery.values('id')),
+        last_message_date=Subquery(last_message_subquery.values('created_at'))
+    )
+    
+    
+    chats_query = chats_query.order_by('-last_message_date', '-created_at')
+    
+    
+    result = []
+    for chat in chats_query:
+        
+        last_message = None
+        if chat.last_message_id:
+            last_message = Message.objects.get(id=chat.last_message_id)
+        
+        result.append({
+            'chat': chat,
+            'messages_count': chat.messages_count,
+            'last_message': last_message,
+            'is_ride_chat': chat.has_ride > 0
+        })
+    
+    return result
+
 def get_chat_stats():
     """
     Obtiene estadísticas generales sobre los chats para el panel de administración.

codigo/dashboard/_utils.py

@@ -1,6 +1,8 @@
 # ==========================================
 # Autor: Alejandro Gasca Mediel
 # ==========================================
+
+
 """
 Utilidades para la aplicación de dashboard
 """
@@ -11,16 +13,14 @@
 from django.db.models.functions import TruncDate, TruncMonth, ExtractHour
 from django.utils import timezone
 
-
+    
 from rides.public import get_rides_in_period, get_ride_stats_for_dashboard, get_recently_published_rides
 from chat.public import get_messages_in_period
 from reports.public import get_reports_in_period
 from accounts.public import get_users_in_period, get_active_users
 from payments.public import get_payments_in_period
-from chat.public import filter_chats_by_criteria, get_chat_stats
+from chat.public import filter_chats_by_criteria_optimized, get_chat_stats
 from django.core.paginator import Paginator
-import json
-from django.core.serializers.json import DjangoJSONEncoder
 from reports.public import (
         get_hourly_report_counts, 
         get_daily_report_counts, 
@@ -296,7 +296,7 @@ def get_messages_data(period=DEFAULT_PERIOD):
         'total': total_messages,
         'labels': labels,
         'data': data,
-        'avg_per_day': round(total_messages / max(1, days_in_period)),  # Redondear a entero, sin el segundo parámetro
+        'avg_per_day': round(total_messages / max(1, days_in_period)),  
         'period': period,
         'period_text': period_text
     }
@@ -1027,7 +1027,8 @@ def get_chat_management_context(request):
         'to_date': request.GET.get('to_date', ''),
     }
 
-    chats_data = filter_chats_by_criteria(
+    
+    chats_data = filter_chats_by_criteria_optimized(
         search=filters['search'],
         chat_type=filters['chat_type'],
         from_date=filters['from_date'],
@@ -1044,10 +1045,20 @@ def get_chat_management_context(request):
 
     stats = get_chat_stats()
 
+    
+    chart_data = {
+        'daily_messages': json.dumps(stats['daily_messages'], cls=DjangoJSONEncoder),
+        'chat_distribution': json.dumps([
+            {'type': 'Chats de viaje', 'count': stats['ride_chats']},
+            {'type': 'Chats directos', 'count': stats['direct_chats']}
+        ], cls=DjangoJSONEncoder)
+    }
+    
     return {
         'chats': chats,
         'filters': filters,
         'stats': stats,
+        'chart_data': chart_data,
         'active_section': 'chats',
         'now': timezone.now()
     }