release: dev → main (dashboard primitives migration + paper exam UX refresh)

.gitignore

@@ -87,8 +87,12 @@ celerybeat.pid
 
 # Database
 *.sql
+*.sql.gz
+*.dump
+*.backup
 *.sqlite
 *.db
+backups/
 artifacts/db_backups/
 
 # Migrations (optional - uncomment if you want to ignore migrations)

backend/apps/contests/exporters/data_service.py

@@ -105,11 +105,12 @@ def get_participants(self) -> List[ParticipantDTO]:
 
         result = []
         for p in participants:
+            profile = getattr(p.user, 'profile', None)
             result.append(ParticipantDTO(
                 user_id=p.user.id,
                 username=p.user.username,
                 email=getattr(p.user, 'email', ''),
-                nickname=getattr(p, 'nickname', '') or '',
+                display_name=getattr(profile, 'display_name', '') or '',
                 exam_status=getattr(p, 'exam_status', ''),
                 started_at=getattr(p, 'started_at', None),
                 left_at=getattr(p, 'left_at', None),
@@ -178,7 +179,7 @@ def calculate_standings(self, user_id: Optional[int] = None) -> StandingsDTO:
             stats[p.user_id] = UserStandingDTO(
                 user_id=p.user_id,
                 username=p.username,
-                display_name=p.nickname or p.username,
+                display_name=p.display_name or p.username,
                 problems=problem_stats,
             )
 

backend/apps/contests/exporters/dto.py

@@ -74,7 +74,7 @@ class ParticipantDTO:
     user_id: int
     username: str
     email: str = ''
-    nickname: str = ''
+    display_name: str = ''
     exam_status: str = ''
     started_at: Optional[datetime] = None
     left_at: Optional[datetime] = None

backend/apps/contests/migrations/0077_remove_contest_anonymous_mode.py

@@ -0,0 +1,19 @@
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("contests", "0076_unique_exam_question_order"),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name="contest",
+            name="anonymous_mode_enabled",
+        ),
+        migrations.RemoveField(
+            model_name="contestparticipant",
+            name="nickname",
+        ),
+    ]

backend/apps/contests/models.py

@@ -215,13 +215,6 @@ class QuestionEditLockTrigger(models.TextChoices):
         help_text='TA 完成批改後手動設為 True，學生才能查看成績'
     )
 
-    # Anonymous mode settings
-    anonymous_mode_enabled = models.BooleanField(
-        default=False,
-        verbose_name='啟用匿名模式',
-        help_text='啟用後學生可使用暱稱參與競賽，排行榜和提交列表顯示暱稱'
-    )
-
     created_at = models.DateTimeField(auto_now_add=True, verbose_name='建立時間')
     updated_at = models.DateTimeField(auto_now=True, verbose_name='更新時間')
 
@@ -483,15 +476,6 @@ class ContestParticipant(models.Model):
         help_text='手動或系統自動交卷原因'
     )
 
