From 6524ef501d99e959dc8d1f1c5b852f8f2849ddc7 Mon Sep 17 00:00:00 2001 From: maurizio Date: Tue, 8 May 2012 17:25:31 +0200 Subject: [PATCH] Comment._get_userinfo is thread safe now --- django/contrib/comments/models.py | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/django/contrib/comments/models.py b/django/contrib/comments/models.py index e0ef8b3d28..475b3c8dea 100644 --- a/django/contrib/comments/models.py +++ b/django/contrib/comments/models.py @@ -92,7 +92,7 @@ class Comment(BaseCommentAbstractModel): This dict will have ``name``, ``email``, and ``url`` fields. """ if not hasattr(self, "_userinfo"): - self._userinfo = { + userinfo = { "name" : self.user_name, "email" : self.user_email, "url" : self.user_url @@ -100,15 +100,16 @@ class Comment(BaseCommentAbstractModel): if self.user_id: u = self.user if u.email: - self._userinfo["email"] = u.email + userinfo["email"] = u.email # If the user has a full name, use that for the user name. # However, a given user_name overrides the raw user.username, # so only use that if this comment has no associated name. if u.get_full_name(): - self._userinfo["name"] = self.user.get_full_name() + userinfo["name"] = self.user.get_full_name() elif not self.user_name: - self._userinfo["name"] = u.username + userinfo["name"] = u.username + self._userinfo = userinfo return self._userinfo userinfo = property(_get_userinfo, doc=_get_userinfo.__doc__)