Merge pull request #26 from udacity/remove_dj_version

[osqa.git] / forum / models / action.py

diff --git a/forum/models/action.py b/forum/models/action.py
index ca92c27c838d008d6b5ecad46c44c8d5690704e5..b3cfd260f33ef8d147e6214245da998f3ac9bd58 100644 (file)
--- a/forum/models/action.py
+++ b/forum/models/action.py
@@ -1,4 +1,6 @@
 from django.utils.translation import ugettext as _
 from django.utils.translation import ugettext as _

+from django.utils.encoding import smart_unicode
+

 from utils import PickledObjectField
 from threading import Thread
 from forum.utils import html
 from utils import PickledObjectField
 from threading import Thread
 from forum.utils import html


@@ -26,7 +28,7 @@ class ActionQuerySet(CachedQuerySet):
 class ActionManager(CachedManager):
     use_for_related_fields = True
 
 class ActionManager(CachedManager):
     use_for_related_fields = True
 

-    def get_query_set(self):
+    def get_queryset(self):

         qs = ActionQuerySet(self.model)
 
         if self.model is not Action:
         qs = ActionQuerySet(self.model)
 
         if self.model is not Action:


@@ -44,7 +46,7 @@ class Action(BaseModel):
     real_user = models.ForeignKey('User', related_name="proxied_actions", null=True)
     ip   = models.CharField(max_length=39)
     node = models.ForeignKey('Node', null=True, related_name="actions")
     real_user = models.ForeignKey('User', related_name="proxied_actions", null=True)
     ip   = models.CharField(max_length=39)
     node = models.ForeignKey('Node', null=True, related_name="actions")

-    action_type = models.CharField(max_length=16)
+    action_type = models.CharField(max_length=32)

     action_date = models.DateTimeField(default=datetime.datetime.now)
 
     extra = PickledObjectField()
     action_date = models.DateTimeField(default=datetime.datetime.now)
 
     extra = PickledObjectField()


@@ -213,10 +215,10 @@ class ActionProxy(Action):
     __metaclass__ = ActionProxyMetaClass
 
     def friendly_username(self, viewer, user):
     __metaclass__ = ActionProxyMetaClass
 
     def friendly_username(self, viewer, user):

-        return (viewer == user) and _('You') or user.username
+        return (viewer == user) and _('You') or smart_unicode(user.username)

 
     def friendly_ownername(self, owner, user):
 
     def friendly_ownername(self, owner, user):

-        return (owner == user) and _('your') or user.username
+        return (owner == user) and _('your') or smart_unicode(user.username)

 
     def viewer_or_user_verb(self, viewer, user, viewer_verb, user_verb):
         return (viewer == user) and viewer_verb or user_verb
 
     def viewer_or_user_verb(self, viewer, user, viewer_verb, user_verb):
         return (viewer == user) and viewer_verb or user_verb


@@ -300,7 +302,7 @@ class ActionRepute(models.Model):
         return 0
 
     def _add_to_rep(self, value):
         return 0
 
     def _add_to_rep(self, value):

-        if (self.user.reputation + value < 1) and not settings.ALLOW_NEGATIVE_REPUTATION:
+        if int(self.user.reputation + value) < 1 and not settings.ALLOW_NEGATIVE_REPUTATION:

             return 0
         else:
             return models.F('reputation') + value
             return 0
         else:
             return models.F('reputation') + value