Move color detection from models.py to template and use type detection instead

aleksis/apps/chronos/models.py

@@ -334,12 +334,12 @@ class LessonSubstitution(models.Model):
             raise ValidationError(_("Lessons can only be either substituted or cancelled."))
 
     @property
-    def color(self):
+    def type(self):
+        # TODO: Add cases events and supervisions
         if self.cancelled:
-            return "green"
-        # Add cases for purple and blue (events and supervisements)
+            return "cancellation"
         else:
-            return "black"
+            return "substitution"
 
     class Meta:
         unique_together = [["lesson_period", "week"]]

aleksis/apps/chronos/templates/chronos/substitution.html

@@ -78,7 +78,7 @@
       </td>
     {% endif %}
     {% for sub in substitutions %}
-      <tr class="{{ sub.color }}-text">
+      <tr class="{% if sub.type == "cancellation" %}green-text{% else %}black-text{% endif %}"> {# TODO: Extend support for blue and purple (supervisions and events) #}
         <td>
           {% include "chronos/partials/groups.html" with groups=sub.lesson_period.lesson.groups.all %}
         </td>