fix: optimize lesson completion counting with prefetching in serializers

apps/course/serializers/course.py

@@ -248,6 +248,16 @@ class CourseDetailSerializer(serializers.ModelSerializer):
 
     def _get_completed_lessons_count(self, student, course):
         """Helper method to count completed lessons for the student in the given course."""
+        # Use prefetched completions if available
+        if hasattr(course, 'lessons') and course.lessons.all():
+            completed_count = 0
+            for lesson in course.lessons.all():
+                if hasattr(lesson, 'completions') and lesson.completions.all():
+                    if any(completion.student_id == student.id for completion in lesson.completions.all()):
+                        completed_count += 1
+            return completed_count
+
+        # Fallback to direct query if not prefetched
         return LessonCompletion.objects.filter(
             student=student,
             course_lesson__course=course
@@ -323,6 +333,16 @@ class MyCourseListSerializer(serializers.ModelSerializer):
 
     def _get_completed_lessons_count(self, student, course):
         """Helper method to count completed lessons for the student in the given course."""
+        # Use prefetched completions if available
+        if hasattr(course, 'lessons') and course.lessons.all():
+            completed_count = 0
+            for lesson in course.lessons.all():
+                if hasattr(lesson, 'completions') and lesson.completions.all():
+                    if any(completion.student_id == student.id for completion in lesson.completions.all()):
+                        completed_count += 1
+            return completed_count
+
+        # Fallback to direct query if not prefetched
         return LessonCompletion.objects.filter(
             student=student,
             course_lesson__course=course

config/test_auth_middleware.py

@@ -10,10 +10,10 @@ def test_auth_middleware(get_response):
     """
 
     def middleware(request):
-        # if "/admin/" not in request.path and request.META.get('HTTP_AUTHORIZATION') is None:
-        #     if request.user.is_authenticated and request.user.is_staff:
-        #         token, _ = Token.objects.get_or_create(user=request.user)
-        #         request.META['HTTP_AUTHORIZATION'] = "Token " + token.key
+        if "/admin/" not in request.path and request.META.get('HTTP_AUTHORIZATION') is None:
+            if request.user.is_authenticated and request.user.is_staff:
+                token, _ = Token.objects.get_or_create(user=request.user)
+                request.META['HTTP_AUTHORIZATION'] = "Token " + token.key
 
 
         # if "/swagger" in request.path or "/redoc" in request.path:
@@ -23,10 +23,10 @@ def test_auth_middleware(get_response):
         #             t, _ = Token.objects.get_or_create(user=user)
         #             request.META['HTTP_AUTHORIZATION'] = f"Token {t}"
 
-        user = User.objects.filter(email="[email protected]").first()
-        if user:
-            t, _ = Token.objects.get_or_create(user=user)
-            request.META['HTTP_AUTHORIZATION'] = f"Token {t}"
+        # user = User.objects.filter(email="[email protected]").first()
+        # if user:
+        #     t, _ = Token.objects.get_or_create(user=user)
+        #     request.META['HTTP_AUTHORIZATION'] = f"Token {t}"
 
         return get_response(request)