diff options
| author | Elrond <elrond+mediagoblin.org@samba-tng.org> | 2011-12-20 18:47:33 +0100 |
|---|---|---|
| committer | Elrond <elrond+mediagoblin.org@samba-tng.org> | 2011-12-27 20:41:31 +0100 |
| commit | 0eb649ff7ac3f1eb71eb1d2cb66019a860b2c5c7 (patch) | |
| tree | da1b3c8f4201da8d3389f07a67d20efa9c9b3172 /mediagoblin/db/mongo/models.py | |
| parent | 479e8a833ba502c976574af77181f60a2a660aec (diff) | |
| download | mediagoblin-0eb649ff7ac3f1eb71eb1d2cb66019a860b2c5c7.tar.lz mediagoblin-0eb649ff7ac3f1eb71eb1d2cb66019a860b2c5c7.tar.xz mediagoblin-0eb649ff7ac3f1eb71eb1d2cb66019a860b2c5c7.zip | |
Use media.url_for_self instead of calling urlgen directly
Replace
urlgen('ID', user=media.get_uploader().username, media=media.*)
by
media.url_for_self(urlgen)
in a few places.
It's just a lot nicer!
Diffstat (limited to 'mediagoblin/db/mongo/models.py')
| -rw-r--r-- | mediagoblin/db/mongo/models.py | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/mediagoblin/db/mongo/models.py b/mediagoblin/db/mongo/models.py index b068fb06..8cd0da1b 100644 --- a/mediagoblin/db/mongo/models.py +++ b/mediagoblin/db/mongo/models.py @@ -282,10 +282,8 @@ class MediaEntry(Document): 'uploader': self.uploader, 'state': 'processed'}).sort( '_id', ASCENDING).limit(1) - if cursor.count(): - return urlgen('mediagoblin.user_pages.media_home', - user=self.get_uploader().username, - media=unicode(cursor[0].slug)) + for media in cursor: + return media.url_for_self(urlgen) def url_to_next(self, urlgen): """ @@ -296,10 +294,8 @@ class MediaEntry(Document): 'state': 'processed'}).sort( '_id', DESCENDING).limit(1) - if cursor.count(): - return urlgen('mediagoblin.user_pages.media_home', - user=self.get_uploader().username, - media=unicode(cursor[0].slug)) + for media in cursor: + return media.url_for_self(urlgen) def get_uploader(self): return self.db.User.find_one({'_id': self.uploader}) |