Generalize datetime formatting function for templates.

1 files changed, 13 insertions, 3 deletions

diff --git a/uploader/background_jobs.py b/uploader/background_jobs.py
index 64d55e9..fc534dc 100644
--- a/uploader/background_jobs.py
+++ b/uploader/background_jobs.py
@@ -130,6 +130,15 @@ def job_error(job_id: uuid.UUID):
             return render_template("jobs/job-not-found.html", job_id=job_id)
 
 
+def make_datetime_formatter(dtformat: str = "") -> str:
+    """Make a datetime formatter with the provided `dtformat`"""
+    def __formatter__(val: str) -> str:
+        dt = datetime.datetime.fromisoformat(val)
+        return dt.strftime(dtformat.strip() or "%A, %d %B %Y at %H:%M %Z")
+
+    return __formatter__
+
+
 @background_jobs_bp.route("/list")
 @require_login
 def list_jobs():
@@ -139,8 +148,7 @@ def list_jobs():
             "background-jobs/list-jobs.html",
             jobs=jobs.jobs_by_external_id(
                 conn, session.user_details()["user_id"]),
-            display_datetime=lambda val: datetime.datetime.fromisoformat(
-                val).strftime("%A, %d %B %Y at %H:%M %Z"))
+            display_datetime=make_datetime_formatter())
 
 
 @background_jobs_bp.route("/summary/<uuid:job_id>")
@@ -153,7 +161,9 @@ def job_summary(job_id: uuid.UUID):
             status = job["metadata"]["status"]
 
             if status in ("completed", "error"):
-                return render_template("background-jobs/job-summary.html", job=job)
+                return render_template("background-jobs/job-summary.html",
+                                       job=job,
+                                       display_datetime=make_datetime_formatter())
             return redirect(url_for("background-jobs.job_status", job["job_id"]))
         except JobNotFound as _jnf:
             return render_template("jobs/job-not-found.html", job_id=job_id)