From 54d181eb299e38118f682378fc988bc1ffed7c45 Mon Sep 17 00:00:00 2001 From: Peter Yee Date: Sat, 19 Mar 2022 09:57:34 -0700 Subject: [PATCH] fix: use approver's identity instead of System 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 495759dd6a..55801ae480 100644 --- a/ietf/submit/utils.py +++ b/ietf/submit/utils.py @@ -18,6 +18,7 @@ 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,