feat: modes and calendar times for attendance code

Author: TigerCodeHub

intranet/apps/eighth/migrations/0072_auto_20250428_1239.py renamed to intranet/apps/eighth/migrations/0072_auto_20250429_1128.py

@@ -1,4 +1,4 @@
-# Generated by Django 3.2.25 on 2025-04-28 16:39
+# Generated by Django 3.2.25 on 2025-04-29 15:28
 
 from django.db import migrations, models
 import intranet.apps.eighth.models
@@ -34,4 +34,14 @@ class Migration(migrations.Migration):
             field=models.CharField(default=intranet.apps.eighth.models.EighthScheduledActivity.random_code, max_length=6),
         ),
         migrations.RunPython(generate_attendance_codes, reverse_code=migrations.RunPython.noop),
+        migrations.AddField(
+            model_name='eighthscheduledactivity',
+            name='code_mode',
+            field=models.IntegerField(choices=[(0, 'Auto'), (1, 'Open'), (2, 'Closed')], default=0),
+        ),
+        migrations.AddField(
+            model_name='historicaleighthscheduledactivity',
+            name='code_mode',
+            field=models.IntegerField(choices=[(0, 'Auto'), (1, 'Open'), (2, 'Closed')], default=0),
+        ),
     ]

intranet/apps/eighth/models.py

@@ -1,8 +1,8 @@
 # pylint: disable=too-many-lines; Allow more than 1000 lines
 import datetime
 import logging
-import string
 import random
+import string
 from collections.abc import Sequence
 from typing import Collection, Iterable, List, Optional, Union
 
@@ -826,6 +826,11 @@ class EighthScheduledActivity(AbstractBaseEighthModel):
         attendance_code
             Random 6-character code of digits and uppercase letters to check attendance
             Distinct for each 8th activity-block
+        code_mode
+            Whether the activity is automatically enabling/disabling the attendance code based on the daily schedule
+            Is an integer:
+            0 = Automatic; 1 = Open; 2 = Closed
+            default = 0
         special
             Whether this scheduled instance of the activity is special. If
             not set, falls back on the EighthActivity's special setting.
@@ -847,16 +852,20 @@ class EighthScheduledActivity(AbstractBaseEighthModel):
     )
 
     def random_code():
-        return ''.join(random.choices(string.ascii_uppercase + string.digits, k=6))
+        return "".join(random.choices(string.ascii_uppercase + string.digits, k=6))
 
-    attendance_code = models.CharField(max_length=6,default=random_code)
-    #Warning: Must manually fix codes for pre-existing entires
+    attendance_code = models.CharField(max_length=6, default=random_code)
+    mode_choices = [
+        (0, "Auto"),
+        (1, "Open"),
+        (2, "Closed"),
+    ]
+    code_mode = models.IntegerField(choices=mode_choices, default=0)
 
     admin_comments = models.CharField(max_length=1000, blank=True)
     title = models.CharField(max_length=1000, blank=True)
     comments = models.CharField(max_length=1000, blank=True)
 
-
     # Overridden attributes
     sponsors = models.ManyToManyField(EighthSponsor, blank=True)
     rooms = models.ManyToManyField(EighthRoom, blank=True)
@@ -1206,8 +1215,9 @@ def set_sticky_students(self, users: "Sequence[AbstractBaseUser]") -> None:
                 bcc=True,
             )
 
-    def get_attendance_code(self):
-        return self.attendance_code
+    def set_code_mode(self, mode):
+        self.code_mode = mode
+        self.save(update_fields=["code_mode"])
 
     @transaction.atomic  # This MUST be run in a transaction. Do NOT remove this decorator.
     def add_user(
@@ -1652,8 +1662,6 @@ def __str__(self):
         return f"{self.activity}{suff} on {self.block}{cancelled_str}"
 
 
-
-
 class EighthSignupManager(Manager):
     """Model manager for EighthSignup."""
 

intranet/apps/eighth/urls.py

@@ -20,7 +20,7 @@
     re_path(r"^/absences$", attendance.eighth_absences_view, name="eighth_absences"),
     re_path(r"^/absences/(?P<user_id>\d+)$", attendance.eighth_absences_view, name="eighth_absences"),
     re_path(r"^/glance$", signup.eighth_location, name="eighth_location"),
-    re_path(r"^/student_attendance$", signup.student_attendance, name="student_attendance"),
+    re_path(r"^/student_attendance$", attendance.student_attendance_view, name="student_attendance"),
     # Teachers
     re_path(r"^/attendance$", attendance.teacher_choose_scheduled_activity_view, name="eighth_attendance_choose_scheduled_activity"),
     re_path(r"^/attendance/(?P<scheduled_activity_id>\d+)$", attendance.take_attendance_view, name="eighth_take_attendance"),

intranet/apps/eighth/views/attendance.py

@@ -1,6 +1,7 @@
 import csv
 import io
 import logging
+from datetime import time as tm
 from html import escape
 
 from cacheops import invalidate_obj
@@ -25,12 +26,14 @@
 from ....utils.date import get_date_range_this_year
 from ...auth.decorators import attendance_taker_required, deny_restricted, eighth_admin_required
 from ...dashboard.views import gen_sponsor_schedule
+from ...schedule.models import Day
 from ...schedule.views import decode_date
 from ..forms.admin.activities import ActivitySelectionForm
 from ..forms.admin.blocks import BlockSelectionForm
 from ..models import EighthActivity, EighthBlock, EighthScheduledActivity, EighthSignup, EighthSponsor, EighthWaitlist
 from ..tasks import email_scheduled_activity_students_task
 from ..utils import get_start_date
+from .signup import shift_time
 
 logger = logging.getLogger(__name__)
 
@@ -304,13 +307,20 @@ def take_attendance_view(request, scheduled_activity_id):
                 status=403,
             )
 
