]> git.openstreetmap.org Git - osqa.git/commitdiff
Remove some attempts to add messages to users
authorTom Hughes <tom@compton.nu>
Tue, 26 Aug 2014 13:49:47 +0000 (14:49 +0100)
committerTom Hughes <tom@compton.nu>
Tue, 12 May 2015 07:56:36 +0000 (08:56 +0100)
forum/actions/user.py

index ce277c32b9e9195d180a63e905f8bb3eb513e4d4..09a66ff05800d9473e6a791c96c64015c19e6638 100644 (file)
@@ -8,8 +8,6 @@ from forum import settings, REQUEST_HOLDER
 from forum.settings import APP_SHORT_NAME
 from forum.utils.mail import send_template_email
 
 from forum.settings import APP_SHORT_NAME
 from forum.utils.mail import send_template_email
 
-from django.contrib import messages
-
 class UserJoinsAction(ActionProxy):
     verb = _("joined")
 
 class UserJoinsAction(ActionProxy):
     verb = _("joined")
 
@@ -74,11 +72,7 @@ class BonusRepAction(ActionProxy):
     def repute_users(self):
         self.repute(self._affected, self._value)
 
     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 +
-                    '<br />%s' % self.extra.get('message', _('Thank you')))
-        else:
+        if self._value < 0:
             messages.info(REQUEST_HOLDER.request, _("You have penalized %s in %s reputation points.") % (self._affected, self._value) +
                     '<br />%s' % self.extra.get('message', ''))
 
             messages.info(REQUEST_HOLDER.request, _("You have penalized %s in %s reputation points.") % (self._affected, self._value) +
                     '<br />%s' % self.extra.get('message', ''))
 
@@ -112,13 +106,6 @@ class AwardPointsAction(ActionProxy):
         self.repute(self._affected, self._value)
         self.repute(self.user, -self._value)
 
         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 <a href=\"%(answer_url)s\">this</a> 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'))
 
     def describe(self, viewer=None):
         value = self.extra.get('value', _('unknown'))
 
@@ -164,12 +151,6 @@ class AwardAction(ActionProxy):
 
         self.user.save()
 
 
         self.user.save()
 
-        self.user.message_set.create(message=_(
-                """Congratulations, you have received a badge '%(badge_name)s'. <a href="%(badge_url)s">Find out who has it, too</a>."""
-        ) % dict(
-            badge_name=award.badge.name,
-            badge_url=award.badge.get_absolute_url()))
-
     def cancel_action(self):
         award = self.award
         badge = award.badge
     def cancel_action(self):
         award = self.award
         badge = award.badge
@@ -242,7 +223,6 @@ class SuspendAction(ActionProxy):
             u.is_active = True
             u._pop_suspension_cache()
             u.save()
             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):
 
     def describe(self, viewer=None):
         if self.extra.get('bantype', 'indefinitely') == 'forxdays' and self.extra.get('forxdays', None):