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
This commit is contained in:
Peter Yee 2022-03-20 05:54:15 -07:00 committed by GitHub
parent 916a3a07e3
commit 4dc0247976
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -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,