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!
This commit is contained in:
@@ -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})
|
||||
|
||||
@@ -136,9 +136,7 @@
|
||||
</div>
|
||||
{% endfor %}
|
||||
{{ render_pagination(request, pagination,
|
||||
request.urlgen('mediagoblin.user_pages.media_home',
|
||||
user = media.get_uploader().username,
|
||||
media = media._id)) }}
|
||||
media.url_for_self(request.urlgen)) }}
|
||||
{% endif %}
|
||||
</div>
|
||||
<div class="grid_5 omega">
|
||||
|
||||
Reference in New Issue
Block a user