Merge pull request #24 from DevOps-Cloud-Team5/SCRUM-90-Course-info

course page + attendence stats

Author: Imafe

api/models.py

@@ -68,10 +68,10 @@ def __str__(self):
         return str(self.name)
 
     def get_enrolled_students(self):
-        return [uc.user for uc in UserCourse.objects.filter(user__role=AccountRoles.STUDENT)]
+        return [uc.user for uc in UserCourse.objects.filter(user__role=AccountRoles.STUDENT, course=self)]
 
     def get_teachers(self):
-        return [uc.user for uc in UserCourse.objects.filter(user__role=AccountRoles.TEACHER)]
+        return [uc.user for uc in UserCourse.objects.filter(user__role=AccountRoles.TEACHER, course=self)]
 
     def get_lectures(self):
         return CourseLecture.objects.filter(course=self)

api/serializers.py

@@ -24,7 +24,12 @@ class UserSerializer(serializers.ModelSerializer):
     class Meta:
         model = User
         fields = '__all__'
-        
+
+class CourseUserSerializer(serializers.ModelSerializer):
+    class Meta:
+        model = User
+        fields = ["username", "first_name", "last_name", "role"]
+
 class CourseSerializer(serializers.ModelSerializer):
     class Meta:
         model = Course

api/urls.py

@@ -20,6 +20,7 @@
 from .views import (
                     AddLectureView,
                     GetCourseLecturesView,
+                    GetFullCoursePage,
                     GetLectureView,
                     GetScheduleView,
                     MassEnrollCourseView,
@@ -75,9 +76,10 @@
     path('course/delete/<pk>', DestroyCourseView.as_view(), name='course_delete'),
     path('course/enroll/<pk>', EnrollCourseView.as_view(), name='course_enroll'),
     path('course/mass_enroll/<pk>', MassEnrollCourseView.as_view(), name='course_mass_enroll'),
-    path('course/get/<pk>', GetCourseByName.as_view(), name='course_get'),
+    path('course/get/<pk>', GetFullCoursePage.as_view(), name='course_get'),
     path('course/getall/', GetCoursesAll.as_view(), name='course_getall'),
 
+    
     path('course/lecture/<pk>/get', GetCourseLecturesView.as_view(), name='course_get_lecture'),
     path('course/lecture/<pk>/add', AddLectureView.as_view(), name='course_add_lecture'),
     # path('course/lecture/<pk>/update', GetCourseByName.as_view(), name='lecture_add'), # TODO

api/views.py

@@ -16,7 +16,10 @@
 
 from .permissions import IsTeacher, IsAdmin, IsStudent
 from .models import Course, AccountRoles, CourseLecture
-from .serializers import AddLectureSerializer, CustomTokenSerializer, CreateUserSerializer, LectureSerializer, MassEnrollSerializer, SetAttendenceTeacherSerializer, UserSerializer, CourseCreateSerializer, CourseSerializer, MailTestSerializer
+
+from .serializers import AddLectureSerializer, CourseUserSerializer, CustomTokenSerializer, CreateUserSerializer, LectureSerializer, MassEnrollSerializer, SetAttendenceTeacherSerializer, UserSerializer, CourseCreateSerializer, CourseSerializer
+
+
 
 from django.core.mail import send_mail
 
@@ -216,6 +219,48 @@ def get(self, _, pk):
         serializer = self.serializer_class(queryset[0])
         return Response(serializer.data, status=status.HTTP_200_OK)
 
+class GetFullCoursePage(generics.RetrieveAPIView):
+    authentication_classes = [JWTAuthentication]
+    permission_classes = [IsStudent]
+    
+    queryset = Course.objects.all()
+    serializer_class = CourseSerializer
+
+    def get_attendence_stats(self, course, user):
+        lectures : List[CourseLecture] = course.get_lectures()
+        attendence_stats = {"attended": 0, "missed": 0}
+        for lecture in lectures:
+            if lecture.end_time > datetime.datetime.now(lecture.end_time.tzinfo): continue
+            att = lecture.get_attendence_user(user)
+            if att is None or not (att.attended_student and att.attended_teacher):
+                attendence_stats["missed"] += 1
+            else: attendence_stats["attended"] += 1
+        return attendence_stats
+    
+    def get(self, request, pk):
+        queryset = self.get_queryset().filter(pk=pk)
+        if not queryset:
+            return Response({"error": f"course id '{pk}' not found"}, status=status.HTTP_404_NOT_FOUND)
+        
+        course : Course = queryset[0]
+        teachers = course.get_teachers()
+        students = course.get_enrolled_students()
+        
+        response_data = {}
+        response_data["id"] = pk
+        response_data["course_name"] = course.course_name
+        response_data["num_teachers"] = len(teachers)
+        response_data["num_students"] = len(students)
+        response_data["attended"] = -1
+        response_data["missed"] = -1
+        response_data["users"] = CourseUserSerializer((teachers + students), many=True).data
+    
+        user = User.objects.all().filter(username=request.user.username)[0]
+        if user.role == AccountRoles.STUDENT: 
+            response_data |= self.get_attendence_stats(course, user)
+        
+        return Response(response_data, status=status.HTTP_200_OK)
+
 class GetCoursesAll(generics.ListAPIView):
     authentication_classes = [JWTAuthentication]
     permission_classes = [IsStudent]
@@ -413,4 +458,5 @@ def get(self, request, year, week):
         # Sort chronological order
         all_lectures.sort(key= lambda x : datetime.datetime.fromisoformat(x["start_time"]))
 
-        return Response(all_lectures)
+        return Response(all_lectures, status=status.HTTP_200_OK)
+