Skip to content
This repository has been archived by the owner on Feb 16, 2023. It is now read-only.

Commit

Permalink
Fix runtime warning when adding log entries
Browse files Browse the repository at this point in the history
LogEntry.action_time expects a Django timezone object instead of a builtin datetime.

This fixes a runtime warning of the following kind:
RuntimeWarning: DateTimeField LogEntry.action_time received a naive datetime (2018-03-28 20:53:01.714173) while time zone support is active.
  • Loading branch information
erikarvstedt committed Mar 29, 2018
1 parent 68251b8 commit fcdcf62
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions src/documents/signals/handlers.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,7 @@
from django.contrib.admin.models import ADDITION, LogEntry
from django.contrib.auth.models import User
from django.contrib.contenttypes.models import ContentType

from datetime import datetime
from django.utils import timezone

from ..models import Correspondent, Document, Tag

Expand Down Expand Up @@ -107,7 +106,7 @@ def set_log_entry(sender, document=None, logging_group=None, **kwargs):

LogEntry.objects.create(
action_flag=ADDITION,
action_time=datetime.now(),
action_time=timezone.now(),
content_type=ct,
object_id=document.id,
user=user,
Expand Down

0 comments on commit fcdcf62

Please sign in to comment.