no email required (#606)

explorer/forms.py

@@ -42,16 +42,12 @@ def __init__(self, *args, **kwargs):
         self.fields["connection"].widget.attrs["class"] = "form-select"
 
     def clean(self):
-        if self.instance and self.data.get("created_by_user", None):
+        # Don't overwrite created_by_user
+        if self.instance and self.instance.created_by_user:
             self.cleaned_data["created_by_user"] = \
                 self.instance.created_by_user
         return super().clean()
 
-    @property
-    def created_by_user_email(self):
-        return self.instance.created_by_user.email if \
-            self.instance.created_by_user else "--"
-
     @property
     def created_at_time(self):
         return self.instance.created_at.strftime("%Y-%m-%d")

explorer/templates/explorer/query.html

@@ -62,7 +62,6 @@ <h2>
                         {% trans "Description"%}
                     </label>
                 </div>
-                <input type="hidden" id="id_created_by_user" name="created_by_user" value="{{ form.created_by_user_email }}" />
                 {% if form.sql.errors %}
                     {% for error in form.sql.errors %}
                         <div class="alert alert-danger">{{ error|escape }}</div>
@@ -142,8 +141,8 @@ <h2>
 
 <div class="container mt-1 text-end small">
     {% if query.avg_duration %}
-        {% blocktrans trimmed with avg_duration_display=query.avg_duration_display user_email=form.created_by_user_email created=form.created_at_time %}
-            Avg. execution: {{ avg_duration_display }}ms. Query created by {{ user_email }} on {{ created }}.
+        {% blocktrans trimmed with avg_duration_display=query.avg_duration_display cuser=query.created_by_user created=form.created_at_time %}
+            Avg. execution: {{ avg_duration_display }}ms. Query created by {{ cuser }} on {{ created }}.
         {% endblocktrans %}
         {% if query %}<a href="{% url 'explorer_logs' %}?query_id={{ query.id }}"> {% trans "History" %}</a>{% endif %}
     {% endif %}