-    # Anonymous mode nickname
-    nickname = models.CharField(
-        max_length=50,
-        blank=True,
-        default='',
-        verbose_name='暱稱',
-        help_text='匿名模式下顯示的名稱，預設為用戶名'
-    )
-
     # Explicit exam state (primary state field for UI)
     exam_status = models.CharField(
         max_length=20,

backend/apps/contests/serializers.py

@@ -95,7 +95,6 @@ class ContestDetailSerializer(serializers.ModelSerializer):
     accepted_at = serializers.SerializerMethodField()
     submitted_at = serializers.SerializerMethodField()
     auto_unlock_at = serializers.SerializerMethodField()
-    my_nickname = serializers.SerializerMethodField()
     problems = serializers.SerializerMethodField()
     participant_count = serializers.SerializerMethodField()
     admins = serializers.SerializerMethodField()
@@ -139,7 +138,6 @@ class Meta:
             'created_at',
             'updated_at',
             # Computed fields
-            'my_nickname',
             'current_user_role',
             'permissions',
             'has_joined',
@@ -159,7 +157,6 @@ class Meta:
             'max_cheat_warnings',
             'allow_auto_unlock',
             'auto_unlock_minutes',
-            'anonymous_mode_enabled',
             'participant_count',
             'admins',
             'is_classroom_bound',
@@ -193,11 +190,6 @@ def _get_current_registration(self, obj):
             cache[cache_key] = obj.registrations.filter(user=user).first()
         return cache[cache_key]
 
-    def get_my_nickname(self, obj):
-        """Get nickname for current user."""
-        registration = self._get_current_registration(obj)
-        return registration.nickname if registration else None
-
     def get_lock_reason(self, obj):
         """Get lock reason for current user."""
         registration = self._get_current_registration(obj)
@@ -414,7 +406,6 @@ class Meta:
             'allow_auto_unlock',
             'auto_unlock_minutes',
             'status',
-            'anonymous_mode_enabled',
             'results_published',
         ]
         read_only_fields = ['id']
@@ -864,25 +855,8 @@ class Meta:
         read_only_fields = ['author', 'status', 'answered_at', 'author_username', 'author_display_name', 'problem_title']
 
     def get_author_display_name(self, obj):
-        """根據匿名模式返回適當的作者名稱"""
-        request = self.context.get('request')
-        contest = obj.contest
-
-        if not contest.anonymous_mode_enabled:
-            return obj.author.username
-
-        # 管理者可見真實名稱
-        if request and request.user.is_authenticated:
-            from .permissions import get_user_role_in_contest
-            role = get_user_role_in_contest(request.user, contest)
-            if role in ['admin', 'owner', 'teacher']:
-                return obj.author.username
-
-        # 查找該用戶的暱稱
-        participant = ContestParticipant.objects.filter(
-            contest=contest, user=obj.author
-        ).first()
-        return participant.nickname if participant else obj.author.username
+        profile = getattr(obj.author, 'profile', None)
+        return getattr(profile, 'display_name', '') or obj.author.username
 
 
 class ClarificationCreateSerializer(serializers.ModelSerializer):
@@ -1070,9 +1044,7 @@ class ContestParticipantSerializer(serializers.ModelSerializer):
     user = UserSerializer(read_only=True)
     user_id = serializers.IntegerField(source='user.id', read_only=True)
     username = serializers.CharField(source='user.username', read_only=True)
-    nickname = serializers.CharField(read_only=True)
     display_name = serializers.SerializerMethodField()
-    user_display_name = serializers.SerializerMethodField()
     account_role = serializers.CharField(source='user.role', read_only=True)
     auth_provider = serializers.CharField(source='user.auth_provider', read_only=True)
     total_score = serializers.SerializerMethodField()
@@ -1090,7 +1062,7 @@ class Meta:
             'user_id', 'username', 'user', 'score', 'total_score', 'rank', 
             'joined_at', 'exam_status',
             'lock_reason', 'violation_count', 'submit_reason', 'auto_unlock_at', 'remaining_unlock_seconds',
-            'nickname', 'display_name', 'user_display_name', 'account_role', 'auth_provider',
+            'display_name', 'account_role', 'auth_provider',
             'connection_status', 'last_heartbeat_at', 'live_monitoring_online', 'live_monitoring_sources',
         ]
 
@@ -1180,23 +1152,8 @@ def get_total_score(self, obj):
         return total
 
     def get_display_name(self, obj):
