Merge pull request #347 from cXhristian/plugins-unicode

Add use __str__ with python_2_unicode_compatible for plugins too

wiki/plugins/images/models.py

@@ -5,6 +5,7 @@ import os.path
 from django.conf import settings as django_settings
 from django.core.exceptions import ImproperlyConfigured
 from django.db import models
+from django.utils.encoding import python_2_unicode_compatible
 from django.utils.translation import ugettext_lazy as _
 
 from . import settings
@@ -26,6 +27,8 @@ def upload_path(instance, filename):
         upload_path = os.path.join(upload_path, uuid.uuid4().hex)
     return os.path.join(upload_path, filename)
 
+
+@python_2_unicode_compatible
 class Image(RevisionPlugin):
     
     # The plugin system is so awesome that the inheritor doesn't need to do
@@ -45,10 +48,12 @@ class Image(RevisionPlugin):
         if settings.APP_LABEL:
             app_label = settings.APP_LABEL
     
-    def __unicode__(self):
+    def __str__(self):
         title = (_('Image: %s') % self.current_revision.imagerevision.get_filename()) if self.current_revision else _('Current revision not set!!')
         return str(title)
 
+
+@python_2_unicode_compatible
 class ImageRevision(RevisionPluginRevision):
     
     image = models.ImageField(upload_to=upload_path,
@@ -100,7 +105,7 @@ class ImageRevision(RevisionPluginRevision):
             app_label = settings.APP_LABEL
         ordering = ('-created',)
 
-    def __unicode__(self):
+    def __str__(self):
         title = _('Image Revsion: %d') % self.revision_number
         return str(title)
 

wiki/plugins/notifications/models.py

@@ -2,6 +2,7 @@
 from __future__ import unicode_literals
 from __future__ import absolute_import
 from django.core.urlresolvers import reverse
+from django.utils.encoding import python_2_unicode_compatible
 from django.utils.translation import ugettext_lazy as _
 from django.db.models import signals
 from django.db import models
@@ -16,11 +17,12 @@ from wiki.plugins.notifications import settings
 from wiki.plugins.notifications.util import get_title
 
 
+@python_2_unicode_compatible
 class ArticleSubscription(ArticlePlugin):
     
     subscription = models.OneToOneField(Subscription)
     
-    def __unicode__(self):
+    def __str__(self):
         title = (_("%(user)s subscribing to %(article)s (%(type)s)") %
                  {'user': self.settings.user.username,
                   'article': self.article.current_revision.title,