From 6ed9045db99d5ec0f9bd3529c9145cacabd5f90e Mon Sep 17 00:00:00 2001 From: jordan Date: Thu, 14 Apr 2011 20:43:44 +0000 Subject: [PATCH] Changing the way we get the User subscribed URL address, before we used the @models.permalink decorator, now we use directly the reverse URL resolver and we make sure that the username is filtered using the smart_unicode function. This used to cause some trouble with Unicode usernames when sending the daily digest. git-svn-id: http://svn.osqa.net/svnroot/osqa/trunk@984 0cfe37f9-358a-4d5e-be75-b63607b5c754 --- forum/models/user.py | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/forum/models/user.py b/forum/models/user.py index afb33d3..b5bef17 100644 --- a/forum/models/user.py +++ b/forum/models/user.py @@ -211,9 +211,15 @@ class User(BaseModel, DjangoUser): def get_answered_url(self): return ('user_questions', (), {'mode': _('answered-by'), 'user': self.id, 'slug': slugify(self.username)}) - @models.permalink def get_subscribed_url(self): - return ('user_questions', (), {'mode': _('subscribed-by'), 'user': self.id, 'slug': slugify(self.username)}) + try: + # Try to retrieve the Subscribed User URL. + url = reverse('user_questions', + kwargs={'mode': _('subscribed-by'), 'user': self.id, 'slug': slugify(smart_unicode(self.username))}) + return url + except Exception, e: + # If some Exception has been raised, don't forget to log it. + logging.error("Error retrieving a subscribed user URL: %s" % e) def get_profile_link(self): profile_link = u'%s' % (self.get_profile_url(), self.username) -- 2.39.5