Intervention model refactoring

* adds UserActionLogEntry model to user/models.py
  * wraps user and timestamp info
  * can be extended for more information in the future
* refactors all filtering and accessing on values

intervention/filters.py

@@ -117,7 +117,7 @@ class InterventionTableFilter(django_filters.FilterSet):
         """
         if not value:
             return queryset.filter(
-                recorded_on=None,
+                recorded=None,
             )
         else:
             return queryset

intervention/models.py

@@ -15,6 +15,7 @@ from intervention.settings import INTERVENTION_IDENTIFIER_LENGTH, INTERVENTION_I
 from konova.models import BaseObject, Geometry
 from konova.utils.generators import generate_random_string
 from organisation.models import Organisation
+from user.models import UserActionLogEntry
 
 
 class Intervention(BaseObject):
@@ -32,27 +33,35 @@ class Intervention(BaseObject):
     geometry = models.ForeignKey(Geometry, null=True, blank=True, on_delete=models.SET_NULL)
     documents = models.ManyToManyField("konova.Document", blank=True)
 
-    # Checks
-    checked_on = models.DateTimeField(default=None, null=True, blank=True)
-    checked_by = models.ForeignKey(User, null=True, blank=True, on_delete=models.SET_NULL, related_name='+')
-
     # Refers to "zugelassen am"
     registration_date = models.DateField(null=True, blank=True)
 
     # Refers to "Bestandskraft am"
     binding_on = models.DateField(null=True, blank=True)
 
+    # Checks - Refers to "Genehmigen" but optional
+    checked = models.OneToOneField(
+        UserActionLogEntry,
+        on_delete=models.SET_NULL,
+        null=True,
+        blank=True,
+        help_text="Holds data on user and timestamp of this action",
+        related_name="+"
+    )
+
     # Refers to "verzeichnen"
-    recorded_on = models.DateTimeField(default=None, null=True, blank=True)
-    recorded_by = models.ForeignKey(User, null=True, blank=True, on_delete=models.SET_NULL, related_name='+')
+    recorded = models.OneToOneField(
+        UserActionLogEntry,
+        on_delete=models.SET_NULL,
+        null=True,
+        blank=True,
+        help_text="Holds data on user and timestamp of this action",
+        related_name="+"
+    )
 
     # Holds which intervention is simply a newer version of this dataset
     next_version = models.ForeignKey("Intervention", null=True, blank=True, on_delete=models.DO_NOTHING)
 
-    # Compensation or payments, one-directional
-    #payments = models.ManyToManyField(Payment, related_name="+", blank=True)
-    #compensations = models.ManyToManyField(Compensation, related_name="+", blank=True)
-
     # Users having access on this object
     users = models.ManyToManyField(User)
 

intervention/tables.py

@@ -33,13 +33,13 @@ class InterventionTable(BaseTable):
         verbose_name=_("Checked"),
         orderable=True,
         empty_values=[],
-        accessor="checked_on",
+        accessor="checked",
     )
     r = tables.Column(
         verbose_name=_("Recorded"),
         orderable=True,
         empty_values=[],
-        accessor="recorded_on",
+        accessor="recorded",
     )
     e = tables.Column(
         verbose_name=_("Editable"),
@@ -110,9 +110,10 @@ class InterventionTable(BaseTable):
         checked = value is not None
         tooltip = _("Not checked yet")
         if checked:
+            value = value.timestamp
             value = localtime(value)
             checked_on = value.strftime(DEFAULT_DATE_TIME_FORMAT)
-            tooltip = _("Checked on {} by {}").format(checked_on, record.checked_by)
+            tooltip = _("Checked on {} by {}").format(checked_on, record.checked.user)
         html += self.render_checked_star(
             tooltip=tooltip,
             icn_filled=checked,
@@ -133,9 +134,10 @@ class InterventionTable(BaseTable):
         checked = value is not None
         tooltip = _("Not registered yet")
         if checked:
+            value = value.timestamp
             value = localtime(value)
             on = value.strftime(DEFAULT_DATE_TIME_FORMAT)
-            tooltip = _("Registered on {} by {}").format(on, record.recorded_by)
+            tooltip = _("Registered on {} by {}").format(on, record.recorded.user)
         html += self.render_bookmark(
             tooltip=tooltip,
             icn_filled=checked,

intervention/templates/intervention/detail/view.html

@@ -93,7 +93,7 @@
                     <tr>
                         <th scope="row">{% trans 'Checked' %}</th>
                         <td class="align-middle">
-                            {% if intervention.checked_on is None %}
+                            {% if intervention.checked is None %}
                                 <span>
                                     {% fa5_icon 'star' 'far' %}
                                 </span>
@@ -107,7 +107,7 @@
                     <tr>
                         <th scope="row">{% trans 'Recorded' %}</th>
                         <td class="align-middle">
-                            {% if intervention.recorded_on is None %}
+                            {% if intervention.recorded is None %}
                                 <span>
                                     {% fa5_icon 'bookmark' 'far' %}
                                 </span>