summaryrefslogtreecommitdiffstats
path: root/askbot/models/__init__.py
diff options
context:
space:
mode:
Diffstat (limited to 'askbot/models/__init__.py')
-rw-r--r--askbot/models/__init__.py143
1 files changed, 112 insertions, 31 deletions
diff --git a/askbot/models/__init__.py b/askbot/models/__init__.py
index d98d4246..9abb4198 100644
--- a/askbot/models/__init__.py
+++ b/askbot/models/__init__.py
@@ -15,10 +15,12 @@ from django.utils.safestring import mark_safe
from django.db import models
from django.conf import settings as django_settings
from django.contrib.contenttypes.models import ContentType
+from django.core import cache
from django.core import exceptions as django_exceptions
from django_countries.fields import CountryField
from askbot import exceptions as askbot_exceptions
from askbot import const
+from askbot.const.message_keys import get_i18n_message
from askbot.conf import settings as askbot_settings
from askbot.models.question import Thread
from askbot.skins import utils as skin_utils
@@ -36,6 +38,7 @@ from askbot import auth
from askbot.utils.decorators import auto_now_timestamp
from askbot.utils.slug import slugify
from askbot.utils.diff import textDiff as htmldiff
+from askbot.utils.url_utils import strip_path
from askbot.utils import mail
def get_model(model_name):
@@ -382,7 +385,9 @@ def user_assert_can_vote_for_post(
:param:post can be instance of question or answer
"""
if self == post.author:
- raise django_exceptions.PermissionDenied(_('cannot vote for own posts'))
+ raise django_exceptions.PermissionDenied(
+ _('Sorry, you cannot vote for your own posts')
+ )
blocked_error_message = _(
'Sorry your account appears to be blocked ' +
@@ -424,8 +429,7 @@ def user_assert_can_upload_file(request_user):
blocked_error_message = _('Sorry, blocked users cannot upload files')
suspended_error_message = _('Sorry, suspended users cannot upload files')
low_rep_error_message = _(
- 'uploading images is limited to users '
- 'with >%(min_rep)s reputation points'
+ 'sorry, file uploading requires karma >%(min_rep)s',
) % {'min_rep': askbot_settings.MIN_REP_TO_UPLOAD_FILES }
_assert_user_can(
@@ -441,10 +445,13 @@ def user_assert_can_post_question(self):
text that has the reason for the denial
"""
+ blocked_message = get_i18n_message('BLOCKED_USERS_CANNOT_POST')
+ suspended_message = get_i18n_message('SUSPENDED_USERS_CANNOT_POST')
+
_assert_user_can(
user = self,
- blocked_error_message = _('blocked users cannot post'),
- suspended_error_message = _('suspended users cannot post'),
+ blocked_error_message = blocked_message,
+ suspended_error_message = suspended_message
)
@@ -488,6 +495,18 @@ def user_assert_can_edit_comment(self, comment = None):
raise django_exceptions.PermissionDenied(error_message)
+def user_can_post_comment(self, parent_post = None):
+ """a simplified method to test ability to comment
+ """
+ if self.reputation >= askbot_settings.MIN_REP_TO_LEAVE_COMMENTS:
+ return True
+ if parent_post and self == parent_post.author:
+ return True
+ if self.is_administrator_or_moderator():
+ return True
+ return False
+
+
def user_assert_can_post_comment(self, parent_post = None):
"""raises exceptions.PermissionDenied if
user cannot post comment
@@ -505,12 +524,14 @@ def user_assert_can_post_comment(self, parent_post = None):
'your own posts and answers to your questions'
) % {'min_rep': askbot_settings.MIN_REP_TO_LEAVE_COMMENTS}
+ blocked_message = get_i18n_message('BLOCKED_USERS_CANNOT_POST')
+
try:
_assert_user_can(
user = self,
post = parent_post,
owner_can = True,
- blocked_error_message = _('blocked users cannot post'),
+ blocked_error_message = blocked_message,
suspended_error_message = suspended_error_message,
min_rep_setting = askbot_settings.MIN_REP_TO_LEAVE_COMMENTS,
low_rep_error_message = low_rep_error_message,
@@ -749,16 +770,29 @@ def user_assert_can_flag_offensive(self, post = None):
assert(post is not None)
- double_flagging_error_message = _('cannot flag message as offensive twice')
+ double_flagging_error_message = _(
+ 'You have flagged this question before and '
+ 'cannot do it more than once'
+ )
if self.get_flags_for_post(post).count() > 0:
raise askbot_exceptions.DuplicateCommand(double_flagging_error_message)
- blocked_error_message = _('blocked users cannot flag posts')
+ blocked_error_message = _(
+ 'Sorry, since your account is blocked '
+ 'you cannot flag posts as offensive'
+ )
- suspended_error_message = _('suspended users cannot flag posts')
+ suspended_error_message = _(
+ 'Sorry, your account appears to be suspended and you cannot make new posts '
+ 'until this issue is resolved. You can, however edit your existing posts. '
+ 'Please contact the forum administrator to reach a resolution.'
+ )
- low_rep_error_message = _('need > %(min_rep)s points to flag spam') % \
+ low_rep_error_message = _(
+ 'Sorry, to flag posts as offensive a minimum reputation '
+ 'of %(min_rep)s is required'
+ ) % \
{'min_rep': askbot_settings.MIN_REP_TO_FLAG_OFFENSIVE}
min_rep_setting = askbot_settings.MIN_REP_TO_FLAG_OFFENSIVE
@@ -777,11 +811,12 @@ def user_assert_can_flag_offensive(self, post = None):
flag_count_today = self.get_flag_count_posted_today()
if flag_count_today >= askbot_settings.MAX_FLAGS_PER_USER_PER_DAY:
flags_exceeded_error_message = _(
- '%(max_flags_per_day)s exceeded'
- ) % {
- 'max_flags_per_day': \
- askbot_settings.MAX_FLAGS_PER_USER_PER_DAY
- }
+ 'Sorry, you have exhausted the maximum number of '
+ '%(max_flags_per_day)s offensive flags per day.'
+ ) % {
+ 'max_flags_per_day': \
+ askbot_settings.MAX_FLAGS_PER_USER_PER_DAY
+ }
raise django_exceptions.PermissionDenied(flags_exceeded_error_message)
def user_assert_can_remove_flag_offensive(self, post = None):
@@ -793,14 +828,19 @@ def user_assert_can_remove_flag_offensive(self, post = None):
if self.get_flags_for_post(post).count() < 1:
raise django_exceptions.PermissionDenied(non_existing_flagging_error_message)
- blocked_error_message = _('blocked users cannot remove flags')
+ blocked_error_message = _(
+ 'Sorry, since your account is blocked you cannot remove flags'
+ )
- suspended_error_message = _('suspended users cannot remove flags')
+ suspended_error_message = _(
+ 'Sorry, your account appears to be suspended and you cannot remove flags. '
+ 'Please contact the forum administrator to reach a resolution.'
+ )
min_rep_setting = askbot_settings.MIN_REP_TO_FLAG_OFFENSIVE
low_rep_error_message = ungettext(
- 'need > %(min_rep)d point to remove flag',
- 'need > %(min_rep)d points to remove flag',
+ 'Sorry, to flag posts a minimum reputation of %(min_rep)d is required',
+ 'Sorry, to flag posts a minimum reputation of %(min_rep)d is required',
min_rep_setting
) % {'min_rep': min_rep_setting}
@@ -908,7 +948,9 @@ def user_assert_can_revoke_old_vote(self, vote):
"""
if (datetime.datetime.now().day - vote.voted_at.day) \
>= askbot_settings.MAX_DAYS_TO_CANCEL_VOTE:
- raise django_exceptions.PermissionDenied(_('cannot revoke old vote'))
+ raise django_exceptions.PermissionDenied(
+ _('sorry, but older votes cannot be revoked')
+ )
def user_get_unused_votes_today(self):
"""returns number of votes that are
@@ -949,6 +991,7 @@ def user_post_comment(
comment = body_text,
added_at = timestamp,
)
+ parent_post.thread.invalidate_cached_data()
award_badges_signal.send(None,
event = 'post_comment',
actor = self,
@@ -976,10 +1019,10 @@ def user_post_anonymous_askbot_content(user, session_key):
#maybe add pending posts message?
else:
if user.is_blocked():
- msg = _('blocked users cannot post')
+ msg = get_i18n_message('BLOCKED_USERS_CANNOT_POST')
user.message_set.create(message = msg)
elif user.is_suspended():
- msg = _('suspended users cannot post')
+ msg = get_i18n_message('SUSPENDED_USERS_CANNOT_POST')
user.message_set.create(message = msg)
else:
for aq in aq_list:
@@ -1061,6 +1104,7 @@ def user_retag_question(
tagnames = tags,
silent = silent
)
+ question.thread.invalidate_cached_data()
award_badges_signal.send(None,
event = 'retag_question',
actor = self,
@@ -1118,6 +1162,7 @@ def user_delete_comment(
):
self.assert_can_delete_comment(comment = comment)
comment.delete()
+ comment.thread.invalidate_cached_data()
@auto_now_timestamp
def user_delete_answer(
@@ -1218,6 +1263,7 @@ def user_delete_post(
self.delete_question(question = post, timestamp = timestamp)
else:
raise TypeError('either Comment, Question or Answer expected')
+ post.thread.invalidate_cached_data()
def user_restore_post(
self,
@@ -1231,6 +1277,7 @@ def user_restore_post(
post.deleted_by = None
post.deleted_at = None
post.save()
+ post.thread.invalidate_cached_data()
if post.post_type == 'answer':
post.thread.update_answer_count()
else:
@@ -1289,10 +1336,13 @@ def user_post_question(
def user_edit_comment(self, comment_post=None, body_text = None):
"""apply edit to a comment, the method does not
change the comments timestamp and no signals are sent
+ todo: see how this can be merged with edit_post
+ todo: add timestamp
"""
self.assert_can_edit_comment(comment_post)
comment_post.text = body_text
comment_post.parse_and_save(author = self)
+ comment_post.thread.invalidate_cached_data()
@auto_now_timestamp
@@ -1321,6 +1371,7 @@ def user_edit_question(
wiki = wiki,
edit_anonymously = edit_anonymously,
)
+ question.thread.invalidate_cached_data()
award_badges_signal.send(None,
event = 'edit_question',
actor = self,
@@ -1347,6 +1398,7 @@ def user_edit_answer(
comment = revision_comment,
wiki = wiki,
)
+ answer.thread.invalidate_cached_data()
award_badges_signal.send(None,
event = 'edit_answer',
actor = self,
@@ -1424,6 +1476,7 @@ def user_post_answer(
email_notify = follow,
wiki = wiki
)
+ answer_post.thread.invalidate_cached_data()
award_badges_signal.send(None,
event = 'post_answer',
actor = self,
@@ -1443,12 +1496,17 @@ def user_visit_question(self, question = None, timestamp = None):
timestamp = datetime.datetime.now()
try:
- question_view = QuestionView.objects.get(who=self, question=question)
+ QuestionView.objects.filter(
+ who=self, question=question
+ ).update(
+ when = timestamp
+ )
except QuestionView.DoesNotExist:
- question_view = QuestionView(who=self, question=question)
-
- question_view.when = timestamp
- question_view.save()
+ QuestionView(
+ who=self,
+ question=question,
+ when = timestamp
+ ).save()
#filter memo objects on response activities directed to the qurrent user
#that refer to the children of the currently
@@ -1910,15 +1968,24 @@ def _process_vote(user, post, timestamp=None, cancel=False, vote_type=None):
if vote_type == Vote.VOTE_UP:
if cancel:
auth.onUpVotedCanceled(vote, post, user, timestamp)
- return None
else:
auth.onUpVoted(vote, post, user, timestamp)
elif vote_type == Vote.VOTE_DOWN:
if cancel:
auth.onDownVotedCanceled(vote, post, user, timestamp)
- return None
else:
auth.onDownVoted(vote, post, user, timestamp)
+
+ if post.post_type == 'question':
+ #denormalize the question post score on the thread
+ post.thread.score = post.score
+ post.thread.save()
+ post.thread.update_summary_html()
+
+ post.thread.invalidate_cached_data()
+
+ if cancel:
+ return None
event = VOTES_TO_EVENTS.get((vote_type, post.post_type), None)
if event:
@@ -2134,6 +2201,7 @@ User.add_to_class('is_following_question', user_is_following_question)
User.add_to_class('mark_tags', user_mark_tags)
User.add_to_class('update_response_counts', user_update_response_counts)
User.add_to_class('can_have_strong_url', user_can_have_strong_url)
+User.add_to_class('can_post_comment', user_can_post_comment)
User.add_to_class('is_administrator', user_is_administrator)
User.add_to_class('is_administrator_or_moderator', user_is_administrator_or_moderator)
User.add_to_class('set_admin_status', user_set_admin_status)
@@ -2279,7 +2347,7 @@ def format_instant_notification_email(
'receiving_user_name': to_user.username,
'content_preview': content_preview,#post.get_snippet()
'update_type': update_type,
- 'post_url': site_url + post.get_absolute_url(),
+ 'post_url': strip_path(site_url) + post.get_absolute_url(),
'origin_post_title': origin_post.thread.title,
'user_subscriptions_url': user_subscriptions_url,
}
@@ -2468,7 +2536,8 @@ def record_user_visit(user, timestamp, **kwargs):
context_object = user,
timestamp = timestamp
)
- user.save()
+ #somehow it saves on the query as compared to user.save()
+ User.objects.filter(id = user.id).update(last_seen = timestamp)
def record_vote(instance, created, **kwargs):
@@ -2653,9 +2722,21 @@ def update_user_avatar_type_flag(instance, **kwargs):
def make_admin_if_first_user(instance, **kwargs):
+ """first user automatically becomes an administrator
+ the function is run only once in the interpreter session
+ """
+ import sys
+ #have to check sys.argv to satisfy the test runner
+ #which fails with the cache-based skipping
+ #for real the setUp() code in the base test case must
+ #clear the cache!!!
+ if 'test' not in sys.argv and cache.cache.get('admin-created'):
+ #no need to hit the database every time!
+ return
user_count = User.objects.all().count()
if user_count == 0:
instance.set_admin_status()
+ cache.cache.set('admin-created', True)
#signal for User model save changes
django_signals.pre_save.connect(make_admin_if_first_user, sender=User)
generated by cgit v1.2.3-1-g7c22 (git 2.25.1) at 2024-07-18 12:10:31 +0000