X-Git-Url: https://git.openstreetmap.org./osqa.git/blobdiff_plain/f982c8513663a0f9927fb04ba650a5f63bb7f21f..2444d6ffd7995be782615589fb1d5aef2c8277ce:/forum/models/user.py diff --git a/forum/models/user.py b/forum/models/user.py index 7f1df5f..9d63248 100644 --- a/forum/models/user.py +++ b/forum/models/user.py @@ -22,6 +22,8 @@ QUESTIONS_PER_PAGE_CHOICES = ( ) class AnonymousUser(DjangoAnonymousUser): + reputation = 0 + def get_visible_answers(self, question): return question.answers.filter_state(deleted=False) @@ -33,6 +35,9 @@ class AnonymousUser(DjangoAnonymousUser): def can_vote_down(self): return False + + def can_vote_count_today(self): + return 0 def can_flag_offensive(self, post=None): return False @@ -54,6 +59,12 @@ class AnonymousUser(DjangoAnonymousUser): def can_convert_to_comment(self, answer): return False + + def can_convert_to_question(self, answer): + return False + + def can_convert_comment_to_answer(self, comment): + return False def can_accept_answer(self, answer): return False @@ -85,6 +96,9 @@ class AnonymousUser(DjangoAnonymousUser): def can_upload_files(self): return False + def is_a_super_user_or_staff(self): + return False + def true_if_is_super_or_staff(fn): def decorated(self, *args, **kwargs): return self.is_superuser or self.is_staff or fn(self, *args, **kwargs) @@ -152,9 +166,16 @@ class User(BaseModel, DjangoUser): return self.username @property - def gravatar(self): - return md5(self.email).hexdigest() + def last_activity(self): + try: + return self.actions.order_by('-action_date')[0].action_date + except: + return self.last_seen + @property + def gravatar(self): + return md5(self.email.lower()).hexdigest() + def save(self, *args, **kwargs): if self.reputation < 0: self.reputation = 0 @@ -222,7 +243,15 @@ class User(BaseModel, DjangoUser): today = datetime.date.today() return self.actions.filter(canceled=False, action_type="flag", action_date__gte=(today - datetime.timedelta(days=1))).count() - + + def can_vote_count_today(self): + votes_today = settings.MAX_VOTES_PER_DAY + + if settings.USER_REPUTATION_TO_MAX_VOTES: + votes_today = votes_today + int(self.reputation) + + return votes_today + @true_if_is_super_or_staff def can_view_deleted_post(self, post): return post.author == self @@ -266,13 +295,21 @@ class User(BaseModel, DjangoUser): def can_delete_comment(self, comment): return self == comment.author or self.reputation >= int(settings.REP_TO_DELETE_COMMENTS) + @true_if_is_super_or_staff + def can_convert_comment_to_answer(self, comment): + return self == comment.author or self.reputation >= int(settings.REP_TO_CONVERT_COMMENTS_TO_ANSWERS) + def can_convert_to_comment(self, answer): return (not answer.marked) and (self.is_superuser or self.is_staff or answer.author == self or self.reputation >= int (settings.REP_TO_CONVERT_TO_COMMENT)) + + def can_convert_to_question(self, answer): + return (not answer.marked) and (self.is_superuser or self.is_staff or answer.author == self or self.reputation >= int + (settings.REP_TO_CONVERT_TO_QUESTION)) @true_if_is_super_or_staff def can_accept_answer(self, answer): - return self == answer.question.author + return self == answer.question.author and (settings.USERS_CAN_ACCEPT_OWN or answer.author != answer.question.author) @true_if_is_super_or_staff def can_create_tags(self): @@ -303,7 +340,7 @@ class User(BaseModel, DjangoUser): @true_if_is_super_or_staff def can_reopen_question(self, question): - return self == question.author and self.reputation >= settings.REP_TO_REOPEN_OWN + return self == question.author and self.reputation >= int(settings.REP_TO_REOPEN_OWN) @true_if_is_super_or_staff def can_delete_post(self, post): @@ -311,12 +348,16 @@ class User(BaseModel, DjangoUser): return self.can_delete_comment(post) return (self == post.author and (post.__class__.__name__ == "Answer" or - not post.answers.exclude(author=self).count())) + not post.answers.exclude(author__id=self.id).count())) @true_if_is_super_or_staff def can_upload_files(self): return self.reputation >= int(settings.REP_TO_UPLOAD) + @true_if_is_super_or_staff + def is_a_super_user_or_staff(self): + return False + def email_valid_and_can_ask(self): return 'ask' not in settings.REQUIRE_EMAIL_VALIDATION_TO or self.email_isvalid @@ -360,7 +401,7 @@ class User(BaseModel, DjangoUser): class Meta: app_label = 'forum' -class UserProperty(models.Model): +class UserProperty(BaseModel): user = models.ForeignKey(User, related_name='properties') key = models.CharField(max_length=16) value = PickledObjectField() @@ -369,6 +410,16 @@ class UserProperty(models.Model): app_label = 'forum' unique_together = ('user', 'key') + def cache_key(self): + return self._generate_cache_key("%s:%s" % (self.user.id, self.key)) + + @classmethod + def infer_cache_key(cls, querydict): + if 'user' in querydict and 'key' in querydict: + return cls._generate_cache_key("%s:%s" % (querydict['user'].id, querydict['key'])) + + return None + class UserPropertyDict(object): def __init__(self, user): self.__dict__['_user'] = user @@ -404,7 +455,7 @@ class UserPropertyDict(object): if current: current.value = value self.__dict__[name] = value - current.save() + current.save(full_save=True) else: user = self.__dict__['_user'] prop = UserProperty(user=user, value=value, key=name)