Merge pull request #201 from PROCOLLAB-github/feature/trajectories

Переработана система доступов к Навыкам

Author: Toksi86

apps/courses/migrations/0018_alter_skill_free_access_alter_task_free_access.py

@@ -0,0 +1,27 @@
+# Generated by Django 5.0.3 on 2025-04-03 08:32
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("courses", "0017_skill_is_from_trajectory_skill_requires_subscription"),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name="skill",
+            name="free_access",
+            field=models.BooleanField(
+                default=True, help_text="Возможность проходить без подписки.", verbose_name="Доступ бесплатно"
+            ),
+        ),
+        migrations.AlterField(
+            model_name="task",
+            name="free_access",
+            field=models.BooleanField(
+                default=True, help_text="Возможность проходить без подписки.", verbose_name="Доступ бесплатно"
+            ),
+        ),
+    ]

apps/courses/models.py

@@ -23,7 +23,7 @@ class AbstractStatusField(models.Model):
         verbose_name="Статус",
     )
     free_access = models.BooleanField(
-        default=False,
+        default=True,
         null=False,
         blank=False,
         verbose_name="Доступ бесплатно",
@@ -93,8 +93,13 @@ def __str__(self):
         return f"{self.name}"
 
     def clean(self):
-        from django.core.exceptions import ValidationError
-
+        if not any(
+            [self.is_from_trajectory, self.requires_subscription, self.free_access]
+        ):
+            raise ValidationError(
+                "Навык должен иметь хотя бы одно из значений: "
+                "'Из траектории', 'Требует подписку' или 'Бесплатный доступ'."
+            )
         if self.free_access and (self.requires_subscription or self.is_from_trajectory):
             raise ValidationError(
                 "Навык не может быть одновременно бесплатным и требовать подписку или принадлежать траектории."

apps/courses/serializers.py

@@ -45,7 +45,7 @@ class Meta:
 
 class SkillsBasicSerializer(serializers.ModelSerializer):
     # Access the link field from the related FileModel
-    file_link = serializers.URLField(source="file.link")
+    file_link = serializers.SerializerMethodField()
     # Просьба захардкодить, статичный 1 уровень для всех навыков
     quantity_of_levels = serializers.SerializerMethodField()
 
@@ -65,6 +65,11 @@ def get_quantity_of_levels(self, obj) -> int:
         # Просьба захардкодить, статичный 1 уровень для всех навыков
         return 1
 
+    def get_file_link(self, obj):
+        if obj.file and obj.file.link:
+            return obj.file.link
+        return None
+
 
 class SkillsDoneSerializer(SkillsBasicSerializer):
     is_done = serializers.BooleanField()

apps/courses/services.py

@@ -7,21 +7,25 @@
                                get_user_available_week)
 
 
-def get_stats(skill: Skill, profile_id: int, request_user: CustomUser | None = None) -> GetStatsDict:
+def get_stats(
+    skill: Skill, profile_id: int, request_user: CustomUser | None = None
+) -> GetStatsDict:
     available_week, user = get_user_available_week(profile_id)
-    skill_status_filter = DBObjectStatusFilters().get_skill_status_for_user(request_user)
+    skill_status_filter = DBObjectStatusFilters().get_skill_status_for_user(
+        request_user
+    )
     available_week: int = available_week if not skill.free_access else 4
 
     tasks_of_skill: QuerySet[Task] = (
-        Task.available
-        .only_awailable_weeks(available_week, user)
+        Task.available.only_awailable_weeks(available_week, user)
         .prefetch_related(
             Prefetch(
                 "task_objects__user_results",
                 queryset=TaskObjUserResult.objects.filter(user_profile_id=profile_id),
                 to_attr="filtered_user_results",
             )
-        ).prefetch_related("task_objects")
+        )
+        .prefetch_related("task_objects")
         .annotate(task_objects_count=Count("task_objects"))
         .filter(skill_id=skill.id)
         .filter(skill_status_filter)
@@ -31,7 +35,9 @@ def get_stats(skill: Skill, profile_id: int, request_user: CustomUser | None = N
     user_done_task_objects: list[int] = []
     all_task_objects: list[int] = []
     for task in tasks_of_skill:
-        user_results_count = sum(1 for obj in task.task_objects.all() if obj.filtered_user_results)
+        user_results_count = sum(
+            1 for obj in task.task_objects.all() if obj.filtered_user_results
+        )
         user_done_task_objects.append(user_results_count)
         all_task_objects.append(task.task_objects_count)
         data.append(
@@ -44,7 +50,9 @@ def get_stats(skill: Skill, profile_id: int, request_user: CustomUser | None = N
                 "free_access": task.free_access,
             }
         )
-    statuses = get_rounded_percentage(sum(user_done_task_objects), sum(all_task_objects))
+    statuses = get_rounded_percentage(
+        sum(user_done_task_objects), sum(all_task_objects)
+    )
     if data:
         try:
             data.sort(key=lambda x: x["level"])
@@ -54,7 +62,9 @@ def get_stats(skill: Skill, profile_id: int, request_user: CustomUser | None = N
     if skill.free_access:
         stats_of_weeks = []
     else:
-        stats_of_weeks: list[WeekStatsDict] = get_stats_of_weeks(skill, profile_id, available_week, request_user)
+        stats_of_weeks: list[WeekStatsDict] = get_stats_of_weeks(
+            skill, profile_id, available_week, request_user
+        )
 
     new_data = {"progress": statuses, "tasks": data, "stats_of_weeks": stats_of_weeks}
     return new_data
@@ -78,25 +88,28 @@ def get_stats_of_weeks(
             "is_done": week.is_done,
             "done_on_time": bool(week.additional_points) if week.is_done else None,
         }
-        for week in UserWeekStat.objects.filter(user_profile__id=profile_id, skill__id=skill.id)
+        for week in UserWeekStat.objects.filter(
+            user_profile__id=profile_id, skill__id=skill.id
+        )
     ]
 
     # TODO Временная мера(2 строчки ниже), как будет нормальное деление по неделям, необходимо убрать.
     # Выбор из минимально доступной недели и максимальной имеющейся у навыка (сейчас на проде все задания - 1неделя)
     # Если не убрать ничего не сломается, но лишний запрос.
     max_skill_week: dict[str:int] = (
-        Task.published
-        .for_user(request_user)
+        Task.published.for_user(request_user)
         .filter(skill__id=skill.id)
         .aggregate(Max("week"))
     )
-    available_week = min(available_week, max_skill_week["week__max"])
+
+    if max_skill_week["week__max"] is not None:
+        available_week = min(available_week, max_skill_week["week__max"])
 
     # Запись в `UserWeekStat` формируется при прохождении, но если юзер вообще не проходил,
     # необходимо дополнить список недостающими неделями (доступными ему сейчас).
     if len(stats_of_weeks) != available_week:
         existing_weeks: set[int] = {week["week"] for week in stats_of_weeks}
-        for week in range(1, available_week+1):
+        for week in range(1, available_week + 1):
             if week not in existing_weeks:
                 stats_of_weeks.append(
                     {

apps/courses/urls.py

@@ -1,10 +1,10 @@
 from django.urls import path
 
-from courses.views import (DoneSkillsList, SkillDetails, SkillsList, TaskList,
-                           TasksOfSkill, TaskStatsGet)
+from courses.views import (DoneSkillsList, SkillDetails, SkillsList,
+                           TaskDetail, TasksOfSkill, TaskStatsGet)
 
 urlpatterns = [
-    path("<int:task_id>", TaskList.as_view(), name="task_list"),
+    path("<int:task_id>", TaskDetail.as_view(), name="task_detail"),
     path("all-skills/", SkillsList.as_view(), name="all-skills"),
     path("choose-skills/", DoneSkillsList.as_view(), name="choose-skills"),
     path("skill-details/<int:skill_id>", SkillDetails.as_view(), name="skill-details"),