diff options
author | Elrond <elrond+mediagoblin.org@samba-tng.org> | 2011-11-21 21:40:48 +0100 |
---|---|---|
committer | Elrond <elrond+mediagoblin.org@samba-tng.org> | 2011-11-22 22:29:37 +0100 |
commit | 30188321531e1b0d3c78166498702bbd8c7dc2bc (patch) | |
tree | 2aaabcf0f3157a032e316066101f2db819ab09c8 /mediagoblin/db/models.py | |
parent | 4d4e5b435b54fcf0920786a40e190c1748cc2ed1 (diff) | |
download | mediagoblin-30188321531e1b0d3c78166498702bbd8c7dc2bc.tar.lz mediagoblin-30188321531e1b0d3c78166498702bbd8c7dc2bc.tar.xz mediagoblin-30188321531e1b0d3c78166498702bbd8c7dc2bc.zip |
Rename MediaEntry.uploader() to .get_uploader()
The .uploader() method conflicts with the uploader database
field. As we're moving to .FIELD for db field access, this
is a relevant conflict.
So renaming .uploader() to .get_uploader()
Diffstat (limited to 'mediagoblin/db/models.py')
-rw-r--r-- | mediagoblin/db/models.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/mediagoblin/db/models.py b/mediagoblin/db/models.py index f13a4457..265fe36d 100644 --- a/mediagoblin/db/models.py +++ b/mediagoblin/db/models.py @@ -263,7 +263,7 @@ class MediaEntry(Document): Use a slug if we have one, else use our '_id'. """ - uploader = self.uploader() + uploader = self.get_uploader() if self.get('slug'): return urlgen( @@ -286,7 +286,7 @@ class MediaEntry(Document): '_id', ASCENDING).limit(1) if cursor.count(): return urlgen('mediagoblin.user_pages.media_home', - user=self.uploader()['username'], + user=self.get_uploader()['username'], media=unicode(cursor[0]['slug'])) def url_to_next(self, urlgen): @@ -300,10 +300,10 @@ class MediaEntry(Document): if cursor.count(): return urlgen('mediagoblin.user_pages.media_home', - user=self.uploader()['username'], + user=self.get_uploader()['username'], media=unicode(cursor[0]['slug'])) - def uploader(self): + def get_uploader(self): return self.db.User.find_one({'_id': self['uploader']}) def get_fail_exception(self): |