-        """根據權限返回適當的顯示名稱"""
-        request = self.context.get('request')
-        contest = obj.contest
-
-        # 非匿名模式，直接返回真實用戶名
-        if not contest.anonymous_mode_enabled:
-            return obj.user.username
-
-        # 管理者可見真實名稱
-        if request and request.user.is_authenticated:
-            from .permissions import get_user_role_in_contest
-            role = get_user_role_in_contest(request.user, contest)
-            if role in ['admin', 'owner', 'teacher']:
-                return obj.user.username
-
-        # 其他用戶看暱稱 (nickname 預設為 username)
-        return obj.nickname or obj.user.username
+        profile = getattr(obj.user, 'profile', None)
+        return getattr(profile, 'display_name', '') or ""
 
     def get_auto_unlock_at(self, obj):
         """Calculate auto unlock time if applicable."""
@@ -1218,13 +1175,6 @@ def get_remaining_unlock_seconds(self, obj):
 
         return int((unlock_at - now).total_seconds())
 
-    def get_user_display_name(self, obj):
-        profile = getattr(obj.user, 'profile', None)
-        if not profile:
-            return ""
-        return profile.display_name or ""
-
-
 # ============================================================================
 # Exam Answer Serializers
 # ============================================================================
@@ -1258,7 +1208,7 @@ class ExamAnswerDetailSerializer(serializers.ModelSerializer):
     )
     participant_user_id = serializers.SerializerMethodField()
     participant_username = serializers.SerializerMethodField()
-    participant_nickname = serializers.SerializerMethodField()
+    participant_display_name = serializers.SerializerMethodField()
 
     class Meta:
         model = ExamAnswer
@@ -1268,7 +1218,7 @@ class Meta:
             'answer', 'is_correct', 'score', 'feedback',
             'question_snapshot',
             'graded_by_username', 'graded_at',
-            'participant_user_id', 'participant_username', 'participant_nickname',
+            'participant_user_id', 'participant_username', 'participant_display_name',
             'created_at', 'updated_at',
         ]
         read_only_fields = fields
@@ -1304,15 +1254,16 @@ def get_participant_user_id(self, obj):
     def get_participant_username(self, obj):
         return obj.participant.user.username
 
-    def get_participant_nickname(self, obj):
-        return obj.participant.nickname or obj.participant.user.username
+    def get_participant_display_name(self, obj):
+        profile = getattr(obj.participant.user, 'profile', None)
+        return getattr(profile, 'display_name', '') or ""
 
 
 class ExamAnswerGradingSerializer(serializers.ModelSerializer):
     """Slim serializer for grading screens.
 
     Drops redundant per-row duplicates (question_prompt/type/options/explanation/
-    max_score/question_snapshot and participant_username/nickname). Consumers
+    max_score/question_snapshot and participant_username/display_name). Consumers
     should join question info via GET /exam-questions/ and participant info via
     contest participants list — both are O(题数) / O(学生数) rather than
     O(answers)."""

backend/apps/contests/services/exam_validation.py

