fix: use approver's identity instead of System (ietf-tools#3701)

Spectre17 · web-flow · commit 4dc024797698 · 2022-03-20T07:54:15.000-05:00
Fixes issue ietf-tools#3545 Conditionally tests if the approver is AnonymousUser, in which case System is still used
diff --git 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,
