From 4dc024797698faab8ae21f634228dd3aefda50f1 Mon Sep 17 00:00:00 2001
From: Peter Yee <github@houseofyee.com>
Date: Sun, 20 Mar 2022 05:54:15 -0700
Subject: [PATCH] fix: use approver's identity instead of System (#3701)
Fixes issue #3545
Conditionally tests if the approver is AnonymousUser, in which case
System is still used
---
ietf/submit/utils.py | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/ietf/submit/utils.py b/ietf/submit/utils.py
index 495759dd6..55801ae48 100644
--- a/ietf/submit/utils.py
+++ b/ietf/submit/utils.py
@@ -18,6 +18,7 @@ from django.db import transaction
from django.http import HttpRequest # pyflakes:ignore
from django.utils.module_loading import import_string
from django.template.loader import render_to_string
+from django.contrib.auth.models import AnonymousUser
import debug # pyflakes:ignore
@@ -330,11 +331,15 @@ def post_submission(request, submission, approved_doc_desc, approved_subm_desc):
# Add all the previous submission events as docevents
events += post_rev00_submission_events(draft, submission, submitter)
+ if isinstance(request.user, AnonymousUser):
+ doer=system
+ else:
+ doer=request.user.person
# Add an approval docevent
e = SubmissionDocEvent.objects.create(
type="new_submission",
doc=draft,
- by=system,
+ by=doer,
desc=approved_doc_desc,
submission=submission,
rev=submission.rev,