@@ -27,10 +27,10 @@ def validate_exam_operation(contest, user, require_in_progress=False, allow_admi
     if allow_admin_bypass and can_manage_contest(user, contest):
         try:
             participant = ContestParticipant.objects.get(contest=contest, user=user)
-            return participant, None
+            return participant
         except ContestParticipant.DoesNotExist:
             # Managers don't need to be registered
-            return None, None
+            return None
 
     # Layer 1: Contest status
     if contest.status != 'published':

backend/apps/contests/services/export_service.py

@@ -163,7 +163,8 @@ def build_paper_exam_results_csv_response(contest):
     participant_count = len(students)
 
     for p in students:
-        display_name = p.nickname or p.user.username
+        profile = getattr(p.user, 'profile', None)
+        display_name = getattr(profile, 'display_name', '') or ''
         status_label = p.get_exam_status_display()
         p_answers = answer_lookup.get(p.id, {})
         graded_count = sum(

backend/apps/contests/services/participant_dashboard.py

@@ -79,9 +79,7 @@ def _serialize_participant(participant: ContestParticipant) -> dict[str, Any]:
     return {
         "user_id": participant.user_id,
         "username": participant.user.username,
-        "nickname": participant.nickname or participant.user.username,
-        "display_name": participant.nickname or participant.user.username,
-        "user_display_name": getattr(profile, "display_name", "") or "",
+        "display_name": getattr(profile, "display_name", "") or "",
         "account_role": getattr(participant.user, "role", "student"),
         "auth_provider": getattr(participant.user, "auth_provider", "email"),
         "email": getattr(participant.user, "email", ""),

backend/apps/contests/services/scoreboard.py

@@ -88,7 +88,6 @@ def calculate(contest: Contest, user_scope: ScoreboardScope) -> ScoreboardResult
             stats[participant.user.id] = {
                 "user": UserSerializer(participant.user).data,
                 "display_name": display_name,
-                "nickname": participant.nickname,
                 "solved": 0,
                 "rank": participant.rank,
                 "score": participant.score,
@@ -177,11 +176,5 @@ def _build_display_name(
         is_privileged: bool,
         use_export_display: bool,
     ) -> str:
-        if use_export_display:
-            return participant.nickname or participant.user.username
-
-        if not contest.anonymous_mode_enabled:
-            return participant.user.username
-        if is_privileged:
-            return participant.user.username
-        return participant.nickname or participant.user.username
+        profile = getattr(participant.user, "profile", None)
+        return getattr(profile, "display_name", "") or participant.user.username

backend/apps/contests/tests/exam/test_exam_answers.py

@@ -441,6 +441,38 @@ def test_teacher_view_dashboard_summary(self):
         self.assertEqual(essay_summary['subjective_stats']['graded_count'], 1)
         self.assertEqual(essay_summary['subjective_stats']['pending_count'], 1)
 
+    def test_participant_can_view_dashboard_summary_after_results_published(self):
+        self.contest.results_published = True
+        self.contest.save(update_fields=['results_published'])
+        ExamAnswer.objects.create(
+            participant=self.participant,
+            question=self.q_single,
+            answer={'selected': 'B'},
+            is_correct=True,
+            score=5,
+        )
+
+        self.client.force_authenticate(user=self.student)
+        url = reverse(
+            'contests:contest-exam-answers-dashboard-summary',
+            kwargs={'contest_pk': self.contest.id},
+        )
+        resp = self.client.get(url)
+
+        self.assertEqual(resp.status_code, status.HTTP_200_OK)
+        self.assertEqual(resp.data['contest']['participant_count'], 1)
+        self.assertEqual(resp.data['score_distribution'][-1]['range_label'], '90-100%')
+
+    def test_participant_cannot_view_dashboard_summary_before_results_published(self):
+        self.client.force_authenticate(user=self.student)
+        url = reverse(
+            'contests:contest-exam-answers-dashboard-summary',
+            kwargs={'contest_pk': self.contest.id},
+        )
+        resp = self.client.get(url)
+
+        self.assertEqual(resp.status_code, status.HTTP_403_FORBIDDEN)
+
     def test_dashboard_summary_cache_invalidates_on_submission(self):
         self.client.force_authenticate(user=self.teacher)
         summary_url = reverse(

backend/apps/contests/tests/exam/test_exam_state.py

@@ -38,6 +38,17 @@ def test_start_exam(self):
 
         p = ContestParticipant.objects.get(user=self.user, contest=self.contest)
         self.assertEqual(p.exam_status, ExamStatus.IN_PROGRESS)
+
+    def test_owner_participant_can_start_exam(self):
+        ContestParticipant.objects.create(contest=self.contest, user=self.admin)
+        self.client.force_authenticate(user=self.admin)
+
+        url = reverse('contests:contest-exam-start-exam', args=[self.contest.id])
+        response = self.client.post(url)
+
+        self.assertEqual(response.status_code, status.HTTP_200_OK)
+        p = ContestParticipant.objects.get(user=self.admin, contest=self.contest)
+        self.assertEqual(p.exam_status, ExamStatus.IN_PROGRESS)
 
     def test_end_exam(self):
         # Start first