+        if "att_code_mode" in request.POST:
+            selected_mode = int(request.POST["att_code_mode"])
+            if scheduled_activity.code_mode != selected_mode:
+                scheduled_activity.set_code_mode(selected_mode)
+                redirect_url = reverse(url_name, args=[scheduled_activity.id])
+                return redirect(redirect_url)
+
         if not scheduled_activity.block.locked and request.user.is_eighth_admin:
             messages.success(request, "Note: Taking attendance on an unlocked block.")
 
-        present_user_ids = list(request.POST.keys()) #list of member.ids checked off
+        present_user_ids = list(request.POST.keys())  # list of member.ids checked off
 
         if request.FILES.get("attendance"):
-            #csv attendance
+            # csv attendance
             try:
                 csv_file = request.FILES["attendance"].read().decode("utf-8")
                 data = csv.DictReader(io.StringIO(csv_file))
@@ -335,6 +345,9 @@ def take_attendance_view(request, scheduled_activity_id):
             except (csv.Error, ValueError, KeyError, IndexError):
                 messages.error(request, "Could not interpret file. Did you upload a Google Meet attendance report without modification?")
 
+        if "att_code_mode" in present_user_ids:
+            present_user_ids.remove("att_code_mode")
+
         csrf = "csrfmiddlewaretoken"
         if csrf in present_user_ids:
             present_user_ids.remove(csrf)
@@ -417,7 +430,7 @@ def take_attendance_view(request, scheduled_activity_id):
             "show_checkboxes": (scheduled_activity.block.locked or request.user.is_eighth_admin),
             "show_icons": (scheduled_activity.block.locked and scheduled_activity.block.attendance_locked() and not request.user.is_eighth_admin),
             "bbcu_script": settings.BBCU_SCRIPT,
-            "attendance_code": scheduled_activity.get_attendance_code(),
+            "is_sponsor": scheduled_activity.user_is_sponsor(request.user),
         }
 
         if request.user.is_eighth_admin:
@@ -757,3 +770,75 @@ def email_students_view(request, scheduled_activity_id):
     context = {"scheduled_activity": scheduled_activity}
 
     return render(request, "eighth/email_students.html", context)
+
+
+@login_required
+@deny_restricted
+def student_attendance_view(request):
+    blocks = EighthBlock.objects.get_blocks_today()
+    attc = None
+    attf = None
+    attimef = None
+    atteachf = None
+    if request.method == "POST":
+        now = timezone.localtime()
+        dayblks = Day.objects.select_related("day_type").get(date=now).day_type.blocks.all()
+        for blk in blocks:
+            blklet = blk.block_letter
+            code = request.POST.get(blklet)
+            if code is None:
+                continue
+            act = request.user.eighthscheduledactivity_set.get(block=blk)
+            if act.get_code_mode_display() == "Auto":
+                dayblk = None
+                for bk in dayblks:
+                    name = bk.name
+                    if name is None:
+                        continue
+                    if blklet in name and "8" in name:
+                        dayblk = bk
+                        break
+                if dayblk is None:
+                    attimef = blk
+                    break
+                start_time = shift_time(tm(hour=dayblk.start.hour, minute=dayblk.start.minute), -20)
+                end_time = shift_time(tm(hour=dayblk.end.hour, minute=dayblk.end.minute), 20)
+                if not start_time <= now.time() <= end_time:
+                    attimef = blk
+                    break
+            elif act.get_code_mode_display() == "Closed":
+                atteachf = blk
+                break
+            code = code.upper()
+            if code == act.attendance_code:
+                present = EighthSignup.objects.filter(scheduled_activity=act, user__in=[request.user.id])
+                present.update(was_absent=False)
+                attc = blk
+                for s in present:
+                    invalidate_obj(s)
+                act.attendance_taken = True
+                act.save()
+                invalidate_obj(act)
+                break
+            else:
+                attf = blk
+                break
+    if blocks:
+        sch_acts = []
+        for b in blocks:
+            try:
+                act = request.user.eighthscheduledactivity_set.get(block=b)
+                if act.activity.name != "z - Hybrid Sticky":
+                    sch_acts.append([b, act, ", ".join([r.name for r in act.get_true_rooms()]), ", ".join([s.name for s in act.get_true_sponsors()])])
+
+            except EighthScheduledActivity.DoesNotExist:
+                sch_acts.append([b, None])
+        response = render(
+            request,
+            "eighth/student_submit_attendance.html",
+            context={"sch_acts": sch_acts, "attc": attc, "attf": attf, "attimef": attimef, "atteachf": atteachf},
+        )
+    else:
+        messages.error(request, "There are no eighth period blocks scheduled today.")
+        response = redirect("index")
+    return response