From 146c00f55e4c7527dff39e0699fe34b37b7a9ef1 Mon Sep 17 00:00:00 2001 From: Pawel Solyga Date: Wed, 27 May 2009 03:22:37 +0200 Subject: [PATCH] Style fixes and wrong function call fix in soc.views.helper.lists module. --- app/soc/views/helper/lists.py | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/app/soc/views/helper/lists.py b/app/soc/views/helper/lists.py index 3cd53832..f9edb30c 100644 --- a/app/soc/views/helper/lists.py +++ b/app/soc/views/helper/lists.py @@ -198,18 +198,19 @@ def getListContent(request, params, filter=None, order=None, i[0].startswith('offset_') or i[0].startswith('limit_')) if params.get('list_key_order'): - export_link = generateLinkForRequest(request, base_params, {'export' : idx}) + export_link = generateLinkForRequest(request, base_params, {'export': idx}) if more: # TODO(dbentley): here we need to implement a new field "last_key" - next = generateLinkForRequest(request, base_params, {offset_key : offset+limit, - limit_key : limit}) + next = generateLinkForRequest(request, base_params, + {offset_key: offset + limit, + limit_key: limit}) if offset > 0: # TODO(dbentley): here we need to implement previous in the good way. prev = generateLinkForRequest(request, base_params, - { offset_key : max(0, offset-limit), - limit_key : limit }) + {offset_key: max(0, offset-limit), + limit_key: limit}) if offset > limit: # Having a link to the first doesn't make sense on the first page (we're on @@ -219,8 +220,8 @@ def getListContent(request, params, filter=None, order=None, # NOTE(dbentley): I personally disagree that it's simpler to do that way, # because sometimes you want to go to the first page without having to # consider what page you're on now. - newest = generateLinkForGetArgs(request, base_params, {offset_key : 0, - limit_key : limit}) + newest = generateLinkForRequest(request, base_params, {offset_key: 0, + limit_key: limit}) content = { 'idx': idx, -- 2.11.4.GIT