X-Git-Url: https://git.openstreetmap.org/osqa.git/blobdiff_plain/b13937220bac0724bd0a580907a3fb4114ead57b..524edb6c52ca7f56e9e032a175dc4b7fba0fca6c:/forum/actions/user.py diff --git a/forum/actions/user.py b/forum/actions/user.py index d9c6be9..09a66ff 100644 --- a/forum/actions/user.py +++ b/forum/actions/user.py @@ -1,9 +1,10 @@ from django.utils.translation import ungettext, ugettext as _ from django.core.urlresolvers import reverse from django.db.models import F +from django.contrib import messages from forum.models.action import ActionProxy from forum.models import Award, Badge, ValidationHash, User -from forum import settings +from forum import settings, REQUEST_HOLDER from forum.settings import APP_SHORT_NAME from forum.utils.mail import send_template_email @@ -19,7 +20,7 @@ class UserJoinsAction(ActionProxy): def describe(self, viewer=None): return _("%(user)s %(have_has)s joined the %(app_name)s Q&A community") % { - 'user': self.hyperlink(self.user.get_profile_url(), self.friendly_username(viewer, self.user)), + 'user': self.hyperlink(self.user.get_absolute_url(), self.friendly_username(viewer, self.user)), 'have_has': self.viewer_or_user_verb(viewer, self.user, _('have'), _('has')), 'app_name': APP_SHORT_NAME, } @@ -29,7 +30,7 @@ class UserLoginAction(ActionProxy): def describe(self, viewer=None): return _("%(user)s %(have_has)s logged in") % { - 'user' : self.hyperlink(self.user.get_profile_url(), self.friendly_username(viewer, self.user)), + 'user' : self.hyperlink(self.user.get_absolute_url(), self.friendly_username(viewer, self.user)), 'have_has': self.viewer_or_user_verb(viewer, self.user, _('have'), _('has')), } @@ -45,7 +46,7 @@ class EmailValidationAction(ActionProxy): def describe(self, viewer=None): return _("%(user)s %(have_has)s validated the e-mail %(email)s") % { - 'user': self.hyperlink(self.user.get_profile_url(), self.friendly_username(viewer, self.user)), + 'user': self.hyperlink(self.user.get_absolute_url(), self.friendly_username(viewer, self.user)), 'have_has': self.viewer_or_user_verb(viewer, self.user, _('have'), _('has')), 'email' : self.user.email if viewer.is_superuser or viewer.is_staff or viewer == self.user else "" } @@ -55,9 +56,9 @@ class EditProfileAction(ActionProxy): def describe(self, viewer=None): return _("%(user)s edited %(hes_or_your)s %(profile_link)s") % { - 'user': self.hyperlink(self.user.get_profile_url(), self.friendly_username(viewer, self.user)), + 'user': self.hyperlink(self.user.get_absolute_url(), self.friendly_username(viewer, self.user)), 'hes_or_your': self.viewer_or_user_verb(viewer, self.user, _('your'), _('his')), - 'profile_link': self.hyperlink(self.user.get_profile_url(), _('profile')), + 'profile_link': self.hyperlink(self.user.get_absolute_url(), _('profile')), } class BonusRepAction(ActionProxy): @@ -71,13 +72,8 @@ class BonusRepAction(ActionProxy): def repute_users(self): self.repute(self._affected, self._value) - if self._value > 0: - self._affected.message_set.create( - message=_("Congratulations, you have been awarded an extra %s reputation points.") % self._value + - '
%s' % self.extra.get('message', _('Thank you'))) - else: - self._affected.message_set.create( - message=_("You have been penalized in %s reputation points.") % self._value + + if self._value < 0: + messages.info(REQUEST_HOLDER.request, _("You have penalized %s in %s reputation points.") % (self._affected, self._value) + '
%s' % self.extra.get('message', '')) def describe(self, viewer=None): @@ -87,12 +83,12 @@ class BonusRepAction(ActionProxy): try: if int(value) > 0: return _("%(user)s awarded an extra %(value)s reputation points to %(users)s: %(message)s") % { - 'user': self.hyperlink(self.user.get_profile_url(), self.friendly_username(viewer, self.user)), + 'user': self.hyperlink(self.user.get_absolute_url(), self.friendly_username(viewer, self.user)), 'value': value, 'users':self.affected_links(viewer), 'message': message } else: return _("%(user)s penalised %(users)s in %(value)s reputation points: %(message)s") % { - 'user': self.hyperlink(self.user.get_profile_url(), self.friendly_username(viewer, self.user)), + 'user': self.hyperlink(self.user.get_absolute_url(), self.friendly_username(viewer, self.user)), 'value': value, 'users':self.affected_links(viewer), 'message': message } except Exception, e: @@ -110,26 +106,18 @@ class AwardPointsAction(ActionProxy): self.repute(self._affected, self._value) self.repute(self.user, -self._value) - - self._affected.message_set.create( - message=_("Congratulations, you have been awarded an extra %(points)s reputation %(points_label)s on this answer.") % { - 'points': self._value, - 'points_label': ungettext('point', 'points', self._value), - 'answer_url': self.node.get_absolute_url() - }) - def describe(self, viewer=None): value = self.extra.get('value', _('unknown')) try: if int(value) > 0: return _("%(user)s awarded an extra %(value)s reputation points to %(users)s") % { - 'user': self.hyperlink(self.user.get_profile_url(), self.friendly_username(viewer, self.user)), + 'user': self.hyperlink(self.user.get_absolute_url(), self.friendly_username(viewer, self.user)), 'value': value, 'users':self.affected_links(viewer), } else: return _("%(user)s penalised %(users)s in %(value)s reputation points") % { - 'user': self.hyperlink(self.user.get_profile_url(), self.friendly_username(viewer, self.user)), + 'user': self.hyperlink(self.user.get_absolute_url(), self.friendly_username(viewer, self.user)), 'value': value, 'users':self.affected_links(viewer), } except Exception, e: @@ -163,12 +151,6 @@ class AwardAction(ActionProxy): self.user.save() - self.user.message_set.create(message=_( - """Congratulations, you have received a badge '%(badge_name)s'. Find out who has it, too.""" - ) % dict( - badge_name=award.badge.name, - badge_url=award.badge.get_absolute_url())) - def cancel_action(self): award = self.award badge = award.badge @@ -188,11 +170,40 @@ class AwardAction(ActionProxy): def describe(self, viewer=None): return _("%(user)s %(were_was)s awarded the %(badge_name)s badge") % { - 'user': self.hyperlink(self.user.get_profile_url(), self.friendly_username(viewer, self.user)), + 'user': self.hyperlink(self.user.get_absolute_url(), self.friendly_username(viewer, self.user)), 'were_was': self.viewer_or_user_verb(viewer, self.user, _('were'), _('was')), 'badge_name': self.award.badge.name, } + +class ReportAction(ActionProxy): + verb = _("suspended") + + def process_data(self, **kwargs): + self.extra = kwargs + # message here? + + + def process_action(self): + + all_superusers = User.objects.filter(is_superuser=True) + + + send_template_email(all_superusers, "notifications/user_reported.html", { + 'reported': self.extra['reported'], + 'user':self.user, + 'message': self.extra['publicmsg'] + } + ) + + def describe(self, viewer=None): + + return _("%(user)s reported %(reported) : %(msg)s") % { + 'user': self.hyperlink(self.user.get_absolute_url(), self.friendly_username(viewer, self.user)), + 'reporter': self.extra.get('reported').username, + 'msg': self.extra.get('publicmsg', _('N/A')) + } + class SuspendAction(ActionProxy): verb = _("suspended") @@ -212,7 +223,6 @@ class SuspendAction(ActionProxy): u.is_active = True u._pop_suspension_cache() u.save() - u.message_set.create(message=_("Your suspension has been removed.")) def describe(self, viewer=None): if self.extra.get('bantype', 'indefinitely') == 'forxdays' and self.extra.get('forxdays', None): @@ -221,6 +231,6 @@ class SuspendAction(ActionProxy): suspension = _("indefinetely") return _("%(user)s suspended %(users)s %(suspension)s: %(msg)s") % { - 'user': self.hyperlink(self.user.get_profile_url(), self.friendly_username(viewer, self.user)), + 'user': self.hyperlink(self.user.get_absolute_url(), self.friendly_username(viewer, self.user)), 'users': self.affected_links(viewer), 'suspension': suspension, 'msg': self.extra.get('publicmsg', _('Bad behaviour')) - } \ No newline at end of file + }