Flag attendance on wrong days

This makes it more clear when someone has e.g. submitted the register a
week early or late.

If people have submitted the same register more than once, we will use
one arbitrary submission, rather than looking for the most correct in
the list.

Author: illicitonion

src/course.rs

@@ -334,6 +334,9 @@ impl TraineeWithSubmissions {
                                 Attendance::Late { .. } => {
                                     numerator += 8;
                                 }
+                                Attendance::WrongDay { .. } => {
+                                    numerator += 3;
+                                }
                                 Attendance::Absent { .. } => {}
                             }
                         }
@@ -382,7 +385,7 @@ impl TraineeWithSubmissions {
                             Attendance::OnTime { .. } | Attendance::Late { .. } => {
                                 numerator += 1;
                             }
-                            Attendance::Absent { .. } => {}
+                            Attendance::Absent { .. } | Attendance::WrongDay { .. } => {}
                         }
                     }
                 }
@@ -435,6 +438,7 @@ impl Submission {
             Self::Attendance(Attendance::Absent { .. }) => String::from("Absent"),
             Self::Attendance(Attendance::OnTime { .. }) => String::from("On time"),
             Self::Attendance(Attendance::Late { .. }) => String::from("Late"),
+            Self::Attendance(Attendance::WrongDay { .. }) => String::from("Wrong day"),
             Self::PullRequest { pull_request } => format!("#{}", pull_request.number),
         }
     }
@@ -452,6 +456,7 @@ pub enum Attendance {
     Absent { register_url: String },
     OnTime { register_url: String },
     Late { register_url: String },
+    WrongDay { register_url: String },
 }
 
 impl Attendance {
@@ -460,6 +465,7 @@ impl Attendance {
             Attendance::Absent { register_url } => &register_url,
             Attendance::OnTime { register_url } => &register_url,
             Attendance::Late { register_url } => &register_url,
+            Attendance::WrongDay { register_url } => &register_url,
         }
     }
 }
@@ -649,7 +655,6 @@ fn get_trainee_module_attendance(
                     .collect::<Vec<chrono::NaiveDate>>();
                 let attendance = match dates.as_slice() {
                     [date] => {
-                        // TODO: Handle Cape Town
                         let start_time = DateTime::<Tz>::from_naive_utc_and_offset(
                             NaiveDateTime::new(
                                 date.clone(),

src/frontend.rs

@@ -139,6 +139,7 @@ impl TraineeBatchTemplate {
             Submission::Attendance(Attendance::Absent { .. }) => String::from("attendance-absent"),
             Submission::Attendance(Attendance::OnTime { .. }) => String::from("attendance-present"),
             Submission::Attendance(Attendance::Late { .. }) => String::from("attendance-late"),
+            Submission::Attendance(Attendance::WrongDay { .. }) => String::from("attendance-wrong-day"),
             Submission::PullRequest { pull_request } => match pull_request.state {
                 PrState::NeedsReview => "pr-needs-review".to_owned(),
                 PrState::Reviewed => "pr-reviewed".to_owned(),

src/register.rs

@@ -35,6 +35,9 @@ pub struct Attendance {
 
 impl Attendance {
     pub fn to_attendance_enum(&self, start_time: DateTime<Utc>) -> crate::course::Attendance {
+        if self.timestamp.date_naive() != start_time.date_naive() {
+            return crate::course::Attendance::WrongDay { register_url: self.register_url.clone() };
+        }
         let late_by = self.timestamp.signed_duration_since(start_time);
         if late_by.num_minutes() > 10 {
             crate::course::Attendance::Late {

templates/trainee-batch.html

@@ -14,7 +14,7 @@
             td.attendance-late {
                 background-color: #e6f4ae;
             }
-            td.attendance-unknown {
+            td.attendance-wrong-day {
                 background-color: grey;
             }
             td.pr-missing {